[PATCH] Unify clang/llvm attributes for asan/tsan/msan (Clang part)

Kostya Serebryany kcc at google.com
Sun Feb 24 23:11:46 PST 2013



================
Comment at: docs/AddressSanitizer.rst:114
@@ -114,1 +113,3 @@
+:ref:`no_sanitize_address <langext-address_sanitizer>`
+(or a synonym `no_address_safety_analysis`)
 to disable instrumentation of a particular function. This attribute may not be
----------------
Sean Silva wrote:
> IMO we should deprecate this in favor of the new more-accurate attribute names. e.g. just say "deprecated synonym" instead of "synonym".
done

================
Comment at: docs/LanguageExtensions.rst:1620
@@ +1619,3 @@
+Use ``__attribute__((no_sanitize_address))``
+(or a synonym ``__attribute__((no_address_safety_analysis))``)
+on a function declaration
----------------
Sean Silva wrote:
> I would remove mention of no_address_safety_analysis here so that we only mention it once in AddressSanitizer.rst (as being deprecated).
done


http://llvm-reviews.chandlerc.com/D449



More information about the llvm-commits mailing list