[PATCH] D56538: [NewPM][TSan] Reiterate the TSan port

Philip Pfaffe via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 11 03:44:47 PST 2019


philip.pfaffe updated this revision to Diff 181237.
philip.pfaffe added a comment.

Insert the constructor and init functions only once per module.

Do you think the current check on the ctor function is sufficient, or should I
also check whether it actually contains the init call as well?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56538/new/

https://reviews.llvm.org/D56538

Files:
  clang/lib/CodeGen/BackendUtil.cpp
  llvm/bindings/go/llvm/InstrumentationBindings.cpp
  llvm/include/llvm/InitializePasses.h
  llvm/include/llvm/Transforms/Instrumentation.h
  llvm/include/llvm/Transforms/Instrumentation/ThreadSanitizer.h
  llvm/include/llvm/Transforms/Utils/ModuleUtils.h
  llvm/lib/Passes/PassBuilder.cpp
  llvm/lib/Passes/PassRegistry.def
  llvm/lib/Transforms/Instrumentation/Instrumentation.cpp
  llvm/lib/Transforms/Instrumentation/ThreadSanitizer.cpp
  llvm/lib/Transforms/Utils/ModuleUtils.cpp
  llvm/test/Instrumentation/ThreadSanitizer/tsan_basic.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D56538.181237.patch
Type: text/x-patch
Size: 15584 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190111/865f0899/attachment-0001.bin>


More information about the llvm-commits mailing list