[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