Re: [PATCH] [PATCH][ARM] Fix thumb-ltu.c testcase.

[Ramana Radhakrishnan <ramana.radhakrishnan@arm.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