OpenROAD
OpenROAD copied to clipboard
gpl: modify routability driven default target RC value
With the previous default of 1.25 no design was being activated to run with routability driven option. With 1.0 we have routability driven mode working, resulting in less DRT iterations on a handful of designs.
clang-tidy review says "All clean, LGTM! :+1:"
@eder-matheus there is a secure CI running for this and so far it looks good. Please take a look tomorrow and see if it mergeable with some minor metrics updates.
@eder-matheus there is a secure CI running for this and so far it looks good. Please take a look tomorrow and see if it mergeable with some minor metrics updates.
It has two metric failures, but ca53 has a timeout during DRT at the 46th iteration. In the master branch, ca53 ends with 30 iterations.
clang-tidy review says "All clean, LGTM! :+1:"
Secure-CI is finishing without errors.
clang-tidy review says "All clean, LGTM! :+1:"
Also, there was some metrics update on the secure-CI:
-
aes_lvt asap7 [ERROR] finish__timing__setup__ws fail test: -135.525 >= -105.98 [ERROR] finish__timing__wns_percent_delay fail test: -22.766924 >= -22.04
-
riscv32i asap7 [ERROR] finish__timing__setup__ws fail test: -553.269 >= -430.53 [ERROR] finish__timing__drv__setup_violation_count fail test: 1069.0 <= 1059.0
A PR for ORFS will be created after this PR is merged.
A PR for ORFS will be created after this PR is merged.
Its good to have both ready so they can be merged at the same time. You only need to update the metrics in that PR.