[llvm-commits] [PATCH] [ASan] fix lifetime intrinsics handling

Alexey Samsonov samsonov at google.com
Wed Dec 26 06:17:37 PST 2012



================
Comment at: lib/Transforms/Instrumentation/AddressSanitizer.cpp:1358
@@ +1357,3 @@
+      if (IncValueAI == 0 || (Res != 0 && IncValueAI != Res)) {
+        Res = 0;
+        break;
----------------
Kostya Serebryany wrote:
> return 0? 
Done

================
Comment at: lib/Transforms/Instrumentation/AddressSanitizer.cpp:328
@@ +327,3 @@
+  typedef DenseMap<AllocaInst*, AllocaPoisonCallVec> AllocaPoisonCallsMapTy;
+  AllocaPoisonCallsMapTy AllocaPoisonCalls;
+
----------------
Kostya Serebryany wrote:
> do you really need this map? 
> w/o it things may become simpler
Replaced map with a vector of AllocaPoisonCall


http://llvm-reviews.chandlerc.com/D244



More information about the llvm-commits mailing list