[PATCH] Small cleanups to cse.c

Andrew Pinski pinskia@gmail.com
Sat Nov 25 21:03:00 GMT 2006


On Sat, 2006-11-25 at 12:32 -0700, Roger Sayle wrote:
> I did do a quick audit of the gcc/ directory to see if I could find
> any instances of "if (foo) gcc_unreachable();"  Although not exhaustive,
> the vast majority of uses of gcc_unreachable are in switch statements
> for handling invalid cases, or at the very end of long
> if-then-else-if-then-else chains.

There are some if foo contains side effects.  I remember one because it
had broke bootstrap when it was converted in the first place.  Though
those should most likely be conveted to:
x = foo;
gcc_assert (x);

If not already done.

Thanks,
Andrew Pinski



More information about the Gcc-patches mailing list