On 3/24/20 3:58 AM, Jakub Jelinek wrote:
+         for (i = tsi_start (stmt); !tsi_end_p (i); tsi_next (&i))
+           {
+             tree t = tsi_stmt (i);
+             if (TREE_CODE (t) != DEBUG_BEGIN_STMT && nondebug_stmts < 2)
+               nondebug_stmts++;
+             cp_walk_tree (tsi_stmt_ptr (i), cp_genericize_r, data, NULL);
+             if (TREE_CODE (t) != DEBUG_BEGIN_STMT
+                 && nondebug_stmts == 1
+                 && TREE_SIDE_EFFECTS (tsi_stmt (i)))

How about (nondebug_stmts > 1 || TREE_SIDE_EFFECTS (tsi_stmt (i))) here...

+               clear_side_effects = false;
+           }
+         if (nondebug_stmts < 2 && clear_side_effects)

So we don't need to look at nondebug_stmts here?  OK with that change.

Jason

Reply via email to