public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* Remove SPARC -mlittle-endian-data spec
@ 2011-02-17  1:12 Joseph S. Myers
  2011-02-17 12:37 ` Eric Botcazou
  0 siblings, 1 reply; 2+ messages in thread
From: Joseph S. Myers @ 2011-02-17  1:12 UTC (permalink / raw)
  To: gcc-patches; +Cc: rth, davem, jakub, ebotcazou

SPARC has a preprocessor spec
%{mlittle-endian-data:-D__LITTLE_ENDIAN_DATA__}.  But there is no
-mlittle-endian-data option in sparc.opt or little-endian.opt, no code
to handle such an option and such a spec is useless in this case since
preprocessing will also pass the -m option down to cc1 which will
reject it.  Thus, this patch removes this spec as useless.

Tested building cc1 and xgcc for cross to sparc-elf.  OK to commit?

2011-02-16  Joseph Myers  <joseph@codesourcery.com>

	* config/sparc/sparc.h (CPP_ENDIAN_SPEC): Don't handle
	-mlittle-endian-data.

Index: gcc/config/sparc/sparc.h
===================================================================
--- gcc/config/sparc/sparc.h	(revision 170230)
+++ gcc/config/sparc/sparc.h	(working copy)
@@ -388,10 +388,9 @@ extern enum cmodel sparc_cmodel;
 %{!m32:%{!m64:%(cpp_arch_default)}} \
 "
 
-/* Macros to distinguish endianness.  */
+/* Macro to distinguish endianness.  */
 #define CPP_ENDIAN_SPEC "\
-%{mlittle-endian:-D__LITTLE_ENDIAN__} \
-%{mlittle-endian-data:-D__LITTLE_ENDIAN_DATA__}"
+%{mlittle-endian:-D__LITTLE_ENDIAN__}"
 
 /* Macros to distinguish the particular subtarget.  */
 #define CPP_SUBTARGET_SPEC ""

-- 
Joseph S. Myers
joseph@codesourcery.com

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

* Re: Remove SPARC -mlittle-endian-data spec
  2011-02-17  1:12 Remove SPARC -mlittle-endian-data spec Joseph S. Myers
@ 2011-02-17 12:37 ` Eric Botcazou
  0 siblings, 0 replies; 2+ messages in thread
From: Eric Botcazou @ 2011-02-17 12:37 UTC (permalink / raw)
  To: Joseph S. Myers; +Cc: gcc-patches, rth, davem, jakub

> 2011-02-16  Joseph Myers  <joseph@codesourcery.com>
>
> 	* config/sparc/sparc.h (CPP_ENDIAN_SPEC): Don't handle
> 	-mlittle-endian-data.

OK, thanks.

-- 
Eric Botcazou

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

end of thread, other threads:[~2011-02-17 12:01 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-02-17  1:12 Remove SPARC -mlittle-endian-data spec Joseph S. Myers
2011-02-17 12:37 ` Eric Botcazou

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