Patchwork [MIPS,committed] Remove 64-bit microMIPS multilibs from mips*-sde-elf

login
register
mail settings
Submitter Richard Sandiford
Date May 19, 2013, 9:51 a.m.
Message ID <8738tjqova.fsf@talisman.default>
Download mbox | patch
Permalink /patch/244796/
State New
Headers show

Comments

Richard Sandiford - May 19, 2013, 9:51 a.m.
The microMIPS patch added microMIPS multilibs to mips*-sde-elf.  The 32-bit
ones build fine, but the 64-bit ones don't, at least not with FSF binutils.
My impression from:

    http://www.cygwin.com/ml/binutils/2010-07/msg00405.html

was that this is expected -- 32-bit has been extensively tested, but the
64-bit side was more theoretical at this stage.

This patch therefore disables the 64-bit microMIPS multilibs,
as is already done for MIPS16.  Tested on mips*-sde-elf and applied.
We should consider reverting it once 64-bit microMIPS is known to work
and be useful for at least one core.

Thanks,
Richard


gcc/
	* config/mips/t-sde: Don't build 64-bit microMIPS multilibs.

Patch

Index: gcc/config/mips/t-sde
===================================================================
--- gcc/config/mips/t-sde	2013-03-20 21:01:21.674584349 +0000
+++ gcc/config/mips/t-sde	2013-05-18 20:47:11.293525699 +0100
@@ -27,9 +27,11 @@  else
 MULTILIB_EXCLUSIONS := !mips32r2/mfp64
 endif
 
-# Don't build 64-bit MIPS16 multilibs.
+# Don't build 64-bit MIPS16 and microMIPS multilibs.
 ifneq ($(filter MIPS_ISA_DEFAULT=6%,$(tm_defines)),)
 MULTILIB_EXCLUSIONS += !mips32/!mips32r2/mips16
+MULTILIB_EXCLUSIONS += !mips32/!mips32r2/mmicromips
 else
 MULTILIB_EXCLUSIONS += mips64/mips16 mips64r2/mips16
+MULTILIB_EXCLUSIONS += mips64/mmicromips mips64r2/mmicromips
 endif