Skip to content

Commit 12a569e

Browse files
authored
Merge pull request #9870 from Divinesoumyadip/fix/grt-cugr-assert-to-logger
grt: replace assert with logger error in CUGR maze route
2 parents a964bd8 + 1e74f46 commit 12a569e

1 file changed

Lines changed: 12 additions & 3 deletions

File tree

src/grt/src/cugr/src/CUGR.cpp

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
#include "CUGR.h"
22

33
#include <algorithm>
4-
#include <cassert>
54
#include <cmath>
65
#include <cstddef>
76
#include <cstdint>
@@ -229,7 +228,12 @@ void CUGR::mazeRoute(std::vector<int>& net_indices)
229228
maze_route.constructSparsifiedGraph(wire_cost_view, grid);
230229
maze_route.run();
231230
std::shared_ptr<SteinerTreeNode> tree = maze_route.getSteinerTree();
232-
assert(tree != nullptr);
231+
if (tree == nullptr) {
232+
logger_->error(GRT,
233+
610,
234+
"Failed to generate Steiner tree for net {}.",
235+
net->getName());
236+
}
233237

234238
PatternRoute pattern_route(
235239
net, grid_graph_.get(), stt_builder_, constants_, logger_);
@@ -414,7 +418,12 @@ void CUGR::getGuides(const GRNet* net,
414418
};
415419

416420
// 1. Pin access patches
417-
assert(constants_.min_routing_layer + 1 < grid_graph_->getNumLayers());
421+
if (constants_.min_routing_layer + 1 >= grid_graph_->getNumLayers()) {
422+
logger_->error(GRT,
423+
611,
424+
"Min routing layer {} exceeds available layers.",
425+
constants_.min_routing_layer);
426+
}
418427
for (auto& gpts : net->getPinAccessPoints()) {
419428
for (auto& gpt : gpts) {
420429
if (gpt.getLayerIdx() < constants_.min_routing_layer) {

0 commit comments

Comments
 (0)