Commit 5717e1f6 by Jakub Jelinek Committed by Jakub Jelinek

re PR tree-optimization/54676 (ICE: in set_value_range, at tree-vrp.c:433)

	PR tree-optimization/54676
	* tree-vrp.c (set_and_canonicalize_value_range): Handle
	one bit precision properly.

	* gcc.dg/pr54676.c: New test.

From-SVN: r191703
parent dadbefd3
2012-09-25 Jakub Jelinek <jakub@redhat.com> 2012-09-25 Jakub Jelinek <jakub@redhat.com>
PR tree-optimization/54676
* tree-vrp.c (set_and_canonicalize_value_range): Handle
one bit precision properly.
PR other/54692 PR other/54692
* configure.ac (CFLAGS, CXXFLAGS): Remove -Ofast or -Og * configure.ac (CFLAGS, CXXFLAGS): Remove -Ofast or -Og
properly. properly.
......
2012-09-25 Jakub Jelinek <jakub@redhat.com>
PR tree-optimization/54676
* gcc.dg/pr54676.c: New test.
2012-09-25 Richard Guenther <rguenther@suse.de> 2012-09-25 Richard Guenther <rguenther@suse.de>
PR tree-optimization/53663 PR tree-optimization/53663
......
/* PR tree-optimization/54676 */
/* { dg-do compile } */
/* { dg-options "-O -fno-tree-ccp -fno-tree-copy-prop -fno-tree-fre -ftree-vrp" } */
struct S
{
int s:1;
};
struct S bar (void);
int a;
void
foo (int x)
{
struct S s = bar ();
while (!a)
{
int l = 94967295;
a = x || (s.s &= l);
}
}
...@@ -501,8 +501,19 @@ set_and_canonicalize_value_range (value_range_t *vr, enum value_range_type t, ...@@ -501,8 +501,19 @@ set_and_canonicalize_value_range (value_range_t *vr, enum value_range_type t,
to adjust them. */ to adjust them. */
if (tree_int_cst_lt (max, min)) if (tree_int_cst_lt (max, min))
{ {
tree one = build_int_cst (TREE_TYPE (min), 1); tree one, tmp;
tree tmp = int_const_binop (PLUS_EXPR, max, one);
/* For one bit precision if max < min, then the swapped
range covers all values, so for VR_RANGE it is varying and
for VR_ANTI_RANGE empty range, so drop to varying as well. */
if (TYPE_PRECISION (TREE_TYPE (min)) == 1)
{
set_value_range_to_varying (vr);
return;
}
one = build_int_cst (TREE_TYPE (min), 1);
tmp = int_const_binop (PLUS_EXPR, max, one);
max = int_const_binop (MINUS_EXPR, min, one); max = int_const_binop (MINUS_EXPR, min, one);
min = tmp; min = tmp;
...@@ -531,6 +542,24 @@ set_and_canonicalize_value_range (value_range_t *vr, enum value_range_type t, ...@@ -531,6 +542,24 @@ set_and_canonicalize_value_range (value_range_t *vr, enum value_range_type t,
set_value_range_to_varying (vr); set_value_range_to_varying (vr);
return; return;
} }
else if (TYPE_PRECISION (TREE_TYPE (min)) == 1
&& !TYPE_UNSIGNED (TREE_TYPE (min))
&& (is_min || is_max))
{
/* For signed 1-bit precision, one is not in-range and
thus adding/subtracting it would result in overflows. */
if (operand_equal_p (min, max, 0))
{
min = max = is_min ? vrp_val_max (TREE_TYPE (min))
: vrp_val_min (TREE_TYPE (min));
t = VR_RANGE;
}
else
{
set_value_range_to_varying (vr);
return;
}
}
else if (is_min else if (is_min
/* As a special exception preserve non-null ranges. */ /* As a special exception preserve non-null ranges. */
&& !(TYPE_UNSIGNED (TREE_TYPE (min)) && !(TYPE_UNSIGNED (TREE_TYPE (min))
......
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