This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch, mips] Patch for new mips triplet - mips-mti-elf
- From: Richard Sandiford <rdsandiford at googlemail dot com>
- To: "Steve Ellcey " <sellcey at mips dot com>
- Cc: <gcc-patches at gcc dot gnu dot org>
- Date: Sun, 16 Sep 2012 09:22:32 +0100
- Subject: Re: [patch, mips] Patch for new mips triplet - mips-mti-elf
- References: <3324456b-606c-494e-93ac-2479868f750d@EXCHHUB01.MIPS.com>
"Steve Ellcey " <sellcey@mips.com> writes:
> +#undef SYSROOT_SUFFIX_SPEC
> +#define SYSROOT_SUFFIX_SPEC \
> + "%{mips32:/mips32}%{mips64:/mips64}%{mips64r2:/mips64r2}%{mips16:/mips16}%{msoft-float:/sof}%{mel|EL:/el}%{mabi=64:/64}%{mabi=n32:/n32}"
This shouldn't be needed for *-elf targets.
> +/* This is idential to sde.h except for the ABI setting which defaults
> + to 32 instead of n32 on 32 bit architectures and the addition of
> + MIPS_ISA_SYNCI_SPEC. */
SDE defaults to o32 (-mabi=32) for 32-bit targets and n32 for 64-bit targets.
I think that's what you'd want here too, otherwise the mips64 multilibs
are really no different from the mips32 ones.
> + /* Remove a redundant -mfp64 for -mabi=n32; we want the !mfp64 \
> + multilibs. There's no need to check whether the architecture \
> + is 64-bit; cc1 will complain if it isn't. */ \
> + "%{mabi=n32: %<mfp64}", \
This bit shouldn't be needed, since there are no mfp64 multilibs.
Richard