From d5418ee66e732da1b227c076431382f813af3b3c Mon Sep 17 00:00:00 2001 From: Vladimir Makarov Date: Fri, 24 Mar 2017 18:47:38 +0000 Subject: 2017-03-24 Vladimir Makarov PR target/80148 * lra-assigns.c (assign_by_spills): Add spilled non-reload pseudos to consider in curr_insn_transform. git-svn-id: https://gcc.gnu.org/svn/gcc/trunk@246467 138bc75d-0d04-0410-961f-82ee72b054a4 --- gcc/ChangeLog | 6 ++++++ gcc/lra-assigns.c | 8 ++++++++ 2 files changed, 14 insertions(+) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index f82afbe9ab4..adf3791dfd9 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2017-03-24 Vladimir Makarov + + PR target/80148 + * lra-assigns.c (assign_by_spills): Add spilled non-reload pseudos + to consider in curr_insn_transform. + 2017-03-24 Jakub Jelinek * genrecog.c (validate_pattern): Add VEC_SELECT validation. diff --git a/gcc/lra-assigns.c b/gcc/lra-assigns.c index 226601bf2e6..75bb630dca7 100644 --- a/gcc/lra-assigns.c +++ b/gcc/lra-assigns.c @@ -1507,6 +1507,14 @@ assign_by_spills (void) sorted_pseudos[nfails++] = conflict_regno; former_reload_pseudo_spill_p = true; } + else + /* It is better to do reloads before spilling as after the + spill-subpass we will reload memory instead of pseudos + and this will make reusing reload pseudos more + complicated. Going directly to the spill pass in such + case might result in worse code performance or even LRA + cycling if we have few registers. */ + bitmap_set_bit (&all_spilled_pseudos, conflict_regno); if (lra_dump_file != NULL) fprintf (lra_dump_file, " Spill %s r%d(hr=%d, freq=%d)\n", pseudo_prefix_title (conflict_regno), conflict_regno, -- cgit v1.2.3