This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Add missing initializers to gcc.c
Caroline Tice <ctice@apple.com> writes:
> Yes, this looks correct to me, but of course I can't properly approve
> anything.
Yes, this is OK. Ian, what wasn't clear about the combinable field?
The comment says:
If non-zero, compiler can deal with multiple source files at once
which seems clear to me in context.
> On Apr 7, 2004, at 5:10 PM, Ian Lance Taylor wrote:
>
> > This patch eliminates a warning when compiling gcc.c. Caroline,
> > please take a look; this seems right to me, since it's copied from the
> > ".i" entry, but I'm not wholly sure what the combinable field means.
> >
> > Ian
> >
> >
> > 2004-04-07 Ian Lance Taylor <ian@wasabisystems.com>
> >
> > * gcc.c (default_compilers): Add missing initializers.
> >
> >
> > Index: gcc.c
> > ===================================================================
> > RCS file: /cvs/gcc/gcc/gcc/gcc.c,v
> > retrieving revision 1.414
> > diff -p -u -r1.414 gcc.c
> > --- gcc.c 7 Apr 2004 19:46:04 -0000 1.414
> > +++ gcc.c 8 Apr 2004 00:02:56 -0000
> > @@ -951,7 +951,7 @@ static const struct compiler default_com
> > %W{o*:--output-pch=%*}%V}}}}}}", 0, 0, 0},
> > {".i", "@cpp-output", 0, 1, 0},
> > {"@cpp-output",
> > - "%{!M:%{!MM:%{!E:cc1 -fpreprocessed %i %(cc1_options)
> > %{!fsyntax-only:%(invoke_as)}}}}", 0},
> > + "%{!M:%{!MM:%{!E:cc1 -fpreprocessed %i %(cc1_options)
> > %{!fsyntax-only:%(invoke_as)}}}}", 0, 1, 0},
> > {".s", "@assembler", 0, 1, 0},
> > {"@assembler",
> > "%{!M:%{!MM:%{!E:%{!S:as %(asm_debug) %(asm_options) %i %A
> > }}}}", 0, 1, 0},
> >
>
>
--
- Geoffrey Keating <geoffk@geoffk.org>