diff options
| author | J. Ryan Stinnett <jryans@gmail.com> | 2020-12-20 02:49:18 +0000 |
|---|---|---|
| committer | J. Ryan Stinnett <jryans@gmail.com> | 2020-12-20 02:55:35 +0000 |
| commit | e628fcfcb591ce78673d2736ebe936873ea5111d (patch) | |
| tree | 64cd42056420a14a70947d5fe6e4ef58ed73d504 /src/bootstrap/builder.rs | |
| parent | 1f5bc176b0e54a8e464704adcd7e571700207fe9 (diff) | |
| download | rust-e628fcfcb591ce78673d2736ebe936873ea5111d.tar.gz rust-e628fcfcb591ce78673d2736ebe936873ea5111d.zip | |
Skip `dsymutil` by default for compiler bootstrap
`dsymutil` adds time to builds on Apple platforms for no clear benefit, and also makes it more difficult for debuggers to find debug info. The compiler currently defaults to running `dsymutil` to preserve its historical default, but when compiling the compiler itself, we skip it by default since we know it's safe to do so in that case.
Diffstat (limited to 'src/bootstrap/builder.rs')
| -rw-r--r-- | src/bootstrap/builder.rs | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/src/bootstrap/builder.rs b/src/bootstrap/builder.rs index 9af79e20630..ab0c4a5c31b 100644 --- a/src/bootstrap/builder.rs +++ b/src/bootstrap/builder.rs @@ -1126,6 +1126,19 @@ impl<'a> Builder<'a> { }, ); + // `dsymutil` adds time to builds on Apple platforms for no clear benefit, and also makes + // it more difficult for debuggers to find debug info. The compiler currently defaults to + // running `dsymutil` to preserve its historical default, but when compiling the compiler + // itself, we skip it by default since we know it's safe to do so in that case. + // See https://github.com/rust-lang/rust/issues/79361 for more info on this flag. + if target.contains("apple") { + if self.config.rust_run_dsymutil { + rustflags.arg("-Zrun-dsymutil=yes"); + } else { + rustflags.arg("-Zrun-dsymutil=no"); + } + } + if self.config.cmd.bless() { // Bless `expect!` tests. cargo.env("UPDATE_EXPECT", "1"); |
