-
Notifications
You must be signed in to change notification settings - Fork 536
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[DON'T MERGE] odb: add command to check if design is routed #5879
base: master
Are you sure you want to change the base?
[DON'T MERGE] odb: add command to check if design is routed #5879
Conversation
Signed-off-by: Eder Monteiro <[email protected]>
…ROAD into odb_is_routed Signed-off-by: Eder Monteiro <[email protected]>
Signed-off-by: Eder Monteiro <[email protected]>
Signed-off-by: Eder Monteiro <[email protected]>
clang-tidy review says "All clean, LGTM! 👍" |
1 similar comment
clang-tidy review says "All clean, LGTM! 👍" |
Signed-off-by: Eder Monteiro <[email protected]>
clang-tidy review says "All clean, LGTM! 👍" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this have a set of CI tests?
Signed-off-by: Eder Monteiro <[email protected]>
Yes, I'm working on it. I'm also testing it in the CI to check if I'm missing some check. |
clang-tidy review says "All clean, LGTM! 👍" |
if (pin_count > 1 && net->getSWires().empty() && net->getWire() == nullptr | ||
&& !net->isConnectedByAbutment()) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if the SWires is not empty, you should probably verify that they actually have routing in them.
Same for getWire (ensure it actually contains something).
No description provided.