aboutsummaryrefslogtreecommitdiff
path: root/gcc
diff options
context:
space:
mode:
authorAndrew MacLeod <amacleod@redhat.com>2021-04-26 17:41:22 -0400
committerAndrew MacLeod <amacleod@redhat.com>2021-05-07 14:53:44 -0400
commit1416a1434c43de0b93b313644f932ac024a94773 (patch)
treee14e4694f814a3fddf533a1406962251ffda9208 /gcc
parente2bc5b6c04df820017c497a2578bd3c4c7b6c89b (diff)
downloadgcc-1416a1434c43de0b93b313644f932ac024a94773.zip
gcc-1416a1434c43de0b93b313644f932ac024a94773.tar.gz
gcc-1416a1434c43de0b93b313644f932ac024a94773.tar.bz2
Enhance initial global value setting.
Incorporate code from vr_values to get safe initial parameter values. If this is a local automatic which is used before defined, use UNDEFINED. * gimple-range.h (gimple_range_global): Pick up parameter initial values, and use-before defined locals are UNDEFINED.
Diffstat (limited to 'gcc')
-rw-r--r--gcc/gimple-range.h47
1 files changed, 32 insertions, 15 deletions
diff --git a/gcc/gimple-range.h b/gcc/gimple-range.h
index 5751b09..f331561 100644
--- a/gcc/gimple-range.h
+++ b/gcc/gimple-range.h
@@ -138,22 +138,39 @@ gimple_range_global (tree name)
{
gcc_checking_assert (gimple_range_ssa_p (name));
tree type = TREE_TYPE (name);
-#if 0
- // Reenable picking up global ranges when we are OK failing tests that look
- // for builtin_unreachable in the code, like
- // RUNTESTFLAGS=dg.exp=pr61034.C check-g++
- // pre-optimizations (inlining) set a global range which causes the ranger
- // to remove the condition which leads to builtin_unreachable.
- if (!POINTER_TYPE_P (type) && SSA_NAME_RANGE_INFO (name))
+
+ if (SSA_NAME_IS_DEFAULT_DEF (name))
{
- // Return a range from an SSA_NAME's available range.
- wide_int min, max;
- enum value_range_kind kind = get_range_info (name, &min, &max);
- return value_range (type, min, max, kind);
- }
-#endif
- // Otherwise return range for the type.
- return value_range (type);
+ tree sym = SSA_NAME_VAR (name);
+ // Adapted from vr_values::get_lattice_entry().
+ // Use a range from an SSA_NAME's available range.
+ if (TREE_CODE (sym) == PARM_DECL)
+ {
+ // Try to use the "nonnull" attribute to create ~[0, 0]
+ // anti-ranges for pointers. Note that this is only valid with
+ // default definitions of PARM_DECLs.
+ if (POINTER_TYPE_P (type)
+ && (nonnull_arg_p (sym) || get_ptr_nonnull (name)))
+ {
+ value_range r;
+ r.set_nonzero (type);
+ return r;
+ }
+ else if (INTEGRAL_TYPE_P (type))
+ {
+ value_range r;
+ get_range_info (name, r);
+ if (r.undefined_p ())
+ r.set_varying (type);
+ return r;
+ }
+ }
+ // If this is a local automatic with no definition, use undefined.
+ else if (TREE_CODE (sym) != RESULT_DECL)
+ return value_range ();
+ }
+ // Otherwise return range for the type.
+ return value_range (type);
}