diff options
| author | Segher Boessenkool <segher@kernel.crashing.org> | 2018-11-11 00:12:48 +0100 |
|---|---|---|
| committer | Segher Boessenkool <segher@gcc.gnu.org> | 2018-11-11 00:12:48 +0100 |
| commit | 16ac6fa5a0537d201abb269ccee396ccb407a770 (patch) | |
| tree | 9dd244f9dddcc99a6057cc50ffa101a39b66adad | |
| parent | babe0c748d341aaa2d18fb901ae3e6eb8af4da1a (diff) | |
| download | gcc-16ac6fa5a0537d201abb269ccee396ccb407a770.zip gcc-16ac6fa5a0537d201abb269ccee396ccb407a770.tar.gz gcc-16ac6fa5a0537d201abb269ccee396ccb407a770.tar.bz2 | |
combine: More make_more_copies
This makes make_more_copies do what its documentation says, that is,
only make an intermediate pseudo if copying to a pseudo.
This regressed generated code quality when we didn't keep the original
notes that were on the copy, but since r265582 we do, and only allowing
pseudos now is a win. It also simplifies the code.
* combine.c (make_more_copies): Only make an intermediate copy if the
dest of a move is a pseudo.
From-SVN: r266004
| -rw-r--r-- | gcc/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/combine.c | 9 |
2 files changed, 9 insertions, 5 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 6b30ada..51c7e42 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2018-11-10 Segher Boessenkool <segher@kernel.crashing.org> + + * combine.c (make_more_copies): Only make an intermediate copy if the + dest of a move is a pseudo. + 2018-11-09 Maya Rashish <coypu@sdf.org> PR target/87221 diff --git a/gcc/combine.c b/gcc/combine.c index 93bd3da..945eca3 100644 --- a/gcc/combine.c +++ b/gcc/combine.c @@ -14956,12 +14956,11 @@ make_more_copies (void) rtx set = single_set (insn); if (!set) continue; + rtx dest = SET_DEST (set); - if (dest == pc_rtx) - continue; - /* See PR87871. */ - if (dest == frame_pointer_rtx) - continue; + if (!(REG_P (dest) && !HARD_REGISTER_P (dest))) + continue; + rtx src = SET_SRC (set); if (!(REG_P (src) && HARD_REGISTER_P (src))) continue; |