Skip to content
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

CI: add GitHub actions for -fsanitze={thread, leak, undefined} #61

Open
marehr opened this issue Dec 14, 2021 · 0 comments
Open

CI: add GitHub actions for -fsanitze={thread, leak, undefined} #61

marehr opened this issue Dec 14, 2021 · 0 comments

Comments

@marehr
Copy link
Member

marehr commented Dec 14, 2021

It would be nice to have automated checks for different address sanitizer options.

I'm not sure whether it is better to have an action for each sanitizer or to combine those that can be combined.

According to:

-fsanitize=address

Enable AddressSanitizer, a fast memory error detector. Memory access instructions are instrumented to detect out-of-bounds and use-after-free bugs. [...] The option cannot be combined with -fsanitize=thread or -fsanitize=hwaddress.

we could combine some sanitizer flags for a single use.

https://gcc.gnu.org/onlinedocs/gcc/Instrumentation-Options.html

We could add a new Build Type -DCMAKE_BUILD_TYPE=DEBUG_ASAN or so like we did for -DCMAKE_BUILD_TYPE=Coverage.


I had some fun with -fsanitize=thread and found some great pointers why my concurrent code didn't work, but it unfortunately produces a lot of "false positives" if it is used with OpenMP.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant