Skip to content

Commit 0bd58d8

Browse files
committed
Apply review comments.
1 parent 87d2e61 commit 0bd58d8

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

src/bootstrap/src/core/build_steps/perf.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ Consider setting `rust.debuginfo-level = 1` in `config.toml`."#);
2222
let sysroot = builder.ensure(Sysroot::new(compiler));
2323
let rustc = sysroot.join("bin/rustc");
2424

25-
let results_dir = builder.build.out.join("rustc-perf");
25+
let results_dir = builder.build.tempdir().join("rustc-perf");
2626

2727
let mut cmd = Command::new(collector);
2828
let cmd = cmd

src/bootstrap/src/core/build_steps/tool.rs

+4-4
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ impl Step for ToolBuild {
129129
if tool == "tidy" {
130130
tool = "rust-tidy";
131131
}
132-
copy_tool_bin(builder, self.compiler, self.target, self.mode, tool)
132+
copy_link_tool_bin(builder, self.compiler, self.target, self.mode, tool)
133133
}
134134
}
135135
}
@@ -214,9 +214,9 @@ pub fn prepare_tool_cargo(
214214
cargo
215215
}
216216

217-
/// Copies a built tool binary with the given `name` from the build directory to the
217+
/// Links a built tool binary with the given `name` from the build directory to the
218218
/// tools directory.
219-
fn copy_tool_bin(
219+
fn copy_link_tool_bin(
220220
builder: &Builder<'_>,
221221
compiler: Compiler,
222222
target: TargetSelection,
@@ -413,7 +413,7 @@ impl Step for RustcPerf {
413413
let collector_bin = builder.ensure(tool.clone());
414414
// We also need to symlink the `rustc-fake` binary to the corresponding directory,
415415
// because `collector` expects it in the same directory.
416-
copy_tool_bin(builder, tool.compiler, tool.target, tool.mode, "rustc-fake");
416+
copy_link_tool_bin(builder, tool.compiler, tool.target, tool.mode, "rustc-fake");
417417

418418
collector_bin
419419
}

0 commit comments

Comments
 (0)