[PATCH] D23052: [Inliner] Add a flag to disable manual alloca merging in the Inliner.

Sean Silva via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 2 01:14:34 PDT 2016


silvas added a subscriber: silvas.
silvas accepted this revision.
silvas added a reviewer: silvas.
silvas added a comment.
This revision is now accepted and ready to land.

Pulling it out into a separate function seems like a nice cleanup anyway. That seems fine regardless of whether we ultimately remove the logic.

And the flag is off by default and easy enough to remove. So LGTM.

Any initial results from disabling the alloca merging?
Also, do you know if we have any PR's related to the information loss due to this alloca merging?


https://reviews.llvm.org/D23052





More information about the llvm-commits mailing list