Skip to content

Commit

Permalink
Auto merge of rust-lang#71444 - RalfJung:test-async-no-opt, r=jonas-s…
Browse files Browse the repository at this point in the history
…chievink

smoke-test for async fn with mir-opt-level=0

MIR opt levels heavily influence which MIR transformations run, and we barely test non-default opt levels. I am particularly worried about `async fn` lowering and how it might (not) work when the set of preceding MIR passes changes -- see rust-lang#70073.

This adds some basic smoke testing, where at least a few `async fn` `run-pass` test are ensured to also work with mir-opt-level=0.
  • Loading branch information
bors committed Apr 28, 2020
2 parents fb5615a + 3a129df commit d7afaa7
Show file tree
Hide file tree
Showing 10 changed files with 30 additions and 0 deletions.
3 changes: 3 additions & 0 deletions src/test/ui/async-await/async-await.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
// run-pass

// revisions: default nomiropt
//[nomiropt]compile-flags: -Z mir-opt-level=0

#![allow(unused)]

// edition:2018
Expand Down
3 changes: 3 additions & 0 deletions src/test/ui/async-await/async-closure.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
// run-pass

// revisions: default nomiropt
//[nomiropt]compile-flags: -Z mir-opt-level=0

// edition:2018
// aux-build:arc_wake.rs

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@
// edition:2018
// run-pass

// revisions: default nomiropt
//[nomiropt]compile-flags: -Z mir-opt-level=0

#![allow(unused_variables)]

// Test that the drop order for parameters in a fn and async fn matches up. Also test that
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@
// edition:2018
// run-pass

// revisions: default nomiropt
//[nomiropt]compile-flags: -Z mir-opt-level=0

#![allow(unused_variables)]

// Test the drop order for parameters relative to local variables and
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@
// edition:2018
// run-pass

// revisions: default nomiropt
//[nomiropt]compile-flags: -Z mir-opt-level=0

// Test that the drop order for parameters in a fn and async fn matches up. Also test that
// parameters (used or unused) are not dropped until the async fn is cancelled.
// This file is mostly copy-pasted from drop-order-for-async-fn-parameters.rs
Expand Down
3 changes: 3 additions & 0 deletions src/test/ui/generator/conditional-drop.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
// run-pass

// revisions: default nomiropt
//[nomiropt]compile-flags: -Z mir-opt-level=0

#![feature(generators, generator_trait)]

use std::ops::Generator;
Expand Down
3 changes: 3 additions & 0 deletions src/test/ui/generator/control-flow.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
// run-pass

// revisions: default nomiropt
//[nomiropt]compile-flags: -Z mir-opt-level=0

#![feature(generators, generator_trait)]

use std::marker::Unpin;
Expand Down
3 changes: 3 additions & 0 deletions src/test/ui/generator/drop-env.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
// run-pass

// revisions: default nomiropt
//[nomiropt]compile-flags: -Z mir-opt-level=0

#![feature(generators, generator_trait)]

use std::ops::Generator;
Expand Down
3 changes: 3 additions & 0 deletions src/test/ui/generator/smoke-resume-args.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
// run-pass

// revisions: default nomiropt
//[nomiropt]compile-flags: -Z mir-opt-level=0

#![feature(generators, generator_trait)]

use std::fmt::Debug;
Expand Down
3 changes: 3 additions & 0 deletions src/test/ui/generator/smoke.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
// run-pass

// revisions: default nomiropt
//[nomiropt]compile-flags: -Z mir-opt-level=0

// ignore-emscripten no threads support
// compile-flags: --test

Expand Down

0 comments on commit d7afaa7

Please sign in to comment.