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
Enable ASan on CI Rocky preset #11332
Conversation
[approve ci] |
Following @bneradt's advice, enabling ASan on rocky instead of adding another preset. |
gcc-toolset-11 doesn't like UBSan :(
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for setting on ci-rocky
. It will be good to get this coverage in PR CI.
CMakePresets.json
Outdated
"CMAKE_CXX_FLAGS_DEBUG": "-g -fsanitize=address", | ||
"CMAKE_C_FLAGS_DEBUG": "-g -fsanitize=address" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should be able to use "ENABLE_ASAN": "ON",
instead of setting the flags. That would be preferable. Can you also use ENABLE_ASAN
for the other places in here that set those -fsanitize=address
flags?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure. For the other places, I'll open another PR.
[approve ci rocky] |
No description provided.