diff options
| author | Jakub Jelinek <jakub@redhat.com> | 2010-10-18 10:26:06 +0200 |
|---|---|---|
| committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2010-10-18 10:26:06 +0200 |
| commit | 8ddf04c2b02d457ee08a8158dab98ec2f3273d66 (patch) | |
| tree | b7acef529c889515ad15e65810919314e6d35650 /gcc | |
| parent | 1d32bbcdcb9839efa24655752ce90ce2ae97fa06 (diff) | |
| download | gcc-8ddf04c2b02d457ee08a8158dab98ec2f3273d66.zip gcc-8ddf04c2b02d457ee08a8158dab98ec2f3273d66.tar.gz gcc-8ddf04c2b02d457ee08a8158dab98ec2f3273d66.tar.bz2 | |
re PR middle-end/46019 (x / (0x200000000ULL << y) miscompilation with 32-bit HWI)
PR middle-end/46019
* fold-const.c (fold_binary_loc): If integer_pow2p has
TREE_INT_CST_LOW zero, look at TREE_INT_CST_HIGH.
* gcc.c-torture/execute/pr46019.c: New test.
From-SVN: r165610
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/fold-const.c | 8 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.c-torture/execute/pr46019.c | 14 |
4 files changed, 32 insertions, 1 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index ff092e7..1665857 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2010-10-18 Jakub Jelinek <jakub@redhat.com> + + PR middle-end/46019 + * fold-const.c (fold_binary_loc): If integer_pow2p has + TREE_INT_CST_LOW zero, look at TREE_INT_CST_HIGH. + 2010-10-18 Basile Starynkevitch <basile@starynkevitch.net> Jeremie Salvucci <jeremie.salvucci@free.fr> diff --git a/gcc/fold-const.c b/gcc/fold-const.c index 8146920..808f491 100644 --- a/gcc/fold-const.c +++ b/gcc/fold-const.c @@ -11611,7 +11611,13 @@ fold_binary_loc (location_t loc, if (integer_pow2p (sval) && tree_int_cst_sgn (sval) > 0) { tree sh_cnt = TREE_OPERAND (arg1, 1); - unsigned long pow2 = exact_log2 (TREE_INT_CST_LOW (sval)); + unsigned long pow2; + + if (TREE_INT_CST_LOW (sval)) + pow2 = exact_log2 (TREE_INT_CST_LOW (sval)); + else + pow2 = exact_log2 (TREE_INT_CST_HIGH (sval)) + + HOST_BITS_PER_WIDE_INT; if (strict_overflow_p) fold_overflow_warning (("assuming signed overflow does not " diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 57a1092..f7f964b 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2010-10-18 Jakub Jelinek <jakub@redhat.com> + + PR middle-end/46019 + * gcc.c-torture/execute/pr46019.c: New test. + 2010-10-17 Nicola Pero <nicola.pero@meta-innovation.com> Merge from 'apple/trunk' branch on FSF servers. diff --git a/gcc/testsuite/gcc.c-torture/execute/pr46019.c b/gcc/testsuite/gcc.c-torture/execute/pr46019.c new file mode 100644 index 0000000..b036557 --- /dev/null +++ b/gcc/testsuite/gcc.c-torture/execute/pr46019.c @@ -0,0 +1,14 @@ +/* PR middle-end/46019 */ + +extern void abort (void); + +int +main (void) +{ + unsigned long long l = 0x40000000000ULL; + int n; + for (n = 0; n < 8; n++) + if (l / (0x200000000ULL << n) != (0x200 >> n)) + abort (); + return 0; +} |