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]

Re: Implement -MF <output-filename>


On Sat, Jan 06, 2001 at 06:37:49PM +0000, Neil Booth wrote:
> Zack Weinberg wrote:-
> 
> > I like it overall.
> > 
> > It is difficult for me to comprehend your patch since it's got
> > substantial code rearrangement on top of the semantic changes.
> 
> Unfortunately, yes.

If you have to respin the patch anyway, you might split it up and
apply just the code rearrangement first.


> > -MD expands to -M -MF %b.d.  %b is the basename of the input file -
> > no directory component.  Is this what we want?  We just went through
> > changing the target name in dependency output to include the
> > directory.  However, since -MD's always done this, we can worry about
> > it later.
> 
> That's what I was doing - going to leave it for a subsequent patch.
> There will probably be overlap with Nathan's patch for that.

Okay.

> > Do we have documentation in cpp.texi for all these new options?
> 
> In cpp.texi, yes, and after this patch in the run-time help too.
> 
> We don't have it in gcc.texi for CPP; this duplication of
> documentation is a pain.  Similarly for extend.texi.  I was going to
> wait until it had settled before updating those.

Fair enough.

> How's your documentation rewrite coming on?

About half done.  I may ask you for help explaining the macro expander
corner cases at some point.

zw

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