for review: use available opt files

Fergus Henderson fjh at cs.mu.OZ.AU
Fri Jun 26 17:42:28 AEST 1998


On 25-Jun-1998, Simon Taylor <stayl at cs.mu.OZ.AU> wrote:
> +++ options.m	1998/06/24 23:25:49
> +        io__write_string("\t--use-opt-files\n"),
> +        io__write_string("\t\tPerform inter-module optimization using any\n"),
> +        io__write_string("\t\t`.opt' files which are already built, but do not\n"),
> +        io__write_string("\t\tbuild any others.\n"),
> +        io__write_string("\t--use-trans-opt-files\n"),
> +        io__write_string("\t\tPerform inter-module optimization using any\n"),
> +        io__write_string("\t\t`.trans_opt' files which are already built, but do\n"),
> +        io__write_string("\t\tnot build any others.\n"),

I suggest you add ", e.g. those for the standard library," after
"already built".

> Index: doc/user_guide.texi
...
> + at item --use-opt-files
> +Perform inter-module optimization using any @samp{.opt} files which are 
> +already built, but do not build any others.
> +
> + at item --use-transopt-files
> +Perform inter-module optimization using any @samp{.trans_opt} files which are 
> +already built, but do not build any others.

Likewise here too.

Also the option name should be `--use-trans-opt-files' rather than
`--use-transopt-files', to match the file names (*.trans_opt) and
the `--make-trans-opt' option.

Apart from that, that change looks fine.  Thanks.

-- 
Fergus Henderson <fjh at cs.mu.oz.au>  |  "I have always known that the pursuit
WWW: <http://www.cs.mu.oz.au/~fjh>  |  of excellence is a lethal habit"
PGP: finger fjh at 128.250.37.3        |     -- the last words of T. S. Garp.



More information about the developers mailing list