[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