drt: decoupling of the Graphics classes and stable drt_lib & graphic Factory

src/dpl/CMakeLists.txt

@@ -40,6 +40,12 @@ include("openroad")
 find_package(TCL)
 find_package(Boost)
 
+swig_lib(NAME         dpl
+         NAMESPACE    dpl
+         I_FILE       src/Opendp.i
+         SCRIPTS      src/Opendp.tcl
+)
+
 add_library(dpl_lib
   src/Opendp.cpp
   src/dbToOpendp.cpp
@@ -61,18 +67,11 @@ target_link_libraries(dpl_lib
     utl_lib
 )
 
-
 target_include_directories(dpl_lib
   PUBLIC
     include
 )
 
-swig_lib(NAME         dpl
-         NAMESPACE    dpl
-         I_FILE       src/Opendp.i
-         SCRIPTS      src/Opendp.tcl
-)
-
 target_sources(dpl
   PRIVATE
     src/MakeOpendp.cpp
@@ -90,9 +89,7 @@ target_include_directories(dpl
 target_link_libraries(dpl
   PRIVATE
     ${TCL_LIBRARY}
-    odb
     dpl_lib
-    OpenSTA
     gui
 )
 

src/drt/CMakeLists.txt

@@ -50,105 +50,99 @@ find_package(Boost REQUIRED COMPONENTS serialization)
 find_package(OpenMP REQUIRED)
 find_package(VTune)
 
+set(FLEXROUTE_HOME ${PROJECT_SOURCE_DIR})
+
 swig_lib(NAME      drt
          NAMESPACE drt
          I_FILE    src/TritonRoute.i
          SCRIPTS   src/TritonRoute.tcl
 )
 
-set(FLEXROUTE_HOME ${PROJECT_SOURCE_DIR})
-
-target_sources(drt
-  PRIVATE
-    src/gr/FlexGRCMap.cpp
-    src/gr/FlexGR.cpp
-    src/gr/FlexGR_end.cpp
-    src/gr/FlexGRGridGraph.cpp
-    src/gr/FlexGRGridGraph_maze.cpp
-    src/gr/FlexGR_init.cpp
-    src/gr/FlexGR_maze.cpp
-    src/gr/FlexGR_rq.cpp
-    src/gr/FlexGR_topo.cpp
-    src/dr/FlexDR_conn.cpp
-    src/dr/FlexDR_init.cpp
-    src/dr/FlexDR.cpp
-    src/db/drObj/drNet.cpp
-    src/dr/FlexDR_maze.cpp
-    src/dr/FlexGridGraph_maze.cpp
-    src/dr/FlexGridGraph.cpp
-    src/dr/FlexDR_rq.cpp
-    src/dr/FlexDR_end.cpp
-    src/dr/FlexDR_graphics.cpp
-    src/ta/FlexTA_end.cpp
-    src/ta/FlexTA_init.cpp
-    src/ta/FlexTA_rq.cpp
-    src/ta/FlexTA_assign.cpp
-    src/ta/FlexTA.cpp
-    src/ta/FlexTA_graphics.cpp
-    src/global.cpp
-    src/gc/FlexGC_end.cpp
-    src/gc/FlexGC_rq.cpp
-    src/gc/FlexGC.cpp
-    src/gc/FlexGC_init.cpp
-    src/gc/FlexGC_main.cpp
-    src/gc/FlexGC_eol.cpp
-    src/gc/FlexGC_inf.cpp
-    src/gc/FlexGC_cut.cpp
-    src/gc/FlexGC_metspc.cpp
-    src/db/drObj/drAccessPattern.cpp
-    src/db/drObj/drPin.cpp
-    src/db/drObj/drShape.cpp
-    src/db/drObj/drVia.cpp
-    src/db/infra/frTime_helper.cpp
-    src/db/infra/frTime.cpp
-    src/db/infra/KDTree.cpp
-    src/db/taObj/taShape.cpp
-    src/db/obj/frShape.cpp
-    src/db/obj/frInst.cpp
-    src/db/obj/frVia.cpp
-    src/db/obj/frAccess.cpp
-    src/db/obj/frRPin.cpp
-    src/db/obj/frNode.cpp
-    src/db/obj/frInstTerm.cpp
-    src/db/obj/frNet.cpp
-    src/db/tech/frConstraint.cc
-    src/db/obj/frMarker.cpp
-    src/db/tech/frLayer.cc
-    src/frRegionQuery.cpp
-    src/io/io_pin.cpp
-    src/io/io.cpp
-    src/io/GuideProcessor.cpp
-    src/io/io_parser_helper.cpp
-    src/pa/FlexPA_init.cpp
-    src/pa/FlexPA.cpp
-    src/pa/FlexPA_acc_point.cpp
-    src/pa/FlexPA_acc_pattern.cpp
-    src/pa/FlexPA_row_pattern.cpp
-    src/pa/FlexPA_unique.cpp
-    src/pa/FlexPA_graphics.cpp
-    src/rp/FlexRP_init.cpp
-    src/rp/FlexRP.cpp
-    src/rp/FlexRP_prep.cpp
-    src/distributed/frArchive.cpp
-    src/distributed/drUpdate.cpp
-    src/distributed/paUpdate.cpp
-    src/TritonRoute.cpp
-    src/MakeTritonRoute.cpp
-    src/frBaseTypes.cpp
-    src/DesignCallBack.cpp
+add_library(drt_lib
+  src/gr/FlexGRCMap.cpp
+  src/gr/FlexGR.cpp
+  src/gr/FlexGR_end.cpp
+  src/gr/FlexGRGridGraph.cpp
+  src/gr/FlexGRGridGraph_maze.cpp
+  src/gr/FlexGR_init.cpp
+  src/gr/FlexGR_maze.cpp
+  src/gr/FlexGR_rq.cpp
+  src/gr/FlexGR_topo.cpp
+  src/dr/FlexDR_conn.cpp
+  src/dr/FlexDR_init.cpp
+  src/dr/FlexDR.cpp
+  src/db/drObj/drNet.cpp
+  src/dr/FlexDR_maze.cpp
+  src/dr/FlexGridGraph_maze.cpp
+  src/dr/FlexGridGraph.cpp
+  src/dr/FlexDR_rq.cpp
+  src/dr/FlexDR_end.cpp
+  src/ta/FlexTA_end.cpp
+  src/ta/FlexTA_init.cpp
+  src/ta/FlexTA_rq.cpp
+  src/ta/FlexTA_assign.cpp
+  src/ta/FlexTA.cpp
+  src/global.cpp
+  src/gc/FlexGC_end.cpp
+  src/gc/FlexGC_rq.cpp
+  src/gc/FlexGC.cpp
+  src/gc/FlexGC_init.cpp
+  src/gc/FlexGC_main.cpp
+  src/gc/FlexGC_eol.cpp
+  src/gc/FlexGC_inf.cpp
+  src/gc/FlexGC_cut.cpp
+  src/gc/FlexGC_metspc.cpp
+  src/db/drObj/drAccessPattern.cpp
+  src/db/drObj/drPin.cpp
+  src/db/drObj/drShape.cpp
+  src/db/drObj/drVia.cpp
+  src/db/infra/frTime_helper.cpp
+  src/db/infra/frTime.cpp
+  src/db/infra/KDTree.cpp
+  src/db/taObj/taShape.cpp
+  src/db/obj/frShape.cpp
+  src/db/obj/frInst.cpp
+  src/db/obj/frVia.cpp
+  src/db/obj/frAccess.cpp
+  src/db/obj/frRPin.cpp
+  src/db/obj/frNode.cpp
+  src/db/obj/frInstTerm.cpp
+  src/db/obj/frNet.cpp
+  src/db/tech/frConstraint.cc
+  src/db/obj/frMarker.cpp
+  src/db/tech/frLayer.cc
+  src/frRegionQuery.cpp
+  src/io/io_pin.cpp
+  src/io/io.cpp
+  src/io/GuideProcessor.cpp
+  src/io/io_parser_helper.cpp
+  src/pa/FlexPA_init.cpp
+  src/pa/FlexPA.cpp
+  src/pa/FlexPA_acc_point.cpp
+  src/pa/FlexPA_acc_pattern.cpp
+  src/pa/FlexPA_row_pattern.cpp
+  src/pa/FlexPA_unique.cpp
+  src/rp/FlexRP_init.cpp
+  src/rp/FlexRP.cpp
+  src/rp/FlexRP_prep.cpp
+  src/distributed/frArchive.cpp
+  src/distributed/drUpdate.cpp
+  src/distributed/paUpdate.cpp
+  src/TritonRoute.cpp
+  src/frBaseTypes.cpp
+  src/DesignCallBack.cpp
 )
 
-target_include_directories(drt
+target_include_directories(drt_lib
   PUBLIC
     include
 
   PRIVATE
     src
 )
 
-target_link_libraries(drt
+target_link_libraries(drt_lib
   PUBLIC
-    gui
     odb
     stt
     OpenSTA
@@ -161,6 +155,29 @@ target_link_libraries(drt
     ZLIB::ZLIB
 )
 
+target_sources(drt
+  PRIVATE
+    src/dr/FlexDR_graphics.cpp
+    src/ta/FlexTA_graphics.cpp
+    src/pa/FlexPA_graphics.cpp
+    src/MakeTritonRoute.cpp
+    src/GraphicsFactory.cpp
+)
+
+target_include_directories(drt
+  PUBLIC
+    include
+
+  PRIVATE
+    src
+)
+
+target_link_libraries(drt
+  PUBLIC
+    drt_lib
+    gui
+)
+
 messages(
   TARGET drt
 )

src/drt/include/triton_route/TritonRoute.h

@@ -69,13 +69,15 @@ namespace drt {
 class frDesign;
 class DesignCallBack;
 class FlexDR;
+class FlexPA;
+class FlexTA;
 class FlexDRWorker;
 class drUpdate;
 struct frDebugSettings;
-class FlexDR;
 struct FlexDRViaData;
 class frMarker;
 struct RouterConfiguration;
+class AbstractGraphicsFactory;
 
 struct ParamStruct
 {
@@ -107,13 +109,14 @@ class TritonRoute
  public:
   TritonRoute();
   ~TritonRoute();
-  void init(Tcl_Interp* tcl_interp,
-            odb::dbDatabase* db,
+  void init(odb::dbDatabase* db,
             utl::Logger* logger,
             dst::Distributed* dist,
-            stt::SteinerTreeBuilder* stt_builder);
+            stt::SteinerTreeBuilder* stt_builder,
+            std::unique_ptr<AbstractGraphicsFactory> graphics_factory);
 
   frDesign* getDesign() const { return design_.get(); }
+  utl::Logger* getLogger() const { return logger_; }
   RouterConfiguration* getRouterConfiguration() const
   {
     return router_cfg_.get();
@@ -221,8 +224,11 @@ class TritonRoute
   int results_sz_{0};
   unsigned int cloud_sz_{0};
   std::optional<boost::asio::thread_pool> dist_pool_;
+  std::unique_ptr<FlexPA> pa_{nullptr};
+  std::unique_ptr<AbstractGraphicsFactory> graphics_factory_{nullptr};
 
   void initDesign();
+  void initGraphics();
   void gr();
   void ta();
   void dr();

src/drt/src/AbstractGraphicsFactory.h

@@ -0,0 +1,61 @@
+//////////////////////////////////////////////////////////////////////////////
+// BSD 3-Clause License
+//
+// Copyright (c) 2025, Precision Innovations Inc.
+// All rights reserved.
+//
+// Redistribution and use in source and binary forms, with or without
+// modification, are permitted provided that the following conditions are met:
+//
+// * Redistributions of source code must retain the above copyright notice, this
+//   list of conditions and the following disclaimer.
+//
+// * Redistributions in binary form must reproduce the above copyright notice,
+//   this list of conditions and the following disclaimer in the documentation
+//   and/or other materials provided with the distribution.
+//
+// * Neither the name of the copyright holder nor the names of its
+//   contributors may be used to endorse or promote products derived from
+//   this software without specific prior written permission.
+//
+// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+// AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+// IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+// ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
+// LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+// CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+// SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+// INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+// CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+// ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+// POSSIBILITY OF SUCH DAMAGE.
+
+#pragma once
+
+#include <memory>
+#include <vector>
+
+#include "db/obj/frBlockObject.h"
+#include "dr/AbstractDRGraphics.h"
+#include "frBaseTypes.h"
+#include "pa/AbstractPAGraphics.h"
+#include "ta/AbstractTAGraphics.h"
+
+namespace drt {
+class AbstractGraphicsFactory
+{
+ public:
+  virtual ~AbstractGraphicsFactory() = default;
+  virtual void reset(frDebugSettings* settings,
+                     frDesign* design,
+                     odb::dbDatabase* db,
+                     Logger* logger,
+                     RouterConfiguration* router_cfg)
+      = 0;
+  virtual bool guiActive() = 0;
+  virtual std::unique_ptr<AbstractDRGraphics> makeUniqueDRGraphics() = 0;
+  virtual std::unique_ptr<AbstractTAGraphics> makeUniqueTAGraphics() = 0;
+  virtual std::unique_ptr<AbstractPAGraphics> makeUniquePAGraphics() = 0;
+};
+
+}  // namespace drt