[PATCH] D108029: [clang][Codegen] Introduce the disable_sanitizer_instrumentation attribute

[Aaron Ballman via Phabricator via llvm-commits]

aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

Clang bits LGTM with a few minor documentation nits.



================
Comment at: clang/include/clang/Basic/AttrDocs.td:2598-2599
+  let Content = [{
+Use the ``disable_sanitizer_instrumentation`` attribute on a function to
+specify that no sanitizer instrumentation should be applied.
+
----------------
function, Objective-C method, or global variable.


================
Comment at: clang/include/clang/Basic/AttrDocs.td:2601-2602
+
+This is not the same as ``__attribute__((no_sanitize(...)))``, which depending
+on the tool may still insert instrumentation to prevent false positive reports.
+  }];
----------------



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108029/new/

https://reviews.llvm.org/D108029