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

Allow sanitizer recovery #18

Open
jayvdb opened this issue May 2, 2022 · 0 comments
Open

Allow sanitizer recovery #18

jayvdb opened this issue May 2, 2022 · 0 comments

Comments

@jayvdb
Copy link
Contributor

jayvdb commented May 2, 2022

It is often useful to allow sanitizer recovery in order to see all errors instead of only the first.
This can be done with varying success using -DCMAKE_{}_FLAGS=-fsanitize-recover=all

Using input cmakeflags to add -DCMAKE_{}_FLAGS=-fsanitize-recover=all does not appear to work with these presets, as sanitizer presets already append -DCMAKE_{}_FLAGS.

The recipe should detect input cmakeflags setting -DCMAKE_{}_FLAGS and append its additions into the existing one rather than add another one.

But even with that fixed, IMO it would be good for the recipe here to allow adding -fsanitize-recover=all easily, or at least document how to do it.

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