diff options
| author | rlibby <rlibby@FreeBSD.org> | 2019-12-21 02:43:20 +0000 |
|---|---|---|
| committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2020-10-26 14:18:46 +0100 |
| commit | b9af5041bd5662748e060d7b716c18a224d91254 (patch) | |
| tree | 85ff8c44f56620b0d1ca52d434c0e492c5347aaf /newlib/libc | |
| parent | b9967c3f9040c25efe0a67b59dbd05ee79d0a97d (diff) | |
| download | newlib-b9af5041bd5662748e060d7b716c18a224d91254.zip newlib-b9af5041bd5662748e060d7b716c18a224d91254.tar.gz newlib-b9af5041bd5662748e060d7b716c18a224d91254.tar.bz2 | |
gcc: quiet Wattribute for no_sanitize("address")
This is an unfortunate instance where the __has_attribute check does
not function usefully. Gcc does have the attribute, but for gcc it only
applies to functions, not variables, and trying to apply it to a
variable generates Wattribute. So far we only apply the attribute to
variables. Only enable the attribute for clang, for now.
Reviewed by: Anton Rang <rang at acm.org>
Sponsored by: Dell EMC Isilon
Differential Revision: https://reviews.freebsd.org/D22875
Diffstat (limited to 'newlib/libc')
| -rw-r--r-- | newlib/libc/include/sys/cdefs.h | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/newlib/libc/include/sys/cdefs.h b/newlib/libc/include/sys/cdefs.h index 1f2ce06..1be994f 100644 --- a/newlib/libc/include/sys/cdefs.h +++ b/newlib/libc/include/sys/cdefs.h @@ -709,8 +709,12 @@ /* Function should not be analyzed. */ #define __no_lock_analysis __lock_annotate(no_thread_safety_analysis) -/* Function or variable should not be sanitized, ie. by AddressSanitizer */ -#if __has_attribute(no_sanitize) +/* + * Function or variable should not be sanitized, i.e. by AddressSanitizer. + * GCC has the nosanitize attribute, but as a function attribute only, and + * warns on use as a variable attribute. + */ +#if __has_attribute(no_sanitize) && defined(__clang__) #define __nosanitizeaddress __attribute__((no_sanitize("address"))) #else #define __nosanitizeaddress |