diff options
| author | Niko Matsakis <niko@alum.mit.edu> | 2020-03-24 15:20:19 -0400 |
|---|---|---|
| committer | Niko Matsakis <niko@alum.mit.edu> | 2020-03-24 15:25:43 -0400 |
| commit | e7f8895359365403a955468156a35bf26a45e0bf (patch) | |
| tree | d2c2de7cb44e5e9037a2f742329b45d268b163f0 | |
| parent | 2dcf54f564c6d8bbf48960fb9aaec88a0e2e062a (diff) | |
| download | rust-e7f8895359365403a955468156a35bf26a45e0bf.tar.gz rust-e7f8895359365403a955468156a35bf26a45e0bf.zip | |
save/restore `pessimistic_yield` when entering bodies
This flag is used to make the execution order around `+=` operators pessimistic. Failure to save/restore the flag was causing independent async blocks to effect one another, leading to strange ICEs and failed assumptions.
| -rw-r--r-- | src/librustc_passes/region.rs | 2 | ||||
| -rw-r--r-- | src/test/ui/async-await/issues/issue-69307.rs | 23 |
2 files changed, 25 insertions, 0 deletions
diff --git a/src/librustc_passes/region.rs b/src/librustc_passes/region.rs index e771696a5b6..1807756fe52 100644 --- a/src/librustc_passes/region.rs +++ b/src/librustc_passes/region.rs @@ -720,6 +720,7 @@ impl<'tcx> Visitor<'tcx> for RegionResolutionVisitor<'tcx> { let outer_ec = mem::replace(&mut self.expr_and_pat_count, 0); let outer_cx = self.cx; let outer_ts = mem::take(&mut self.terminating_scopes); + let outer_pessimistic_yield = mem::replace(&mut self.pessimistic_yield, false); self.terminating_scopes.insert(body.value.hir_id.local_id); if let Some(root_id) = self.cx.root_id { @@ -771,6 +772,7 @@ impl<'tcx> Visitor<'tcx> for RegionResolutionVisitor<'tcx> { self.expr_and_pat_count = outer_ec; self.cx = outer_cx; self.terminating_scopes = outer_ts; + self.pessimistic_yield = outer_pessimistic_yield; } fn visit_arm(&mut self, a: &'tcx Arm<'tcx>) { diff --git a/src/test/ui/async-await/issues/issue-69307.rs b/src/test/ui/async-await/issues/issue-69307.rs new file mode 100644 index 00000000000..4dae96ec8a6 --- /dev/null +++ b/src/test/ui/async-await/issues/issue-69307.rs @@ -0,0 +1,23 @@ +// Regression test for #69307 +// +// Having a `async { .. foo.await .. }` block appear inside of a `+=` +// expression was causing an ICE due to a failure to save/restore +// state in the AST numbering pass when entering a nested body. +// +// check-pass +// edition:2018 + +fn block_on<F>(_: F) -> usize { + 0 +} + +fn main() {} + +async fn bar() { + let mut sum = 0; + sum += block_on(async { + baz().await; + }); +} + +async fn baz() {} |
