[PATCH] D87777: [ASAN] Properly deal with musttail calls in ASAN
Xun Li via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 16 17:34:41 PDT 2020
lxfind added inline comments.
================
Comment at: llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp:562
+// point.
+static Instruction *adjustForMusttailCall(Instruction *I) {
+ ReturnInst *RI = dyn_cast<ReturnInst>(I);
----------------
wenlei wrote:
> Is there a better place for this helper and avoid duplication? We have a few copies of it now, TSAN, ASAN and EntryExitInstrumenter...
Yeah I have been wondering about this too. There are more duplicates than TSAN and ASAN.
Do you have any suggestions? I couldn't find a good place for this.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87777/new/
https://reviews.llvm.org/D87777
More information about the llvm-commits
mailing list