Skip to content

Commit a027474

Browse files
committed
Don't run llvm-config in dry runs
1 parent c327719 commit a027474

File tree

1 file changed

+10
-5
lines changed

1 file changed

+10
-5
lines changed

src/bootstrap/native.rs

+10-5
Original file line numberDiff line numberDiff line change
@@ -498,11 +498,16 @@ impl Step for Llvm {
498498
// https://llvm.org/docs/HowToCrossCompileLLVM.html
499499
if target != builder.config.build {
500500
let llvm_config = builder.ensure(Llvm { target: builder.config.build });
501-
let llvm_bindir = output(Command::new(&llvm_config).arg("--bindir"));
502-
let host_bin = Path::new(llvm_bindir.trim());
503-
cfg.define("LLVM_TABLEGEN", host_bin.join("llvm-tblgen").with_extension(EXE_EXTENSION));
504-
// LLVM_NM is required for cross compiling using MSVC
505-
cfg.define("LLVM_NM", host_bin.join("llvm-nm").with_extension(EXE_EXTENSION));
501+
if !builder.config.dry_run {
502+
let llvm_bindir = output(Command::new(&llvm_config).arg("--bindir"));
503+
let host_bin = Path::new(llvm_bindir.trim());
504+
cfg.define(
505+
"LLVM_TABLEGEN",
506+
host_bin.join("llvm-tblgen").with_extension(EXE_EXTENSION),
507+
);
508+
// LLVM_NM is required for cross compiling using MSVC
509+
cfg.define("LLVM_NM", host_bin.join("llvm-nm").with_extension(EXE_EXTENSION));
510+
}
506511
cfg.define("LLVM_CONFIG_PATH", llvm_config);
507512
if builder.config.llvm_clang {
508513
let build_bin = builder.llvm_out(builder.config.build).join("build").join("bin");

0 commit comments

Comments
 (0)