This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] [PATCH][ARM] Fix thumb-ltu.c testcase.
- From: Ramana Radhakrishnan <ramana dot radhakrishnan at arm dot com>
- To: Alex Velenko <Alex dot Velenko at arm dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 01 Jun 2015 10:50:31 +0100
- Subject: Re: [PATCH] [PATCH][ARM] Fix thumb-ltu.c testcase.
- Authentication-results: sourceware.org; auth=none
- References: <1433152137-12570-1-git-send-email-alex dot velenko at arm dot com>
On 01/06/15 10:48, Alex Velenko wrote:
Hi,
This patch fix thumb-ltu.c to pass excess error test.
Without default -std=gnu90 flag, this testcase started failing
as some functions were called before being predefined.
Is patch ok?
gcc/testsuite
2015-06-01 Alex Velenko <Alex.Velenko@arm.com>
* gcc.target/arm/thumb-ltu.c (foo): Predefined.
(bar): Predefined.
---
gcc/testsuite/gcc.target/arm/thumb-ltu.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/gcc/testsuite/gcc.target/arm/thumb-ltu.c b/gcc/testsuite/gcc.target/arm/thumb-ltu.c
index d057ea3..124b025 100644
--- a/gcc/testsuite/gcc.target/arm/thumb-ltu.c
+++ b/gcc/testsuite/gcc.target/arm/thumb-ltu.c
@@ -2,6 +2,9 @@
/* { dg-require-effective-target arm_thumb1_ok } */
/* { dg-options "-mcpu=arm1136jf-s -mthumb -O2" } */
+int foo();
+int bar();
+
Surely this is,
extern int foo (void);
extern int bar (void);
void f(unsigned a, unsigned b, unsigned c, unsigned d)
{
if (a <= b || c > d)
OK with that change.
Ramana