Skip to content

update metric for DPL change - #4446

Open
gudeh wants to merge 10 commits into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:secure-dpl-remove-negotiation-padding
Open

update metric for DPL change#4446
gudeh wants to merge 10 commits into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:secure-dpl-remove-negotiation-padding

Conversation

@gudeh

@gudeh gudeh commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

update metrics for OR PR: The-OpenROAD-Project/OpenROAD#11089

designs/gf12/tinyRocket/rules-base.json updates:

Metric Old New Type
synth__design__instance__area__stdcell 13122.1 10800.0 Tighten
placeopt__design__instance__area 16686 16624 Tighten
cts__timing__setup__tns -868.0 -1430.0 Failing
finish__design__instance__area 17217 17174 Tighten

gudeh added 5 commits August 6, 2026 19:52
@gudeh
master version
15f5ca3 
Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>
@gudeh
update OR
14499a6 
Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>
@gudeh
update OR
7fda039 
Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>
@gudeh
Merge remote-tracking branch 'private/master' into secure-dpl-remove-…
88e68e4 
…negotiation-padding
@gudeh
update metric after DPL change
ebc7be3 
designs/gf12/tinyRocket/rules-base.json updates:
| Metric                                        | Old        | New        | Type     |
| ------                                        | ---        | ---        | ----     |
| synth__design__instance__area__stdcell        |    13122.1 |    10800.0 | Tighten  |
| placeopt__design__instance__area              |      16686 |      16624 | Tighten  |
| cts__timing__setup__tns                       |     -868.0 |    -1430.0 | Failing  |
| finish__design__instance__area                |      17217 |      17174 | Tighten  |

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>
@gudeh gudeh self-assigned this Aug 19, 2026
gemini-code-assist[bot]
gemini-code-assist Bot reviewed Aug 19, 2026
View reviewed changes

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the tools/OpenROAD subproject commit and adjusts several baseline rules in flow/designs/gf12/tinyRocket/rules-base.json. Specifically, it adds netlist hash warnings, decreases the standard cell area and placement/finish instance areas, and adjusts the CTS setup TNS limit. There are no review comments, and I have no feedback to provide.

@openroad-ci

openroad-ci commented Aug 19, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

🔍 QoR check

Metrics reflect the PR merge build — i.e. what will land on the target branch. Advisory — results are log-only and do not affect build status. The authoritative QoR gate remains the local rules-file check.

Commit 5bf8c56 · Jenkins build #4 · Baseline: build

62 design(s) checked — 0 with regression(s), 0 without a comparable baseline.
Design Variant Verdict local vs inline passed / failed / missing
asap7/aes base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/aes-block base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/aes-mbff base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/aes_lvt base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/cva6 base ✅ pass ✅ MATCH 12 / 0 / 0
asap7/ethmac base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/ethmac_lvt base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/gcd base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/gcd-ccs base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/ibex base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/jpeg base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/jpeg_lvt base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/mock-alu base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/mock-cpu base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/riscv32i base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/riscv32i-mock-sram base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/swerv_wrapper base ✅ pass ✅ MATCH 25 / 0 / 0
asap7/uart base ✅ pass ✅ MATCH 25 / 0 / 0
gf180/aes base ✅ pass ✅ MATCH 25 / 0 / 0
gf180/aes-hybrid base ✅ pass ✅ MATCH 25 / 0 / 0
gf180/ibex base ✅ pass ✅ MATCH 25 / 0 / 0
gf180/jpeg base ✅ pass ✅ MATCH 25 / 0 / 0
gf180/riscv32i base ✅ pass ✅ MATCH 25 / 0 / 0
gf180/uart-blocks base ✅ pass ✅ MATCH 25 / 0 / 0
gt2n/aes base ✅ pass ✅ MATCH 25 / 0 / 0
gt2n/gcd base ✅ pass ✅ MATCH 25 / 0 / 0
gt2n/jpeg base ✅ pass ✅ MATCH 25 / 0 / 0
ihp-sg13g2/aes base ✅ pass ✅ MATCH 25 / 0 / 0
ihp-sg13g2/gcd base ✅ pass ✅ MATCH 25 / 0 / 0
ihp-sg13g2/i2c-gpio-expander base ✅ pass ✅ MATCH 25 / 0 / 0
ihp-sg13g2/ibex base ✅ pass ✅ MATCH 25 / 0 / 0
ihp-sg13g2/jpeg base ✅ pass ✅ MATCH 25 / 0 / 0
ihp-sg13g2/riscv32i base ✅ pass ✅ MATCH 25 / 0 / 0
ihp-sg13g2/spi base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/aes base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/ariane133 base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/ariane136 base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/black_parrot base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/bp_be_top base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/bp_fe_top base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/bp_multi_top base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/cva6 base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/dynamic_node base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/gcd base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/ibex base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/jpeg base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/mempool_group base ✅ pass ✅ MATCH 12 / 0 / 0
nangate45/swerv base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/swerv_wrapper base ✅ pass ✅ MATCH 25 / 0 / 0
nangate45/tinyRocket base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hd/aes base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hd/chameleon base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hd/gcd base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hd/ibex base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hd/jpeg base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hd/microwatt base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hd/riscv32i base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hs/aes base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hs/gcd base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hs/ibex base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hs/jpeg base ✅ pass ✅ MATCH 25 / 0 / 0
sky130hs/riscv32i base ✅ pass ✅ MATCH 25 / 0 / 0

gudeh added 3 commits August 20, 2026 13:29
@gudeh
update OR
075197c 
Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>
@gudeh
Merge remote-tracking branch 'private/master' into secure-dpl-remove-…
c65cf86 
…negotiation-padding
@gudeh
update OR
dbaec5a 
Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>
@gudeh
gudeh requested a review from eder-matheus August 20, 2026 21:42
@gudeh

gudeh commented Aug 20, 2026

Copy link
Copy Markdown
Contributor Author

🔍 QoR check

Metrics reflect the PR merge build — i.e. what will land on the target branch. Advisory — results are log-only and do not affect build status. The authoritative QoR gate remains the local rules-file check.

Commit bb868b5 · Jenkins build #3 · Baseline: build

62 design(s) checked — 3 with regression(s), 1 without a comparable baseline.
❌ asap7/ethmac base — 2 failing metric(s)
Metric target base delta limit band
cts__timing__setup__tns -1438.99 -1238.26 16.21065042882755% -1298.26 PeriodPadding 20%
finish__timing__setup__tns -1430.55 -1337.97 6.919437655552815% -1397.97 PeriodPadding 20%
❌ nangate45/mempool_group base — 2 failing metric(s)
Metric target base delta limit band
finish__timing__setup__tns -11523.9 -11440.5 0.7289891176084962% -11441.1 PeriodPadding 20%
globalroute__timing__setup__tns -11626.2 -11501.8 1.0815698412422403% -11502.4 PeriodPadding 20%
❌ sky130hd/microwatt base — 2 failing metric(s)

This seems incorrect. I see only uW failing

@eder-matheus

eder-matheus commented Aug 20, 2026

Copy link
Copy Markdown
Member

@gudeh Please wait until #4452 is merged, then update your branch to get the fixes for microwatt and ca53

gudeh added 2 commits August 21, 2026 00:32
@gudeh
update OR
c4159a1 
Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>
@gudeh
Merge remote-tracking branch 'private/master' into secure-dpl-remove-…
e32927f 
…negotiation-padding
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@eder-matheus eder-matheus Awaiting requested review from eder-matheus
1 more reviewer
@gemini-code-assist gemini-code-assist[bot] gemini-code-assist[bot] left review comments
Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

@gudeh gudeh

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@gudeh @openroad-ci @eder-matheus