diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 1e2601bac49..be1f4fba562 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2006-12-31 Roger Sayle + + PR middle-end/30322 + * fold-const.c (fold_binary): Fold X + ~X and ~X + X into -1 when + we don't care about trapping overflow. Only convert -A - 1 into ~A + when we don't care about trapping overflow. + 2006-12-31 Richard Guenther PR middle-end/30338 diff --git a/gcc/fold-const.c b/gcc/fold-const.c index dbdc9c26367..da9579e6b4e 100644 --- a/gcc/fold-const.c +++ b/gcc/fold-const.c @@ -8662,6 +8662,26 @@ fold_binary (enum tree_code code, tree type, tree op0, tree op1) if (integer_zerop (arg1)) return non_lvalue (fold_convert (type, arg0)); + /* ~X + X is -1. */ + if (TREE_CODE (arg0) == BIT_NOT_EXPR + && operand_equal_p (TREE_OPERAND (arg0, 0), arg1, 0) + && !TYPE_TRAP_SIGNED (type)) + { + t1 = build_int_cst (type, -1); + t1 = force_fit_type (t1, 0, false, false); + return omit_one_operand (type, t1, arg1); + } + + /* X + ~X is -1. */ + if (TREE_CODE (arg1) == BIT_NOT_EXPR + && operand_equal_p (arg0, TREE_OPERAND (arg1, 0), 0) + && !TYPE_TRAP_SIGNED (type)) + { + t1 = build_int_cst (type, -1); + t1 = force_fit_type (t1, 0, false, false); + return omit_one_operand (type, t1, arg0); + } + /* If we are adding two BIT_AND_EXPR's, both of which are and'ing with a constant, and the two constants have no bits in common, we should treat this as a BIT_IOR_EXPR since this may produce more @@ -9002,7 +9022,8 @@ fold_binary (enum tree_code code, tree type, tree op0, tree op1) /* Convert -A - 1 to ~A. */ if (INTEGRAL_TYPE_P (type) && TREE_CODE (arg0) == NEGATE_EXPR - && integer_onep (arg1)) + && integer_onep (arg1) + && !TYPE_TRAP_SIGNED (type)) return fold_build1 (BIT_NOT_EXPR, type, fold_convert (type, TREE_OPERAND (arg0, 0))); diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index df63017b96c..30d25ea11dd 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2006-12-31 Roger Sayle + + PR middle-end/30322 + * gcc.dg/fold-plusnot-1.c: New test case. + 2006-12-31 Simon Martin PR c++/29731 diff --git a/gcc/testsuite/gcc.dg/fold-plusnot-1.c b/gcc/testsuite/gcc.dg/fold-plusnot-1.c new file mode 100644 index 00000000000..81ddf46d6d0 --- /dev/null +++ b/gcc/testsuite/gcc.dg/fold-plusnot-1.c @@ -0,0 +1,30 @@ +/* PR middle-end/30322 */ +/* { dg-do compile } */ +/* { dg-options "-O2 -fdump-tree-original" } */ + +int test1(int a) +{ + return a + ~a; +} + +int test2(int b) +{ + return ~b + b; +} + +unsigned int test3(unsigned int c) +{ + return c + ~c; +} + +unsigned int test4(unsigned int d) +{ + return ~d + d; +} + +/* { dg-final { scan-tree-dump-times "\\+ a" 0 "original" } } */ +/* { dg-final { scan-tree-dump-times "\\+ b" 0 "original" } } */ +/* { dg-final { scan-tree-dump-times "\\+ c" 0 "original" } } */ +/* { dg-final { scan-tree-dump-times "\\+ d" 0 "original" } } */ +/* { dg-final { cleanup-tree-dump "original" } } */ +