Skip to content

Commit 56e0afe

Browse files
[CI] Added System Verilog Regression Tests to GitHub Runners
The system verilog regression tests were put into the currently not-working nightly tests. Since they are so small, putting them in the GitHub runners tests instead so they do not regress. The strong parmys test also has system verilog tests (I think); however, that test may be too large for the GitHub runners. We may consider moving that test to the SAVI workflows in the future.
1 parent 1279554 commit 56e0afe

File tree

2 files changed

+36
-4
lines changed

2 files changed

+36
-4
lines changed

.github/workflows/nightly_test.yml

+1-2
Original file line numberDiff line numberDiff line change
@@ -65,9 +65,8 @@ jobs:
6565
- {test: "vtr_reg_strong", cores: "16", options: "", cmake: "-DVTR_ASSERT_LEVEL=3", extra_pkgs: "libeigen3-dev"}
6666
- {test: "vtr_reg_strong_odin", cores: "16", options: "", cmake: "-DVTR_ASSERT_LEVEL=3 -DWITH_ODIN=ON", extra_pkgs: "libeigen3-dev"}
6767
- {test: "vtr_reg_strong_odin", cores: "16", options: "-skip_qor", cmake: "-DVTR_ASSERT_LEVEL=3 -DVTR_ENABLE_SANITIZE=ON -DWITH_ODIN=ON", extra_pkgs: "libeigen3-dev"}
68-
- {test: "vtr_reg_system_verilog", cores: "16", options: "", cmake: "-DSYNLIG_SYSTEMVERILOG=ON", extra_pkgs: ""}
6968
- {test: "odin_reg_strong", cores: "16", options: "", cmake: "-DWITH_ODIN=ON", extra_pkgs: ""}
70-
- {test: "parmys_reg_strong", cores: "16", options: "", cmake: "-DSYNLIG_SYSTEMVERILOG=ON", extra_pkgs: ""}
69+
- {test: "parmys_reg_strong", cores: "16", options: "", cmake: "-DSYNLIG_SYSTEMVERILOG=ON", extra_pkgs: ""}
7170

7271
env:
7372
DEBIAN_FRONTEND: "noninteractive"

.github/workflows/test.yml

+35-2
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ jobs:
9797
run: ./dev/${{ matrix.script }}
9898

9999

100-
UniTests:
100+
UnitTests:
101101
name: 'U: C++ Unit Tests'
102102
runs-on: ubuntu-24.04
103103
steps:
@@ -366,6 +366,38 @@ jobs:
366366
./.github/scripts/build.sh
367367
./run_reg_test.py parmys_reg_basic -show_failures -j${{ steps.cpu-cores.outputs.count }}
368368
369+
SystemVerilog:
370+
name: 'System Verilog Regression'
371+
runs-on: ubuntu-24.04
372+
steps:
373+
374+
- uses: actions/setup-python@v5
375+
with:
376+
python-version: 3.12.3
377+
378+
# NOTE: This is using recursive on purpose. In order to use the System
379+
# Verilog feature, need submodules inside of a submodule.
380+
- uses: actions/checkout@v4
381+
with:
382+
submodules: 'recursive'
383+
384+
- name: Get number of CPU cores
385+
uses: SimenB/github-actions-cpu-cores@v2
386+
id: cpu-cores
387+
388+
- name: Install dependencies
389+
run: ./.github/scripts/install_dependencies.sh
390+
391+
- uses: hendrikmuhs/[email protected]
392+
393+
- name: Test
394+
env:
395+
CMAKE_PARAMS: '-DSYNLIG_SYSTEMVERILOG=ON'
396+
NUM_PROC: ${{ steps.cpu-cores.outputs.count }}
397+
run: |
398+
export PATH="/usr/lib/ccache:/usr/local/opt/ccache/libexec:$PATH"
399+
./.github/scripts/build.sh
400+
./run_reg_test.py vtr_reg_system_verilog -show_failures -j${{ steps.cpu-cores.outputs.count}}
369401
370402
ODINII:
371403
name: 'ODIN-II Basic Test'
@@ -508,11 +540,12 @@ jobs:
508540
needs:
509541
- Build
510542
- Format
511-
- UniTests
543+
- UnitTests
512544
- Warnings
513545
- Regression
514546
- Sanitized
515547
- Parmys
548+
- SystemVerilog
516549
- ODINII
517550
- VQM2BLIF
518551
- Compatibility

0 commit comments

Comments
 (0)