public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [Patch, MIPS] Remove some multilibs from mips-mti-linux-gnu target
@ 2014-12-24 17:24 Steve Ellcey 
  2014-12-29 10:07 ` Matthew Fortune
  0 siblings, 1 reply; 3+ messages in thread
From: Steve Ellcey  @ 2014-12-24 17:24 UTC (permalink / raw)
  To: gcc-patches; +Cc: clm, matthew.fortune

This patch removes mips32[r1] and mips64[r1] from the list of
mips-mti-linux-gnu architectures built with -mnan=2008.  They
should never have been included since these architectures do not
support the ieee 2008 format (GCC warns and ignores the nan flag).

The recent changes to glibc to use -Werror mean that the warning
GCC gives is treated as an error and that glibc will not build with
these flag combinations so I would like to remove them from the
GCC mips-mti-linux-gnu multilib list.

This does not affect any GCC target other then mips-mti-linux.gnu.

OK to checkin?

Steve Ellcey
sellcey@imgtec.com


2014-12-24  Steve Ellcey  <sellcey@mips.com>

	* config/mips/t-mti-linux (MULTILIB_EXCEPTIONS): Add exceptions
	for mips32[r1] and mips64[r1] with -mnan=2008.


diff --git a/gcc/config/mips/t-mti-linux b/gcc/config/mips/t-mti-linux
index 487a015..c111532 100644
--- a/gcc/config/mips/t-mti-linux
+++ b/gcc/config/mips/t-mti-linux
@@ -41,5 +41,8 @@ MULTILIB_EXCEPTIONS += *mips32/mmicromips*
 MULTILIB_EXCEPTIONS += *mips64*/mmicromips*
 MULTILIB_EXCEPTIONS += *mmicromips/mabi=64*
 
-# We do not want nan2008 libraries for soft-float.
+# We do not want nan2008 libraries for soft-float,
+# mips32[r1], or mips64[r1].
 MULTILIB_EXCEPTIONS += *msoft-float*/*mnan=2008*
+MULTILIB_EXCEPTIONS += *mips32/*mnan=2008*
+MULTILIB_EXCEPTIONS += *mips64/*mnan=2008*

^ permalink raw reply	[flat|nested] 3+ messages in thread

* RE: [Patch, MIPS] Remove some multilibs from mips-mti-linux-gnu target
  2014-12-24 17:24 [Patch, MIPS] Remove some multilibs from mips-mti-linux-gnu target Steve Ellcey 
@ 2014-12-29 10:07 ` Matthew Fortune
  2014-12-29 17:09   ` Steve Ellcey
  0 siblings, 1 reply; 3+ messages in thread
From: Matthew Fortune @ 2014-12-29 10:07 UTC (permalink / raw)
  To: Steve Ellcey, gcc-patches; +Cc: clm

> 2014-12-24  Steve Ellcey  <sellcey@mips.com>
> 
> 	* config/mips/t-mti-linux (MULTILIB_EXCEPTIONS): Add exceptions
> 	for mips32[r1] and mips64[r1] with -mnan=2008.

This is OK, but I think it may be best to fix t-mti-elf at the same
time even though it is probably building but with warnings.

Thanks,
Matthew

^ permalink raw reply	[flat|nested] 3+ messages in thread

* RE: [Patch, MIPS] Remove some multilibs from mips-mti-linux-gnu target
  2014-12-29 10:07 ` Matthew Fortune
@ 2014-12-29 17:09   ` Steve Ellcey
  0 siblings, 0 replies; 3+ messages in thread
From: Steve Ellcey @ 2014-12-29 17:09 UTC (permalink / raw)
  To: Matthew Fortune, gcc-patches; +Cc: clm

> From: Matthew Fortune

> > 2014-12-24  Steve Ellcey  <sellcey@mips.com>
> >
> >       * config/mips/t-mti-linux (MULTILIB_EXCEPTIONS): Add exceptions
> >       for mips32[r1] and mips64[r1] with -mnan=2008.

> This is OK, but I think it may be best to fix t-mti-elf at the same
> time even though it is probably building but with warnings.

Good point.  I made the same change to t-mti-elf and checked them both in.

Steve Ellcey
sellcey@imgtec.com

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2014-12-29 16:46 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-12-24 17:24 [Patch, MIPS] Remove some multilibs from mips-mti-linux-gnu target Steve Ellcey 
2014-12-29 10:07 ` Matthew Fortune
2014-12-29 17:09   ` Steve Ellcey

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).