diff options
| author | Ramana Radhakrishnan <ramana.radhakrishnan@arm.com> | 2014-04-22 10:05:48 +0000 |
|---|---|---|
| committer | Marcus Shawcroft <mshawcroft@gcc.gnu.org> | 2014-04-22 10:05:48 +0000 |
| commit | 9d8b4d1ce91511ae92cd746c10a05e16215f1940 (patch) | |
| tree | 922c651f349f67863242079c3a242cc17fd315a4 /gcc | |
| parent | a01be1aeb2df00d1af7dfca6f3355db1a12a25c8 (diff) | |
| download | gcc-9d8b4d1ce91511ae92cd746c10a05e16215f1940.zip gcc-9d8b4d1ce91511ae92cd746c10a05e16215f1940.tar.gz gcc-9d8b4d1ce91511ae92cd746c10a05e16215f1940.tar.bz2 | |
[ARM] Allow any register for DImode values in Thumb2
Ramana commented in the submission email:
I noticed that for T32 we don't allow any old register for DImode
values. The restriction of an even register is true only for ARM state
because the ISA doesn't allow any old register in this place. In a few
large .i files that I had knocking about, noticed a nice drop in stack
usage and a generally improved register allocation strategy.
From-SVN: r209615
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/config/arm/arm.c | 17 |
2 files changed, 17 insertions, 5 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index e640faf..80ffa58 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2014-04-22 Ramana Radhakrishnan <ramana.radhakrishnan@arm.com> + + * config/arm/arm.c (arm_hard_regno_mode_ok): Loosen + restrictions on core registers for DImode values in Thumb2. + 2014-04-22 Ian Bolton <ian.bolton@arm.com> * config/arm/arm.md (*anddi_notdi_zesidi): New pattern. diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c index 08b5255..88d957a 100644 --- a/gcc/config/arm/arm.c +++ b/gcc/config/arm/arm.c @@ -22646,12 +22646,19 @@ arm_hard_regno_mode_ok (unsigned int regno, enum machine_mode mode) } /* We allow almost any value to be stored in the general registers. - Restrict doubleword quantities to even register pairs so that we can - use ldrd. Do not allow very large Neon structure opaque modes in - general registers; they would use too many. */ + Restrict doubleword quantities to even register pairs in ARM state + so that we can use ldrd. Do not allow very large Neon structure + opaque modes in general registers; they would use too many. */ if (regno <= LAST_ARM_REGNUM) - return !(TARGET_LDRD && GET_MODE_SIZE (mode) > 4 && (regno & 1) != 0) - && ARM_NUM_REGS (mode) <= 4; + { + if (ARM_NUM_REGS (mode) > 4) + return FALSE; + + if (TARGET_THUMB2) + return TRUE; + + return !(TARGET_LDRD && GET_MODE_SIZE (mode) > 4 && (regno & 1) != 0); + } if (regno == FRAME_POINTER_REGNUM || regno == ARG_POINTER_REGNUM) |