Re: PATCH to split up cpp_read_main_file

[Geoff Keating <geoffk at geoffk dot org>]

Per Bothner <per@bothner.com> writes:

> The name --enable-intermodule flag is misleading - at least it confused
> me.  I thought it would enable "intermodule optimization".  Instead,
> it doesn't change any gcc feature - it only changes how gcc is built
> (if I'm reading the Makefile correctly).  I think a --with option
> would be more correct, perhaps --with-onestep.

--with is for external packages, like zlib; the option is

     --with-PACKAGE[=ARG]
     --without-PACKAGE

It is clearly more appropriate to use --enable, which is for features
(in this case, the feature of being built with intermodule analysis
and hopefully going faster).

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