-
-
Notifications
You must be signed in to change notification settings - Fork 223
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add extra_compiler_files
attribute to make ignorelist files available to sanitizers during compilation
#245
Add extra_compiler_files
attribute to make ignorelist files available to sanitizers during compilation
#245
Conversation
…ble to sanitizers during compilation
eb4e353
to
db06ce7
Compare
db06ce7
to
b47cd41
Compare
extra_compiler_files
attribute to make suppression files available to sanitizers during compilationextra_compiler_files
attribute to make ignorelist files available to sanitizers during compilation
71d550e
to
63f552c
Compare
63f552c
to
adbb14e
Compare
Thanks for the contribution. |
This helps inject files into the compiler action sandbox. The files can then be used in the compiler command line, e.g. the sanitizer ignore list. --------- Co-authored-by: Siddhartha Bagaria <starsid@gmail.com>
Hey @lalten. Does this work with legacy non-MODULE builds? I've tried integrating it into our codebase, but having this file in project root, bazel complains about
i.e. it tries to find the file in the external repository rather than in the project. Specifying the repository explicitly e.g. |
I have not tried using this on WORKSPACE :/ |
Did you find a solution?
Unfortunately, no, our SDO team promised to have a look. We'll possibly just patch the file inside the toolchain itself
|
I'm adding sanitizers into our build and I need the compilation action to have access to the sanitizer blacklist.
The compiletime_ignorelist.txt is a https://releases.llvm.org/17.0.1/tools/clang/docs/SanitizerSpecialCaseList.html
I'd like to hear feedback if you think this is a good idea, maybe there are better alternatives?