-
Notifications
You must be signed in to change notification settings - Fork 508
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
sanitizers: Add documentation for the no_sanitize
attribute
#1701
base: master
Are you sure you want to change the base?
Conversation
Add documentation for the `no_sanitize` attribute, being stabilized in rust-lang/rust#123617 along with AddressSanitizer and LeakSanitizer.
8907f6d
to
a29174a
Compare
Some drive-by questions:
|
☔ The latest upstream changes (possibly f80986b) made this pull request unmergeable. Please resolve the merge conflicts. |
Thanks for pointing it out! I copied the documentation from The Unstable Book and missed it. I'll fix it. (Yes,
It can be placed on globals and functions.
No, I believe it takes the last, but I'll double check.
Yes.
No, it provides incremental value without requiring rebuilding the Rust Standard Library (i.e., Cargo build-std feature), and it's usually where it interests more the user (i.e., the code being developed). That said, we'll provide pre-build standard libraries for each sanitizer when stabilizing the MemorySanitizer and ThreadSanitizer next. I've created a Zulip thread for us to discuss the next steps and stabilizing the |
Add documentation for the
no_sanitize
attribute, being stabilized in rust-lang/rust#123617 along with AddressSanitizer and LeakSanitizer.