Cleanup(Option<_>) -> Cleanup(BasicBlock), Skip

Author: hyd-dev

compiler/rustc_mir/src/interpret/eval_context.rs

@@ -138,7 +138,9 @@ pub struct FrameInfo<'tcx> {
 #[derive(Clone, Copy, Eq, PartialEq, Debug, HashStable)]
 pub enum StackPopUnwind {
     /// The cleanup block.
-    Cleanup(Option<mir::BasicBlock>),
+    Cleanup(mir::BasicBlock),
+    /// No cleanup needs to be done.
+    Skip,
     /// Unwinding is not allowed (UB).
     NotAllowed,
 }
@@ -820,7 +822,8 @@ impl<'mir, 'tcx: 'mir, M: Machine<'mir, 'tcx>> InterpCx<'mir, 'tcx, M> {
             (StackPopCleanup::Goto { unwind, .. }, true) => (
                 true,
                 Some(match unwind {
-                    StackPopUnwind::Cleanup(unwind) => unwind,
+                    StackPopUnwind::Cleanup(unwind) => Some(unwind),
+                    StackPopUnwind::Skip => None,
                     StackPopUnwind::NotAllowed => {
                         throw_ub_format!("unwind past a frame that does not allow unwinding")
                     }

compiler/rustc_mir/src/interpret/terminator.rs

@@ -316,10 +316,10 @@ impl<'mir, 'tcx: 'mir, M: Machine<'mir, 'tcx>> InterpCx<'mir, 'tcx, M> {
                     ret.map(|p| p.0),
                     StackPopCleanup::Goto {
                         ret: ret.map(|p| p.1),
-                        unwind: if can_unwind {
-                            StackPopUnwind::Cleanup(unwind)
-                        } else {
-                            StackPopUnwind::NotAllowed
+                        unwind: match (unwind, can_unwind) {
+                            (Some(unwind), true) => StackPopUnwind::Cleanup(unwind),
+                            (None, true) => StackPopUnwind::Skip,
+                            (_, false) => StackPopUnwind::NotAllowed,
                         },
                     },
                 )?;