From: Andrew Pinski <andrew.pinski@oss.qualcomm.com>
Date: Mon, 13 Oct 2025 23:47:55 +0000 (-0700)
Subject: phi-opt: Disable parts of it for -Og
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=6b5e2f6e5bad9c320c6c40fd51dc70f741d96933;p=thirdparty%2Fgcc.git

phi-opt: Disable parts of it for -Og

While working on the cselim limited part of phiopt, I noticed
that the debugging experience for -Og case would cause jumping
execution in some cases. So this disables the store and operation
factoring parts for -Og since those 2 can cause the line information
of the debugging to be off.

Bootstrapped and tested on x86_64-linux-gnu.

gcc/ChangeLog:

	* tree-ssa-phiopt.cc (pass_phiopt::execute): Disable
	cselim-limited and factor out operations for -Og.

Signed-off-by: Andrew Pinski <andrew.pinski@oss.qualcomm.com>
---

diff --git a/gcc/tree-ssa-phiopt.cc b/gcc/tree-ssa-phiopt.cc
index 9f7f662d7cf..031184d9e58 100644
--- a/gcc/tree-ssa-phiopt.cc
+++ b/gcc/tree-ssa-phiopt.cc
@@ -4569,8 +4569,8 @@ pass_phiopt::execute (function *)
 	    hoist_adjacent_loads (bb, bb1, bb2, bb3);
 
 	  /* Try to see if there are only store in each side of the if
-	     and try to remove that.  */
-	  if (EDGE_COUNT (bb3->preds) == 2)
+	     and try to remove that; don't do this for -Og.  */
+	  if (EDGE_COUNT (bb3->preds) == 2 && !optimize_debug)
 	    while (cond_if_else_store_replacement_limited (bb1, bb2, bb3))
 	      ;
 	}
@@ -4586,7 +4586,8 @@ pass_phiopt::execute (function *)
 
       /* Factor out operations from the phi if possible. */
       if (single_pred_p (bb1)
-	  && EDGE_COUNT (merge->preds) == 2)
+	  && EDGE_COUNT (merge->preds) == 2
+	  && !optimize_debug)
 	{
 	  for (gsi = gsi_start (phis); !gsi_end_p (gsi); )
 	    {