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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits