]> git.lizzy.rs Git - rust.git/commit - src/tools/rustfmt
Rollup merge of #79155 - 12101111:fix-profiler-config, r=Mark-Simulacrum
authorDylan DPC <dylan.dpc@gmail.com>
Thu, 19 Nov 2020 15:26:40 +0000 (16:26 +0100)
committerGitHub <noreply@github.com>
Thu, 19 Nov 2020 15:26:40 +0000 (16:26 +0100)
commit3b857a5a3aa49b3ec1f69ad0e62812e7ae26d7b9
tree77813221e7ff6906dd35514e4545cf733f8ba46e
parent5c7d530b5ec520073f7388519f39033a7b342eef
parent36972b0d63ad4af65bb7bef6a05df4e1c726fbf0
Rollup merge of #79155 - 12101111:fix-profiler-config, r=Mark-Simulacrum

fix handling the default config for profiler and sanitizers

#78354 don't handle the case that user don't add any target-specific config in `[target.*]` of `config.toml`:
```toml
changelog-seen = 2
[llvm]
link-shared = true
[build]
sanitizers = true
profiler = true
[install]
[rust]
[dist]
```

The previes code handle the default config in `Config::prase()`:

```rust
target.sanitizers = cfg.sanitizers.unwrap_or(build.sanitizers.unwrap_or_default());
target.profiler = cfg.profiler.unwrap_or(build.profiler.unwrap_or_default());
config.target_config.insert(TargetSelection::from_user(&triple), target);
```

In this case, `toml.target` don't contain any target, so the above code won't execute. Instead, a default `Target` is insert in https://github.com/rust-lang/rust/blob/c919f490bbcd2b29b74016101f7ec71aaa24bdbb/src/bootstrap/sanity.rs#L162-L166

The default value for `bool` is false, hence the issue in #79124

This fix change the type of `sanitizers` and `profiler` to `Option<bool>`, so the default value is `None`, and fallback config is handled in  `Config::sanitizers_enabled` and `Config::profiler_enabled`

fix #79124

cc `@Mark-Simulacrum` `@richkadel`