From 7898840fde320acf0c3e8d3c35ec40fcff7221a8 Mon Sep 17 00:00:00 2001 From: Andrew Pinski Date: Fri, 1 Jul 2005 22:01:17 +0000 Subject: [PATCH] re PR tree-optimization/22269 (ICE in tree-ssa-reassoc with uninitialized variables) 2005-07-01 Andrew Pinski PR tree-opt/22269 * tree-ssa-reassoc.c (should_transpose): Fix which operand we check for SSA_NAME for. 2005-07-01 Andrew Pinski PR tree-opt/22269 * gcc.c-torture/compile/pr22269.c: New test. From-SVN: r101519 --- gcc/ChangeLog | 6 ++++++ gcc/testsuite/ChangeLog | 5 +++++ gcc/testsuite/gcc.c-torture/compile/pr22269.c | 16 ++++++++++++++++ gcc/tree-ssa-reassoc.c | 2 +- 4 files changed, 28 insertions(+), 1 deletion(-) create mode 100644 gcc/testsuite/gcc.c-torture/compile/pr22269.c diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 70be4195356..8236a2fd350 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2005-07-01 Andrew Pinski + + PR tree-opt/22269 + * tree-ssa-reassoc.c (should_transpose): Fix which operand + we check for SSA_NAME for. + 2005-06-29 Daniel Berlin Fix PR tree-optimization/22071 diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 47001d30381..32ef7e1e213 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2005-07-01 Andrew Pinski + + PR tree-opt/22269 + * gcc.c-torture/compile/pr22269.c: New test. + 2005-06-30 Diego Novillo 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 00000000000..bfcf209266c --- /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 5f0e8fc6187..395c5495455 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)