[PATCH] D34789: Copy arguments passed by value into explicit allocas for ASan

[Vitaly Buka via Phabricator via llvm-commits]

vitalybuka added inline comments.


================
Comment at: lib/Transforms/Instrumentation/AddressSanitizer.cpp:2541
 
+void FunctionStackPoisoner::copyArgsPassedByValToAllocas() {
+  BasicBlock &FirstBB = *F.begin();
----------------
eugenis wrote:
> vitalybuka wrote:
> > Should we poison original location?
> I'm thinking not, because it is not adjacent to any alloca - it's just a random location on the stack at this point.
But access to that location is a sign of a bug


https://reviews.llvm.org/D34789