Re: [Committed/Obvious] Fix PR 49474: ICE on ppc-linux with -O3 in cprop.c

2011-07-12 Thread Andrew Pinski
On Tue, Jul 12, 2011 at 11:42 AM, Andrew Pinski wrote: > Hi, >  The problem here is the code reads: >       /* Check for more than one successor.  */ >     if (! EDGE_COUNT (bb->succs) > 1) > But that expression is always false as ! has a higher precedence than >> does.  So the obvious thing is to

[Committed/Obvious] Fix PR 49474: ICE on ppc-linux with -O3 in cprop.c

2011-07-12 Thread Andrew Pinski
Hi, The problem here is the code reads: /* Check for more than one successor. */ if (! EDGE_COUNT (bb->succs) > 1) But that expression is always false as ! has a higher precedence than > does. So the obvious thing is to rewrite this statement as: if (EDGE_COUNT (bb->succs) <= 1) A