about summary refs log tree commit diff
path: root/compiler/rustc_llvm/llvm-wrapper/RustWrapper.cpp
diff options
context:
space:
mode:
authorMatthias Krüger <matthias.krueger@famsik.de>2023-07-27 06:04:12 +0200
committerGitHub <noreply@github.com>2023-07-27 06:04:12 +0200
commitc3cd05198a629aa744f8f8a4067f52e710d262ba (patch)
treecd9b512736afa574bb7794c8131ede10f1b0188c /compiler/rustc_llvm/llvm-wrapper/RustWrapper.cpp
parente7d6ce3a6f6b14b1839142cff5ac7d566e90417e (diff)
parent8c31219d5c6fe11654060ab968285705f00b1ec8 (diff)
downloadrust-c3cd05198a629aa744f8f8a4067f52e710d262ba.tar.gz
rust-c3cd05198a629aa744f8f8a4067f52e710d262ba.zip
Rollup merge of #113872 - nnethercote:tweak-cgu-sorting, r=pnkfelix
Tweak CGU sorting in a couple of places.

In `base.rs`, tweak how the CGU size interleaving works. Since #113777, it's much more common to have multiple CGUs with identical sizes. With the existing code these same-sized items ended up in the opposite-to-desired order due to the stable sorting. The code now starts with a reverse sort (like is done in `partitioning.rs`) which gives the behaviour we want. This doesn't matter much for perf, but makes profiles in `samply` look more like what we expect.

In `partitioning.rs`, we can use `sort_by_key` instead of `sort_by_cached_key` because `CGU::size_estimate()` is cheap. (There is an identical CGU sort earlier in that function that already uses `sort_by_key`.)

r? `@pnkfelix`
Diffstat (limited to 'compiler/rustc_llvm/llvm-wrapper/RustWrapper.cpp')
0 files changed, 0 insertions, 0 deletions