aboutsummaryrefslogtreecommitdiff
path: root/gcc
diff options
context:
space:
mode:
authorAldy Hernandez <aldyh@redhat.com>2021-11-25 17:30:07 +0100
committerAldy Hernandez <aldyh@redhat.com>2021-11-25 17:34:31 +0100
commit415f9ee404dc9e8ae1cade160a33400b10abb18a (patch)
treeb8fb9fa2e7d9408758281ae5770bbb951902fa08 /gcc
parent58d50a5dd6344179eebaeb6fd2f895e59463cf74 (diff)
downloadgcc-415f9ee404dc9e8ae1cade160a33400b10abb18a.zip
gcc-415f9ee404dc9e8ae1cade160a33400b10abb18a.tar.gz
gcc-415f9ee404dc9e8ae1cade160a33400b10abb18a.tar.bz2
path solver: Revert computation of ranges in gimple order.
Revert the patch below, as it may slow down compilation with large CFGs. commit 8acbd7bef6edbf537e3037174907029b530212f6 Author: Aldy Hernandez <aldyh@redhat.com> Date: Wed Nov 24 09:43:36 2021 +0100 path solver: Compute ranges in path in gimple order. gcc/ChangeLog: * gimple-range-path.cc (path_range_query::compute_ranges_defined): Remove. (path_range_query::compute_ranges_in_block): Revert to bitmap order. * gimple-range-path.h: Remove compute_ranges_defined.
Diffstat (limited to 'gcc')
-rw-r--r--gcc/gimple-range-path.cc33
-rw-r--r--gcc/gimple-range-path.h1
2 files changed, 11 insertions, 23 deletions
diff --git a/gcc/gimple-range-path.cc b/gcc/gimple-range-path.cc
index 806bce9..b9c7122 100644
--- a/gcc/gimple-range-path.cc
+++ b/gcc/gimple-range-path.cc
@@ -401,27 +401,6 @@ path_range_query::compute_ranges_in_phis (basic_block bb)
}
}
-// Compute ranges defined in block.
-
-void
-path_range_query::compute_ranges_defined (basic_block bb)
-{
- int_range_max r;
-
- compute_ranges_in_phis (bb);
-
- // Iterate in gimple order to minimize recursion.
- for (auto gsi = gsi_start_nondebug_bb (bb); !gsi_end_p (gsi); gsi_next (&gsi))
- if (gimple_has_lhs (gsi_stmt (gsi)))
- {
- tree name = gimple_get_lhs (gsi_stmt (gsi));
- if (TREE_CODE (name) == SSA_NAME
- && bitmap_bit_p (m_imports, SSA_NAME_VERSION (name))
- && range_defined_in_block (r, name, bb))
- set_cache (r, name);
- }
-}
-
// Compute ranges defined in the current block, or exported to the
// next block.
@@ -444,7 +423,17 @@ path_range_query::compute_ranges_in_block (basic_block bb)
clear_cache (name);
}
- compute_ranges_defined (bb);
+ // Solve imports defined in this block, starting with the PHIs...
+ compute_ranges_in_phis (bb);
+ // ...and then the rest of the imports.
+ EXECUTE_IF_SET_IN_BITMAP (m_imports, 0, i, bi)
+ {
+ tree name = ssa_name (i);
+
+ if (gimple_code (SSA_NAME_DEF_STMT (name)) != GIMPLE_PHI
+ && range_defined_in_block (r, name, bb))
+ set_cache (r, name);
+ }
if (at_exit ())
return;
diff --git a/gcc/gimple-range-path.h b/gcc/gimple-range-path.h
index 81c87d4..57a9ae9 100644
--- a/gcc/gimple-range-path.h
+++ b/gcc/gimple-range-path.h
@@ -58,7 +58,6 @@ private:
// Methods to compute ranges for the given path.
bool range_defined_in_block (irange &, tree name, basic_block bb);
void compute_ranges_in_block (basic_block bb);
- void compute_ranges_defined (basic_block bb);
void compute_ranges_in_phis (basic_block bb);
void adjust_for_non_null_uses (basic_block bb);
void ssa_range_in_phi (irange &r, gphi *phi);