Skip to content

Commit 37dc976

Browse files
authored
Merge pull request #2051 from Kobzol/fix-bootstrap-rustc
Fix bootstrap rustc benchmark
2 parents 2d85e53 + 522533c commit 37dc976

File tree

1 file changed

+9
-1
lines changed

1 file changed

+9
-1
lines changed

collector/src/compile/execute/rustc.rs

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ async fn record(
117117
.env("BOOTSTRAP_SKIP_TARGET_SANITY", "1")
118118
.arg("build")
119119
.arg("--stage")
120-
.arg("0")
120+
.arg("1")
121121
// We want bootstrap and the Cargos it spawns to have no parallelism --
122122
// if multiple rustcs are competing for jobserver tokens, we introduce
123123
// quite a bit of variance.
@@ -144,6 +144,14 @@ async fn record(
144144
}
145145
}
146146

147+
// Sanity check
148+
if timing_data.is_empty() {
149+
return Err(anyhow::anyhow!(
150+
"rustc benchmark failed to produce timing data\nSTDOUT:\n{}\n\nSTDERR:{timings}\n",
151+
String::from_utf8_lossy(&output.stdout)
152+
));
153+
}
154+
147155
let version = get_rustc_perf_commit();
148156
let collection = conn.collection_id(&version).await;
149157

0 commit comments

Comments
 (0)