diff --git a/src/librustc_mir/monomorphize/partitioning.rs b/src/librustc_mir/monomorphize/partitioning.rs index 4150f9f95485bde1a82ab2c6f57bca0aae7f4d9e..806d787c84522601c69c4a56fde4acbc7f547ba4 100644 --- a/src/librustc_mir/monomorphize/partitioning.rs +++ b/src/librustc_mir/monomorphize/partitioning.rs @@ -427,8 +427,6 @@ fn merge_codegen_units<'tcx>(initial_partitioning: &mut PreInliningPartitioning< codegen_units.sort_by_key(|cgu| cgu.name().clone()); // Merge the two smallest codegen units until the target size is reached. - // Note that "size" is estimated here rather inaccurately as the number of - // translation items in a given unit. This could be improved on. while codegen_units.len() > target_cgu_count { // Sort small cgus to the back codegen_units.sort_by_key(|cgu| usize::MAX - cgu.size_estimate()); diff --git a/src/librustc_trans/base.rs b/src/librustc_trans/base.rs index e03b6ee794d67d64ee578a6ba39a953436554786..1126c9d2c8cdbddf686df67bae32d26f863354f6 100644 --- a/src/librustc_trans/base.rs +++ b/src/librustc_trans/base.rs @@ -824,9 +824,7 @@ pub fn trans_crate<'a, 'tcx>(tcx: TyCtxt<'a, 'tcx, 'tcx>, ongoing_translation.submit_pre_translated_module_to_llvm(tcx, metadata_module); // We sort the codegen units by size. This way we can schedule work for LLVM - // a bit more efficiently. Note that "size" is defined rather crudely at the - // moment as it is just the number of TransItems in the CGU, not taking into - // account the size of each TransItem. + // a bit more efficiently. let codegen_units = { let mut codegen_units = codegen_units; codegen_units.sort_by_key(|cgu| usize::MAX - cgu.size_estimate());