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

bpo-30364: replace deprecated no_address_safety_analysis with no_sanitize_address #17702

Open
wants to merge 1 commit into
base: master
from

Conversation

@isidentical
Copy link
Contributor

isidentical commented Dec 25, 2019

# define _Py_NO_ADDRESS_SAFETY_ANALYSIS \
__attribute__((no_address_safety_analysis))
# define _Py_NO_SANITIZE_ADDRESS \
__attribute__((no_sanitize("address")))

This comment has been minimized.

Copy link
@benjaminp

benjaminp Dec 25, 2019

Contributor

Shouldn't this be no_sanitize_address?

This comment has been minimized.

Copy link
@isidentical

isidentical Dec 25, 2019

Author Contributor

The page issue refers suggests usage of this instead of other deprecated aliases

One may use the attribute __attribute__((no_sanitize("address"))) (which has deprecated synonyms no_sanitize_address and no_address_safety_analysis) to disable instrumentation of a particular function.

@isidentical isidentical requested a review from benjaminp Dec 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.