Hello Richard, Thanks for reply!
> > Well, it's enough to delay that to later passes that do this, so I'd prefer > to not change this in this patch. > OK, I'll fix it in next patch version. > I suppose yes. Maybe the cases are also obsoleted by the improved > loop PHI handling. > I noticed that PHI node special cases are caught only when we have three operands in operands vector which is not a case for new expr rewritter. Grouping of operands with the same rank is a good thing to do though, so I moved this part of code into a separate function. I've sent new patch in this mail: http://gcc.gnu.org/ml/gcc-patches/2011-07/msg02240.html It has all comments fixed except TODO_remove_unused_locals flags. Thanks Ilya