diff mbox series

GORI cleanups

Message ID 20240625134429.9EF1F38438A1@sourceware.org
State New
Headers show
Series GORI cleanups | expand

Commit Message

Richard Biener June 25, 2024, 1:44 p.m. UTC 
The following replaces conditional is_export_p calls as is_export_p
handles a NULL bb itself.

Bootstrap running on x86_64-unknown-linux-gnu, OK?

Thanks,
Richard.

	* gimple-range-gori.cc (gori_compute::may_recompute_p):
	Call is_export_p with NULL bb.
---
 gcc/gimple-range-gori.cc | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)
diff mbox series

Patch

diff --git a/gcc/gimple-range-gori.cc b/gcc/gimple-range-gori.cc
index 275283a424f..a31e3be65f7 100644
--- a/gcc/gimple-range-gori.cc
+++ b/gcc/gimple-range-gori.cc
@@ -1332,18 +1332,14 @@  gori_compute::may_recompute_p (tree name, basic_block bb, int depth)
 	  gcc_checking_assert (depth >= 1);
 	}
 
-      bool res = (bb ? m_map.is_export_p (dep1, bb)
-		     : m_map.is_export_p (dep1));
+      bool res = m_map.is_export_p (dep1, bb);
       if (res || depth <= 1)
 	return res;
       // Check another level of recomputation.
       return may_recompute_p (dep1, bb, --depth);
     }
   // Two dependencies terminate the depth of the search.
-  if (bb)
-    return m_map.is_export_p (dep1, bb) || m_map.is_export_p (dep2, bb);
-  else
-    return m_map.is_export_p (dep1) || m_map.is_export_p (dep2);
+  return m_map.is_export_p (dep1, bb) || m_map.is_export_p (dep2, bb);
 }
 
 // Return TRUE if NAME can be recomputed on edge E.  If any direct dependent