ci: Test UBSAN + ASAN#1704
Conversation
|
Author jpr42 not on autobuild list. Waiting for curator authorization before starting CI build. |
1 similar comment
|
Author jpr42 not on autobuild list. Waiting for curator authorization before starting CI build. |
Currently UBSAN isn't being tested on CI
|
Author jpr42 not on autobuild list. Waiting for curator authorization before starting CI build. |
1 similar comment
|
Author jpr42 not on autobuild list. Waiting for curator authorization before starting CI build. |
| target_compile_options(vulkan PUBLIC -fsanitize=address,undefined) | ||
| target_link_options(vulkan PUBLIC -fsanitize=address,undefined) |
There was a problem hiding this comment.
To save CI time just combine them.
There was a problem hiding this comment.
This affects regular developers (so, ME). Which is fine, I should be using this locally anyhow.
| # This is to combat a bug when using 6.6 linux kernels with thread/address sanitizer | ||
| # https://github.com/google/sanitizers/issues/1716 | ||
| - run: sudo sysctl vm.mmap_rnd_bits=28 |
There was a problem hiding this comment.
Definitely safe to remove - I love removing workarounds for bugs that no longer affect us.
| target_compile_options(vulkan PUBLIC -fsanitize=address,undefined) | ||
| target_link_options(vulkan PUBLIC -fsanitize=address,undefined) |
There was a problem hiding this comment.
This affects regular developers (so, ME). Which is fine, I should be using this locally anyhow.
Currently UBSAN isn't being tested on CI