diff options
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/lra-constraints.c | 8 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.target/i386/pr91223.c | 11 |
4 files changed, 30 insertions, 0 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 336e9ee..42beda4 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2019-07-25 Vladimir Makarov <vmakarov@redhat.com> + + PR rtl-optimization/91223 + * lra-constraints.c (process_alt_operands): Fail for unsuccessful + matching with INOUT operand. + 2019-07-25 Eric Botcazou <ebotcazou@adacore.com> * stmt.c (expand_case): Try to narrow the index type if it's larger diff --git a/gcc/lra-constraints.c b/gcc/lra-constraints.c index 55d8d13..bd9f85d 100644 --- a/gcc/lra-constraints.c +++ b/gcc/lra-constraints.c @@ -2171,6 +2171,14 @@ process_alt_operands (int only_alternative) } else { + /* If the operands do not match and one + operand is INOUT, we can not match them. + Try other possibilities, e.g. other + alternatives or commutative operand + exchange. */ + if (curr_static_id->operand[nop].type == OP_INOUT + || curr_static_id->operand[m].type == OP_INOUT) + break; /* Operands don't match. If the operands are different user defined explicit hard registers, then we cannot make them match diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 6fc4931..0ff528b 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2019-07-25 Vladimir Makarov <vmakarov@redhat.com> + + PR rtl-optimization/91223 + * gcc.target/i386/pr91223.c: New test. + 2019-07-25 Iain Sandoe <iain@sandoe.co.uk> PR gcov-profile/91087 diff --git a/gcc/testsuite/gcc.target/i386/pr91223.c b/gcc/testsuite/gcc.target/i386/pr91223.c new file mode 100644 index 0000000..1b9e85b --- /dev/null +++ b/gcc/testsuite/gcc.target/i386/pr91223.c @@ -0,0 +1,11 @@ +/* { dg-do compile } */ +/* { dg-options "-Og -fno-tree-fre" } */ +int a; +void fn2(short, short); + +void fn1(void) { + short b[8]; + b[0] |= a & 3; + b[1] = a; + fn2(b[0], b[1]); +} |