This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG8dc7dcdca1e0: [msan] Add support for
disable_sanitizer_instrumentation attribute (authored by glider).
Repository:
rG LLVM Github Monorepo
CHANGE
glider added inline comments.
Comment at: clang/docs/MemorySanitizer.rst:91
+
+The ``disable_sanitizer_instrumentation`` attribute can be applied to a certain
+function to prevent all kinds of instrumentation. This attribute overrides
melver wrote:
> Could apply
glider updated this revision to Diff 367774.
glider marked 2 inline comments as done.
glider added a comment.
Addressed Marco's comments
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D108199/new/
https://reviews.llvm.org/D108199
Files:
clang/docs
glider updated this revision to Diff 367767.
glider added a comment.
Updated after landing the disable_sanitizer_instrumentation to LLVM
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D108199/new/
https://reviews.llvm.org/D108199
Files:
clang/docs
melver accepted this revision.
melver added inline comments.
Comment at: clang/docs/MemorySanitizer.rst:91
+
+The ``disable_sanitizer_instrumentation`` attribute can be applied to a certain
+function to prevent all kinds of instrumentation. This attribute overrides
--
eugenis accepted this revision.
eugenis added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D108199/new/
https://reviews.llvm.org/D108199
_
glider created this revision.
glider added reviewers: eugenis, melver.
Herald added a subscriber: hiraditya.
glider requested review of this revision.
Herald added projects: clang, LLVM.
Herald added subscribers: llvm-commits, cfe-commits.
Unlike __attribute__((no_sanitize("memory"))), this one wi