On 6/25/24 09:44, Richard Biener wrote:
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?
Absolutely. Thanks Andrew
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 --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
