Hi! On Thu, 28 Jun 2012 12:42:23 +0200, Matthias Klose <d...@ubuntu.com> wrote: > On 28.06.2012 12:01, Thomas Schwinge wrote: > > On Mon, 25 Jun 2012 18:19:26 +0200, Matthias Klose <d...@ubuntu.com> > > wrote: > >> On 25.06.2012 15:56, Joseph S. Myers wrote: > >>> On Mon, 25 Jun 2012, Matthias Klose wrote: > >>> > >>>> Please find attached the patch updated for trunk 20120625, x86 only, > >>>> tested on x86-linux-gnu, KFreeBSD and the Hurd. > > > >> 2012-06-25 Matthias Klose <d...@ubuntu.com> > >> > >> * doc/invoke.texi: Document -print-multiarch. * doc/install.texi: > >> Document --enable-multiarch. * doc/fragments.texi: Document > >> MULTILIB_OSDIRNAMES, MULTIARCH_DIRNAME. * configure.ac: Add > >> --enable-multiarch option. * configure.in: Regenerate. * Makefile.in > >> (s-mlib): Pass MULTIARCH_DIRNAME to genmultilib. enable_multiarch, > >> with_float: New macros. if_multiarch: New macro, define in terms of > >> enable_multiarch. * genmultilib: Add new argument for the multiarch > >> name. * gcc.c (multiarch_dir): Define. (for_each_path): Search for > >> multiarch suffixes. (driver_handle_option): Handle multiarch option. > >> (do_spec_1): Pass -imultiarch if defined. (main): Print multiarch. > >> (set_multilib_dir): Separate multilib and multiarch names from > >> multilib_select. (print_multilib_info): Ignore multiarch names in > >> multilib_select. * incpath.c (add_standard_paths): Search the multiarch > >> include dirs. * cppdeault.h (default_include): Document multiarch in > >> multilib member. * cppdefault.c: [LOCAL_INCLUDE_DIR, > >> STANDARD_INCLUDE_DIR] Add an include directory for multiarch > >> directories. * common.opt: New options --print-multiarch and -imultilib. > >> * config.gcc: Add tmake fragments to tmake_file ( i386/t-kfreebsd for > >> i[34567]86-*-kfreebsd*-gnu and x86_64-*-kfreebsd*-gnu, i386/t-gnu for > >> i[34567]86-*-gnu*). * config/i386/t-kfreebsd: Add multiarch names in > >> MULTILIB_OSDIRNAMES, define MULTIARCH_DIRNAME. * config/i386/t-linux64: > >> Likewise. * config/i386/t-linux: Define MULTIARCH_DIRNAME. * > >> config/i386/t-gnu: Likewise. > > > > As I said before, »config/i386/t-{gnu,kfreebsd,linux}« are new files. > > Instead of repeating: my comments from > > <http://news.gmane.org/find-root.php?message_id=%3C87zk94cg1h.fsf%40schwinge.name%3E> > > > > > as well as the follow-up still hold. > > Like > > * config/i386/t-gnu: New, define MULTIARCH_DIRNAME. > > ?
I'd use: * config/i386/t-gnu: New file. * config/i386/t-kfreebsd: Likewise. * config/i386/t-linux: Likewise. Plus the following instead of your changes: gcc/ * config.gcc <i[34567]86-*-linux* | x86_64-*-linux*> (tmake_file): Include i386/t-linux. <i[34567]86-*-kfreebsd*-gnu | x86_64-*-kfreebsd*-gnu> (tmake_file): Include i386/t-kfreebsd. <i[34567]86-*-gnu*> (tmake_file): Include i386/t-gnu. diff --git a/gcc/config.gcc b/gcc/config.gcc index 7ec184c..39c70f2 100644 --- a/gcc/config.gcc +++ b/gcc/config.gcc @@ -3481,9 +3481,14 @@ case ${target} in i[34567]86-*-darwin* | x86_64-*-darwin*) ;; - i[34567]86-*-linux* | x86_64-*-linux* | \ - i[34567]86-*-kfreebsd*-gnu | x86_64-*-kfreebsd*-gnu | \ - i[34567]86-*-gnu*) + i[34567]86-*-linux* | x86_64-*-linux*) + tmake_file="$tmake_file i386/t-linux" + ;; + i[34567]86-*-kfreebsd*-gnu | x86_64-*-kfreebsd*-gnu) + tmake_file="$tmake_file i386/t-kfreebsd" + ;; + i[34567]86-*-gnu*) + tmake_file="$tmake_file i386/t-gnu" ;; i[34567]86-*-solaris2* | x86_64-*-solaris2.1[0-9]*) ;; Otherwise, I can't imagine how that would work. Grüße, Thomas
pgpJmqjTH8LJD.pgp
Description: PGP signature