From 82396b8cfb651c9d7ae108d33648ff5cb7b9bce0 Mon Sep 17 00:00:00 2001 From: Vladimir Makarov Date: Fri, 8 Feb 2019 19:01:10 +0000 Subject: [PATCH] re PR middle-end/88560 (armv8_2-fp16-move-1.c and related regressions after r266385) 2019-02-08 Vladimir Makarov PR middle-end/88560 * lra-constraints.c (process_alt_operands): Don't increase reject for memory when offset memory is required. From-SVN: r268705 --- gcc/ChangeLog | 6 ++++++ gcc/lra-constraints.c | 43 +++++++++++++++++++++++-------------------- 2 files changed, 29 insertions(+), 20 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 7bdcc07def1..9e5a140b0e1 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2019-02-08 Vladimir Makarov + + PR middle-end/88560 + * lra-constraints.c (process_alt_operands): Don't increase reject + for memory when offset memory is required. + 2019-02-08 Robin Dapp * config/s390/vector.md: Implement vector copysign. diff --git a/gcc/lra-constraints.c b/gcc/lra-constraints.c index 3b975cc2c9a..17ea2c53bbc 100644 --- a/gcc/lra-constraints.c +++ b/gcc/lra-constraints.c @@ -2796,29 +2796,32 @@ process_alt_operands (int only_alternative) (GET_MODE (op), this_alternative, cl))))) losers++; - /* Input reloads can be inherited more often than output - reloads can be removed, so penalize output - reloads. */ - if (!REG_P (op) || curr_static_id->operand[nop].type != OP_IN) - { - if (lra_dump_file != NULL) - fprintf - (lra_dump_file, - " %d Non input pseudo reload: reject++\n", - nop); - reject++; - } - if (MEM_P (op) && offmemok) addr_losers++; - else if (curr_static_id->operand[nop].type == OP_INOUT) + else { - if (lra_dump_file != NULL) - fprintf - (lra_dump_file, - " %d Input/Output reload: reject+=%d\n", - nop, LRA_LOSER_COST_FACTOR); - reject += LRA_LOSER_COST_FACTOR; + /* Input reloads can be inherited more often than + output reloads can be removed, so penalize output + reloads. */ + if (!REG_P (op) || curr_static_id->operand[nop].type != OP_IN) + { + if (lra_dump_file != NULL) + fprintf + (lra_dump_file, + " %d Non input pseudo reload: reject++\n", + nop); + reject++; + } + + if (curr_static_id->operand[nop].type == OP_INOUT) + { + if (lra_dump_file != NULL) + fprintf + (lra_dump_file, + " %d Input/Output reload: reject+=%d\n", + nop, LRA_LOSER_COST_FACTOR); + reject += LRA_LOSER_COST_FACTOR; + } } }