aboutsummaryrefslogtreecommitdiff
path: root/gcc/ira.c
diff options
context:
space:
mode:
authorOmar Tahir <omar.tahir@arm.com>2020-07-09 10:14:19 +0100
committerRichard Sandiford <richard.sandiford@arm.com>2020-07-09 10:14:19 +0100
commitedf95e51e53697f3050f076675c26a4cece17741 (patch)
tree5197c9858a0ea13331a4b0d246500dc7a2bc0975 /gcc/ira.c
parent319078dad62eba942d33c8975bdcbb09d1c68ba6 (diff)
downloadgcc-edf95e51e53697f3050f076675c26a4cece17741.zip
gcc-edf95e51e53697f3050f076675c26a4cece17741.tar.gz
gcc-edf95e51e53697f3050f076675c26a4cece17741.tar.bz2
ira: Fix unnecessary register spill
The variables first_moveable_pseudo and last_moveable_pseudo aren't reset after compiling a function, which means they leak into the first scheduler pass of the following function. In some cases, this can cause an extra spill during register allocation of the second function. gcc/ChangeLog: * ira.c (move_unallocated_pseudos): Zero first_moveable_pseudo and last_moveable_pseudo before returning. gcc/testsuite/ChangeLog: * gcc.target/aarch64/nospill.c: New test.
Diffstat (limited to 'gcc/ira.c')
-rw-r--r--gcc/ira.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/gcc/ira.c b/gcc/ira.c
index a655ae1..b748a6d 100644
--- a/gcc/ira.c
+++ b/gcc/ira.c
@@ -5126,6 +5126,8 @@ move_unallocated_pseudos (void)
INSN_UID (newinsn), i);
SET_REG_N_REFS (i, 0);
}
+
+ first_moveable_pseudo = last_moveable_pseudo = 0;
}
/* If the backend knows where to allocate pseudos for hard