[PATCH] D83337: [MSAN] Instrument libatomic load/store calls

[Gui Andrade via Phabricator via llvm-commits]

guiand marked 5 inline comments as done.
guiand added inline comments.


================
Comment at: llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp:3559
+                                                   kMinOriginAlignment);
+      Value *NewOrigin = updateOrigin(SrcOrigin, NextIRB);
+      NextIRB.CreateCall(MS.MsanSetOriginFn, {DstPtr, Size, NewOrigin});
----------------
eugenis wrote:
> Why not storeOrigin? It even has an option to use a runtime helper, a better one (conditional).
Since this function isn't necessarily called with a constant size, I needed some way to set the origin for a variable-sized region too. It seems like `storeOrigin` and the maybe-store functions use constant sizes?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83337