diff options
| author | Nicholas Nethercote <n.nethercote@gmail.com> | 2023-05-24 10:08:40 +1000 |
|---|---|---|
| committer | Nicholas Nethercote <n.nethercote@gmail.com> | 2023-05-24 12:25:58 +1000 |
| commit | b39b7098eaaf5eb99e2f210ad98307ab310c1eee (patch) | |
| tree | 9d88fbeff4913c0c8828f8f968bd35dade712951 /compiler/rustc_monomorphize/src/partitioning/mod.rs | |
| parent | e26c0c92bda6a66a5ea2f308b28e53a0a9bfba7f (diff) | |
| download | rust-b39b7098eaaf5eb99e2f210ad98307ab310c1eee.tar.gz rust-b39b7098eaaf5eb99e2f210ad98307ab310c1eee.zip | |
Remove the `merging` module.
Three of the four methods in `DefaultPartitioning` are defined in `default.rs`. But `merge_codegen_units` is defined in a separate module, `merging`, even though it's less than 100 lines of code and roughly the same size as the other three methods. (Also, the `merging` module currently sits alongside `default`, when it should be a submodule of `default`, adding to the confusion.) In #74275 this explanation was given: > I pulled this out into a separate module since it seemed like we might > want a few different merge algorithms to choose from. But in the three years since there have been no additional merging algorithms, and there is no mechanism for choosing between different merging algorithms. (There is a mechanism, `-Zcgu-partitioning-strategy`, for choosing between different partitioning strategies, but the merging algorithm is just one piece of a partitioning strategy.) This commit merges `merging` into `default`, making the code easier to navigate and read.
Diffstat (limited to 'compiler/rustc_monomorphize/src/partitioning/mod.rs')
| -rw-r--r-- | compiler/rustc_monomorphize/src/partitioning/mod.rs | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/compiler/rustc_monomorphize/src/partitioning/mod.rs b/compiler/rustc_monomorphize/src/partitioning/mod.rs index eafe57a0c02..8ea82b39534 100644 --- a/compiler/rustc_monomorphize/src/partitioning/mod.rs +++ b/compiler/rustc_monomorphize/src/partitioning/mod.rs @@ -93,7 +93,6 @@ //! inlining, even when they are not marked `#[inline]`. mod default; -mod merging; use std::cmp; use std::fs::{self, File}; |
