On 01/06/15 10:50, Ramana Radhakrishnan wrote:
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.vele...@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
Committed with said change r223982. Is patch ok for fsf-5 backport? Alex
