[avr-gcc-list] Re: Updates needed to AVR test files

[hutchinsonandy]

Im rather busy with GCC ugs/patches.

I was hoping to get a volunteer!

Note I have also seen "exit" with warning that this noreturn function 
will return. (ie declaration mismatches with stdio.h wrt to noreturn)

I would be grateful if someone could fix that too.

Any warning emiited by gcc can trigger a testsuite failure, so it has 
to be squeaky clean !

best regards



----------------------------------------------
Sent from my Dingleberry wired device.


-----Original Message-----
From: Weddington, Eric <address@hidden>
To: Andy H <address@hidden>; AVR-GCC <address@hidden>; 
Paulo Marques <address@hidden>; Mike Stein 
<address@hidden>; Anatoly Sokolov <address@hidden>
Sent: Mon, 2 Jun 2008 11:59 am
Subject: RE: Updates needed to AVR test files





> -----Original Message-----
> From: Andy H [mailto:address@hidden
> Sent: Friday, May 30, 2008 8:59 PM
> To: AVR-GCC; Paulo Marques; Mike Stein; Weddington, Eric;
> Anatoly Sokolov
> Subject: Updates needed to AVR test files
>
> There are a couple of changes needed to AVR test files to
> pass a few tests.
>
> Compatibility tests default to no optimization and maximum
> tests - this
> can easily overflow 128K code area.
> Add these lines to end board file (mine is called
> atmega128-simnew.exp).
> They set environment vars that control these tests and get
> many more to
> work. (Some still need other fixes).
>
> # Restrict compatibility tests. And optimise to reduce size.
> set COMPAT_SKIPS [list {VA} {COMPLEX_INTS}]
> set COMPAT_OPTIONS [list [list {-Os -mcall-prologues} {-Os
> -mcall-prologues}]]
>
> Dummy io/exit/abort  file exit.c  has unused parameter stream. The
> warning created  then causes a failure in some tests. Hack as
> follows to
> create
> dummy reference to stream, thus removing the warning.
>
> int putchar_exit_c(char c, FILE *stream)
> {
>     *((volatile unsigned char *) STDIO_PORT) = c;
>     stream = NULL;
>     return 0;
> }

Hi Andy,

Thanks for your work on this. Could you provide a patch for these
changes? I want to make sure that everything gets done correctly.

Paulo, Mike, Anatoly, do any of you see any problems with these changes?
If not, then I will commit them.

Eric