P&R: Move remove_buffers pass to floorplan stage

mtdudek · mtdudek · commit 59290d44989d · 2023-12-18T11:25:56.000+01:00
Move `remove_buffers` pass to run just after floorplan
initialization. It `remove_buffers` should only remove
clock/data buffers added by synthesis process.

Signed-off-by: Maciej Dudek &lt;mdudek@antmicro.com&gt;
diff --git a/place_and_route/private/floorplan.bzl b/place_and_route/private/floorplan.bzl
@@ -91,6 +91,7 @@ def init_floor_plan(ctx):
         "source {tracks_file}".format(
             tracks_file = open_road_configuration.tracks_file.path,
         ),
+        "remove_buffers",
         "insert_tiecells {port} -prefix \"TIE_ONE_\"".format(
             port = tieoneport,
         ),
diff --git a/place_and_route/private/global_placement.bzl b/place_and_route/private/global_placement.bzl
@@ -40,7 +40,6 @@ def global_placement(ctx, open_road_info):
             pad_left = open_road_configuration.global_placement_cell_pad,
             pad_right = open_road_configuration.global_placement_cell_pad,
         ),
-        "remove_buffers",
     ]
 
     command_output = openroad_command(

Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,6 @@ def global_placement(ctx, open_road_info):`
`40`	`40`	`pad_left = open_road_configuration.global_placement_cell_pad,`
`41`	`41`	`pad_right = open_road_configuration.global_placement_cell_pad,`
`42`	`42`	`),`
`43`		`- "remove_buffers",`
`44`	`43`	`]`
`45`	`44`
`46`	`45`	`command_output = openroad_command(`