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

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 29 12:24:35 PDT 2017


eugenis added inline comments.


================
Comment at: lib/Transforms/Instrumentation/AddressSanitizer.cpp:2541
 
+void FunctionStackPoisoner::copyArgsPassedByValToAllocas() {
+  BasicBlock &FirstBB = *F.begin();
----------------
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.


https://reviews.llvm.org/D34789





More information about the llvm-commits mailing list