Skip to content

Commit

Permalink
Auto merge of #7428 - alexcrichton:nocapture, r=ehuss
Browse files Browse the repository at this point in the history
Don't ever capture CPU state if timings are disabled

This should initially help with the panic in #7427 but doesn't solve it.
  • Loading branch information
bors committed Sep 25, 2019
2 parents c59e1f4 + fb80b9e commit 249b31b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/cargo/core/compiler/timings.rs
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,7 @@ impl<'a, 'cfg> Timings<'a, 'cfg> {
unit_times: Vec::new(),
active: HashMap::new(),
concurrency: Vec::new(),
last_cpu_state: State::current().ok(),
last_cpu_state: if enabled { State::current().ok() } else { None },
last_cpu_recording: Instant::now(),
cpu_usage: Vec::new(),
}
Expand Down

0 comments on commit 249b31b

Please sign in to comment.