Merge pull request #9057 from osamahammad21/drt-unidirectional-layers

DRT: set unidirectional layers in router_cfg_

Author: maliberty

src/drt/src/TritonRoute.cpp

@@ -572,13 +572,14 @@ void TritonRoute::initDesign()
       || db_->getChip()->getBlock() == nullptr) {
     logger_->error(utl::DRT, 151, "Database, chip or block not initialized.");
   }
+  const bool design_exists = getDesign()->getTopBlock() != nullptr;
   io::Parser parser(db_, getDesign(), logger_, router_cfg_.get());
-  if (getDesign()->getTopBlock() != nullptr) {
+  if (design_exists) {
     parser.updateDesign();
-    return;
+  } else {
+    parser.readTechAndLibs(db_);
+    parser.readDesign(db_);
   }
-  parser.readTechAndLibs(db_);
-  parser.readDesign(db_);
   auto tech = getDesign()->getTech();
 
   if (!router_cfg_->VIAINPIN_BOTTOMLAYER_NAME.empty()) {
@@ -628,9 +629,11 @@ void TritonRoute::initDesign()
                     router_cfg_->REPAIR_PDN_LAYER_NAME);
     }
   }
-  parser.postProcess();
-  db_callback_->addOwner(db_->getChip()->getBlock());
-  initGraphics();
+  if (!design_exists) {
+    parser.postProcess();
+    db_callback_->addOwner(db_->getChip()->getBlock());
+    initGraphics();
+  }
 }
 
 void TritonRoute::initGraphics()
@@ -1277,7 +1280,7 @@ void TritonRoute::setUnidirectionalLayer(const std::string& layerName)
                    "Non-routing layer {} can't be set unidirectional",
                    layerName);
   }
-  design_->getTech()->setUnidirectionalLayer(dbLayer);
+  router_cfg_->unidirectional_layers_.insert(dbLayer);
 }
 
 void TritonRoute::setParams(const ParamStruct& params)

src/drt/src/db/tech/frTechObject.h

@@ -59,10 +59,6 @@ class frTechObject
   {
     return viaRuleGenerates_;
   }
-  bool hasUnidirectionalLayer(odb::dbTechLayer* dbLayer) const
-  {
-    return unidirectional_layers_.find(dbLayer) != unidirectional_layers_.end();
-  }
   bool hasMaxSpacingConstraints() const
   {
     for (const auto& layer : layers_) {
@@ -126,10 +122,6 @@ class frTechObject
     }
     return nullptr;
   }
-  void setUnidirectionalLayer(odb::dbTechLayer* dbLayer)
-  {
-    unidirectional_layers_.insert(dbLayer);
-  }
 
   // forbidden length table related
   bool isVia2ViaForbiddenLen(int tableLayerIdx,
@@ -406,8 +398,6 @@ class frTechObject
   ByLayer<std::array<bool, 4>> viaForbiddenThrough_;
   bool hasVia2viaMinStep_ = false;
   bool hasCornerSpacingConstraint_ = false;
-  // unidirectional layers
-  std::set<odb::dbTechLayer*> unidirectional_layers_;
 
   friend class FlexRP;
 };

src/drt/src/global.h

@@ -9,13 +9,15 @@
 #include <memory>
 #include <optional>
 #include <string>
+#include <unordered_set>
 
 #include "db/obj/frMarker.h"
 #include "frBaseTypes.h"
 
 namespace odb {
 class Point;
 class Rect;
+class dbTechLayer;
 }  // namespace odb
 
 namespace drt {
@@ -97,6 +99,9 @@ struct RouterConfiguration
   std::string REPAIR_PDN_LAYER_NAME;
   frLayerNum REPAIR_PDN_LAYER_NUM = -1;
   frLayerNum GC_IGNORE_PDN_LAYER_NUM = -1;
+
+  // unidirectional layers
+  std::unordered_set<odb::dbTechLayer*> unidirectional_layers_;
 };
 
 constexpr int DIRBITSIZE = 3;

src/drt/src/io/io.cpp

@@ -1574,7 +1574,7 @@ void io::Parser::setRoutingLayerProperties(odb::dbTechLayer* layer,
     getTech()->addUConstraint(std::move(uCon));
     tmpLayer->addLef58SpacingWrongDirConstraint(rptr);
   }
-  if (getTech()->hasUnidirectionalLayer(layer)) {
+  if (router_cfg_->unidirectional_layers_.contains(layer)) {
     tmpLayer->setUnidirectional(true);
   }
   if (layer->isRectOnly()) {