On 29/07/14 10:21, Christophe Lyon wrote:
Hi all,


On 23 July 2014 11:26, Kyrill Tkachov <kyrylo.tkac...@arm.com> wrote:
Hi all,

I've committed this on Jiongs' behalf as r212927 with some minor ChangeLog
modifications to make it fit into the 80 character column limit:

2014-07-23  Jiong Wang  <jiong.w...@arm.com>


     * config/arm/arm.c (arm_get_frame_offsets): If both r3 and other
     callee-saved registers are available for padding purpose
     and r3 is not mandatory, then prefer use those callee-saved
     instead of r3.

2014-07-23  Jiong Wang  <jiong.w...@arm.com>

     * gcc.dg/ira-shrinkwrap-prep-1.c (target): Add arm_nothumb.
     * gcc.dg/ira-shrinkwrap-prep-2.c (target): Likewise.
     * gcc.dg/pr10474.c (target): Likewise.


Kyrill

Since this commit, I've noticed a few regressions:

   gcc.target/arm/pr43920-2.c object-size text <= 54
   gcc.target/arm/pr45701-1.c scan-assembler push\t{r3
   gcc.target/arm/pr45701-1.c scan-assembler-not r8
   gcc.target/arm/pr45701-2.c scan-assembler push\t{r3
   gcc.target/arm/pr45701-2.c scan-assembler-not r8

(targets arm-none-linux-gnueabi, arm-none-linux-gnueabihf).

Note that these tests still PASS if one forces -marm or -march=arm-v5t
in the multilib flags, or if the compiler is configured
--with-cpu=cortex-a57 --with-arch=armv8.

Hi Christophe,

  Indeed. I noticed these regressions also, and just posted a fix, maybe
  during you writing this email:)

  Thanks.

-- Jiong


Christophe.


thanks.

-- Jiong

gcc/
    * config/arm/arm.c (arm_get_frame_offsets): If both r3 and other
callee-saved
      registers are available for padding purpose, and r3 is not
mandatory,
then
      prefer use those callee-saved instead of r3.

gcc/testsuite/
    * gcc.dg/ira-shrinkwrap-prep-1.c (target): Add arm_nothumb
    * gcc.dg/ira-shrinkwrap-prep-2.c (target): Add arm_nothumb
    * gcc.dg/pr10474.c (target): Add arm_nothumb




Reply via email to