[PATCH] D60617: MSan: handle llvm.lifetime.start intrinsic

[Evgenii Stepanov via Phabricator via llvm-commits]

eugenis added inline comments.


================
Comment at: llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp:1299
+    for (auto Item : LifetimeStartList) {
+      Instruction *InsertAt = InstrumentLifetimeStart ? Item.first : nullptr;
+      instrumentAlloca(*Item.second, InsertAt);
----------------
What if InstrumentLifetimeStart == false, but there is an alloca with multiple lifetime.start ? It looks like you'll poison it several times in a row.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60617