[m-rev.] for review: Use --trans-opt-deps-spec option.

Julien Fischer jfischer at opturion.com
Wed Jan 25 17:00:41 AEDT 2023


On Wed, 25 Jan 2023, Peter Wang wrote:

> Suggestions for the name of the spec file are welcome.
>
> I've attached the module order when the spec file is used,
> and the changes to the analysis results.
>
> ---
>
> Use the recently added --trans-opt-deps-spec option to break cycles in
> the trans-opt dependency graph for the standard library. This enables
> more parallelism when making the .trans_opt files; it now takes about
> half as long as before.
>
> Ordering modules sensibly, so that .trans_opt files are created in a
> logical order, also improves analysis results for many predicates and
> functions. The only results which show a regression with this change are
> for deprecated forwarding predicates/functions.
>
> In future, we will probably be able to trim more dependencies to further
> improve parallelism, without impacting analysis results.
>
> configure.ac:
>    Check that the bootstrap compiler supports --trans-opt-deps-spec.
>
> library/mer_std.trans_opt_deps_spec:
>    Add the spec file that adjusts dependencies in the trans-opt
>    dependency graph.
>
> library/INTER_FLAGS:
>    Use the --trans-opt-deps-spec option when building with mmake.
>
> scripts/prepare_install_dir.in:
> tools/binary:
> tools/bootcheck:
> tools/unary:
>    Copy mer_std.trans_opt_deps_spec when preparing a copy of the
>    library directory.

That's fine.

Julien.


More information about the reviews mailing list