Add sanitizers#200
Conversation
License Check Results🚀 The license check job ran with the Bazel command: bazel run --lockfile_mode=error //:license-checkStatus: Click to expand output |
|
The created documentation from the pull request is available at: docu-html |
|
be aware there is also going work in that direction in score_cpp_policies repo |
9d8bad6 to
c564c1f
Compare
Should lifecycle continue with the self-contained sanitizer implementation in this PR as a short-term unblocker, planning a later migration to the shared score_cpp_policies module once its sanitizer structure stabilizes or do we want to pivot now and integrate directly with the centralized sanitizer infrastructure despite the potential dependency and release-readiness risks? |
|
My opinion: I would say we should bring in our thoughts into score_cpp_policies and not create silos. @NicolasFussberger @S-MOHAMD @pawelrutkaq any thoughts on this? |
c564c1f to
337377e
Compare
If we can bring in these changes into score_cpp_policies, I agree that this should be the preferred solution. I cannot judge how realistic this is. |
This PR adds sanitizer support and CI integration for ASan, UBSan, LSan, and TSan.
Changes
quality/sanitizer/.bazelrccc_test_with_common_optswrapper to simplify test setupSanitizer builds now run automatically for PRs, merge groups, and pushes to
main.Closes #50.