public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [patch] fix multiarch definition for powerpcspe-linux-gnu
@ 2012-12-19 16:47 Matthias Klose
  2012-12-20  4:28 ` David Edelsohn
  0 siblings, 1 reply; 2+ messages in thread
From: Matthias Klose @ 2012-12-19 16:47 UTC (permalink / raw)
  To: GCC Patches, David Edelsohn; +Cc: Roland Stigge

[-- Attachment #1: Type: text/plain, Size: 251 bytes --]

The definition of the multiarch tuple for powerpcspe-linux-gnu was wrong. The
t-spe fragment isn't included for the powerpc*-linux* case, so move it to
t-linux, and use tm_file_list (tm_file is only used in config.gcc).

Ok for the trunk?

  Matthias

[-- Attachment #2: spe.diff --]
[-- Type: text/x-diff, Size: 1226 bytes --]

2012-12-19  Roland Stigge  <stigge@debian.org>
	    Matthias Klose  <doko@ubuntu.com>

	* config/rs6000/t-spe (MULTIARCH_DIRNAME): Remove.
	* config/rs6000/t-linux (MULTIARCH_DIRNAME): Define name for
	powerpc-linux-gnuspe.

Index: config/rs6000/t-spe
===================================================================
--- config/rs6000/t-spe	(Revision 194615)
+++ config/rs6000/t-spe	(Arbeitskopie)
@@ -71,7 +71,3 @@
 			  mabi=altivec/mlittle \
 			  maltivec/mlittle \
 			  maltivec/mabi=altivec/mlittle
-
-ifneq (,$(findstring linux, $(target)))
-MULTIARCH_DIRNAME = powerpc-linux-gnuspe$(if $(findstring rs6000/e500-double.h, $(tm_file)),,v1)
-endif
Index: config/rs6000/t-linux
===================================================================
--- config/rs6000/t-linux	(Revision 194615)
+++ config/rs6000/t-linux	(Arbeitskopie)
@@ -1,5 +1,9 @@
 # do not define the multiarch name if configured for a soft-float cpu
 # or soft-float.
 ifeq (,$(filter $(with_cpu),$(SOFT_FLOAT_CPUS))$(findstring soft,$(with_float)))
+ifneq (,$(findstring spe,$(target)))
+MULTIARCH_DIRNAME = powerpc-linux-gnuspe$(if $(findstring rs6000/e500-double.h, $(tm_file_list)),,v1)
+else
 MULTIARCH_DIRNAME = powerpc-linux-gnu
 endif
+endif

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

* Re: [patch] fix multiarch definition for powerpcspe-linux-gnu
  2012-12-19 16:47 [patch] fix multiarch definition for powerpcspe-linux-gnu Matthias Klose
@ 2012-12-20  4:28 ` David Edelsohn
  0 siblings, 0 replies; 2+ messages in thread
From: David Edelsohn @ 2012-12-20  4:28 UTC (permalink / raw)
  To: Matthias Klose; +Cc: GCC Patches, Roland Stigge

On Wed, Dec 19, 2012 at 11:47 AM, Matthias Klose <doko@ubuntu.com> wrote:
> The definition of the multiarch tuple for powerpcspe-linux-gnu was wrong. The
> t-spe fragment isn't included for the powerpc*-linux* case, so move it to
> t-linux, and use tm_file_list (tm_file is only used in config.gcc).
>
> Ok for the trunk?

Okay.

Thanks, David

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

end of thread, other threads:[~2012-12-20  4:28 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-12-19 16:47 [patch] fix multiarch definition for powerpcspe-linux-gnu Matthias Klose
2012-12-20  4:28 ` David Edelsohn

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).