On January 27, 2016 8:20:47 PM GMT+01:00, Jeff Law <l...@redhat.com> wrote:

>
>This also fixes 3 minor issues Bernd spotted in the last round of
>changes.

Bernhard, not Bernd i suppose. Note that the !is_gimple_debug (stmt) should now 
be redundant in the hunk below.
Thanks,

@@ -280,33 +280,19 @@ fsm_find_control_statement_thread_paths (tree name,
                  break;
                }
 
-             for (gsi = gsi_start_bb (bb); !gsi_end_p (gsi); gsi_next (&gsi))
+             for (gsi = gsi_after_labels (bb);
+                  !gsi_end_p (gsi);
+                  gsi_next_nondebug (&gsi))
                {
                  gimple *stmt = gsi_stmt (gsi);
                  /* Do not count empty statements and labels.  */
                  if (gimple_code (stmt) != GIMPLE_NOP
-                     && gimple_code (stmt) != GIMPLE_LABEL
                      && !(gimple_code (stmt) == GIMPLE_ASSIGN
                           && gimple_assign_rhs_code (stmt) == ASSERT_EXPR)
                      && !is_gimple_debug (stmt))
                    ++n_insns;
                }

>
>Jeff


Reply via email to