http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51521

Dmitry Vyukov <dvyukov at google dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dvyukov at google dot com

--- Comment #3 from Dmitry Vyukov <dvyukov at google dot com> 2011-12-13 
11:15:18 UTC ---
Hi Richard,

May you please take a look at this message where I describe why I had done the
change:
http://codereview.appspot.com/5486043/#msg6

The edge w/o EDGE_FALLTHRU also causes internal errors when I try to split it.

> The EDGE_FALLTHRU flag is wrong if any of EDGE_TRUE/FALSE_* is added which
is probably what happens.  Thus the patch was wrong.

But when I set EDGE_FALLTHRU there is no EDGE_TRUE/FALSE_* yet (the flags were
0). So, isn't it who adds EDGE_TRUE/FALSE_* must reset EDGE_FALLTHRU?

Reply via email to