[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