[PATCH] Use getFirstNonPHI to set the insertion point for PHIs

Arnold Schwaighofer aschwaighofer at apple.com
Fri Sep 27 06:47:21 PDT 2013


Thanks. People normally don’t go this far in also reducing the data layout (they just leave it as is).


LGTM.

On Sep 26, 2013, at 10:53 PM, Justin Bogner <mail at justinbogner.com> wrote:

> Eli Friedman <eli.friedman at gmail.com> writes:
>> Please try to hand-reduce the testcase a bit more; it looks like bugpoint
>> output at the moment. Otherwise, looks fine; please commit.
> 
> Patch with hand-reduced testcase attached. The only part I'm not sure
> about is the datalayout - I reduced it to only make sure that a double
> has the same alignments as v64. Is this the right thing to do?
> 
> <phi_landingpad_2.patch>_______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits





More information about the llvm-commits mailing list