about summary refs log tree commit diff
path: root/compiler/rustc_llvm/llvm-wrapper/RustWrapper.cpp
diff options
context:
space:
mode:
authorNicholas Nethercote <n.nethercote@gmail.com>2023-05-24 10:08:40 +1000
committerNicholas Nethercote <n.nethercote@gmail.com>2023-05-24 12:25:58 +1000
commitb39b7098eaaf5eb99e2f210ad98307ab310c1eee (patch)
tree9d88fbeff4913c0c8828f8f968bd35dade712951 /compiler/rustc_llvm/llvm-wrapper/RustWrapper.cpp
parente26c0c92bda6a66a5ea2f308b28e53a0a9bfba7f (diff)
downloadrust-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_llvm/llvm-wrapper/RustWrapper.cpp')
0 files changed, 0 insertions, 0 deletions