[PATCH] D79537: Add NoMerge MIFlag to avoid MIR branch folding
Reid Kleckner via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 13 13:36:37 PDT 2020
rnk added a subscriber: vitalybuka.
rnk added a comment.
After this lands, can you follow this up by migrating the sanitizer passes to using the attribute instead of using empty inline asm blobs? Send the patch to @vitalybuka. This is the code I'm thinking of:
https://github.com/llvm/llvm-project/blob/028bfdd8913616f7a3e57e8ef5c2a9990e528ff0/llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp#L1626
https://github.com/llvm/llvm-project/blob/d6695e18763a05b30cb336c18157175277da8f4b/llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp#L864
... and any similar uses.
Run `check-asan`, `check-msan`, etc to run the sanitizer integration tests, and confirm that they pass. You can run `check-all` to run everything and be sure as well.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79537/new/
https://reviews.llvm.org/D79537
More information about the llvm-commits
mailing list