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]

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>


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