Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions flow/designs/ihp-sg13g2/ibex/rules-base.json
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@
"compare": ">="
},
"globalroute__timing__setup__tns": {
"value": -1.6,
"value": -4.99,

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.

medium

The global routing setup TNS has degraded significantly from -1.6 to -4.99 (more than a 3x increase in total negative slack). While the final timing (finish__timing__setup__tns) shows an improvement, such a large degradation at the global routing stage indicates a potential regression in the early optimization or clock tree synthesis phases of the updated OpenROAD tool.

"compare": ">="
},
"globalroute__timing__hold__ws": {
Expand All @@ -74,7 +74,7 @@
"compare": ">="
},
"detailedroute__route__wirelength": {
"value": 1012494,
"value": 1000731,
"compare": "<="
},
"detailedroute__route__drc_errors": {
Expand All @@ -94,7 +94,7 @@
"compare": ">="
},
"finish__timing__setup__tns": {
"value": -5.34,
"value": -3.37,
"compare": ">="
},
"finish__timing__hold__ws": {
Expand All @@ -106,7 +106,7 @@
"compare": ">="
},
"finish__design__instance__area": {
"value": 320304,
"value": 320199,
"compare": "<="
}
}
14 changes: 7 additions & 7 deletions flow/designs/nangate45/swerv_wrapper/rules-base.json
Original file line number Diff line number Diff line change
Expand Up @@ -38,11 +38,11 @@
"compare": "<="
},
"cts__timing__setup__ws": {
"value": -0.431,
"value": -0.429,
"compare": ">="
},
"cts__timing__setup__tns": {
"value": -415.0,
"value": -459.0,
"compare": ">="
},
"cts__timing__hold__ws": {
Expand All @@ -58,11 +58,11 @@
"compare": "<="
},
"globalroute__timing__setup__ws": {
"value": -0.466,
"value": -0.428,
"compare": ">="
},
"globalroute__timing__setup__tns": {
"value": -438.0,
"value": -509.0,

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.

medium

The global routing setup TNS has degraded by over 16% (from -438.0 to -509.0). Combined with the 16.2% increase in detailed routing wirelength, this indicates a substantial degradation in physical synthesis quality for the swerv_wrapper design under the new OpenROAD commit.

"compare": ">="
},
"globalroute__timing__hold__ws": {
Expand All @@ -74,7 +74,7 @@
"compare": ">="
},
"detailedroute__route__wirelength": {
"value": 4160494,
"value": 4835341,

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.

high

The detailed routing wirelength has increased significantly by approximately 16.2% (from 4160494 to 4835341). Since there is no corresponding improvement in area or other metrics for this design, this represents a severe routing regression introduced by the OpenROAD submodule update. Accepting this change permanently lowers the quality bar for this design in CI.

"compare": "<="
},
"detailedroute__route__drc_errors": {
Expand All @@ -90,11 +90,11 @@
"compare": "<="
},
"finish__timing__setup__ws": {
"value": -0.425,
"value": -0.397,
"compare": ">="
},
"finish__timing__setup__tns": {
"value": -384.0,
"value": -426.0,
"compare": ">="
},
"finish__timing__hold__ws": {
Expand Down
14 changes: 7 additions & 7 deletions flow/designs/sky130hd/gcd/rules-base.json
Original file line number Diff line number Diff line change
Expand Up @@ -38,11 +38,11 @@
"compare": "<="
},
"cts__timing__setup__ws": {
"value": -1.45,
"value": -1.35,
"compare": ">="
},
"cts__timing__setup__tns": {
"value": -60.9,
"value": -58.9,
"compare": ">="
},
"cts__timing__hold__ws": {
Expand All @@ -62,7 +62,7 @@
"compare": ">="
},
"globalroute__timing__setup__tns": {
"value": -71.6,
"value": -71.1,
"compare": ">="
},
"globalroute__timing__hold__ws": {
Expand All @@ -74,7 +74,7 @@
"compare": ">="
},
"detailedroute__route__wirelength": {
"value": 13026,
"value": 12929,
"compare": "<="
},
"detailedroute__route__drc_errors": {
Expand All @@ -90,11 +90,11 @@
"compare": "<="
},
"finish__timing__setup__ws": {
"value": -1.53,
"value": -1.63,
"compare": ">="
},
"finish__timing__setup__tns": {
"value": -67.0,
"value": -66.0,
"compare": ">="
},
"finish__timing__hold__ws": {
Expand All @@ -106,7 +106,7 @@
"compare": ">="
},
"finish__design__instance__area": {
"value": 5996,
"value": 5950,
"compare": "<="
}
}
20 changes: 10 additions & 10 deletions flow/designs/sky130hd/microwatt/rules-base.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
"level": "warning"
},
"synth__design__instance__area__stdcell": {
"value": 677000.0,
"value": 676000.0,
"compare": "<="
},
"constraints__clocks__count": {
Expand Down Expand Up @@ -38,11 +38,11 @@
"compare": "<="
},
"cts__timing__setup__ws": {
"value": -2.32,
"value": -2.28,
"compare": ">="
},
"cts__timing__setup__tns": {
"value": -254.0,
"value": -221.0,
"compare": ">="
},
"cts__timing__hold__ws": {
Expand All @@ -54,15 +54,15 @@
"compare": ">="
},
"globalroute__antenna_diodes_count": {
"value": 4314,
"value": 4204,
"compare": "<="
},
"globalroute__timing__setup__ws": {
"value": -2.21,
"compare": ">="
},
"globalroute__timing__setup__tns": {
"value": -269.0,
"value": -261.0,
"compare": ">="
},
"globalroute__timing__hold__ws": {
Expand All @@ -86,27 +86,27 @@
"compare": "<="
},
"detailedroute__antenna_diodes_count": {
"value": 2725,
"value": 2747,
"compare": "<="
},
"finish__timing__setup__ws": {
"value": -2.16,
"compare": ">="
},
"finish__timing__setup__tns": {
"value": -227.0,
"value": -215.0,
"compare": ">="
},
"finish__timing__hold__ws": {
"value": -0.883,
"value": -0.808,
"compare": ">="
},
"finish__timing__hold__tns": {
"value": -3.42,
"value": -3.1,
"compare": ">="
},
"finish__design__instance__area": {
"value": 5568932,
"value": 5567346,
"compare": "<="
}
}
2 changes: 1 addition & 1 deletion tools/OpenROAD
Submodule OpenROAD updated 131 files