Skip to content

Commit 7965fea

Browse files
[CI] Made the Regression Tests Error on Warning
To prevent warnings from showing up in the build in the future, made the CI error on any warnings during the regression tests. It looks like VPR originally had the ability to error on warning; however, it was specific to only VPR. This CMAKE method will now work for all executables compiled during the tests. So I removed the old Warnings test since it was redundant.
1 parent 0c25ec3 commit 7965fea

File tree

1 file changed

+9
-35
lines changed

1 file changed

+9
-35
lines changed

.github/workflows/test.yml

Lines changed: 9 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -151,31 +151,6 @@ jobs:
151151
run: ./.github/scripts/unittest.sh
152152

153153

154-
Warnings:
155-
name: 'W: Check Compilation Warnings'
156-
runs-on: ubuntu-22.04
157-
steps:
158-
159-
- uses: actions/setup-python@v4
160-
with:
161-
python-version: 3.10.10
162-
- uses: actions/checkout@v3
163-
with:
164-
submodules: 'true'
165-
- run: ./.github/scripts/install_dependencies.sh
166-
167-
- uses: hendrikmuhs/[email protected]
168-
169-
- name: Test
170-
env:
171-
#In order to get compilation warnings produced per source file, we must do a non-IPO build
172-
#We also turn warnings into errors for this target by doing a strict compile
173-
CMAKE_PARAMS: "-DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DVTR_ENABLE_STRICT_COMPILE=on -DVTR_IPO_BUILD=off"
174-
run: |
175-
export PATH="/usr/lib/ccache:/usr/local/opt/ccache/libexec:$PATH"
176-
./.github/scripts/build.sh
177-
178-
179154
Regression:
180155
runs-on: ubuntu-22.04
181156
strategy:
@@ -184,47 +159,47 @@ jobs:
184159
include: [
185160
{
186161
name: 'Basic',
187-
params: '-DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on',
162+
params: '-DCMAKE_COMPILE_WARNING_AS_ERROR=on -DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on',
188163
suite: 'vtr_reg_basic'
189164
},
190165
{
191166
name: 'Basic_odin',
192-
params: '-DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DWITH_PARMYS=OFF -DWITH_ODIN=on',
167+
params: '-DCMAKE_COMPILE_WARNING_AS_ERROR=on -DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DWITH_PARMYS=OFF -DWITH_ODIN=on',
193168
suite: 'vtr_reg_basic_odin'
194169
},
195170
{
196171
name: 'Basic with NO_GRAPHICS',
197-
params: '-DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DVPR_USE_EZGL=off',
172+
params: '-DCMAKE_COMPILE_WARNING_AS_ERROR=on -DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DVPR_USE_EZGL=off',
198173
suite: 'vtr_reg_basic'
199174
},
200175
{
201176
name: 'Basic with CAPNPROTO disabled',
202-
params: '-DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DVTR_ENABLE_CAPNPROTO=off',
177+
params: '-DCMAKE_COMPILE_WARNING_AS_ERROR=on -DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DVTR_ENABLE_CAPNPROTO=off',
203178
suite: 'vtr_reg_basic'
204179
},
205180
{
206181
name: 'Basic with VTR_ENABLE_DEBUG_LOGGING',
207-
params: '-DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DVTR_ENABLE_DEBUG_LOGGING=on',
182+
params: '-DCMAKE_COMPILE_WARNING_AS_ERROR=on -DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DVTR_ENABLE_DEBUG_LOGGING=on',
208183
suite: 'vtr_reg_basic'
209184
},
210185
{
211186
name: 'Basic_odin with VTR_ENABLE_DEBUG_LOGGING',
212-
params: '-DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DVTR_ENABLE_DEBUG_LOGGING=on -DWITH_PARMYS=OFF -DWITH_ODIN=on',
187+
params: '-DCMAKE_COMPILE_WARNING_AS_ERROR=on -DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DVTR_ENABLE_DEBUG_LOGGING=on -DWITH_PARMYS=OFF -DWITH_ODIN=on',
213188
suite: 'vtr_reg_basic_odin'
214189
},
215190
{
216191
name: 'Strong',
217-
params: '-DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on',
192+
params: '-DCMAKE_COMPILE_WARNING_AS_ERROR=on -DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on',
218193
suite: 'vtr_reg_strong'
219194
},
220195
{
221196
name: 'Strong_odin',
222-
params: '-DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DWITH_PARMYS=OFF -DWITH_ODIN=on',
197+
params: '-DCMAKE_COMPILE_WARNING_AS_ERROR=on -DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DWITH_PARMYS=OFF -DWITH_ODIN=on',
223198
suite: 'vtr_reg_strong_odin'
224199
},
225200
{
226201
name: 'Valgrind Memory',
227-
params: '-DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DWITH_ODIN=on',
202+
params: '-DCMAKE_COMPILE_WARNING_AS_ERROR=on -DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DWITH_ODIN=on',
228203
suite: 'vtr_reg_valgrind_small'
229204
}
230205
]
@@ -432,7 +407,6 @@ jobs:
432407
- Build
433408
- Format
434409
- UniTests
435-
- Warnings
436410
- Regression
437411
- Sanitized
438412
- Parmys

0 commit comments

Comments
 (0)