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

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 29 14:40:20 PDT 2017


vitalybuka added a comment.

In https://reviews.llvm.org/D34789#795718, @eugenis wrote:

> Right, but then why don't we add 5 new allocas in each function, poison them and never use them. Access to those would also be a bug.


This is not the same. We already have original space allocate on the stack, and we know that this should not be accessed.
We can investigate usefulness of this later.


https://reviews.llvm.org/D34789





More information about the llvm-commits mailing list