From 8d8c2abc14d7727e10fbe5437134f57ecb54cf72 Mon Sep 17 00:00:00 2001 From: jiez Date: Mon, 12 Jul 2010 14:02:08 +0000 Subject: [PATCH] * config/arm/arm.c (arm_get_frame_offsets): Don't use r3 to align the stack when it's going to be saved. testsuite/ * gcc.target/arm/interrupt-1.c: New test. * gcc.target/arm/interrupt-2.c: New test. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@162078 138bc75d-0d04-0410-961f-82ee72b054a4 --- gcc/ChangeLog | 5 +++++ gcc/config/arm/arm.c | 3 ++- gcc/testsuite/ChangeLog | 5 +++++ gcc/testsuite/gcc.target/arm/interrupt-1.c | 23 +++++++++++++++++++++++ gcc/testsuite/gcc.target/arm/interrupt-2.c | 26 ++++++++++++++++++++++++++ 5 files changed, 61 insertions(+), 1 deletion(-) create mode 100644 gcc/testsuite/gcc.target/arm/interrupt-1.c create mode 100644 gcc/testsuite/gcc.target/arm/interrupt-2.c diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 47a87affe3f..5a446a9da08 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2010-07-12 Jie Zhang + + * config/arm/arm.c (arm_get_frame_offsets): Don't use r3 to + align the stack when it's going to be saved. + 2010-07-12 Rainer Orth PR pch/14940 diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c index db4f701e068..aa0f9cc7590 100644 --- a/gcc/config/arm/arm.c +++ b/gcc/config/arm/arm.c @@ -14691,7 +14691,8 @@ arm_get_frame_offsets (void) generates better code on Thumb-2 by avoiding the need to use 32-bit push/pop instructions. */ if (!crtl->tail_call_emit - && arm_size_return_regs () <= 12) + && arm_size_return_regs () <= 12 + && (offsets->saved_regs_mask & (1 << 3)) == 0) { reg = 3; } diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 5957c707bf8..7d0fb14c001 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2010-07-12 Jie Zhang + + * gcc.target/arm/interrupt-1.c: New test. + * gcc.target/arm/interrupt-2.c: New test. + 2010-07-12 Rainer Orth * gcc.dg/pr42427.c: Require c99_runtime. diff --git a/gcc/testsuite/gcc.target/arm/interrupt-1.c b/gcc/testsuite/gcc.target/arm/interrupt-1.c new file mode 100644 index 00000000000..18379de33d0 --- /dev/null +++ b/gcc/testsuite/gcc.target/arm/interrupt-1.c @@ -0,0 +1,23 @@ +/* Verify that prologue and epilogue are correct for functions with + __attribute__ ((interrupt)). */ +/* { dg-do compile } */ +/* { dg-options "-O0" } */ + +/* This test is not valid when -mthumb. We just cheat. */ +#ifndef __thumb__ +extern void bar (int); +extern void foo (void) __attribute__ ((interrupt("IRQ"))); + +void foo () +{ + bar (0); +} +#else +void foo () +{ + asm ("stmfd\tsp!, {r0, r1, r2, r3, r4, fp, ip, lr}"); + asm ("ldmfd\tsp!, {r0, r1, r2, r3, r4, fp, ip, pc}^"); +} +#endif +/* { dg-final { scan-assembler "stmfd\tsp!, {r0, r1, r2, r3, r4, fp, ip, lr}" } } */ +/* { dg-final { scan-assembler "ldmfd\tsp!, {r0, r1, r2, r3, r4, fp, ip, pc}\\^" } } */ diff --git a/gcc/testsuite/gcc.target/arm/interrupt-2.c b/gcc/testsuite/gcc.target/arm/interrupt-2.c new file mode 100644 index 00000000000..b979bf17e8a --- /dev/null +++ b/gcc/testsuite/gcc.target/arm/interrupt-2.c @@ -0,0 +1,26 @@ +/* Verify that prologue and epilogue are correct for functions with + __attribute__ ((interrupt)). */ +/* { dg-do compile } */ +/* { dg-options "-O1" } */ + +/* This test is not valid when -mthum. We just cheat. */ +#ifndef __thumb__ +extern void bar (int); +extern void test (void) __attribute__((__interrupt__)); + +int foo; +void test() +{ + funcptrs(foo); + foo = 0; +} +#else +void test () +{ + asm ("stmfd\tsp!, {r0, r1, r2, r3, r4, r5, ip, lr}"); + asm ("ldmfd\tsp!, {r0, r1, r2, r3, r4, r5, ip, pc}^"); +} +#endif + +/* { dg-final { scan-assembler "stmfd\tsp!, {r0, r1, r2, r3, r4, r5, ip, lr}" } } */ +/* { dg-final { scan-assembler "ldmfd\tsp!, {r0, r1, r2, r3, r4, r5, ip, pc}\\^" } } */ -- 2.11.4.GIT