Skip to content

Commit f41ea8c

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 f41ea8c

File tree

2 files changed

+38
-4
lines changed

2 files changed

+38
-4
lines changed

.github/workflows/nightly_test.yml

Lines changed: 1 addition & 2 deletions
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

Lines changed: 37 additions & 2 deletions
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,40 @@ 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+
git submodule sync --recursive # Sync submodules
399+
git submodule update --init --recursive # Ensure they are initialized & updated'
400+
export PATH="/usr/lib/ccache:/usr/local/opt/ccache/libexec:$PATH"
401+
./.github/scripts/build.sh
402+
./run_reg_test.py vtr_reg_system_verilog -show_failures -j${{ steps.cpu-cores.outputs.count}}
369403
370404
ODINII:
371405
name: 'ODIN-II Basic Test'
@@ -508,11 +542,12 @@ jobs:
508542
needs:
509543
- Build
510544
- Format
511-
- UniTests
545+
- UnitTests
512546
- Warnings
513547
- Regression
514548
- Sanitized
515549
- Parmys
550+
- SystemVerilog
516551
- ODINII
517552
- VQM2BLIF
518553
- Compatibility

0 commit comments

Comments
 (0)