Summary: | ADDRESS_SANITIZER macro is overloaded | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Alexey Proskuryakov <ap> | ||||
Component: | WebKit Misc. | Assignee: | Alexey Proskuryakov <ap> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | andersca, ayao, benjamin, cmarcelo, commit-queue, ddkilzer | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=128819 | ||||||
Attachments: |
|
Description
Alexey Proskuryakov
2015-01-06 10:48:06 PST
Created attachment 244074 [details]
proposed fix
Comment on attachment 244074 [details] proposed fix View in context: https://bugs.webkit.org/attachment.cgi?id=244074&action=review > Source/WTF/wtf/Compiler.h:143 > +#if defined(__has_feature) > +#define ASAN_ENABLED __has_feature(address_sanitizer) > +#else > +#define ASAN_ENABLED false > +#endif Maybe this should be #define ENABLE_ASAN? so #if ENABLE(ASAN) could be used? ENABLE() macros are for things that can be enabled by defining the macro, which is why I intentionally stayed away from using this form - ASan won't appear if you pass -DENABLE_ASAN. Committed <http://trac.webkit.org/r177995>. |