Auto merge of #70175 - Amanieu:remove_nlp, r=pnkfelix

Remove -Z no-landing-pads flag

Since #67502, `-Z no-landing-pads` will cause all attempted unwinds to abort since we don't generate a `try` / `catch`. This previously worked because `__rust_try` was located in libpanic_unwind which is always compiled with `-C panic=unwind`, but `__rust_try` is now directly inline into the crate that uses `catch_unwind`.

As such, `-Z no-landing-pads` is now mostly useless and people should use `-C panic=abort` instead.

4 files changed, 43 insertions, 96 deletions

diff --git a/src/test/mir-opt/generator-tiny.rs b/src/test/mir-opt/generator-tiny.rs
index b537263dd1b..c86e2865ca8 100644
--- a/src/test/mir-opt/generator-tiny.rs
+++ b/src/test/mir-opt/generator-tiny.rs
@@ -1,7 +1,8 @@
 //! Tests that generators that cannot return or unwind don't have unnecessary
 //! panic branches.
 
-// compile-flags: -Zno-landing-pads
+// compile-flags: -C panic=abort
+// no-prefer-dynamic
 
 #![feature(generators, generator_trait)]
 
diff --git a/src/test/mir-opt/generator-tiny/rustc.main-{{closure}}.generator_resume.0.mir b/src/test/mir-opt/generator-tiny/rustc.main-{{closure}}.generator_resume.0.mir
index 84b8af5d6b7..75c2fb3d130 100644
--- a/src/test/mir-opt/generator-tiny/rustc.main-{{closure}}.generator_resume.0.mir
+++ b/src/test/mir-opt/generator-tiny/rustc.main-{{closure}}.generator_resume.0.mir
@@ -1,78 +1,78 @@
 // MIR for `main::{{closure}}#0` 0 generator_resume
 // generator_layout = GeneratorLayout { field_tys: [HasDrop], variant_fields: [[], [], [], [_0]], storage_conflicts: BitMatrix { num_rows: 1, num_columns: 1, words: [1], marker: PhantomData } }
 
-fn main::{{closure}}#0(_1: std::pin::Pin<&mut [generator@$DIR/generator-tiny.rs:18:16: 24:6 {u8, HasDrop, ()}]>, _2: u8) -> std::ops::GeneratorState<(), ()> {
-    debug _x => _10;                     // in scope 0 at $DIR/generator-tiny.rs:18:17: 18:19
-    let mut _0: std::ops::GeneratorState<(), ()>; // return place in scope 0 at $DIR/generator-tiny.rs:18:16: 24:6
-    let _3: HasDrop;                     // in scope 0 at $DIR/generator-tiny.rs:19:13: 19:15
-    let mut _4: !;                       // in scope 0 at $DIR/generator-tiny.rs:20:9: 23:10
-    let mut _5: ();                      // in scope 0 at $DIR/generator-tiny.rs:18:16: 24:6
-    let _6: u8;                          // in scope 0 at $DIR/generator-tiny.rs:21:13: 21:18
-    let mut _7: ();                      // in scope 0 at $DIR/generator-tiny.rs:21:13: 21:18
-    let _8: ();                          // in scope 0 at $DIR/generator-tiny.rs:22:13: 22:21
-    let mut _9: ();                      // in scope 0 at $DIR/generator-tiny.rs:18:25: 18:25
-    let _10: u8;                         // in scope 0 at $DIR/generator-tiny.rs:18:17: 18:19
-    let mut _11: isize;                  // in scope 0 at $DIR/generator-tiny.rs:18:16: 24:6
+fn main::{{closure}}#0(_1: std::pin::Pin<&mut [generator@$DIR/generator-tiny.rs:19:16: 25:6 {u8, HasDrop, ()}]>, _2: u8) -> std::ops::GeneratorState<(), ()> {
+    debug _x => _10;                     // in scope 0 at $DIR/generator-tiny.rs:19:17: 19:19
+    let mut _0: std::ops::GeneratorState<(), ()>; // return place in scope 0 at $DIR/generator-tiny.rs:19:16: 25:6
+    let _3: HasDrop;                     // in scope 0 at $DIR/generator-tiny.rs:20:13: 20:15
+    let mut _4: !;                       // in scope 0 at $DIR/generator-tiny.rs:21:9: 24:10
+    let mut _5: ();                      // in scope 0 at $DIR/generator-tiny.rs:19:16: 25:6
+    let _6: u8;                          // in scope 0 at $DIR/generator-tiny.rs:22:13: 22:18
+    let mut _7: ();                      // in scope 0 at $DIR/generator-tiny.rs:22:13: 22:18
+    let _8: ();                          // in scope 0 at $DIR/generator-tiny.rs:23:13: 23:21
+    let mut _9: ();                      // in scope 0 at $DIR/generator-tiny.rs:19:25: 19:25
+    let _10: u8;                         // in scope 0 at $DIR/generator-tiny.rs:19:17: 19:19
+    let mut _11: isize;                  // in scope 0 at $DIR/generator-tiny.rs:19:16: 25:6
     scope 1 {
-        debug _d => (((*(_1.0: &mut [generator@$DIR/generator-tiny.rs:18:16: 24:6 {u8, HasDrop, ()}])) as variant#3).0: HasDrop); // in scope 1 at $DIR/generator-tiny.rs:19:13: 19:15
+        debug _d => (((*(_1.0: &mut [generator@$DIR/generator-tiny.rs:19:16: 25:6 {u8, HasDrop, ()}])) as variant#3).0: HasDrop); // in scope 1 at $DIR/generator-tiny.rs:20:13: 20:15
     }
 
     bb0: {
-        _11 = discriminant((*(_1.0: &mut [generator@$DIR/generator-tiny.rs:18:16: 24:6 {u8, HasDrop, ()}]))); // scope 0 at $DIR/generator-tiny.rs:18:16: 24:6
-        switchInt(move _11) -> [0u32: bb1, 3u32: bb5, otherwise: bb6]; // scope 0 at $DIR/generator-tiny.rs:18:16: 24:6
+        _11 = discriminant((*(_1.0: &mut [generator@$DIR/generator-tiny.rs:19:16: 25:6 {u8, HasDrop, ()}]))); // scope 0 at $DIR/generator-tiny.rs:19:16: 25:6
+        switchInt(move _11) -> [0u32: bb1, 3u32: bb5, otherwise: bb6]; // scope 0 at $DIR/generator-tiny.rs:19:16: 25:6
     }
 
     bb1: {
-        _10 = move _2;                   // scope 0 at $DIR/generator-tiny.rs:18:16: 24:6
-        nop;                             // scope 0 at $DIR/generator-tiny.rs:19:13: 19:15
-        (((*(_1.0: &mut [generator@$DIR/generator-tiny.rs:18:16: 24:6 {u8, HasDrop, ()}])) as variant#3).0: HasDrop) = HasDrop; // scope 0 at $DIR/generator-tiny.rs:19:18: 19:25
-        StorageLive(_4);                 // scope 1 at $DIR/generator-tiny.rs:20:9: 23:10
-        goto -> bb2;                     // scope 1 at $DIR/generator-tiny.rs:20:9: 23:10
+        _10 = move _2;                   // scope 0 at $DIR/generator-tiny.rs:19:16: 25:6
+        nop;                             // scope 0 at $DIR/generator-tiny.rs:20:13: 20:15
+        (((*(_1.0: &mut [generator@$DIR/generator-tiny.rs:19:16: 25:6 {u8, HasDrop, ()}])) as variant#3).0: HasDrop) = HasDrop; // scope 0 at $DIR/generator-tiny.rs:20:18: 20:25
+        StorageLive(_4);                 // scope 1 at $DIR/generator-tiny.rs:21:9: 24:10
+        goto -> bb2;                     // scope 1 at $DIR/generator-tiny.rs:21:9: 24:10
     }
 
     bb2: {
-        StorageLive(_6);                 // scope 1 at $DIR/generator-tiny.rs:21:13: 21:18
-        StorageLive(_7);                 // scope 1 at $DIR/generator-tiny.rs:21:13: 21:18
-        _7 = ();                         // scope 1 at $DIR/generator-tiny.rs:21:13: 21:18
-        _0 = std::ops::GeneratorState::<(), ()>::Yielded(move _7); // scope 1 at $DIR/generator-tiny.rs:21:13: 21:18
-        discriminant((*(_1.0: &mut [generator@$DIR/generator-tiny.rs:18:16: 24:6 {u8, HasDrop, ()}]))) = 3; // scope 1 at $DIR/generator-tiny.rs:21:13: 21:18
-        return;                          // scope 1 at $DIR/generator-tiny.rs:21:13: 21:18
+        StorageLive(_6);                 // scope 1 at $DIR/generator-tiny.rs:22:13: 22:18
+        StorageLive(_7);                 // scope 1 at $DIR/generator-tiny.rs:22:13: 22:18
+        _7 = ();                         // scope 1 at $DIR/generator-tiny.rs:22:13: 22:18
+        _0 = std::ops::GeneratorState::<(), ()>::Yielded(move _7); // scope 1 at $DIR/generator-tiny.rs:22:13: 22:18
+        discriminant((*(_1.0: &mut [generator@$DIR/generator-tiny.rs:19:16: 25:6 {u8, HasDrop, ()}]))) = 3; // scope 1 at $DIR/generator-tiny.rs:22:13: 22:18
+        return;                          // scope 1 at $DIR/generator-tiny.rs:22:13: 22:18
     }
 
     bb3: {
-        StorageDead(_7);                 // scope 1 at $DIR/generator-tiny.rs:21:17: 21:18
-        StorageDead(_6);                 // scope 1 at $DIR/generator-tiny.rs:21:18: 21:19
-        StorageLive(_8);                 // scope 1 at $DIR/generator-tiny.rs:22:13: 22:21
-        _8 = const callee() -> bb4;      // scope 1 at $DIR/generator-tiny.rs:22:13: 22:21
+        StorageDead(_7);                 // scope 1 at $DIR/generator-tiny.rs:22:17: 22:18
+        StorageDead(_6);                 // scope 1 at $DIR/generator-tiny.rs:22:18: 22:19
+        StorageLive(_8);                 // scope 1 at $DIR/generator-tiny.rs:23:13: 23:21
+        _8 = const callee() -> bb4;      // scope 1 at $DIR/generator-tiny.rs:23:13: 23:21
                                          // ty::Const
                                          // + ty: fn() {callee}
                                          // + val: Value(Scalar(<ZST>))
                                          // mir::Constant
-                                         // + span: $DIR/generator-tiny.rs:22:13: 22:19
+                                         // + span: $DIR/generator-tiny.rs:23:13: 23:19
                                          // + literal: Const { ty: fn() {callee}, val: Value(Scalar(<ZST>)) }
     }
 
     bb4: {
-        StorageDead(_8);                 // scope 1 at $DIR/generator-tiny.rs:22:21: 22:22
-        _5 = const ();                   // scope 1 at $DIR/generator-tiny.rs:20:14: 23:10
+        StorageDead(_8);                 // scope 1 at $DIR/generator-tiny.rs:23:21: 23:22
+        _5 = const ();                   // scope 1 at $DIR/generator-tiny.rs:21:14: 24:10
                                          // ty::Const
                                          // + ty: ()
                                          // + val: Value(Scalar(<ZST>))
                                          // mir::Constant
-                                         // + span: $DIR/generator-tiny.rs:20:14: 23:10
+                                         // + span: $DIR/generator-tiny.rs:21:14: 24:10
                                          // + literal: Const { ty: (), val: Value(Scalar(<ZST>)) }
-        goto -> bb2;                     // scope 1 at $DIR/generator-tiny.rs:20:9: 23:10
+        goto -> bb2;                     // scope 1 at $DIR/generator-tiny.rs:21:9: 24:10
     }
 
     bb5: {
-        StorageLive(_4);                 // scope 0 at $DIR/generator-tiny.rs:18:16: 24:6
-        StorageLive(_6);                 // scope 0 at $DIR/generator-tiny.rs:18:16: 24:6
-        StorageLive(_7);                 // scope 0 at $DIR/generator-tiny.rs:18:16: 24:6
-        _6 = move _2;                    // scope 0 at $DIR/generator-tiny.rs:18:16: 24:6
-        goto -> bb3;                     // scope 0 at $DIR/generator-tiny.rs:18:16: 24:6
+        StorageLive(_4);                 // scope 0 at $DIR/generator-tiny.rs:19:16: 25:6
+        StorageLive(_6);                 // scope 0 at $DIR/generator-tiny.rs:19:16: 25:6
+        StorageLive(_7);                 // scope 0 at $DIR/generator-tiny.rs:19:16: 25:6
+        _6 = move _2;                    // scope 0 at $DIR/generator-tiny.rs:19:16: 25:6
+        goto -> bb3;                     // scope 0 at $DIR/generator-tiny.rs:19:16: 25:6
     }
 
     bb6: {
-        unreachable;                     // scope 0 at $DIR/generator-tiny.rs:18:16: 24:6
+        unreachable;                     // scope 0 at $DIR/generator-tiny.rs:19:16: 25:6
     }
 }
diff --git a/src/test/run-fail/mir_codegen_no_landing_pads.rs b/src/test/run-fail/mir_codegen_no_landing_pads.rs
deleted file mode 100644
index f3384dc45f3..00000000000
--- a/src/test/run-fail/mir_codegen_no_landing_pads.rs
+++ /dev/null
@@ -1,27 +0,0 @@
-// compile-flags: -Z no-landing-pads -C codegen-units=1
-// error-pattern:converging_fn called
-// ignore-cloudabi no std::process
-
-use std::io::{self, Write};
-
-struct Droppable;
-impl Drop for Droppable {
-    fn drop(&mut self) {
-        ::std::process::exit(1)
-    }
-}
-
-fn converging_fn() {
-    panic!("converging_fn called")
-}
-
-fn mir(d: Droppable) {
-    let x = Droppable;
-    converging_fn();
-    drop(x);
-    drop(d);
-}
-
-fn main() {
-    mir(Droppable);
-}
diff --git a/src/test/run-fail/mir_codegen_no_landing_pads_diverging.rs b/src/test/run-fail/mir_codegen_no_landing_pads_diverging.rs
deleted file mode 100644
index 08f6d578bb2..00000000000
--- a/src/test/run-fail/mir_codegen_no_landing_pads_diverging.rs
+++ /dev/null
@@ -1,27 +0,0 @@
-// compile-flags: -Z no-landing-pads -C codegen-units=1
-// error-pattern:diverging_fn called
-// ignore-cloudabi no std::process
-
-use std::io::{self, Write};
-
-struct Droppable;
-impl Drop for Droppable {
-    fn drop(&mut self) {
-        ::std::process::exit(1)
-    }
-}
-
-fn diverging_fn() -> ! {
-    panic!("diverging_fn called")
-}
-
-fn mir(d: Droppable) {
-    let x = Droppable;
-    diverging_fn();
-    drop(x);
-    drop(d);
-}
-
-fn main() {
-    mir(Droppable);
-}