Commit aab162fb by Aldy Hernandez Committed by Aldy Hernandez

tree-vrp.c (value_range::check): Do not access internals directly.

	* tree-vrp.c (value_range::check): Do not access internals
	directly.
	(value_range::singleton_p): Same.
	(value_range::type): Same.
	(vrp_finalize): Use value_range API.

From-SVN: r265955
parent 6e40dcae
2018-11-08 Aldy Hernandez <aldyh@redhat.com> 2018-11-08 Aldy Hernandez <aldyh@redhat.com>
* tree-vrp.c (value_range::check): Do not access internals
directly.
(value_range::singleton_p): Same.
(value_range::type): Same.
(vrp_finalize): Use value_range API.
2018-11-08 Aldy Hernandez <aldyh@redhat.com>
* tree-vrp.c (may_contain_p): Do not access m_min/m_max directly. * tree-vrp.c (may_contain_p): Do not access m_min/m_max directly.
2018-11-08 Aldy Hernandez <aldyh@redhat.com> 2018-11-08 Aldy Hernandez <aldyh@redhat.com>
...@@ -157,7 +157,7 @@ value_range::check () ...@@ -157,7 +157,7 @@ value_range::check ()
} }
case VR_UNDEFINED: case VR_UNDEFINED:
case VR_VARYING: case VR_VARYING:
gcc_assert (!m_min && !m_max); gcc_assert (!min () && !max ());
gcc_assert (!m_equiv || bitmap_empty_p (m_equiv)); gcc_assert (!m_equiv || bitmap_empty_p (m_equiv));
break; break;
default: default:
...@@ -291,11 +291,11 @@ bool ...@@ -291,11 +291,11 @@ bool
value_range::singleton_p (tree *result) const value_range::singleton_p (tree *result) const
{ {
if (m_kind == VR_RANGE if (m_kind == VR_RANGE
&& vrp_operand_equal_p (m_min, m_max) && vrp_operand_equal_p (min (), max ())
&& is_gimple_min_invariant (m_min)) && is_gimple_min_invariant (min ()))
{ {
if (result) if (result)
*result = m_min; *result = min ();
return true; return true;
} }
return false; return false;
...@@ -306,8 +306,8 @@ value_range::type () const ...@@ -306,8 +306,8 @@ value_range::type () const
{ {
/* Types are only valid for VR_RANGE and VR_ANTI_RANGE, which are /* Types are only valid for VR_RANGE and VR_ANTI_RANGE, which are
known to have non-zero min/max. */ known to have non-zero min/max. */
gcc_assert (m_min); gcc_assert (min ());
return TREE_TYPE (m_min); return TREE_TYPE (min ());
} }
/* Dump value range to FILE. */ /* Dump value range to FILE. */
...@@ -6558,9 +6558,7 @@ vrp_prop::vrp_finalize (bool warn_array_bounds_p) ...@@ -6558,9 +6558,7 @@ vrp_prop::vrp_finalize (bool warn_array_bounds_p)
&& range_includes_zero_p (vr) == 0) && range_includes_zero_p (vr) == 0)
set_ptr_nonnull (name); set_ptr_nonnull (name);
else if (!POINTER_TYPE_P (TREE_TYPE (name))) else if (!POINTER_TYPE_P (TREE_TYPE (name)))
set_range_info (name, vr->kind (), set_range_info (name, *vr);
wi::to_wide (vr->min ()),
wi::to_wide (vr->max ()));
} }
/* If we're checking array refs, we want to merge information on /* If we're checking array refs, we want to merge information on
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment