diff options
author | Andrew MacLeod <amacleod@redhat.com> | 2021-04-26 19:23:25 -0400 |
---|---|---|
committer | Andrew MacLeod <amacleod@redhat.com> | 2021-05-07 15:00:21 -0400 |
commit | d942d7332d3dbd93609f5b5706df5990fd63dcfc (patch) | |
tree | 64b7da43580eaa104a7da35f1f34aeacc08f4644 /gcc | |
parent | 12f0a54ba9fa25aa13af368af5bb07e345c33dff (diff) | |
download | gcc-d942d7332d3dbd93609f5b5706df5990fd63dcfc.zip gcc-d942d7332d3dbd93609f5b5706df5990fd63dcfc.tar.gz gcc-d942d7332d3dbd93609f5b5706df5990fd63dcfc.tar.bz2 |
Fix range_on_exit for PHI stmts when there are no other stmts in the block.
last_stmt(bb) returns NULL for blocks which only have PHI stmts, and
range_on_exit would trigger a cache fill all the way to the top of the
program for the SSA_NAME.
* gimple-range.cc (gimple_ranger::range_on_exit): Handle block with
only PHI nodes better.
Diffstat (limited to 'gcc')
-rw-r--r-- | gcc/gimple-range.cc | 23 |
1 files changed, 16 insertions, 7 deletions
diff --git a/gcc/gimple-range.cc b/gcc/gimple-range.cc index 6158a75..e94bb35 100644 --- a/gcc/gimple-range.cc +++ b/gcc/gimple-range.cc @@ -1003,14 +1003,23 @@ gimple_ranger::range_on_exit (irange &r, basic_block bb, tree name) gcc_checking_assert (bb != EXIT_BLOCK_PTR_FOR_FN (cfun)); gcc_checking_assert (gimple_range_ssa_p (name)); - gimple *s = last_stmt (bb); - // If there is no statement in the block and this isn't the entry - // block, go get the range_on_entry for this block. For the entry - // block, a NULL stmt will return the global value for NAME. - if (!s && bb != ENTRY_BLOCK_PTR_FOR_FN (cfun)) - range_on_entry (r, bb, name); - else + gimple *s = SSA_NAME_DEF_STMT (name); + basic_block def_bb = gimple_bb (s); + // If this is not the definition block, get the range on the last stmt in + // the block... if there is one. + if (def_bb != bb) + s = last_stmt (bb); + // If there is no statement provided, get the range_on_entry for this block. + if (s) range_of_expr (r, name, s); + else + { + range_on_entry (r, bb, name); + // See if there was a deref in this block, if applicable + if (!cfun->can_throw_non_call_exceptions && r.varying_p () && + m_cache.m_non_null.non_null_deref_p (name, bb)) + r = range_nonzero (TREE_TYPE (name)); + } gcc_checking_assert (r.undefined_p () || range_compatible_p (r.type (), TREE_TYPE (name))); } |