Re: CVS commit: src/external/gpl3/gcc/dist/gcc/config/i386

[David Laight <david%l8s.co.uk@localhost>]

On Fri, Sep 14, 2012 at 01:00:01PM +0000, Joerg Sonnenberger wrote:
> Module Name:  src
> Committed By: joerg
> Date:         Fri Sep 14 13:00:01 UTC 2012
> 
> Modified Files:
>       src/external/gpl3/gcc/dist/gcc/config/i386: i386.c netbsd-elf.h
>           netbsd64.h
> 
> Log Message:
> Fix GCC to correctly implement the i386 psABI on NetBSD.
> This consists of two parts:
> - for NetBSD/i386 and NetBSD/AMD64 with -m32, use a default stack
>   alignment of 23bit as specified by the ABI
> - ensure that double and long long variables on the stack are by default
>   only aligned to 32bit, if there is nothing else in the function
>   needing a larger stack alignment
> 
> The combination ensures that SSE variables on the stack trigger the
> realignment logic, but just using double or long long doesn't.
> 
> 
> To generate a diff of this commit:
> cvs rdiff -u -r1.1.1.1 -r1.2 \
>     src/external/gpl3/gcc/dist/gcc/config/i386/i386.c \
>     src/external/gpl3/gcc/dist/gcc/config/i386/netbsd64.h
> cvs rdiff -u -r1.2 -r1.3 \
>     src/external/gpl3/gcc/dist/gcc/config/i386/netbsd-elf.h
> 
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.

I'm still to be convinced that the patches are correct.

        David

-- 
David Laight: david%l8s.co.uk@localhost