DPL negotiation default algorithm by openroad-ci · Pull Request #4186 · The-OpenROAD-Project/OpenROAD-flow-scripts

openroad-ci · 2026-04-24T20:48:02Z

This PR updates metrics for OR PR The-OpenROAD-Project/OpenROAD#10226 to make new negotiation legalizer algorithm into the default one.

This PR also replaces USE_NEGOTIATION for DPL_USE_OLD_DIAMOND, since the OR submodule now does not have -use_negotiation anymore for dpl, and has -use_diamond_legalizer.

I also modified the utilization of a few failing designs.

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

…tion-default

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

…tion-default

replace USE_NEGOTIATION for DPL_USE_OLD_DIAMOND, since the OR submodule now does not have -use_negotiation anymore for dpl, and has -use_old_diamond, the default is negotiation legalizer Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

…tion-default

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

…tion-default

gemini-code-assist

Code Review

This pull request updates the OpenROAD subproject, reduces core utilization for several designs, and renames the USE_NEGOTIATION variable to DPL_USE_OLD_DIAMOND to reflect that the negotiation legalizer is now the default. Feedback suggests removing leftover debug code in global_route.tcl and adding the stages field to the new variable definition in the configuration files for improved documentation and consistency.

gemini-code-assist · 2026-04-24T20:50:06Z

+    #set_debug_level DPL negotiation 1
+    dpl::detailed_placement_debug
    log_cmd detailed_placement {*}$dpl_args


These lines appear to be leftover debug code. Commented-out code and explicit calls to dpl::detailed_placement_debug should be removed to keep the scripts clean and consistent with other stages (like CTS) that also perform detailed placement.

log_cmd detailed_placement {*}$dpl_args

gemini-code-assist · 2026-04-24T20:50:06Z

+  "DPL_USE_OLD_DIAMOND": {
    "default": 0,
-    "description": "Enable using negotiation legalizer for detailed placement.\n"
+    "description": "Use the former diamond search legalizer for detailed placement instead of the default negotiation legalizer.\n"
  },


It is recommended to include the stages field for the new DPL_USE_OLD_DIAMOND variable to document where it is used (place, cts, and grt) and to maintain consistency with other variables in this file.

"DPL_USE_OLD_DIAMOND": { "default": 0, "description": "Use the former diamond search legalizer for detailed placement instead of the default negotiation legalizer.\n", "stages": [ "place", "cts", "grt" ] },

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

…tion-default

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

…tion-default

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

…tion-default

===================================================== make update_ok for ibex (gf180)... ===================================================== designs/gf180/ibex/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | finish__timing__setup__tns | -2.22 | -4.76 | Failing | | finish__design__instance__area | 764974 | 764104 | Tighten | ===================================================== make update_ok for gcd (asap7)... ===================================================== designs/asap7/gcd/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | globalroute__timing__setup__tns | -1790.0 | -2050.0 | Failing | | detailedroute__route__wirelength | 1324 | 1947 | Failing | | finish__timing__setup__tns | -1570.0 | -1820.0 | Failing | ===================================================== make update_ok for riscv32i-mock-sram (asap7)... ===================================================== designs/asap7/riscv32i-mock-sram/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | globalroute__timing__setup__ws | -56.5 | -47.5 | Tighten | | globalroute__timing__setup__tns | -407.0 | -190.0 | Tighten | | finish__timing__setup__tns | -298.0 | -1330.0 | Failing | ===================================================== make update_ok for ibex (asap7)... ===================================================== designs/asap7/ibex/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__setup__tns | -1630.0 | -13700.0 | Failing | | finish__design__instance__area | 2816 | 2810 | Tighten | ===================================================== make update_ok for uart (asap7)... ===================================================== designs/asap7/uart/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | detailedroute__route__wirelength | 1673 | 1973 | Failing | | finish__timing__setup__ws | -52.4 | -47.3 | Tighten | | finish__timing__setup__tns | -1320.0 | -1210.0 | Tighten | ===================================================== make update_ok for gcd-ccs (asap7)... ===================================================== designs/asap7/gcd-ccs/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__setup__ws | -63.5 | -63.4 | Tighten | | cts__timing__setup__tns | -773.0 | -768.0 | Tighten | | globalroute__timing__setup__ws | -63.5 | -63.4 | Tighten | | globalroute__timing__setup__tns | -771.0 | -766.0 | Tighten | | finish__timing__setup__tns | -1270.0 | -1430.0 | Failing | ===================================================== make update_ok for chameleon (sky130hd)... ===================================================== designs/sky130hd/chameleon/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__setup__ws | -0.962 | -0.945 | Tighten | | cts__timing__setup__tns | -6.72 | -8.5 | Failing | | globalroute__antenna_diodes_count | 196 | 194 | Tighten | | globalroute__timing__setup__ws | -0.944 | -0.909 | Tighten | | globalroute__timing__setup__tns | -5.91 | -5.68 | Tighten | | finish__timing__setup__ws | -0.916 | -0.906 | Tighten | | finish__timing__setup__tns | -7.59 | -6.54 | Tighten | ===================================================== make update_ok for gcd (sky130hd)... ===================================================== designs/sky130hd/gcd/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__setup__tns | -68.8 | -69.9 | Failing | | globalroute__timing__setup__ws | -1.9 | -2.25 | Failing | | globalroute__timing__setup__tns | -88.8 | -93.7 | Failing | | detailedroute__route__wirelength | 14932 | 17828 | Failing | | finish__timing__setup__ws | -1.74 | -1.86 | Failing | | finish__timing__setup__tns | -81.2 | -79.8 | Tighten | ===================================================== make update_ok for microwatt (sky130hd)... ===================================================== designs/sky130hd/microwatt/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__setup__tns | -345.0 | -335.0 | Tighten | | globalroute__antenna_diodes_count | 1426 | 1421 | Tighten | | globalroute__timing__setup__tns | -322.0 | -288.0 | Tighten | | detailedroute__antenna__violating__nets | 5 | 1 | Tighten | | detailedroute__antenna_diodes_count | 1451 | 1477 | Failing | | finish__timing__setup__tns | -158.0 | -140.0 | Tighten | | finish__design__instance__area | 5572106 | 5571256 | Tighten | ===================================================== make update_ok for aes (sky130hd)... ===================================================== designs/sky130hd/aes/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | globalroute__timing__setup__ws | -1.3 | -0.523 | Tighten | | globalroute__timing__setup__tns | -9.57 | -12.5 | Failing | | finish__timing__setup__ws | -1.17 | -0.461 | Tighten | | finish__timing__setup__tns | -2.8 | -4.58 | Failing | ===================================================== make update_ok for gcd (sky130hs)... ===================================================== designs/sky130hs/gcd/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__setup__tns | -11.4 | -12.3 | Failing | | globalroute__timing__setup__tns | -19.2 | -20.3 | Failing | | detailedroute__route__wirelength | 14238 | 16488 | Failing | | finish__timing__setup__tns | -15.8 | -17.2 | Failing | ===================================================== make update_ok for aes (sky130hs)... ===================================================== designs/sky130hs/aes/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | placeopt__design__instance__area | 160499 | 159595 | Tighten | | placeopt__design__instance__count__stdcell | 19517 | 19233 | Tighten | | globalroute__timing__setup__tns | -1.92 | -3.16 | Failing | | finish__design__instance__area | 176489 | 174570 | Tighten | ===================================================== make update_ok for swerv (nangate45)... ===================================================== designs/nangate45/swerv/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | synth__design__instance__area__stdcell | 178043.59 | 166000.0 | Tighten | | placeopt__design__instance__area | 179149 | 177861 | Tighten | | placeopt__design__instance__count__stdcell | 99342 | 98391 | Tighten | | cts__design__instance__count__setup_buffer | 8638 | 8556 | Tighten | | cts__design__instance__count__hold_buffer | 8638 | 8556 | Tighten | | cts__timing__setup__tns | -363.0 | -398.0 | Failing | | globalroute__antenna_diodes_count | 102 | 101 | Tighten | | globalroute__timing__setup__tns | -420.0 | -498.0 | Failing | | detailedroute__route__wirelength | 2659376 | 2435536 | Tighten | | finish__timing__setup__tns | -396.0 | -438.0 | Failing | ===================================================== make update_ok for bp_fe_top (nangate45)... ===================================================== designs/nangate45/bp_fe_top/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | finish__timing__setup__ws | -0.15 | -0.14 | Tighten | | finish__timing__setup__tns | -1.58 | -1.95 | Failing | ===================================================== make update_ok for jpeg (nangate45)... ===================================================== designs/nangate45/jpeg/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | synth__design__instance__area__stdcell | 102000.0 | 99800.0 | Tighten | | placeopt__design__instance__count__stdcell | 68509 | 68139 | Tighten | | cts__design__instance__count__setup_buffer | 5957 | 5925 | Tighten | | cts__design__instance__count__hold_buffer | 5957 | 5925 | Tighten | | cts__timing__setup__tns | -54.6 | -43.0 | Tighten | | globalroute__timing__setup__tns | -66.5 | -52.7 | Tighten | | detailedroute__route__wirelength | 631144 | 767282 | Failing | | finish__timing__setup__tns | -53.3 | -46.7 | Tighten | ===================================================== make update_ok for tinyRocket (nangate45)... ===================================================== designs/nangate45/tinyRocket/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__setup__tns | -30.0 | -37.4 | Failing | | globalroute__timing__setup__tns | -50.3 | -64.7 | Failing | | finish__timing__setup__tns | -42.8 | -57.6 | Failing | ===================================================== make update_ok for mempool_group (nangate45)... ===================================================== designs/nangate45/mempool_group/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__setup__tns | -12000.0 | -12100.0 | Failing | | globalroute__timing__setup__tns | -14000.0 | -12100.0 | Tighten | ===================================================== make update_ok for ariane133 (nangate45)... ===================================================== designs/nangate45/ariane133/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | placeopt__design__instance__area | 827643 | 827361 | Tighten | | cts__timing__setup__tns | -502.0 | -556.0 | Failing | | globalroute__timing__setup__tns | -556.0 | -642.0 | Failing | | finish__timing__setup__tns | -549.0 | -604.0 | Failing | | finish__design__instance__area | 837050 | 836289 | Tighten | ===================================================== make update_ok for spi (ihp-sg13g2)... ===================================================== designs/ihp-sg13g2/spi/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | detailedroute__route__wirelength | 3972 | 4623 | Failing | ===================================================== make update_ok for i2c-gpio-expander (ihp-sg13g2)... ===================================================== designs/ihp-sg13g2/i2c-gpio-expander/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | detailedroute__route__wirelength | 37489 | 44583 | Failing | ===================================================== make update_ok for ibex (ihp-sg13g2)... ===================================================== designs/ihp-sg13g2/ibex/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | synth__design__instance__area__stdcell | 280000.0 | 278000.0 | Tighten | | placeopt__design__instance__count__stdcell | 20659 | 20256 | Tighten | | detailedroute__route__wirelength | 895142 | 1137972 | Failing | Large percentage changes in failing metrics (>50%): - ibex (asap7) cts__timing__setup__tns 740.49% (-1630.0 → -13700.0) - riscv32i-mock-sram (asap7) finish__timing__setup__tns 346.31% (-298.0 → -1330.0) - ibex (gf180) finish__timing__setup__tns 114.41% (-2.22 → -4.76) - aes (sky130hs) globalroute__timing__setup__tns 64.58% (-1.92 → -3.16) - aes (sky130hd) finish__timing__setup__tns 63.57% (-2.8 → -4.58) Large percentage improvements in tighten metrics (>50%): - microwatt (sky130hd) detailedroute__antenna__violating__nets -80.00% (5 → 1) - aes (sky130hd) finish__timing__setup__ws -60.60% (-1.17 → -0.461) - aes (sky130hd) globalroute__timing__setup__ws -59.77% (-1.3 → -0.523) - riscv32i-mock-sram (asap7) globalroute__timing__setup__tns -53.32% (-407.0 → -190.0) Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

gudeh · 2026-05-06T19:17:58Z

All green. Ignore the updates from the bot, they were wrong, I did the updates manually

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

…tion-default

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

…tion-default

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

…tion-default

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

…tion-default

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

…tion-default

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

…tion-default

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

… the default Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

gudeh · 2026-07-13T22:38:51Z

1 secure-CI fatal:

[2026-07-13T21:48:45.343Z] ihp-sg13g2 i2c-gpio-expander (base)
[2026-07-13T21:48:45.343Z]   Last log file 3_5_place_dp.log
[2026-07-13T21:48:45.343Z]   Found 1 errors in the logs.
[2026-07-13T21:48:45.343Z]       [ERROR DPL-0038] Utilization greater than 100%, impossible to legalize

gudeh · 2026-07-13T23:14:02Z

This valid utilization check is not considering the case of i2c having pads outside the core area:

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

## make update_ok for aes-hybrid (gf180)... designs/gf180/aes-hybrid/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__setup__tns | -122.0 | -124.0 | Failing | | globalroute__timing__setup__tns | -135.0 | -141.0 | Failing | | finish__timing__setup__tns | -130.0 | -138.0 | Failing | ## make update_ok for aes (gf180)... designs/gf180/aes/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__setup__tns | -87.3 | -85.9 | Tighten | | globalroute__timing__setup__tns | -100.0 | -105.0 | Failing | | finish__timing__setup__tns | -97.6 | -103.0 | Failing | ## make update_ok for ethmac_lvt (asap7)... designs/asap7/ethmac_lvt/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | globalroute__timing__setup__tns | -308.0 | -303.0 | Tighten | | finish__timing__setup__tns | -60.2 | -121.0 | Failing | ## make update_ok for ethmac (asap7)... designs/asap7/ethmac/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | globalroute__timing__setup__tns | -1870.0 | -2120.0 | Failing | | finish__timing__setup__tns | -1500.0 | -1650.0 | Failing | ## make update_ok for mock-alu (asap7)... designs/asap7/mock-alu/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__setup__tns | -18100.0 | -18500.0 | Failing | | globalroute__timing__setup__tns | -19400.0 | -20000.0 | Failing | | finish__timing__setup__ws | -300.0 | -299.0 | Tighten | | finish__timing__setup__tns | -19700.0 | -19900.0 | Failing | ## make update_ok for gcd (sky130hd)... designs/sky130hd/gcd/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__setup__ws | -1.36 | -1.45 | Failing | | cts__timing__setup__tns | -58.6 | -60.9 | Failing | | globalroute__timing__setup__ws | -1.58 | -1.66 | Failing | | globalroute__timing__setup__tns | -67.4 | -71.6 | Failing | | detailedroute__route__wirelength | 11052 | 13026 | Failing | | finish__timing__setup__ws | -1.47 | -1.53 | Failing | | finish__timing__setup__tns | -62.0 | -67.0 | Failing | ## make update_ok for jpeg (sky130hd)... designs/sky130hd/jpeg/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__setup__ws | -0.716 | -0.692 | Tighten | | cts__timing__setup__tns | -78.6 | -81.3 | Failing | | globalroute__antenna_diodes_count | 100 | 102 | Failing | | globalroute__timing__setup__ws | -0.877 | -0.853 | Tighten | | globalroute__timing__setup__tns | -139.0 | -129.0 | Tighten | | finish__timing__setup__tns | -53.5 | -52.9 | Tighten | ## make update_ok for microwatt (sky130hd)... designs/sky130hd/microwatt/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__setup__tns | -299.0 | -290.0 | Tighten | | globalroute__antenna_diodes_count | 1497 | 1595 | Failing | | globalroute__timing__setup__ws | -2.43 | -2.4 | Tighten | | globalroute__timing__setup__tns | -277.0 | -261.0 | Tighten | | detailedroute__antenna_diodes_count | 1425 | 1220 | Tighten | | finish__timing__setup__tns | -326.0 | -270.0 | Tighten | | finish__timing__hold__tns | -9.05 | -4.46 | Tighten | ## make update_ok for riscv32i (sky130hd)... designs/sky130hd/riscv32i/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | placeopt__design__instance__area | 81836 | 81677 | Tighten | | placeopt__design__instance__count__stdcell | 8404 | 8384 | Tighten | | cts__design__instance__count__setup_buffer | 731 | 729 | Tighten | | cts__design__instance__count__hold_buffer | 731 | 729 | Tighten | | cts__timing__setup__ws | -1.41 | -1.31 | Tighten | | cts__timing__setup__tns | -128.0 | -88.1 | Tighten | | globalroute__timing__setup__ws | -1.55 | -1.53 | Tighten | | globalroute__timing__setup__tns | -224.0 | -255.0 | Failing | | detailedroute__route__wirelength | 312952 | 309885 | Tighten | | finish__timing__setup__tns | -114.0 | -136.0 | Failing | ## make update_ok for aes (sky130hd)... designs/sky130hd/aes/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | globalroute__timing__setup__ws | -0.554 | -0.46 | Tighten | | globalroute__timing__setup__tns | -4.6 | -8.39 | Failing | | finish__timing__setup__ws | -0.425 | -0.406 | Tighten | | finish__timing__setup__tns | -2.38 | -3.38 | Failing | ## make update_ok for jpeg (sky130hs)... designs/sky130hs/jpeg/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | globalroute__timing__setup__ws | -0.268 | -0.267 | Tighten | | detailedroute__antenna_diodes_count | 100 | 108 | Failing | ## make update_ok for aes (sky130hs)... designs/sky130hs/aes/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | globalroute__timing__setup__ws | -0.206 | -0.351 | Failing | | globalroute__timing__setup__tns | -1.64 | -3.45 | Failing | | finish__timing__setup__ws | -0.145 | -0.313 | Failing | ## make update_ok for swerv (nangate45)... designs/nangate45/swerv/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | globalroute__timing__setup__ws | -0.699 | -0.671 | Tighten | | globalroute__timing__setup__tns | -299.0 | -351.0 | Failing | | finish__timing__setup__ws | -0.672 | -0.627 | Tighten | | finish__timing__setup__tns | -359.0 | -352.0 | Tighten | ## make update_ok for jpeg (nangate45)... designs/nangate45/jpeg/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | placeopt__design__instance__area | 103034 | 102921 | Tighten | | placeopt__design__instance__count__stdcell | 68147 | 68062 | Tighten | | cts__design__instance__count__setup_buffer | 5926 | 5918 | Tighten | | cts__design__instance__count__hold_buffer | 5926 | 5918 | Tighten | | cts__timing__setup__tns | -39.1 | -42.1 | Failing | | globalroute__timing__setup__tns | -47.7 | -52.9 | Failing | | detailedroute__route__wirelength | 654704 | 757284 | Failing | | finish__timing__setup__tns | -43.5 | -49.1 | Failing | ## make update_ok for tinyRocket (nangate45)... designs/nangate45/tinyRocket/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__setup__tns | -37.6 | -40.8 | Failing | | globalroute__timing__setup__tns | -56.6 | -59.2 | Failing | | finish__timing__setup__tns | -48.2 | -52.4 | Failing | ## make update_ok for mempool_group (nangate45)... designs/nangate45/mempool_group/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__setup__tns | -14900.0 | -12200.0 | Tighten | | globalroute__timing__setup__tns | -12400.0 | -12200.0 | Tighten | | finish__timing__setup__tns | -13600.0 | -11900.0 | Tighten | | finish__timing__hold__tns | -1.02 | -1.7 | Failing | | finish__design__instance__area | 453697 | 453619 | Tighten | ## make update_ok for swerv_wrapper (nangate45)... designs/nangate45/swerv_wrapper/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__setup__ws | -0.335 | -0.331 | Tighten | | cts__timing__setup__tns | -237.0 | -238.0 | Failing | | globalroute__timing__setup__tns | -230.0 | -229.0 | Tighten | | finish__timing__setup__tns | -211.0 | -208.0 | Tighten | ## make update_ok for ariane133 (nangate45)... designs/nangate45/ariane133/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | cts__timing__setup__ws | -0.574 | -0.569 | Tighten | | cts__timing__setup__tns | -743.0 | -760.0 | Failing | | globalroute__timing__setup__tns | -994.0 | -798.0 | Tighten | | finish__timing__setup__tns | -1010.0 | -881.0 | Tighten | ## make update_ok for spi (ihp-sg13g2)... designs/ihp-sg13g2/spi/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | detailedroute__route__wirelength | 3972 | 4630 | Failing | ## make update_ok for ibex (ihp-sg13g2)... designs/ihp-sg13g2/ibex/rules-base.json updates: | Metric | Old | New | Type | | ------ | --- | --- | ---- | | placeopt__design__instance__area | 306794 | 306205 | Tighten | | placeopt__design__instance__count__stdcell | 20278 | 20217 | Tighten | | cts__design__instance__count__setup_buffer | 1763 | 1758 | Tighten | | cts__design__instance__count__hold_buffer | 1763 | 1758 | Tighten | | finish__timing__setup__tns | -1.6 | -3.55 | Failing | | finish__design__instance__area | 327350 | 326753 | Tighten | Large percentage changes in failing metrics (>50%): - ibex (ihp-sg13g2) finish__timing__setup__tns 121.87% (-1.6 → -3.55) - aes (sky130hs) finish__timing__setup__ws 115.86% (-0.145 → -0.313) - aes (sky130hs) globalroute__timing__setup__tns 110.37% (-1.64 → -3.45) - ethmac_lvt (asap7) finish__timing__setup__tns 101.00% (-60.2 → -121.0) - aes (sky130hd) globalroute__timing__setup__tns 82.39% (-4.6 → -8.39) - aes (sky130hs) globalroute__timing__setup__ws 70.39% (-0.206 → -0.351) - mempool_group (nangate45) finish__timing__hold__tns 66.67% (-1.02 → -1.7) Large percentage improvements in tighten metrics (>50%): - microwatt (sky130hd) finish__timing__hold__tns -50.72% (-9.05 → -4.46) Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

gudeh · 2026-07-14T17:27:25Z

I opted to remove the global variable to switch the DPL algorithm for legalization.

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

gudeh · 2026-07-14T20:29:55Z

@osamahammad21 @eder-matheus
Metrics are green here.

I chose to remove the env var for switching the DPL algorithm along ORFS, I suppose it would be just an unused value.

gudeh added 12 commits April 22, 2026 08:50

update OR

39a5107

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

update OR

d5ea65a

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

update OR

9920eab

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

Merge remote-tracking branch 'private/master' into secure-dpl-negotia…

cc5508f

…tion-default

decrease utilization to enable DPL negotiation as default

0ba57a8

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

update OR

fb0786e

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

Merge remote-tracking branch 'private/master' into secure-dpl-negotia…

bc4f077

…tion-default

update OR

d641ef3

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

Merge remote-tracking branch 'private/master' into secure-dpl-negotia…

eff1eb4

…tion-default

update OR

afbcb1a

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

Merge remote-tracking branch 'private/master' into secure-dpl-negotia…

ee257a7

…tion-default

openroad-ci assigned gudeh Apr 24, 2026

gemini-code-assist Bot reviewed Apr 24, 2026

View reviewed changes

gudeh added 9 commits April 24, 2026 21:00

remove accidental debug code

635593a

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

update docs for DPL_USE_OLD_DIAMOND

1357354

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

fix lint

d3fc6ad

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

update OR

d54ae36

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

Merge remote-tracking branch 'private/master' into secure-dpl-negotia…

c7a84ed

…tion-default

update OR

321d857

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

Merge remote-tracking branch 'private/master' into secure-dpl-negotia…

e9deaa3

…tion-default

update OR

32810dd

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

Merge remote-tracking branch 'private/master' into secure-dpl-negotia…

401c1cf

…tion-default

gudeh added the UpdateRules Starts GHA to update rules label May 6, 2026

openroad-ci removed the UpdateRules Starts GHA to update rules label May 6, 2026

openroad-ci force-pushed the secure-dpl-negotiation-default branch from f282ea5 to 75c8d96 Compare May 6, 2026 14:28

gudeh requested review from eder-matheus and maliberty May 6, 2026 19:17

gudeh added 19 commits May 27, 2026 22:27

update OR

e4bbdfb

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

Merge remote-tracking branch 'private/master' into secure-dpl-negotia…

d535404

…tion-default

small decrease in nangate45/swerv utilization to avoid grt congestion

39234eb

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

update OR

6d1a9b9

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

Merge remote-tracking branch 'private/master' into secure-dpl-negotia…

3e83127

…tion-default

update OR

63b0764

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

Merge remote-tracking branch 'private/master' into secure-dpl-negotia…

3d9e4ee

…tion-default

update OR

1985643

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

Merge remote-tracking branch 'private/master' into secure-dpl-negotia…

296c455

…tion-default

update OR

28c49c0

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

remove debug messages

bf5270c

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

Merge remote-tracking branch 'private/master' into secure-dpl-negotia…

a090441

…tion-default

clean up comments

88e17a3

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

update OR

bf3945f

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

Merge remote-tracking branch 'private/master' into secure-dpl-negotia…

ca0377e

…tion-default

update OR

27aa4cf

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

better wording for using legacy diamond legalizer

805fe1e

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

undo ultilziation modification

357dddc

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

remove env var for midifying legalizing algorithm. Now negotiation is…

4c28721

… the default Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

gudeh added 3 commits July 13, 2026 23:15

update OR

226c39d

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

Update OK metrics

ee8b4d9

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

undo incorrect removal of flow/BUILD

63100c9

Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>

maliberty approved these changes Jul 15, 2026

View reviewed changes

maliberty merged commit bea7dcd into The-OpenROAD-Project:master Jul 15, 2026
8 checks passed

maliberty deleted the secure-dpl-negotiation-default branch July 15, 2026 02:59

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

DPL negotiation default algorithm#4186

DPL negotiation default algorithm#4186
maliberty merged 63 commits into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:secure-dpl-negotiation-default

openroad-ci commented Apr 24, 2026 •

edited by maliberty

Loading

Uh oh!

gemini-code-assist Bot left a comment

Uh oh!

gemini-code-assist Bot Apr 24, 2026

Uh oh!

gemini-code-assist Bot Apr 24, 2026

Uh oh!

Uh oh!

gudeh commented May 6, 2026

Uh oh!

gudeh commented Jul 13, 2026

Uh oh!

gudeh commented Jul 13, 2026

Uh oh!

gudeh commented Jul 14, 2026

Uh oh!

gudeh commented Jul 14, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

openroad-ci commented Apr 24, 2026 • edited by maliberty Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

gemini-code-assist Bot Apr 24, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Apr 24, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

gudeh commented May 6, 2026

Uh oh!

gudeh commented Jul 13, 2026

Uh oh!

gudeh commented Jul 13, 2026

Uh oh!

gudeh commented Jul 14, 2026

Uh oh!

gudeh commented Jul 14, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

openroad-ci commented Apr 24, 2026 •

edited by maliberty

Loading