summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorjakub <jakub@138bc75d-0d04-0410-961f-82ee72b054a4>2010-08-12 15:21:34 +0000
committerjakub <jakub@138bc75d-0d04-0410-961f-82ee72b054a4>2010-08-12 15:21:34 +0000
commitcab932e0f7e8c311fd9fac996bb95d1cec76bb64 (patch)
tree639f56221290e502c4e4cfc6c62da003331396f5
parent12c655b1e4ab9e522d575335b14d4c6fc6afc72e (diff)
downloadppe42-gcc-cab932e0f7e8c311fd9fac996bb95d1cec76bb64.tar.gz
ppe42-gcc-cab932e0f7e8c311fd9fac996bb95d1cec76bb64.zip
PR middle-end/45262
* fold-const.c (make_range) <case NEGATE_EXPR>: Punt if -a overflows. Normalize the range. * gcc.c-torture/execute/pr45262.c: New test. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@163193 138bc75d-0d04-0410-961f-82ee72b054a4
-rw-r--r--gcc/ChangeLog4
-rw-r--r--gcc/fold-const.c7
-rw-r--r--gcc/testsuite/ChangeLog5
-rw-r--r--gcc/testsuite/gcc.c-torture/execute/pr45262.c33
4 files changed, 46 insertions, 3 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 09da08d6176..31dd288030f 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -3,6 +3,10 @@
* builtins.c (fold_builtin_memory_op): Avoid -Wsign-compare
warning.
+ PR middle-end/45262
+ * fold-const.c (make_range) <case NEGATE_EXPR>: Punt if
+ -a overflows. Normalize the range.
+
2010-08-12 Richard Guenther <rguenther@suse.de>
PR tree-optimization/45232
diff --git a/gcc/fold-const.c b/gcc/fold-const.c
index 9ca5eff554e..d63411ea89a 100644
--- a/gcc/fold-const.c
+++ b/gcc/fold-const.c
@@ -3985,9 +3985,9 @@ make_range (tree exp, int *pin_p, tree *plow, tree *phigh,
n_high = range_binop (MINUS_EXPR, exp_type,
build_int_cst (exp_type, 0),
0, low, 0);
- low = n_low, high = n_high;
- exp = arg0;
- continue;
+ if (n_high != 0 && TREE_OVERFLOW (n_high))
+ break;
+ goto normalize;
case BIT_NOT_EXPR:
/* ~ X -> -X - 1 */
@@ -4021,6 +4021,7 @@ make_range (tree exp, int *pin_p, tree *plow, tree *phigh,
if (TYPE_OVERFLOW_UNDEFINED (arg0_type))
*strict_overflow_p = true;
+ normalize:
/* Check for an unsigned range which has wrapped around the maximum
value thus making n_high < n_low, and normalize it. */
if (n_low && n_high && tree_int_cst_lt (n_high, n_low))
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index dd180c655e5..840253dcaf3 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2010-08-12 Jakub Jelinek <jakub@redhat.com>
+
+ PR middle-end/45262
+ * gcc.c-torture/execute/pr45262.c: New test.
+
2010-08-12 Uros Bizjak <ubizjak@gmail.com>
* gcc.dg/profile-generate-3.c: Call dg-require-profiling with
diff --git a/gcc/testsuite/gcc.c-torture/execute/pr45262.c b/gcc/testsuite/gcc.c-torture/execute/pr45262.c
new file mode 100644
index 00000000000..72e186bf287
--- /dev/null
+++ b/gcc/testsuite/gcc.c-torture/execute/pr45262.c
@@ -0,0 +1,33 @@
+/* PR middle-end/45262 */
+
+extern void abort (void);
+
+int
+foo (unsigned int x)
+{
+ return ((int) x < 0) || ((int) (-x) < 0);
+}
+
+int
+bar (unsigned int x)
+{
+ return x >> 31 || (-x) >> 31;
+}
+
+int
+main (void)
+{
+ if (foo (1) != 1)
+ abort ();
+ if (foo (0) != 0)
+ abort ();
+ if (foo (-1) != 1)
+ abort ();
+ if (bar (1) != 1)
+ abort ();
+ if (bar (0) != 0)
+ abort ();
+ if (bar (-1) != 1)
+ abort ();
+ return 0;
+}
OpenPOWER on IntegriCloud