diff options
author | Andrew Pinski <pinskia@physics.uc.edu> | 2005-07-01 22:01:17 +0000 |
---|---|---|
committer | Andrew Pinski <pinskia@gcc.gnu.org> | 2005-07-01 15:01:17 -0700 |
commit | 7898840fde320acf0c3e8d3c35ec40fcff7221a8 (patch) | |
tree | 550a195febd08e1cb26ae88ec4c1db7334116f48 | |
parent | a04eae1bf3cfb764120e74b5167b6207da11f091 (diff) | |
download | gcc-7898840fde320acf0c3e8d3c35ec40fcff7221a8.zip gcc-7898840fde320acf0c3e8d3c35ec40fcff7221a8.tar.gz gcc-7898840fde320acf0c3e8d3c35ec40fcff7221a8.tar.bz2 |
re PR tree-optimization/22269 (ICE in tree-ssa-reassoc with uninitialized variables)
2005-07-01 Andrew Pinski <pinskia@physics.uc.edu>
PR tree-opt/22269
* tree-ssa-reassoc.c (should_transpose): Fix which operand
we check for SSA_NAME for.
2005-07-01 Andrew Pinski <pinskia@physics.uc.edu>
PR tree-opt/22269
* gcc.c-torture/compile/pr22269.c: New test.
From-SVN: r101519
-rw-r--r-- | gcc/ChangeLog | 6 | ||||
-rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
-rw-r--r-- | gcc/testsuite/gcc.c-torture/compile/pr22269.c | 16 | ||||
-rw-r--r-- | gcc/tree-ssa-reassoc.c | 2 |
4 files changed, 28 insertions, 1 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 70be419..8236a2f 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2005-07-01 Andrew Pinski <pinskia@physics.uc.edu> + + PR tree-opt/22269 + * tree-ssa-reassoc.c (should_transpose): Fix which operand + we check for SSA_NAME for. + 2005-06-29 Daniel Berlin <dberlin@dberlin.org> Fix PR tree-optimization/22071 diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 47001d3..32ef7e1 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2005-07-01 Andrew Pinski <pinskia@physics.uc.edu> + + PR tree-opt/22269 + * gcc.c-torture/compile/pr22269.c: New test. + 2005-06-30 Diego Novillo <dnovillo@redhat.com> PR 21584 diff --git a/gcc/testsuite/gcc.c-torture/compile/pr22269.c b/gcc/testsuite/gcc.c-torture/compile/pr22269.c new file mode 100644 index 0000000..bfcf209 --- /dev/null +++ b/gcc/testsuite/gcc.c-torture/compile/pr22269.c @@ -0,0 +1,16 @@ +/* We used to ICE in tree-ssa-reassoc because we did look at the correct operand to + see if it was a SSA_NAME. */ +int printf(const char*, ...); +int main(int argv, char*argc) { + + int d1; + int d2; + int s1, s2; + int b; + ((d1)&=(int)0x0000ffffL, (d1)|=((int)(short)(0x344))<<16); + ((d1)&=(int)0xffff0000UL, (d1)|=(int)(unsigned short)(0x4567)); + ((d2)&=(int)0x0000ffffL, (d2)|=((int)(short)(0))<<16); + ((d2)&=(int)0xffff0000UL, (d2)|=(int)(unsigned short)(0x3b9a)); + printf(" dividend >>: %ld\n", d1); + printf(" divisor >>: %ld\n", d2); +} diff --git a/gcc/tree-ssa-reassoc.c b/gcc/tree-ssa-reassoc.c index 5f0e8fc..395c549 100644 --- a/gcc/tree-ssa-reassoc.c +++ b/gcc/tree-ssa-reassoc.c @@ -406,7 +406,7 @@ should_transpose (tree rhs ATTRIBUTE_UNUSED, /* Also, see if the LHS's high ranked op should be switched with our RHS simply because it is greater in rank than our current RHS. */ - if (TREE_CODE (TREE_OPERAND (lhsdefop, 0)) == SSA_NAME) + if (TREE_CODE (TREE_OPERAND (lhsdefop, highrankop)) == SSA_NAME) { tree iop = SSA_NAME_DEF_STMT (TREE_OPERAND (lhsdefop, highrankop)); if (TREE_CODE (iop) == MODIFY_EXPR) |