Patchwork [RFA/arm] Fix gcc.target/arm/pr42835.c testcase

login
register
mail settings
Submitter Matthew Gretton-Dann
Date Sept. 28, 2011, 4:31 p.m.
Message ID <4E834BFC.1020205@arm.com>
Download mbox | patch
Permalink /patch/116837/
State New
Headers show

Comments

Matthew Gretton-Dann - Sept. 28, 2011, 4:31 p.m.
All,

gcc.target/arm/pr42835.c started failing as a result of the following
change which add tree-tail merging:
   http://gcc.gnu.org/viewcvs?view=revision&revision=179275

The behaviour of the testcase with tree-tail merging is correct, but not 
what is expected.

The attached patch adds -fno-tree-tail-merge to the test options.

Tested arm-none-eabi.

Can someone review and approve please?

Thanks,

Matt

gcc/testsuite/ChangeLog:

2011-09-28  Matthew Gretton-Dann  <matthew.gretton-dann@arm.com>

	* gcc.target/arm/pr42835.c: Add -fno-tree-tail-merge.
Ramana Radhakrishnan - Sept. 28, 2011, 4:34 p.m.
> gcc/testsuite/ChangeLog:
>
> 2011-09-28  Matthew Gretton-Dann  <matthew.gretton-dann@arm.com>
>
>        * gcc.target/arm/pr42835.c: Add -fno-tree-tail-merge.

This is OK.

Ramana

Patch

diff --git a/gcc/testsuite/gcc.target/arm/pr42835.c b/gcc/testsuite/gcc.target/arm/pr42835.c
index 71c51eb..867dd02 100644
--- a/gcc/testsuite/gcc.target/arm/pr42835.c
+++ b/gcc/testsuite/gcc.target/arm/pr42835.c
@@ -1,5 +1,5 @@ 
 /* { dg-do compile } */
-/* { dg-options "-mthumb -Os" }  */
+/* { dg-options "-mthumb -Os -fno-tree-tail-merge" }  */
 /* { dg-require-effective-target arm_thumb2_ok } */
 
 int foo(int *p, int i)