Skip to content

Commit 01ee930

Browse files
committed
Review fixes
1 parent c33c16b commit 01ee930

File tree

4 files changed

+16
-17
lines changed

4 files changed

+16
-17
lines changed

src/bootstrap/src/core/build_steps/test.rs

+4-3
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,8 @@ use crate::utils::cache::{Interned, INTERNER};
3030
use crate::utils::exec::BootstrapCommand;
3131
use crate::utils::helpers::{
3232
self, add_link_lib_path, add_rustdoc_cargo_linker_args, dylib_path, dylib_path_var,
33-
linker_args, output, t, target_supports_cranelift_backend, up_to_date, LldThreads,
33+
linker_args, linker_flags, output, t, target_supports_cranelift_backend, up_to_date,
34+
LldThreads,
3435
};
3536
use crate::utils::render_tests::{add_flags_and_try_run_tests, try_run_tests};
3637
use crate::{envify, CLang, DocTests, GitRepo, Mode};
@@ -1746,14 +1747,14 @@ NOTE: if you're sure you want to do this, please open an issue as to why. In the
17461747

17471748
let mut hostflags = flags.clone();
17481749
hostflags.push(format!("-Lnative={}", builder.test_helpers_out(compiler.host).display()));
1749-
hostflags.extend(linker_args(builder, compiler.host, LldThreads::No));
1750+
hostflags.extend(linker_flags(builder, compiler.host, LldThreads::No));
17501751
for flag in hostflags {
17511752
cmd.arg("--host-rustcflags").arg(flag);
17521753
}
17531754

17541755
let mut targetflags = flags;
17551756
targetflags.push(format!("-Lnative={}", builder.test_helpers_out(target).display()));
1756-
targetflags.extend(linker_args(builder, compiler.host, LldThreads::No));
1757+
targetflags.extend(linker_flags(builder, compiler.host, LldThreads::No));
17571758
for flag in targetflags {
17581759
cmd.arg("--target-rustcflags").arg(flag);
17591760
}

src/bootstrap/src/core/builder.rs

+10-12
Original file line numberDiff line numberDiff line change
@@ -19,10 +19,8 @@ use crate::core::build_steps::{check, clean, compile, dist, doc, install, run, s
1919
use crate::core::config::flags::{Color, Subcommand};
2020
use crate::core::config::{DryRun, SplitDebuginfo, TargetSelection};
2121
use crate::utils::cache::{Cache, Interned, INTERNER};
22-
use crate::utils::helpers::{
23-
self, add_dylib_path, add_link_lib_path, exe, linker_args, linker_flags,
24-
};
25-
use crate::utils::helpers::{libdir, output, t, LldThreads};
22+
use crate::utils::helpers::{self, add_dylib_path, add_link_lib_path, exe, linker_args};
23+
use crate::utils::helpers::{libdir, linker_flags, output, t, LldThreads};
2624
use crate::Crate;
2725
use crate::EXTRA_CHECK_CFGS;
2826
use crate::{Build, CLang, DocTests, GitRepo, Mode};
@@ -1669,22 +1667,22 @@ impl<'a> Builder<'a> {
16691667
}
16701668
}
16711669

1672-
linker_args(self, compiler.host, LldThreads::Yes).into_iter().for_each(|flag| {
1673-
hostflags.arg(flag);
1674-
});
1670+
for arg in linker_args(self, compiler.host, LldThreads::Yes) {
1671+
hostflags.arg(&arg);
1672+
}
16751673

16761674
if let Some(target_linker) = self.linker(target) {
16771675
let target = crate::envify(&target.triple);
16781676
cargo.env(&format!("CARGO_TARGET_{target}_LINKER"), target_linker);
16791677
}
16801678
// We want to set -Clinker using Cargo, therefore we only call `linker_flags` and not
16811679
// `linker_args` here.
1682-
linker_flags(self, target, LldThreads::Yes).into_iter().for_each(|flag| {
1680+
for flag in linker_flags(self, target, LldThreads::Yes) {
16831681
rustflags.arg(&flag);
1684-
});
1685-
linker_args(self, target, LldThreads::Yes).into_iter().for_each(|flag| {
1686-
rustdocflags.arg(&flag);
1687-
});
1682+
}
1683+
for arg in linker_args(self, target, LldThreads::Yes) {
1684+
rustdocflags.arg(&arg);
1685+
}
16881686

16891687
if !(["build", "check", "clippy", "fix", "rustc"].contains(&cmd)) && want_rustdoc {
16901688
cargo.env("RUSTDOC_LIBDIR", self.rustc_libdir(compiler));

src/bootstrap/src/core/config/config.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ impl Display for DebuginfoLevel {
116116
/// 2) MSVC
117117
/// - Self-contained: -Clinker=<path to rust-lld>
118118
/// - External: -Clinker=lld
119-
#[derive(Default, Clone)]
119+
#[derive(Default, Copy, Clone)]
120120
pub enum LldMode {
121121
/// Do not use LLD
122122
#[default]

src/bootstrap/src/utils/helpers.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -526,7 +526,7 @@ pub fn linker_flags(
526526
if matches!(lld_threads, LldThreads::No) {
527527
args.push(format!(
528528
"-Clink-arg=-Wl,{}",
529-
lld_flag_no_threads(builder.config.lld_mode.clone(), target.is_msvc())
529+
lld_flag_no_threads(builder.config.lld_mode, target.is_msvc())
530530
));
531531
}
532532
}

0 commit comments

Comments
 (0)