rename OpenROAD.RepairDesign to OpenROAD.RepairDesignPostGPL

efabless · Oct 2, 2023 · 1422fc8 · 1422fc8
1 parent 0fde233
commit 1422fc8
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 5 deletions.
diff --git a/openlane/flows/classic.py b/openlane/flows/classic.py
@@ -134,11 +134,11 @@ class Classic(SequentialFlow):
             deprecated_names=["TAP_DECAP_INSERTION", "RUN_TAP_DECAP_INSERTION"],
         ),
         Variable(
-            "RUN_REPAIR_DESIGN",
+            "RUN_REPAIR_DESIGN_POST_GPL",
             bool,
             "Enables resizer design repair using the OpenROAD.RepairDesign step.",
             default=True,
-            deprecated_names=["PL_RESIZER_DESIGN_OPTIMIZATIONS"],
+            deprecated_names=["PL_RESIZER_DESIGN_OPTIMIZATIONS", "RUN_REPAIR_DESIN"],
         ),
         Variable(
             "RUN_CTS",
@@ -315,7 +315,7 @@ class Classic(SequentialFlow):
             "RUN_LINTER",
             "QUIT_ON_LINTER_TIMING_CONSTRUCTS",
         ],
-        "OpenROAD.RepairDesign": ["RUN_REPAIR_DESIGN"],
+        "OpenROAD.RepairDesign": ["RUN_REPAIR_DESIGN_POST_GPL"],
         "OpenROAD.CTS": ["RUN_CTS"],
         "OpenROAD.ResizerTimingPostCTS": ["RUN_POST_CTS_RESIZER_TIMING"],
         "OpenROAD.RCX": ["RUN_SPEF_EXTRACTION"],

diff --git a/openlane/steps/openroad.py b/openlane/steps/openroad.py
@@ -1437,12 +1437,12 @@ def run(self, state_in: State, **kwargs) -> Tuple[ViewsUpdate, MetricsUpdate]:
 
 
 @Step.factory.register()
-class RepairDesign(ResizerStep):
+class RepairDesignPostGPL(ResizerStep):
     """
     Runs a number of design "repairs" on a global-placed ODB file.
     """
 
-    id = "OpenROAD.RepairDesign"
+    id = "OpenROAD.RepairDesignPostGPL"
     name = "Repair Design (Post-Global Placement)"
 
     config_vars = ResizerStep.config_vars + [
@@ -1504,6 +1504,16 @@ def get_script_path(self):
         return os.path.join(get_script_dir(), "openroad", "repair_design.tcl")
 
 
+@Step.factory.register()
+class RepairDesign(RepairDesignPostGPL):
+    """
+    This is identical to OpenROAD.RepairDesignPostGPL. It is retained for backwards compatibility.
+    """
+
+    id = "OpenROAD.RepairDesign"
+    name = "Repair Design (Post-Global Placement)"
+
+
 @Step.factory.register()
 class RepairDesignPostGRT(ResizerStep):
     """