This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
RFA: Define MULTILIB_ABI_DEFAULT for mips64vr*-*-elf* targets
- From: Nick Clifton <nickc at redhat dot com>
- To: echristo at apple dot com, rdsandiford at googlemail dot com
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Fri, 17 Jul 2009 09:28:28 +0100
- Subject: RFA: Define MULTILIB_ABI_DEFAULT for mips64vr*-*-elf* targets
Hi Eric, Hi Richard,
This probably counts as an obvious patch, but I thought that I would
check with you first:
The mips64vr-*-elf* and mips64vrel-*-elf* targets make use of the
mips/vr.h header file. This redefines MIPS_ABI_DEFAULT to be
ABI_EABI. Unfortunately it does not, currently, redefine
MULTILIB_ABI_DEFAULT as well. This means that in the definition of
MULTILIB_DEFAULTS that follows the default abi is still set to
mips32 which means that the wrong set of multilibs are built.
Fixed by applying the patch below and tested by building a
mips64vrel-elf toolchain and running the gcc testsuite with a
-mabi=32 selector.
OK to apply ?
Cheers
Nick
gcc/ChangeLog
2009-07-16 Nick Clifton <nickc@redhat.com>
* config/mips/vr.h (MULTILIB_ABI_DEFAULT): Override.
Index: gcc/config/mips/vr.h
===================================================================
--- gcc/config/mips/vr.h (revision 149734)
+++ gcc/config/mips/vr.h (working copy)
@@ -22,6 +22,8 @@
#define DEFAULT_VR_ARCH "mfix-vr4130"
#undef MIPS_ABI_DEFAULT
#define MIPS_ABI_DEFAULT ABI_EABI
+#undef MULTILIB_ABI_DEFAULT
+#define MULTILIB_ABI_DEFAULT "mabi=eabi"
#undef MULTILIB_DEFAULTS
#define MULTILIB_DEFAULTS \
{ MULTILIB_ENDIAN_DEFAULT, \