Skip to content

update gf180 layer resistance with segment-based regression results#4228

Merged
eder-matheus merged 1 commit into
The-OpenROAD-Project:masterfrom
AcKoucher:gf180-new-r-values
May 13, 2026
Merged

update gf180 layer resistance with segment-based regression results#4228
eder-matheus merged 1 commit into
The-OpenROAD-Project:masterfrom
AcKoucher:gf180-new-r-values

Conversation

@AcKoucher
Copy link
Copy Markdown
Contributor

Correlation Data

Mode: segment
Designs (6): aes, aes-hybrid, ibex, jpeg, riscv32i, uart-blocks
OR Version: 390e0d62
Units: R [kΩ/μm], C [pF/μ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. I also removed the comments with redundant tech data.

Metal1 is not used for routing on gf180 so there's no data for that layer.

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 GF180 platform configuration and removes the file's descriptive header. Review feedback recommends restoring the header with specific details about the data source for better maintainability and correcting a slight inconsistency in the Metal4 resistance value to match the other identical metal layers.

Comment thread flow/platforms/gf180/setRC.tcl
Comment thread flow/platforms/gf180/setRC.tcl
@AcKoucher AcKoucher requested a review from eder-matheus May 13, 2026 15:34
@eder-matheus eder-matheus merged commit 6c587df into The-OpenROAD-Project:master May 13, 2026
7 of 8 checks passed
@AcKoucher AcKoucher deleted the gf180-new-r-values branch May 13, 2026 17:16
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.

2 participants