On February 16, 2015 10:11:07 PM CET, Jakub Jelinek <ja...@redhat.com> wrote: >On Mon, Feb 16, 2015 at 02:00:32PM -0700, Jeff Law wrote: >> --- a/gcc/tree-vrp.c >> +++ b/gcc/tree-vrp.c >> @@ -10176,13 +10176,20 @@ identify_jump_threads (void) >> /* We only care about blocks ending in a COND_EXPR. While >there >> may be some value in handling SWITCH_EXPR here, I doubt it's >> terribly important. */ >> - last = gsi_stmt (gsi_last_bb (bb)); >> + last = gsi_stmt (gsi_last_nondebug_bb (bb));
And if the comment is correct then it should not even matter as the condition ends a basic block. Richard. >Isn't that just > last = last_stmt (bb); >equivalent? > > Jakub
