[llvm-commits] LCSSA avoiding creation of unused PHI nodes

Cameron Zwarich zwarich at apple.com
Mon Mar 14 21:24:14 PDT 2011


Sorry to be a pain, but do you want to use the new FileCheck style of test here?

I will try to take a look at the LoopSimplify patch later today or tomorrow. There was something I wanted to test with it. It is unfortunate that block layout is so brittle right now, but that's the way it is for the time being.

Cameron

On Mar 14, 2011, at 9:18 PM, Andrew Clinton wrote:

> Thanks for the review - here is the update.
> 
> Could you also review/commit the updated loop-simplify patch that I
> send to the llvm-commits list?
> 
> On Mon, Mar 14, 2011 at 11:50 PM, Cameron Zwarich <zwarich at apple.com> wrote:
>> I would add a test for this. Also, AvailablePHIs is a bit of a vague name. Maybe AddedPHIs would be better? Other than that, it looks good.
>> 
>> Cameron
> <lcssa_unused_phis.diff>





More information about the llvm-commits mailing list