basic-improvements call for testers

[Geoff Keating]

> Date: Mon, 16 Dec 2002 14:55:03 -0500
> From: Daniel Jacobowitz <drow@mvista.com>
> Cc: zack@codesourcery.com, toa@pop.agri.ch, gcc@gcc.gnu.org
> Mail-Followup-To: Geoff Keating <geoffk@geoffk.org>,
> 	zack@codesourcery.com, toa@pop.agri.ch, gcc@gcc.gnu.org
> Content-Disposition: inline
> User-Agent: Mutt/1.5.1i
> X-OriginalArrivalTime: 16 Dec 2002 19:54:18.0435 (UTC) FILETIME=[EB50B130:01C2A53C]
> 
> On Mon, Dec 16, 2002 at 11:45:22AM -0800, Geoff Keating wrote:
> > > Cc: Andreas Tobler <toa@pop.agri.ch>, gcc@gcc.gnu.org
> > > From: Zack Weinberg <zack@codesourcery.com>
> > > Date: Sun, 15 Dec 2002 15:09:12 -0800
> > > User-Agent: Gnus/5.090008 (Oort Gnus v0.08) Emacs/21.2 (i386-pc-linux-gnu)
> > > X-OriginalArrivalTime: 15 Dec 2002 23:08:47.0062 (UTC) FILETIME=[EBF21F60:01C2A48E]
> > > 
> > > Geoff Keating <geoffk@geoffk.org> writes:
> > > > Andreas Tobler <toa@pop.agri.ch> writes:
> > > >> stage1/xgcc -Bstage1/
> > > >> -B/Volumes/reserved1/gcctmp/powerpc-apple-darwin6.2/bin/   -g -O2
> > > >> -DIN_GCC   -W -Wall -Wwrite-strings -Wstrict-prototypes
> > > >> -Wmissing-prototypes -Wtraditional -pedantic -Wno-long-long
> > > >> -fno-common -DHAVE_CONFIG_H -DGENERATOR_FILE  -o gengenrtl \
> > > >>   gengenrtl.o ../libiberty/libiberty.a
> > > >> make[2]: *** [gengenrtl] Abort trap
> > > >> make[1]: *** [stage2_build] Error 2
> > > >> make: *** [bootstrap] Error 2
> > > >
> > > > I now see exactly this on Darwin too, after the last merge into BIB.
> > > 
> > > I don't have access to a Darwin machine.  Would you mind trying to
> > > debug this a bit more?
> > 
> > It happens in the driver:
> > 
> > #0  0x9005d478 in abort ()
> > #1  0x00011bd8 in handle_braces (p=0x63bb4 "@:-o %f%u.out}%{!@:%{o*}%{!o:-o a.out}}     %{!Zdynamiclib:%{!A:%{!nostdlib:%{!nostartfiles:%S}}}}     %{L*} %(link_libgcc) %o %{!nostdlib:%{!nodefaultlibs:%G %L}}     %{!A:%{!nostdlib:%{!nostartfiles"...) at /Network/Servers/cauchy/homes/thorin/gkeating/co/basic-improvement/gcc/gcc/gcc.c:5703
> > 
> > I don't see anything wrong with the specs string at this point,
> > it looks like an example of the %{S:X} rule.
> 
> @ isn't legal there according to the driver.  The test for the atom is:
> 
>       while (ISIDNUM(*p) || *p == '-' || *p == '+' || *p == '='
>              || *p == ',' || *p == '.')
>         p++;
> 
> If -@ is a valid option here that if statement should be extended to
> include it.

-@ is certainly a valid option.  I think we also have -#.

-- 
- Geoffrey Keating <geoffk@geoffk.org>