[PATCH] D85559: [MSAN] Reintroduce libatomic load/store instrumentation

Gui Andrade via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 7 14:33:46 PDT 2020


guiand created this revision.
guiand added reviewers: eugenis, vitalybuka.
Herald added subscribers: llvm-commits, Sanitizers, jfb, hiraditya.
Herald added projects: Sanitizers, LLVM.
guiand requested review of this revision.

  Calls to __atomic_load now get converted to use a thunk call, which is instrumented with the relevant
  shadow/origin code.
  This is because, when compiled with exceptions, a call to __libatomic_load might
  come at the end of the function, with no succeeding BB. So the previous
  approach of instrumenting right after the call didn't work in that case.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D85559

Files:
  compiler-rt/test/msan/libatomic.c
  llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp
  llvm/test/Instrumentation/MemorySanitizer/libatomic.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D85559.284060.patch
Type: text/x-patch
Size: 12971 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200807/8956d767/attachment.bin>


More information about the llvm-commits mailing list