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

[hutchinsonandy]

BTW Mike Stein has patched his as his latest compares are showing 
updated cflags on Compat tests. Way to go Mike!

The compat test are potentially VERY USEFUL. It has the ability to 
compare and link 2 different compilers. For example, gcc 4.3 with gcc 
4.4 to check ABI.

For now it compares same version - with same flags.

The remaining failures in compat are due to memory overflow (complex 
float vectors  with inline initialisation and checking) and ints <32 
bits.  Seem they forgot to allow any of these to be skipped or marked 
XFAIL. So I cant turn them off. Hopefully Janis will fix these in due 
course.


Andy



----------------------------------------------
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