Skip to content

Commit 30e538d

Browse files
[CI] Moved Sanitized Test from Debug to RelWithDebInfo
I expect the sanitized test to be the tall pole for the speed of the CI due to its long run time on the testcases. After investigating with Vaughn, we realized that this test was using the debug build of VTR. This was likely making it much slower than it needed to be. To improve the speed of this test, moved it to use the release build. We still want symbols to give good error messages.
1 parent 605fb00 commit 30e538d

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/workflows/test.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -483,7 +483,7 @@ jobs:
483483
- name: Test
484484
env:
485485
CMAKE_PARAMS: ${{ matrix.params }}
486-
BUILD_TYPE: debug
486+
BUILD_TYPE: RelWithDebInfo
487487
LSAN_OPTIONS: 'exitcode=42' #Use a non-standard exit code to ensure LSAN errors are detected
488488
# In Ubuntu 20240310.1.0, the entropy of ASLR has increased (28 -> 32). LLVM 14 in this
489489
# image is not compatible with this increased ASLR entropy. Apparently, memory sanitizer

0 commit comments

Comments
 (0)