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

Justin Bogner mail at justinbogner.com
Fri Sep 27 08:37:12 PDT 2013


Committed in r191526

Arnold Schwaighofer <aschwaighofer at apple.com> writes:
> 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