public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [RFA/arm] Fix gcc.target/arm/pr42835.c testcase
@ 2011-09-28 18:41 Matthew Gretton-Dann
  2011-09-28 18:42 ` Ramana Radhakrishnan
  0 siblings, 1 reply; 2+ messages in thread
From: Matthew Gretton-Dann @ 2011-09-28 18:41 UTC (permalink / raw)
  To: gcc-patches; +Cc: Ramana Radhakrishnan

[-- Attachment #1: Type: text/plain, Size: 659 bytes --]

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.

-- 
Matthew Gretton-Dann
Principal Engineer, PD Software - Tools, ARM Ltd

[-- Attachment #2: 1109-pr42835-fixup.txt --]
[-- Type: text/plain, Size: 437 bytes --]

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)

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [RFA/arm] Fix gcc.target/arm/pr42835.c testcase
  2011-09-28 18:41 [RFA/arm] Fix gcc.target/arm/pr42835.c testcase Matthew Gretton-Dann
@ 2011-09-28 18:42 ` Ramana Radhakrishnan
  0 siblings, 0 replies; 2+ messages in thread
From: Ramana Radhakrishnan @ 2011-09-28 18:42 UTC (permalink / raw)
  To: Matthew Gretton-Dann; +Cc: gcc-patches

> 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

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-09-28 16:35 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-09-28 18:41 [RFA/arm] Fix gcc.target/arm/pr42835.c testcase Matthew Gretton-Dann
2011-09-28 18:42 ` Ramana Radhakrishnan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).