Skip to content

Commit b844d41

Browse files
committed
ifp: Fix gap on the right of the power domain.
We want at least power_domain_y_space * site_dy. The ceil function should be used instead of the implicit floor one. Also make odb::makeSiteLoc take care of the that (currently it's noop as the input is already snapped to site width). Signed-off-by: Christian COSTA <[email protected]>
1 parent 7fe9b26 commit b844d41

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

src/ifp/src/InitFloorplan.cc

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -552,11 +552,7 @@ void InitFloorplan::updateVoltageDomain(const int core_lx,
552552
}
553553

554554
// rcr stands for right core row
555-
// rcr_dx_site_number is the max number of site_dx that is less than
556-
// power_domain_y_space * site_dy. This helps align the rcr_x_min on
557-
// the multiple of site_dx.
558-
int rcr_dx_site_number = (power_domain_y_space * site_dy) / site_dx;
559-
int rcr_x_min = domain_x_max + rcr_dx_site_number * site_dx;
555+
int rcr_x_min = domain_x_max + power_domain_y_space * site_dy;
560556
// snap to the site grid rightward
561557
rcr_x_min = odb::makeSiteLoc(rcr_x_min, site_dx, false, 0);
562558

0 commit comments

Comments
 (0)