tightens gf12/aes, gf12/gcd and sky130hs/ibex by AcKoucher · Pull Request #4507 · The-OpenROAD-Project/OpenROAD-flow-scripts

AcKoucher · 2026-09-03T19:33:40Z

New Metrics

sky130hs/ibex:

Metric	Old	New	Type
cts__timing__setup__tns	-3.2	-57.7	Failing
cts__timing__hold__ws	-0.35	-0.325	Tighten
cts__timing__hold__tns	-1.4	-1.3	Tighten
globalroute__timing__setup__tns	-338.0	-604.0	Failing
globalroute__timing__hold__ws	-0.35	-0.325	Tighten
globalroute__timing__hold__tns	-1.4	-1.3	Tighten
detailedroute__route__wirelength	824415	783228	Tighten
finish__timing__setup__ws	-0.35	-0.861	Failing
finish__timing__setup__tns	-62.1	-291.0	Failing
finish__timing__hold__ws	-0.35	-0.325	Tighten
finish__timing__hold__tns	-1.4	-1.3	Tighten

gf12/aes:

Metric	Old	New	Type
cts__timing__hold__ws	-21.0	-18.0	Tighten
cts__timing__hold__tns	-84.0	-72.0	Tighten
globalroute__timing__setup__tns	-84.0	-368.0	Failing
globalroute__timing__hold__ws	-21.0	-18.0	Tighten
globalroute__timing__hold__tns	-84.0	-72.0	Tighten
detailedroute__route__wirelength	126525	125219	Tighten
finish__timing__setup__tns	-84.0	-171.0	Failing
finish__timing__hold__ws	-21.0	-18.0	Tighten
finish__timing__hold__tns	-84.0	-72.0	Tighten

gf12/gcd:

Metric	Old	New	Type
synth__canonical_netlist__hash	a5874bf0 [... see commit message]	d8877e14 [... see commit message]	Failing
synth__netlist__hash	4289d976 [... see commit message]	308043d4 [... see commit message]	Failing
placeopt__design__instance__area	472	228	Tighten
placeopt__design__instance__count__stdcell	974	607	Tighten
cts__design__instance__count__setup_buffer	85	53	Tighten
cts__design__instance__count__hold_buffer	85	53	Tighten
cts__timing__setup__ws	-21.5	-21.3	Tighten
cts__timing__hold__ws	-14.0	-12.0	Tighten
globalroute__timing__setup__ws	-26.3	-26.0	Tighten
globalroute__timing__setup__tns	-250.0	-435.0	Failing
globalroute__timing__hold__ws	-14.0	-12.0	Tighten
detailedroute__route__wirelength	4449	2026	Tighten
finish__timing__setup__tns	-56.0	-113.0	Failing
finish__timing__hold__ws	-14.0	-12.0	Tighten
finish__design__instance__area	490	240	Tighten

1) Increase core utilization; 2) Use LB_ADDON instead of raw density; 3) Lower clock period. Signed-off-by: Arthur Koucher <arthurkoucher@precisioninno.com>

1) Use CORE_UTILIZATION instead of hard-coded dimensions; 2) Use PLACE_DENSITY_LB_ADDON instead of raw placement density; 3) Reduce clock period to increase negative slack. Signed-off-by: Arthur Koucher <arthurkoucher@precisioninno.com>

1) Increase CORE_UTILIZATION; 2) Lower clock period. Signed-off-by: Arthur Koucher <arthurkoucher@precisioninno.com>

gemini-code-assist

Code Review

This pull request updates configuration and constraint files for several designs (aes, gcd, and ibex) across different platforms (gf12 and sky130hs). Specifically, it adjusts core utilization, replaces or updates placement density parameters, and reduces clock periods to tighten timing constraints. In the gcd design, fixed die and core areas are replaced with dynamic utilization parameters. There are no review comments, so I have no feedback to provide.

| Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__hold__ws | -21.0 | -18.0 | Tighten | | cts__timing__hold__tns | -84.0 | -72.0 | Tighten | | globalroute__timing__setup__tns | -84.0 | -368.0 | Failing | | globalroute__timing__hold__ws | -21.0 | -18.0 | Tighten | | globalroute__timing__hold__tns | -84.0 | -72.0 | Tighten | | detailedroute__route__wirelength | 126525 | 125219 | Tighten | | finish__timing__setup__tns | -84.0 | -171.0 | Failing | | finish__timing__hold__ws | -21.0 | -18.0 | Tighten | | finish__timing__hold__tns | -84.0 | -72.0 | Tighten | gf12/gcd: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | synth__canonical_netlist__hash | a5874bf0d7a4676c0e25a99a5c7f6fcc237ed682 | d8877e14f6cc83df1d67733f6bf3d9e8a9dac8bd | Failing | | synth__netlist__hash | 4289d9760a66cd6331d0d0a75d97005969ecace5 | 308043d4a1cccf693410f3548be8b367ee18ffa4 | Failing | | placeopt__design__instance__area | 472 | 228 | Tighten | | placeopt__design__instance__count__stdcell | 974 | 607 | Tighten | | cts__design__instance__count__setup_buffer | 85 | 53 | Tighten | | cts__design__instance__count__hold_buffer | 85 | 53 | Tighten | | cts__timing__setup__ws | -21.5 | -21.3 | Tighten | | cts__timing__hold__ws | -14.0 | -12.0 | Tighten | | globalroute__timing__setup__ws | -26.3 | -26.0 | Tighten | | globalroute__timing__setup__tns | -250.0 | -435.0 | Failing | | globalroute__timing__hold__ws | -14.0 | -12.0 | Tighten | | detailedroute__route__wirelength | 4449 | 2026 | Tighten | | finish__timing__setup__tns | -56.0 | -113.0 | Failing | | finish__timing__hold__ws | -14.0 | -12.0 | Tighten | | finish__design__instance__area | 490 | 240 | Tighten | Signed-off-by: Arthur Koucher <arthurkoucher@precisioninno.com>

openroad-ci · 2026-09-03T21:36:37Z

🔍 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 51a5b77 · Jenkins build #3 · Baseline: build

62 design(s) checked — 1 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	❌ fail	⚠️ MISMATCH	22 / 3 / 0
sky130hs/jpeg	base	✅ pass	✅ MATCH	25 / 0 / 0
sky130hs/riscv32i	base	✅ pass	✅ MATCH	25 / 0 / 0

❌ sky130hs/ibex base — 3 failing metric(s)

Metric	target	base	delta	limit	band
`cts__timing__setup__tns`	-56.4332	-1.96242	2775.694295818428%	-3.36242	PeriodPadding 20%
`finish__timing__setup__tns`	-289.801	-39.3099	637.221412417737%	-40.7099	PeriodPadding 20%
`globalroute__timing__setup__tns`	-602.407	-119.866	402.5670331870589%	-121.266	PeriodPadding 20%

| Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__setup__tns | -3.2 | -57.7 | Failing | | cts__timing__hold__ws | -0.35 | -0.325 | Tighten | | cts__timing__hold__tns | -1.4 | -1.3 | Tighten | | globalroute__timing__setup__tns | -338.0 | -604.0 | Failing | | globalroute__timing__hold__ws | -0.35 | -0.325 | Tighten | | globalroute__timing__hold__tns | -1.4 | -1.3 | Tighten | | detailedroute__route__wirelength | 824415 | 783228 | Tighten | | finish__timing__setup__ws | -0.35 | -0.861 | Failing | | finish__timing__setup__tns | -62.1 | -291.0 | Failing | | finish__timing__hold__ws | -0.35 | -0.325 | Tighten | | finish__timing__hold__tns | -1.4 | -1.3 | Tighten | Signed-off-by: Arthur Koucher <arthurkoucher@precisioninno.com>

AcKoucher added 4 commits August 31, 2026 23:16

gf12/aes:

cd04e0d

1) Increase core utilization; 2) Use LB_ADDON instead of raw density; 3) Lower clock period. Signed-off-by: Arthur Koucher <arthurkoucher@precisioninno.com>

gf12/gcd:

3d0e09c

1) Use CORE_UTILIZATION instead of hard-coded dimensions; 2) Use PLACE_DENSITY_LB_ADDON instead of raw placement density; 3) Reduce clock period to increase negative slack. Signed-off-by: Arthur Koucher <arthurkoucher@precisioninno.com>

sky130hs/ibex:

0fa8a6a

1) Increase CORE_UTILIZATION; 2) Lower clock period. Signed-off-by: Arthur Koucher <arthurkoucher@precisioninno.com>

Merge branch 'master' into arthur-koucher-designs

8052fc8

AcKoucher self-assigned this Sep 3, 2026

gemini-code-assist Bot reviewed Sep 3, 2026

View reviewed changes

AcKoucher added UpdateRules Starts GHA to update rules and removed UpdateRules Starts GHA to update rules labels Sep 3, 2026

AcKoucher requested a review from eder-matheus September 4, 2026 13:52

eder-matheus approved these changes Sep 4, 2026

View reviewed changes

eder-matheus merged commit a16fb34 into The-OpenROAD-Project:master Sep 4, 2026
12 checks passed

eder-matheus deleted the arthur-koucher-designs branch September 4, 2026 17:14

Navigation Menu

Uh oh!

tightens gf12/aes, gf12/gcd and sky130hs/ibex - #4507

tightens gf12/aes, gf12/gcd and sky130hs/ibex#4507
eder-matheus merged 6 commits into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:arthur-koucher-designs

AcKoucher commented Sep 3, 2026 •

edited

Loading

Uh oh!

gemini-code-assist Bot left a comment

Uh oh!

openroad-ci commented Sep 3, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

AcKoucher commented Sep 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

New Metrics

sky130hs/ibex:

gf12/aes:

gf12/gcd:

Uh oh!

gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

openroad-ci commented Sep 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🔍 QoR check

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

AcKoucher commented Sep 3, 2026 •

edited

Loading

openroad-ci commented Sep 3, 2026 •

edited

Loading