Hallo! First, Joseph, thanks for working on this clean-up! I'm sorry for the delay, but I have now reviewed all these patches (including those you have committed by now); looks all good.
On Tue, 12 Apr 2011 18:37:50 +0000 (UTC), "Joseph S. Myers" <jos...@codesourcery.com> wrote: > This patch stops targets not using the Linux kernel from using > linux*.h config headers. [...] > 2011-04-12 Joseph Myers <jos...@codesourcery.com> > > * config.gcc (*-*-kfreebsd*-gnu | *-*-knetbsd*-gnu | *-*-gnu* | > *-*-kopensolaris*-gnu): Don't define SINGLE_LIBC. > (i[34567]86-*-kfreebsd*-gnu | i[34567]86-*-knetbsd*-gnu | > i[34567]86-*-gnu* | i[34567]86-*-kopensolaris*-gnu, > x86_64-*-kfreebsd*-gnu | x86_64-*-knetbsd*-gnu): Don't use > linux*.h headers. > * config/gnu-user.h (TARGET_C99_FUNCTIONS, TARGET_HAS_SINCOS): > Define. > * config/i386/gnu.h (MD_UNWIND_SUPPORT): Don't undefine. > * config/i386/kfreebsd-gnu.h (MD_UNWIND_SUPPORT): Don't undefine. > * config/i386/knetbsd-gnu.h (MD_UNWIND_SUPPORT): Don't undefine. > * config/i386/kopensolaris-gnu.h (MD_UNWIND_SUPPORT): Don't > undefine. > * config/i386/linux-unwind.h (x86_fallback_frame_state): Don't use > REG_NAME. > * config/i386/linux.h (REG_NAME): Don't define. > * config/i386/linux64.h (REG_NAME): Don't define. > * config/linux.h (TARGET_C99_FUNCTIONS, TARGET_HAS_SINCOS): > Undefine before defining. Looks good, too. I have not yet built-tested this (and it's too late by now), but I'll do that in the next days. One (minor) additional idea: for clarity, shouldn't it now be possible to remove any #undef GNU_USER_TARGET_OS_CPP_BUILTINS, #undef GNU_USER_DYNAMIC_LINKER, #undef GNU_USER_LINK_EMULATION, etc. from *gnu.h? I don't expect you to test this; I'll have a look later on. I'll also test whether gnu.h:LIB_SPEC is still needed, or if the gnu-user.h definition contains all we need. Grüße, Thomas
pgpUo74PLWe9F.pgp
Description: PGP signature