[PATCH] D83595: [Draft][MSAN] Optimize away poisoning allocas that are always written before load

Gui Andrade via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Aug 22 12:32:16 PDT 2020


guiand added inline comments.


================
Comment at: llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp:3927
+      StoredSet StoredBytes(TypeSize, false);
+      if (firstUsesAreStore(&I, InsPoint, StoredBytes))
+        return;
----------------
vitalybuka wrote:
> not sure why StoredBytes is a parameter and not just a local var in firstUsesAreStore 
True, that was leftover from a previous version of this patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83595



More information about the llvm-commits mailing list