From 1422fc86445e3fb5c192471863447bf352d77b8a Mon Sep 17 00:00:00 2001
From: kareefardi <kareem.farid@efabless.com>
Date: Mon, 2 Oct 2023 16:37:13 +0300
Subject: [PATCH] rename OpenROAD.RepairDesign to OpenROAD.RepairDesignPostGPL

---
 openlane/flows/classic.py  |  6 +++---
 openlane/steps/openroad.py | 14 ++++++++++++--
 2 files changed, 15 insertions(+), 5 deletions(-)

diff --git a/openlane/flows/classic.py b/openlane/flows/classic.py
index 864dde2d9..a34d1f3a8 100644
--- 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
index 8e06ade62..f2c2d1461 100644
--- 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):
     """