Skip to content

Conversation

@openroad-ci
Copy link
Collaborator

For #8729.

Shrink the macro placement area for gf12/bp_single and update needed metrics.

gf12/ariane:

Metric Old New Type
placeopt__design__instance__area 215149 214773 Tighten
globalroute__antenna_diodes_count 182 180 Tighten
globalroute__timing__setup__tns -719.0 -717.0 Tighten
detailedroute__route__wirelength 3818585 3507181 Tighten
detailedroute__route__drc_errors 1 2 Failing
detailedroute__antenna_diodes_count 182 180 Tighten
detailedroute__timing__setup__tns -702.0 -701.0 Tighten
finish__design__instance__area 217366 217050 Tighten

gf12/bp_single:

Metric Old New Type
detailedroute__flow__warnings__count:DRT-0120 7 5 Tighten
globalroute__flow__warnings__count:DRT-0120 7 5 Tighten
cts__timing__setup__ws -211.0 -100.0 Tighten
cts__timing__setup__tns -2620.0 -400.0 Tighten
cts__timing__hold__ws -100.0 -355.0 Failing
cts__timing__hold__tns -400.0 -5460.0 Failing
globalroute__timing__hold__tns -569.0 -1880.0 Failing
detailedroute__timing__setup__ws -277.0 -237.0 Tighten
detailedroute__timing__setup__tns -2680.0 -2050.0 Tighten
detailedroute__timing__hold__tns -1310.0 -7300.0 Failing
finish__timing__setup__ws -222.0 -166.0 Tighten
finish__timing__setup__tns -1600.0 -931.0 Tighten

asap7/swerv_wrapper:

Metric Old New Type
finish__timing__setup__ws -546.0 -441.0 Tighten
finish__timing__setup__tns -206000.0 -223000.0 Failing
finish__timing__hold__tns -94300.0 -86400.0 Tighten

nangate45/swerv_wrapper:

Metric Old New Type
cts__timing__setup__tns -4.31 -40.3 Failing
globalroute__timing__setup__tns -9.52 -42.4 Failing
detailedroute__timing__hold__tns -1.92 -0.893 Tighten
finish__timing__setup__tns -88.0 -115.0 Failing
finish__timing__hold__tns -7.63 -4.07 Tighten

gf12/ariane:
| Metric                                        | Old      | New      | Type     |
| ------                                        | ---      | ---      | ----     |
| placeopt__design__instance__area              |   215149 |   214773 | Tighten  |
| globalroute__antenna_diodes_count             |      182 |      180 | Tighten  |
| globalroute__timing__setup__tns               |   -719.0 |   -717.0 | Tighten  |
| detailedroute__route__wirelength              |  3818585 |  3507181 | Tighten  |
| detailedroute__route__drc_errors              |        1 |        2 | Failing  |
| detailedroute__antenna_diodes_count           |      182 |      180 | Tighten  |
| detailedroute__timing__setup__tns             |   -702.0 |   -701.0 | Tighten  |
| finish__design__instance__area                |   217366 |   217050 | Tighten  |

gf12/bp_single:
| Metric                                        | Old      | New      | Type     |
| ------                                        | ---      | ---      | ----     |
| detailedroute__flow__warnings__count:DRT-0120 |        7 |        5 | Tighten  |
| globalroute__flow__warnings__count:DRT-0120   |        7 |        5 | Tighten  |
| cts__timing__setup__ws                        |   -211.0 |   -100.0 | Tighten  |
| cts__timing__setup__tns                       |  -2620.0 |   -400.0 | Tighten  |
| cts__timing__hold__ws                         |   -100.0 |   -355.0 | Failing  |
| cts__timing__hold__tns                        |   -400.0 |  -5460.0 | Failing  |
| globalroute__timing__hold__tns                |   -569.0 |  -1880.0 | Failing  |
| detailedroute__timing__setup__ws              |   -277.0 |   -237.0 | Tighten  |
| detailedroute__timing__setup__tns             |  -2680.0 |  -2050.0 | Tighten  |
| detailedroute__timing__hold__tns              |  -1310.0 |  -7300.0 | Failing  |
| finish__timing__setup__ws                     |   -222.0 |   -166.0 | Tighten  |
| finish__timing__setup__tns                    |  -1600.0 |   -931.0 | Tighten  |

asap7/swerv_wrapper:
| Metric                                        | Old      | New      | Type     |
| ------                                        | ---      | ---      | ----     |
| finish__timing__setup__ws                     |   -546.0 |   -441.0 | Tighten  |
| finish__timing__setup__tns                    | -206000.0 | -223000.0 | Failing  |
| finish__timing__hold__tns                     | -94300.0 | -86400.0 | Tighten  |

nangate45/swerv_wrapper:
| Metric                                        | Old      | New      | Type     |
| ------                                        | ---      | ---      | ----     |
| cts__timing__setup__tns                       |    -4.31 |    -40.3 | Failing  |
| globalroute__timing__setup__tns               |    -9.52 |    -42.4 | Failing  |
| detailedroute__timing__hold__tns              |    -1.92 |   -0.893 | Tighten  |
| finish__timing__setup__tns                    |    -88.0 |   -115.0 | Failing  |
| finish__timing__hold__tns                     |    -7.63 |    -4.07 | Tighten  |

Signed-off-by: Arthur Koucher <[email protected]>
@AcKoucher
Copy link
Contributor

AcKoucher commented Oct 29, 2025

@osamahammad21 The new DRC seems to be another occurrence of the one that was already on master (there's a GH issue to investigate it, but, please, let me know if I should open another one for this new DRC).

@maliberty The large TNS in asap7/swerv_wrapper is already being looked into (excessive empty area).

@AcKoucher
Copy link
Contributor

This is for #8729.

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