[PATCH] [SROA] Fold a PHI node if all its incoming values are the same

Duncan Sands duncan.sands at gmail.com
Sat Jul 26 09:13:23 PDT 2014


Hi Chandler,

On 26/07/14 07:45, Chandler Carruth wrote:
> Yea, this looks nice. I have a bunch of nit-picky comments below... Nothing really significant.

SimplifyPHINode in InstructionSimplify has this logic already, only it correctly 
handles the case of phi(x, undef) where undef corresponds to a block not 
dominated by x.  In this case it is wrong to replace the phi by x.  I suggest 
you just use SimplifyPHINode.

Ciao, Duncan.



More information about the llvm-commits mailing list