about summary refs log tree commit diff
diff options
context:
space:
mode:
authorDylan MacKenzie <ecstaticmorse@gmail.com>2019-12-01 11:43:19 -0800
committerDylan MacKenzie <ecstaticmorse@gmail.com>2019-12-01 11:43:24 -0800
commitccb4eed3529eaf610f476dd6d78ecd159a897b64 (patch)
treedaac97fb1e96ee1f9c445b2e4430ab66105ab56f
parent846be82277fef758e033538d7d49a2e834e472c4 (diff)
downloadrust-ccb4eed3529eaf610f476dd6d78ecd159a897b64.tar.gz
rust-ccb4eed3529eaf610f476dd6d78ecd159a897b64.zip
Incorporate fixes from review
-rw-r--r--src/librustc_mir/transform/check_consts/validation.rs8
1 files changed, 3 insertions, 5 deletions
diff --git a/src/librustc_mir/transform/check_consts/validation.rs b/src/librustc_mir/transform/check_consts/validation.rs
index bc13764f0a3..68bfa50a952 100644
--- a/src/librustc_mir/transform/check_consts/validation.rs
+++ b/src/librustc_mir/transform/check_consts/validation.rs
@@ -364,12 +364,11 @@ impl Visitor<'tcx> for Validator<'_, 'mir, 'tcx> {
                 }
             }
 
-            // Taking a shared borrow of a `static` is always legal, even if that `static` has
-            // interior mutability.
+            // At the moment, `PlaceBase::Static` is only used for promoted MIR.
             | Rvalue::Ref(_, BorrowKind::Shared, ref place)
             | Rvalue::Ref(_, BorrowKind::Shallow, ref place)
             if matches!(place.base, PlaceBase::Static(_))
-            => {}
+            => bug!("Saw a promoted during const-checking, which must run before promotion"),
 
             | Rvalue::Ref(_, kind @ BorrowKind::Shared, ref place)
             | Rvalue::Ref(_, kind @ BorrowKind::Shallow, ref place)
@@ -708,8 +707,7 @@ fn place_as_reborrow(
             // A borrow of a `static` also looks like `&(*_1)` in the MIR, but `_1` is a `const`
             // that points to the allocation for the static. Don't treat these as reborrows.
             if let PlaceBase::Local(local) = place.base {
-                let decl = &body.local_decls[local];
-                if let LocalInfo::StaticRef { .. } = decl.local_info {
+                if body.local_decls[local].is_ref_to_static() {
                     return None;
                 }
             }