[PATCH] D11179: [asan] Invalid debug info for promotable allocas
Kostya Serebryany
kcc at google.com
Tue Jul 14 09:37:54 PDT 2015
kcc added a comment.
Two style comments.
Also, please add a test.
================
Comment at: lib/Transforms/Instrumentation/AddressSanitizer.cpp:528
@@ -527,2 +527,3 @@
SmallVector<AllocaInst *, 16> AllocaVec;
+ SmallVector<AllocaInst *, 16> NonInstrumentedAllocaVec;
SmallVector<Instruction *, 8> RetVec;
----------------
NonInstrumentedStaticAllocaVec
================
Comment at: lib/Transforms/Instrumentation/AddressSanitizer.cpp:1741
@@ -1736,1 +1740,3 @@
+ for (auto *AI : NonInstrumentedAllocaVec) {
+ AI->moveBefore(AllocaVec[0]);
----------------
This code tends to not have {} in such case.
http://reviews.llvm.org/D11179
More information about the llvm-commits
mailing list