Patchwork [18/25] Specs cleanup: rs6000/vxworks.h -fvec*

login
register
mail settings
Submitter Joseph S. Myers
Date Jan. 14, 2011, 12:15 a.m.
Message ID <Pine.LNX.4.64.1101140014270.6059@digraph.polyomino.org.uk>
Download mbox | patch
Permalink /patch/78846/
State New
Headers show

Comments

Joseph S. Myers - Jan. 14, 2011, 12:15 a.m.
config/rs6000/vxworks.h defines CC1_SPEC to handle -fvec and
-fvec-eabi, translating them to -m options.  But since the original
options would still be passed to cc1 (all -f options are), this is
useless in practice; the -f options would result in errors as they are
in no .opt file.  This patch removes the useless specs.  OK to commit?

2011-01-13  Joseph Myers  <joseph@codesourcery.com>

	* config/rs6000/vxworks.h (CC1_SPEC): Don't handle -fvec or
	-fvec-eabi.
Nathan Sidwell - Jan. 14, 2011, 7:58 a.m.
On 01/14/11 00:15, Joseph S. Myers wrote:
> config/rs6000/vxworks.h defines CC1_SPEC to handle -fvec and
> -fvec-eabi, translating them to -m options.  But since the original
> options would still be passed to cc1 (all -f options are), this is
> useless in practice; the -f options would result in errors as they are
> in no .opt file.  This patch removes the useless specs.  OK to commit?

ok

Patch

diff -rupN --exclude=.svn gcc-mainline-17/gcc/config/rs6000/vxworks.h gcc-mainline/gcc/config/rs6000/vxworks.h
--- gcc-mainline-17/gcc/config/rs6000/vxworks.h	2011-01-12 14:43:16.000000000 -0800
+++ gcc-mainline/gcc/config/rs6000/vxworks.h	2011-01-12 14:44:59.000000000 -0800
@@ -80,8 +80,7 @@  VXWORKS_ADDITIONAL_CPP_SPEC
 
 #define CC1_SPEC						\
 "%{G*} %{mno-sdata:-msdata=none} %{msdata:-msdata=default}	\
- %{mlittle|mlittle-endian:-mstrict-align}			\
- %{fvec:-maltivec} %{fvec-eabi:-maltivec -mabi=altivec}"
+ %{mlittle|mlittle-endian:-mstrict-align}"
 
 #define ASM_SPEC \
 "%(asm_cpu) \