Skip checking for Storage* statements in constants/statics

rust-lang · Mar 22, 2018 · 9fa14e4 · 9fa14e4
1 parent 9839e5f
commit 9fa14e4
Showing 1 changed file with 17 additions and 10 deletions.
diff --git a/src/librustc_mir/interpret/eval_context.rs b/src/librustc_mir/interpret/eval_context.rs
@@ -1,6 +1,7 @@
 use std::fmt::Write;
 
 use rustc::hir::def_id::DefId;
+use rustc::hir::def::Def;
 use rustc::hir::map::definitions::DefPathData;
 use rustc::middle::const_val::{ConstVal, ErrKind};
 use rustc::mir;
@@ -387,17 +388,23 @@ impl<'a, 'mir, 'tcx: 'mir, M: Machine<'mir, 'tcx>> EvalContext<'a, 'mir, 'tcx, M
         let num_locals = mir.local_decls.len() - 1;
 
         let mut locals = vec![Some(Value::ByVal(PrimVal::Undef)); num_locals];
-        trace!("push_stack_frame: {:?}: num_bbs: {}", span, mir.basic_blocks().len());
-        for block in mir.basic_blocks() {
-            for stmt in block.statements.iter() {
-                use rustc::mir::StatementKind::{StorageDead, StorageLive};
-                match stmt.kind {
-                    StorageLive(local) | StorageDead(local) => if local.index() > 0 {
-                        locals[local.index() - 1] = None;
-                    },
-                    _ => {}
+        match self.tcx.describe_def(instance.def_id()) {
+            // statics and constants don't have `Storage*` statements, no need to look for them
+            Some(Def::Static(..)) | Some(Def::Const(..)) | Some(Def::AssociatedConst(..)) => {},
+            _ => {
+                trace!("push_stack_frame: {:?}: num_bbs: {}", span, mir.basic_blocks().len());
+                for block in mir.basic_blocks() {
+                    for stmt in block.statements.iter() {
+                        use rustc::mir::StatementKind::{StorageDead, StorageLive};
+                        match stmt.kind {
+                            StorageLive(local) | StorageDead(local) => if local.index() > 0 {
+                                locals[local.index() - 1] = None;
+                            },
+                            _ => {}
+                        }
+                    }
                 }
-            }
+            },
         }
 
         self.stack.push(Frame {