This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[BIB] Fix specs for NEC VR series


[ Hmm, mails from my USAFA email address get rejected as spam, even after
mailing to global-allow-subscribe-blabla... ]

This patch fixes the DRIVER_SELF_SPECS for the NEC VR series processors. The
fix is only needed on the BIB because that's Zack's changes to the specs
language are.

Richard, this patch reverts your change:
2002-10-15  Richard Sandiford  <rsandifo at redhat dot com>

     * config/mips/vr.h (DRIVER_SELF_SPECS): Change %<mgp32 to %{<mgp32}.

I can't test this patch, could you make sure this works for the branch?

Greetz
Steven



Index: vr.h
===================================================================
RCS file: /cvsroot/gcc/gcc/gcc/config/mips/vr.h,v
retrieving revision 1.1.6.2
diff -c -r1.1.6.2 vr.h
*** vr.h	22 Oct 2002 18:28:33 -0000	1.1.6.2
--- vr.h	19 Nov 2002 19:43:33 -0000
***************
*** 28,31 ****
     if it is redundant.  */
  #define DRIVER_SELF_SPECS \
  	"%{mabi=eabi:%{!mlong*:%{!mgp32:-mlong64}}}", \
! 	"%{mabi=32:%{<mgp32}}"
--- 28,31 ----
     if it is redundant.  */
  #define DRIVER_SELF_SPECS \
  	"%{mabi=eabi:%{!mlong*:%{!mgp32:-mlong64}}}", \
! 	"%{mabi=32:%<mgp32}"
-------- 8< --------

-----Original Message-----
From: Zack Weinberg [mailto:zack@codesourcery.com] 
Sent: Tuesday, November 19, 2002 11:29 AM
To: Bosscher Steven Civ USAFA/DFAN
Cc: 'gcc-gnats@gcc.gnu.org'; 'gcc-bugs@gcc.gnu.org'; 'rsandifo@redhat.com'
Subject: Re: other/633: In specs, %{<XXX*} does NOT remove switches from c
ommand line

Bosscher Steven Civ USAFA/DFAN <Steven.Bosscher@usafa.af.mil> writes:

>
http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&p
> r=633
>
> It seems that PR other/633 ("In specs, %{<XXX*} does NOT remove switches
> from command line") was fixed long ago by:
>
> 2000-12-13  Tom Tromey  <tromey@redhat.com>
>
>         * gcc.c (handle_braces): Handle %{<S*} spec correctly.
>
> Maybe somebody with GNATS write access can close the bug?

Done.

> But it seems that all occurrences of %{<S*} should be replaced with %<S*
> since the changes on 2002-09-19 by Zack Weinberg.
>
> The %{<S*} spec is still used (or again, actually, was introduced
> 2002-10-15) in config/mips/vr.h: "%{mabi=32:%{<mgp32}}". Is this a bug?

My change is only present on the basic-improvements branch.
config/mips/vr.h does need to be changed there.  I have a lot of
delicate changes in my copy of that branch right now -- would you mind
submitting a patch?

zw


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]