diff options
author | Vladimir Makarov <vmakarov@redhat.com> | 2017-03-24 18:47:38 +0000 |
---|---|---|
committer | Vladimir Makarov <vmakarov@redhat.com> | 2017-03-24 18:47:38 +0000 |
commit | d5418ee66e732da1b227c076431382f813af3b3c (patch) | |
tree | e7c09a30300cf2cb2cf94d0f5e7329a635a8b1a3 | |
parent | 2389170ad6d9339291dadbf22bc29edc2ff49aae (diff) |
2017-03-24 Vladimir Makarov <vmakarov@redhat.com>
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
-rw-r--r-- | gcc/ChangeLog | 6 | ||||
-rw-r--r-- | gcc/lra-assigns.c | 8 |
2 files changed, 14 insertions, 0 deletions
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 <vmakarov@redhat.com> + + 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 <jakub@redhat.com> * 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, |