Skip to content

update nangate45 layer resistance with segment-based regression results#4225

Merged
eder-matheus merged 2 commits into
The-OpenROAD-Project:masterfrom
AcKoucher:nangate45-new-r-values
May 12, 2026
Merged

update nangate45 layer resistance with segment-based regression results#4225
eder-matheus merged 2 commits into
The-OpenROAD-Project:masterfrom
AcKoucher:nangate45-new-r-values

Conversation

@AcKoucher
Copy link
Copy Markdown
Contributor

@AcKoucher AcKoucher commented May 12, 2026

Correlation Data

Mode: segment
Designs (15): aes, ariane133, ariane136, bp, bp_be, bp_fe, bp_multi, dynamic_node, gcd, ibex, jpeg, mempool_group, swerv, swerv_wrapper, tinyRocket.
OR Version: 390e0d62
Units: R [kΩ/μm], C [fF/μm]

The resistance values that we were using weren't actually precise. Probably something off with the net-based regression script. The resistance fits obtained with new segment-based regression have perfect (1.00) R².

I cleaned up the setRC.tcl script so that the data w.r.t. the correlation is tracked here rather than having comments in the file.

Observations Regarding M1

  • There was no regression data for this layer (nangate45 doesn't route on M1 — pins only).
  • The old values were kept.

Updated Rules

[WARNING] Multiple clocks not supported. Will use first clock: mrx_clk_pad_i: 300.0000.
[WARNING] Multiple clocks not supported. Will use first clock: mrx_clk_pad_i: 300.0000.
[WARNING] Multiple clocks not supported. Will use first clock: clk: 333.0000.
designs/nangate45/aes/rules-base.json updates:

Metric Old New Type
globalroute__timing__hold__tns -0.398 -0.726 Failing

designs/nangate45/ariane133/rules-base.json updates:

Metric Old New Type
globalroute__timing__setup__tns -556.0 -570.0 Failing
finish__timing__setup__tns -549.0 -577.0 Failing

[WARNING] Multiple clocks not supported. Will use first clock: clk_i: 3.0000.
designs/nangate45/swerv/rules-base.json updates:

Metric Old New Type
cts__timing__setup__tns -363.0 -497.0 Failing
globalroute__timing__setup__tns -420.0 -512.0 Failing
finish__timing__setup__tns -396.0 -460.0 Failing

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

Metric Old New Type
cts__timing__setup__tns -30.0 -32.4 Failing
globalroute__timing__setup__tns -50.3 -53.9 Failing
finish__timing__setup__tns -42.8 -46.0 Failing

[WARNING] Multiple clocks not supported. Will use first clock: ext_clk: 15.0000.

Messages from CI

[INFO] asap7/minimal not included in CI.
[INFO] gf12 not included in the update.
[INFO] gf55 not included in the update.
[INFO] nangate45/bp_quad not included in CI.

Signed-off-by: Arthur Koucher <arthurkoucher@precisioninno.com>
Copy link
Copy Markdown

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

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 resistance values for metal layers in the Nangate45 platform's RC configuration. The reviewer identified minor inconsistencies in the resistance values for layers with identical physical characteristics (metal4-6, metal7-8, and metal9-10) and suggested using consistent, rounded values to improve maintainability and accurately reflect the PDK's physical symmetry.

Comment thread flow/platforms/nangate45/setRC.tcl
@AcKoucher AcKoucher added the UpdateRules Starts GHA to update rules label May 12, 2026
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@openroad-ci openroad-ci removed the UpdateRules Starts GHA to update rules label May 12, 2026
@AcKoucher AcKoucher requested a review from eder-matheus May 12, 2026 20:33
@eder-matheus eder-matheus enabled auto-merge May 12, 2026 21:24
@eder-matheus eder-matheus merged commit ae34c73 into The-OpenROAD-Project:master May 12, 2026
7 of 8 checks passed
@AcKoucher AcKoucher deleted the nangate45-new-r-values branch May 12, 2026 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants