PR92085

Jakub Jelinek jakub@redhat.com
Tue Oct 15 07:04:00 GMT 2019


On Mon, Oct 14, 2019 at 03:24:02PM -0700, Prathamesh Kulkarni wrote:
> The patch fixes this by simply putting gsi_next in else, which avoids
> the above issue.
> Bootstrap+test in progress on x86_64-unknown-linux-gnu.
> OK to commit if passes ?

No ChangeLog entry.

--- a/gcc/tree-if-conv.c
+++ b/gcc/tree-if-conv.c
@@ -2976,7 +2976,8 @@ ifcvt_local_dce (class loop *loop)
           if (dse_classify_store (&write, stmt, false, NULL, NULL, latch_vdef)
               == DSE_STORE_DEAD)
             delete_dead_or_redundant_assignment (&gsi, "dead");
-	  gsi_next (&gsi);
+	  else
+	    gsi_next (&gsi);
 	  continue;
 	}
 
The patch clearly shows that the 3 lines above it are incorrectly indented
(spaces instead of tabs), can you please fix that up while touching this?

Ok for trunk with ChangeLog added and the indentation fixed.  Thanks.

	Jakub



More information about the Gcc-patches mailing list