[PATCH] Add a pass to transform aggregate loads and stores into scalar loads and stores.

Chandler Carruth chandlerc at google.com
Tue Mar 3 20:08:39 PST 2015


On Tue, Mar 3, 2015 at 8:04 PM, Chandler Carruth <chandlerc at google.com>
wrote:

> Ok, so what I recall about this is that the problem was that these were
> not single-instruction replacements because there was the desire to not
> introduce insertvalue / extractvalue? Am I on the right path or not?


Looking carefully at this code, it is using insertvalue and extract value
as well, so that doesn't seem to explain it.

Also, I think your comments about recursion Philip really need to be
addressed before this goes into the tree. We should not let code which is
likely to exhaust the stack into the tree, that will destabilize it for
many users but not all in hard to track down ways.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150303/d9eaee4d/attachment.html>


More information about the llvm-commits mailing list