Skip to content

Commit

Permalink
Update OpenROAD (The-OpenROAD-Project#2062)
Browse files Browse the repository at this point in the history
~ openroad_app -> `75f2f32`

## CI
~ Always compare regression and create reproducibles
~ Always escape design name
+ Add tests for attached crashes by adding issues reproducibles in designs submodules and symlink them to interactive scripts under ./tests
+ Fix failing designs in extended test:
  * salsa20: setup violations
  * y_huff: routing congestion
  * aes_core: pin antenna violations benchmark mismatch
  • Loading branch information
kareefardi authored Dec 26, 2023
1 parent 892d791 commit 8bea8c5
Show file tree
Hide file tree
Showing 11 changed files with 20 additions and 7 deletions.
8 changes: 4 additions & 4 deletions .github/scripts/run_tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -104,10 +104,6 @@ def cat(x):
)
print("Created ./reproducible.tar.gz.")

if run_return_code != 0:
print("Run failed")
raise run_exception

difference_reports = glob.glob(os.path.join(results_folder, f"{test_name}*.rpt"))
if len(difference_reports):
print("Verbose differences within the benchmark:")
Expand All @@ -125,4 +121,8 @@ def cat(x):
print("Testing the design has failed.")
exit(-1)

if run_return_code != 0:
print("Run failed")
raise run_exception

print("Done.")
3 changes: 3 additions & 0 deletions .github/workflows/openlane_ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -141,6 +141,8 @@ jobs:
matrix: ${{ fromJSON(needs.prepare_test_matrices.outputs.issue_regression_matrix) }}
steps:
- uses: actions/checkout@v3
with:
submodules: true

- name: Set up environment variables
uses: ./.github/actions/set_env_variables
Expand Down Expand Up @@ -233,6 +235,7 @@ jobs:
python3 ${GITHUB_WORKSPACE}/.github/scripts/run_tests.py ${{ matrix.design.name }}
- name: Escape Design Name
if: ${{ always() }}
run: |
design_name=${{ matrix.design.name }}
escaped_design_name=${design_name//\//_}
Expand Down
2 changes: 1 addition & 1 deletion dependencies/tool_metadata.yml
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@
in_install: false
- name: openroad_app
repo: https://github.com/The-OpenROAD-Project/OpenROAD
commit: 6f9b2bb8b808b1bb5831d4525d868212ae50517a
commit: 75f2f325b7a42e56a92404f33af8e96530d9b202
in_install: false
- name: git
repo: https://github.com/git/git
Expand Down
2 changes: 1 addition & 1 deletion designs/ci
Submodule ci updated 45 files
+23 −18 aes_core/config.json
+ reproducibles/1935/issue.tar.gz
+56 −0 reproducibles/2060/issue_reproducible/env.gdb
+56 −0 reproducibles/2060/issue_reproducible/env.lldb
+60 −0 reproducibles/2060/issue_reproducible/home/yunus/caravel_user_project/openlane/fpga_core_v2/grid_clb.tcon
+24 −0 reproducibles/2060/issue_reproducible/openlane/scripts/openroad/common/dpl_cell_pad.tcl
+337 −0 reproducibles/2060/issue_reproducible/openlane/scripts/openroad/common/io.tcl
+65 −0 reproducibles/2060/issue_reproducible/openlane/scripts/openroad/common/resizer.tcl
+75 −0 reproducibles/2060/issue_reproducible/openlane/scripts/openroad/common/set_global_connections.tcl
+43 −0 reproducibles/2060/issue_reproducible/openlane/scripts/openroad/common/set_rc.tcl
+89 −0 reproducibles/2060/issue_reproducible/openlane/scripts/openroad/resizer.tcl
+344,411 −0 ...producible/pdk/gf180mcuD/libs.ref/gf180mcu_fd_sc_mcu7t5v0/liberty/gf180mcu_fd_sc_mcu7t5v0__tt_025C_5v00.lib
+344,419 −0 ...producible/pdk/gf180mcuD/libs.ref/gf180mcu_fd_sc_mcu9t5v0/liberty/gf180mcu_fd_sc_mcu9t5v0__ff_n40C_5v50.lib
+344,414 −0 ...producible/pdk/gf180mcuD/libs.ref/gf180mcu_fd_sc_mcu9t5v0/liberty/gf180mcu_fd_sc_mcu9t5v0__ss_125C_4v50.lib
+344,413 −0 ...producible/pdk/gf180mcuD/libs.ref/gf180mcu_fd_sc_mcu9t5v0/liberty/gf180mcu_fd_sc_mcu9t5v0__tt_025C_5v00.lib
+61 −0 reproducibles/2060/issue_reproducible/run.sh
+58 −0 reproducibles/2060/issue_reproducible/run.tcl
+52,247 −0 reproducibles/2060/issue_reproducible/tmp/merged.nom.lef
+86,369 −0 reproducibles/2060/issue_reproducible/tmp/placement/10-global.def
+54,754 −0 reproducibles/2060/issue_reproducible/tmp/placement/10-global.nl.v
+ reproducibles/2060/issue_reproducible/tmp/placement/10-global.odb
+108,462 −0 reproducibles/2060/issue_reproducible/tmp/placement/10-global.pnl.v
+65 −0 reproducibles/2065/issue_reproducible/env.gdb
+65 −0 reproducibles/2065/issue_reproducible/env.lldb
+144 −0 ...pp/libresilicon/StdCellLib/Catalog/gf180_stdcelllib_1/openlane/user_proj_example/base_user_proj_example.sdc
+9 −0 ..._reproducible/home/philipp/libresilicon/StdCellLib/Catalog/gf180_stdcelllib_1/verilog/rtl/user_proj_cells.v
+24 −0 reproducibles/2065/issue_reproducible/openlane/scripts/openroad/common/dpl_cell_pad.tcl
+337 −0 reproducibles/2065/issue_reproducible/openlane/scripts/openroad/common/io.tcl
+65 −0 reproducibles/2065/issue_reproducible/openlane/scripts/openroad/common/resizer.tcl
+75 −0 reproducibles/2065/issue_reproducible/openlane/scripts/openroad/common/set_global_connections.tcl
+43 −0 reproducibles/2065/issue_reproducible/openlane/scripts/openroad/common/set_rc.tcl
+89 −0 reproducibles/2065/issue_reproducible/openlane/scripts/openroad/resizer.tcl
+344,411 −0 ...producible/pdk/gf180mcuD/libs.ref/gf180mcu_fd_sc_mcu7t5v0/liberty/gf180mcu_fd_sc_mcu7t5v0__tt_025C_5v00.lib
+344,419 −0 ...producible/pdk/gf180mcuD/libs.ref/gf180mcu_fd_sc_mcu9t5v0/liberty/gf180mcu_fd_sc_mcu9t5v0__ff_n40C_5v50.lib
+344,414 −0 ...producible/pdk/gf180mcuD/libs.ref/gf180mcu_fd_sc_mcu9t5v0/liberty/gf180mcu_fd_sc_mcu9t5v0__ss_125C_4v50.lib
+344,413 −0 ...producible/pdk/gf180mcuD/libs.ref/gf180mcu_fd_sc_mcu9t5v0/liberty/gf180mcu_fd_sc_mcu9t5v0__tt_025C_5v00.lib
+70 −0 reproducibles/2065/issue_reproducible/run.sh
+67 −0 reproducibles/2065/issue_reproducible/run.tcl
+52,247 −0 reproducibles/2065/issue_reproducible/tmp/merged.nom.lef
+48,225 −0 reproducibles/2065/issue_reproducible/tmp/placement/10-global.def
+25,393 −0 reproducibles/2065/issue_reproducible/tmp/placement/10-global.nl.v
+ reproducibles/2065/issue_reproducible/tmp/placement/10-global.odb
+51,473 −0 reproducibles/2065/issue_reproducible/tmp/placement/10-global.pnl.v
+18 −17 salsa20/config.json
+25 −18 y_huff/config.json
2 changes: 1 addition & 1 deletion run_designs.py
Original file line number Diff line number Diff line change
Expand Up @@ -398,7 +398,7 @@ def run_design(designs_queue):
design,
f"Failed to compare with benchmark: {error_msg}",
)
flow_failure_flag = True
design_failure_flag = True

if delete:
try:
Expand Down
3 changes: 3 additions & 0 deletions tests/1935-cts-crash/interactive.tcl
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
exec bash -c "set -e && \
cd [file dirname [file normalize [info script]]]/reproducible && \
tar xvf issue.tar.gz && cd issue_reproducible && bash run.sh"
1 change: 1 addition & 0 deletions tests/1935-cts-crash/reproducible
2 changes: 2 additions & 0 deletions tests/2060-resizer-crash/interactive.tcl
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
exec bash -c "set -e && \
cd [file dirname [file normalize [info script]]]/reproducible && bash run.sh"
1 change: 1 addition & 0 deletions tests/2060-resizer-crash/reproducible
2 changes: 2 additions & 0 deletions tests/2065-resizer-crash/interactive.tcl
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
exec bash -c "set -e && \
cd [file dirname [file normalize [info script]]]/reproducible && bash run.sh"
1 change: 1 addition & 0 deletions tests/2065-resizer-crash/reproducible

0 comments on commit 8bea8c5

Please sign in to comment.