On Wed, Jul 16, 2014 at 05:18:06AM -0400, David Edelsohn wrote:
> > This passed bootstrap and regtesting on powerpc64-linux with no regressions.
> > Ok for mainline?
> >
> > Peter
> >
> >         * config.gcc (powerpc*-*-linux*): Include gnu-user.h in tm_file.
> >         * config/rs6000/sysv4.h (CC!_SPEC): Undefine it before defining it.
> 
> Typo in ChangeLog (CC!)?
> 
> This seems weird. Why wasn't this file included before or whenever it
> was added for other *-linux targets?  This seems to define SPECs that
> should have been necessary before now.

All other Linux targets where asan is supported got the right definitions
from gnu-user.h, it was needed even on the older release branches.

As including gnu-user.h there might be too risky for the release branches,
perhaps it would be better to copy the LIB[AT]SAN* macros from gnu-user.h
to say rs6000/linux.h or rs6000/linux64.h on the release branches (and in 4.8 
also
STATIC_LIB[AT]SAN_LIBS).

        Jakub

Reply via email to