]> git.lizzy.rs Git - rust.git/blob - src/librustc_mir/transform/const_prop.rs
Rollup merge of #59315 - Zoxc:move-query, r=oli-obk
[rust.git] / src / librustc_mir / transform / const_prop.rs
1 //! Propagates constants for early reporting of statically known
2 //! assertion failures
3
4
5 use rustc::hir::def::Def;
6 use rustc::mir::{Constant, Location, Place, PlaceBase, Mir, Operand, Rvalue, Local};
7 use rustc::mir::{NullOp, UnOp, StatementKind, Statement, BasicBlock, LocalKind, Static, StaticKind};
8 use rustc::mir::{TerminatorKind, ClearCrossCrate, SourceInfo, BinOp, ProjectionElem};
9 use rustc::mir::visit::{Visitor, PlaceContext, MutatingUseContext, NonMutatingUseContext};
10 use rustc::mir::interpret::{EvalErrorKind, Scalar, GlobalId, EvalResult};
11 use rustc::ty::{TyCtxt, self, Instance};
12 use syntax::source_map::{Span, DUMMY_SP};
13 use rustc::ty::subst::InternalSubsts;
14 use rustc_data_structures::indexed_vec::IndexVec;
15 use rustc::ty::ParamEnv;
16 use rustc::ty::layout::{
17     LayoutOf, TyLayout, LayoutError,
18     HasTyCtxt, TargetDataLayout, HasDataLayout,
19 };
20
21 use crate::interpret::{EvalContext, ScalarMaybeUndef, Immediate, OpTy, ImmTy, MemoryKind};
22 use crate::const_eval::{
23     CompileTimeInterpreter, error_to_const_error, eval_promoted, mk_eval_cx,
24 };
25 use crate::transform::{MirPass, MirSource};
26
27 pub struct ConstProp;
28
29 impl MirPass for ConstProp {
30     fn run_pass<'a, 'tcx>(&self,
31                           tcx: TyCtxt<'a, 'tcx, 'tcx>,
32                           source: MirSource<'tcx>,
33                           mir: &mut Mir<'tcx>) {
34         // will be evaluated by miri and produce its errors there
35         if source.promoted.is_some() {
36             return;
37         }
38
39         use rustc::hir::map::blocks::FnLikeNode;
40         let hir_id = tcx.hir().as_local_hir_id(source.def_id())
41                               .expect("Non-local call to local provider is_const_fn");
42
43         let is_fn_like = FnLikeNode::from_node(tcx.hir().get_by_hir_id(hir_id)).is_some();
44         let is_assoc_const = match tcx.describe_def(source.def_id()) {
45             Some(Def::AssociatedConst(_)) => true,
46             _ => false,
47         };
48
49         // Only run const prop on functions, methods, closures and associated constants
50         if !is_fn_like && !is_assoc_const  {
51             // skip anon_const/statics/consts because they'll be evaluated by miri anyway
52             trace!("ConstProp skipped for {:?}", source.def_id());
53             return
54         }
55
56         trace!("ConstProp starting for {:?}", source.def_id());
57
58         // FIXME(oli-obk, eddyb) Optimize locals (or even local paths) to hold
59         // constants, instead of just checking for const-folding succeeding.
60         // That would require an uniform one-def no-mutation analysis
61         // and RPO (or recursing when needing the value of a local).
62         let mut optimization_finder = ConstPropagator::new(mir, tcx, source);
63         optimization_finder.visit_mir(mir);
64
65         trace!("ConstProp done for {:?}", source.def_id());
66     }
67 }
68
69 type Const<'tcx> = (OpTy<'tcx>, Span);
70
71 /// Finds optimization opportunities on the MIR.
72 struct ConstPropagator<'a, 'mir, 'tcx:'a+'mir> {
73     ecx: EvalContext<'a, 'mir, 'tcx, CompileTimeInterpreter<'a, 'mir, 'tcx>>,
74     mir: &'mir Mir<'tcx>,
75     tcx: TyCtxt<'a, 'tcx, 'tcx>,
76     source: MirSource<'tcx>,
77     places: IndexVec<Local, Option<Const<'tcx>>>,
78     can_const_prop: IndexVec<Local, bool>,
79     param_env: ParamEnv<'tcx>,
80 }
81
82 impl<'a, 'b, 'tcx> LayoutOf for ConstPropagator<'a, 'b, 'tcx> {
83     type Ty = ty::Ty<'tcx>;
84     type TyLayout = Result<TyLayout<'tcx>, LayoutError<'tcx>>;
85
86     fn layout_of(&self, ty: ty::Ty<'tcx>) -> Self::TyLayout {
87         self.tcx.layout_of(self.param_env.and(ty))
88     }
89 }
90
91 impl<'a, 'b, 'tcx> HasDataLayout for ConstPropagator<'a, 'b, 'tcx> {
92     #[inline]
93     fn data_layout(&self) -> &TargetDataLayout {
94         &self.tcx.data_layout
95     }
96 }
97
98 impl<'a, 'b, 'tcx> HasTyCtxt<'tcx> for ConstPropagator<'a, 'b, 'tcx> {
99     #[inline]
100     fn tcx<'c>(&'c self) -> TyCtxt<'c, 'tcx, 'tcx> {
101         self.tcx
102     }
103 }
104
105 impl<'a, 'mir, 'tcx> ConstPropagator<'a, 'mir, 'tcx> {
106     fn new(
107         mir: &'mir Mir<'tcx>,
108         tcx: TyCtxt<'a, 'tcx, 'tcx>,
109         source: MirSource<'tcx>,
110     ) -> ConstPropagator<'a, 'mir, 'tcx> {
111         let param_env = tcx.param_env(source.def_id());
112         let ecx = mk_eval_cx(tcx, tcx.def_span(source.def_id()), param_env);
113         ConstPropagator {
114             ecx,
115             mir,
116             tcx,
117             source,
118             param_env,
119             can_const_prop: CanConstProp::check(mir),
120             places: IndexVec::from_elem(None, &mir.local_decls),
121         }
122     }
123
124     fn use_ecx<F, T>(
125         &mut self,
126         source_info: SourceInfo,
127         f: F
128     ) -> Option<T>
129     where
130         F: FnOnce(&mut Self) -> EvalResult<'tcx, T>,
131     {
132         self.ecx.tcx.span = source_info.span;
133         let lint_root = match self.mir.source_scope_local_data {
134             ClearCrossCrate::Set(ref ivs) => {
135                 //FIXME(#51314): remove this check
136                 if source_info.scope.index() >= ivs.len() {
137                     return None;
138                 }
139                 ivs[source_info.scope].lint_root
140             },
141             ClearCrossCrate::Clear => return None,
142         };
143         let r = match f(self) {
144             Ok(val) => Some(val),
145             Err(error) => {
146                 let diagnostic = error_to_const_error(&self.ecx, error);
147                 use rustc::mir::interpret::EvalErrorKind::*;
148                 match diagnostic.error {
149                     // don't report these, they make no sense in a const prop context
150                     | MachineError(_)
151                     // at runtime these transformations might make sense
152                     // FIXME: figure out the rules and start linting
153                     | FunctionAbiMismatch(..)
154                     | FunctionArgMismatch(..)
155                     | FunctionRetMismatch(..)
156                     | FunctionArgCountMismatch
157                     // fine at runtime, might be a register address or sth
158                     | ReadBytesAsPointer
159                     // fine at runtime
160                     | ReadForeignStatic
161                     | Unimplemented(_)
162                     // don't report const evaluator limits
163                     | StackFrameLimitReached
164                     | NoMirFor(..)
165                     | InlineAsm
166                     => {},
167
168                     | InvalidMemoryAccess
169                     | DanglingPointerDeref
170                     | DoubleFree
171                     | InvalidFunctionPointer
172                     | InvalidBool
173                     | InvalidDiscriminant(..)
174                     | PointerOutOfBounds { .. }
175                     | InvalidNullPointerUsage
176                     | ValidationFailure(..)
177                     | InvalidPointerMath
178                     | ReadUndefBytes(_)
179                     | DeadLocal
180                     | InvalidBoolOp(_)
181                     | DerefFunctionPointer
182                     | ExecuteMemory
183                     | Intrinsic(..)
184                     | InvalidChar(..)
185                     | AbiViolation(_)
186                     | AlignmentCheckFailed{..}
187                     | CalledClosureAsFunction
188                     | VtableForArgumentlessMethod
189                     | ModifiedConstantMemory
190                     | ModifiedStatic
191                     | AssumptionNotHeld
192                     // FIXME: should probably be removed and turned into a bug! call
193                     | TypeNotPrimitive(_)
194                     | ReallocatedWrongMemoryKind(_, _)
195                     | DeallocatedWrongMemoryKind(_, _)
196                     | ReallocateNonBasePtr
197                     | DeallocateNonBasePtr
198                     | IncorrectAllocationInformation(..)
199                     | UnterminatedCString(_)
200                     | HeapAllocZeroBytes
201                     | HeapAllocNonPowerOfTwoAlignment(_)
202                     | Unreachable
203                     | ReadFromReturnPointer
204                     | GeneratorResumedAfterReturn
205                     | GeneratorResumedAfterPanic
206                     | ReferencedConstant
207                     | InfiniteLoop
208                     => {
209                         // FIXME: report UB here
210                     },
211
212                     | OutOfTls
213                     | TlsOutOfBounds
214                     | PathNotFound(_)
215                     => bug!("these should not be in rustc, but in miri's machine errors"),
216
217                     | Layout(_)
218                     | UnimplementedTraitSelection
219                     | TypeckError
220                     | TooGeneric
221                     // these are just noise
222                     => {},
223
224                     // non deterministic
225                     | ReadPointerAsBytes
226                     // FIXME: implement
227                     => {},
228
229                     | Panic { .. }
230                     | BoundsCheck{..}
231                     | Overflow(_)
232                     | OverflowNeg
233                     | DivisionByZero
234                     | RemainderByZero
235                     => {
236                         diagnostic.report_as_lint(
237                             self.ecx.tcx,
238                             "this expression will panic at runtime",
239                             lint_root,
240                             None,
241                         );
242                     }
243                 }
244                 None
245             },
246         };
247         self.ecx.tcx.span = DUMMY_SP;
248         r
249     }
250
251     fn eval_constant(
252         &mut self,
253         c: &Constant<'tcx>,
254         source_info: SourceInfo,
255     ) -> Option<Const<'tcx>> {
256         self.ecx.tcx.span = source_info.span;
257         match self.ecx.eval_const_to_op(*c.literal, None) {
258             Ok(op) => {
259                 Some((op, c.span))
260             },
261             Err(error) => {
262                 let err = error_to_const_error(&self.ecx, error);
263                 err.report_as_error(self.ecx.tcx, "erroneous constant used");
264                 None
265             },
266         }
267     }
268
269     fn eval_place(&mut self, place: &Place<'tcx>, source_info: SourceInfo) -> Option<Const<'tcx>> {
270         match *place {
271             Place::Base(PlaceBase::Local(loc)) => self.places[loc].clone(),
272             Place::Projection(ref proj) => match proj.elem {
273                 ProjectionElem::Field(field, _) => {
274                     trace!("field proj on {:?}", proj.base);
275                     let (base, span) = self.eval_place(&proj.base, source_info)?;
276                     let res = self.use_ecx(source_info, |this| {
277                         this.ecx.operand_field(base, field.index() as u64)
278                     })?;
279                     Some((res, span))
280                 },
281                 // We could get more projections by using e.g., `operand_projection`,
282                 // but we do not even have the stack frame set up properly so
283                 // an `Index` projection would throw us off-track.
284                 _ => None,
285             },
286             Place::Base(
287                 PlaceBase::Static(box Static {kind: StaticKind::Promoted(promoted), ..})
288             ) => {
289                 let generics = self.tcx.generics_of(self.source.def_id());
290                 if generics.requires_monomorphization(self.tcx) {
291                     // FIXME: can't handle code with generics
292                     return None;
293                 }
294                 let substs = InternalSubsts::identity_for_item(self.tcx, self.source.def_id());
295                 let instance = Instance::new(self.source.def_id(), substs);
296                 let cid = GlobalId {
297                     instance,
298                     promoted: Some(promoted),
299                 };
300                 // cannot use `const_eval` here, because that would require having the MIR
301                 // for the current function available, but we're producing said MIR right now
302                 let res = self.use_ecx(source_info, |this| {
303                     eval_promoted(this.tcx, cid, this.mir, this.param_env)
304                 })?;
305                 trace!("evaluated promoted {:?} to {:?}", promoted, res);
306                 Some((res.into(), source_info.span))
307             },
308             _ => None,
309         }
310     }
311
312     fn eval_operand(&mut self, op: &Operand<'tcx>, source_info: SourceInfo) -> Option<Const<'tcx>> {
313         match *op {
314             Operand::Constant(ref c) => self.eval_constant(c, source_info),
315             | Operand::Move(ref place)
316             | Operand::Copy(ref place) => self.eval_place(place, source_info),
317         }
318     }
319
320     fn const_prop(
321         &mut self,
322         rvalue: &Rvalue<'tcx>,
323         place_layout: TyLayout<'tcx>,
324         source_info: SourceInfo,
325     ) -> Option<Const<'tcx>> {
326         let span = source_info.span;
327         match *rvalue {
328             Rvalue::Use(ref op) => {
329                 self.eval_operand(op, source_info)
330             },
331             Rvalue::Repeat(..) |
332             Rvalue::Ref(..) |
333             Rvalue::Aggregate(..) |
334             Rvalue::NullaryOp(NullOp::Box, _) |
335             Rvalue::Discriminant(..) => None,
336
337             Rvalue::Cast(kind, ref operand, _) => {
338                 let (op, span) = self.eval_operand(operand, source_info)?;
339                 self.use_ecx(source_info, |this| {
340                     let dest = this.ecx.allocate(place_layout, MemoryKind::Stack);
341                     this.ecx.cast(op, kind, dest.into())?;
342                     Ok((dest.into(), span))
343                 })
344             }
345
346             // FIXME(oli-obk): evaluate static/constant slice lengths
347             Rvalue::Len(_) => None,
348             Rvalue::NullaryOp(NullOp::SizeOf, ty) => {
349                 type_size_of(self.tcx, self.param_env, ty).and_then(|n| Some((
350                     ImmTy {
351                         imm: Immediate::Scalar(
352                             Scalar::Bits {
353                                 bits: n as u128,
354                                 size: self.tcx.data_layout.pointer_size.bytes() as u8,
355                             }.into()
356                         ),
357                         layout: self.tcx.layout_of(self.param_env.and(self.tcx.types.usize)).ok()?,
358                     }.into(),
359                     span,
360                 )))
361             }
362             Rvalue::UnaryOp(op, ref arg) => {
363                 let def_id = if self.tcx.is_closure(self.source.def_id()) {
364                     self.tcx.closure_base_def_id(self.source.def_id())
365                 } else {
366                     self.source.def_id()
367                 };
368                 let generics = self.tcx.generics_of(def_id);
369                 if generics.requires_monomorphization(self.tcx) {
370                     // FIXME: can't handle code with generics
371                     return None;
372                 }
373
374                 let (arg, _) = self.eval_operand(arg, source_info)?;
375                 let val = self.use_ecx(source_info, |this| {
376                     let prim = this.ecx.read_immediate(arg)?;
377                     match op {
378                         UnOp::Neg => {
379                             // Need to do overflow check here: For actual CTFE, MIR
380                             // generation emits code that does this before calling the op.
381                             if prim.to_bits()? == (1 << (prim.layout.size.bits() - 1)) {
382                                 return err!(OverflowNeg);
383                             }
384                         }
385                         UnOp::Not => {
386                             // Cannot overflow
387                         }
388                     }
389                     // Now run the actual operation.
390                     this.ecx.unary_op(op, prim)
391                 })?;
392                 let res = ImmTy {
393                     imm: Immediate::Scalar(val.into()),
394                     layout: place_layout,
395                 };
396                 Some((res.into(), span))
397             }
398             Rvalue::CheckedBinaryOp(op, ref left, ref right) |
399             Rvalue::BinaryOp(op, ref left, ref right) => {
400                 trace!("rvalue binop {:?} for {:?} and {:?}", op, left, right);
401                 let right = self.eval_operand(right, source_info)?;
402                 let def_id = if self.tcx.is_closure(self.source.def_id()) {
403                     self.tcx.closure_base_def_id(self.source.def_id())
404                 } else {
405                     self.source.def_id()
406                 };
407                 let generics = self.tcx.generics_of(def_id);
408                 if generics.requires_monomorphization(self.tcx) {
409                     // FIXME: can't handle code with generics
410                     return None;
411                 }
412
413                 let r = self.use_ecx(source_info, |this| {
414                     this.ecx.read_immediate(right.0)
415                 })?;
416                 if op == BinOp::Shr || op == BinOp::Shl {
417                     let left_ty = left.ty(self.mir, self.tcx);
418                     let left_bits = self
419                         .tcx
420                         .layout_of(self.param_env.and(left_ty))
421                         .unwrap()
422                         .size
423                         .bits();
424                     let right_size = right.0.layout.size;
425                     let r_bits = r.to_scalar().and_then(|r| r.to_bits(right_size));
426                     if r_bits.ok().map_or(false, |b| b >= left_bits as u128) {
427                         let source_scope_local_data = match self.mir.source_scope_local_data {
428                             ClearCrossCrate::Set(ref data) => data,
429                             ClearCrossCrate::Clear => return None,
430                         };
431                         let dir = if op == BinOp::Shr {
432                             "right"
433                         } else {
434                             "left"
435                         };
436                         let hir_id = source_scope_local_data[source_info.scope].lint_root;
437                         self.tcx.lint_hir(
438                             ::rustc::lint::builtin::EXCEEDING_BITSHIFTS,
439                             hir_id,
440                             span,
441                             &format!("attempt to shift {} with overflow", dir));
442                         return None;
443                     }
444                 }
445                 let left = self.eval_operand(left, source_info)?;
446                 let l = self.use_ecx(source_info, |this| {
447                     this.ecx.read_immediate(left.0)
448                 })?;
449                 trace!("const evaluating {:?} for {:?} and {:?}", op, left, right);
450                 let (val, overflow) = self.use_ecx(source_info, |this| {
451                     this.ecx.binary_op(op, l, r)
452                 })?;
453                 let val = if let Rvalue::CheckedBinaryOp(..) = *rvalue {
454                     Immediate::ScalarPair(
455                         val.into(),
456                         Scalar::from_bool(overflow).into(),
457                     )
458                 } else {
459                     if overflow {
460                         let err = EvalErrorKind::Overflow(op).into();
461                         let _: Option<()> = self.use_ecx(source_info, |_| Err(err));
462                         return None;
463                     }
464                     Immediate::Scalar(val.into())
465                 };
466                 let res = ImmTy {
467                     imm: val,
468                     layout: place_layout,
469                 };
470                 Some((res.into(), span))
471             },
472         }
473     }
474 }
475
476 fn type_size_of<'a, 'tcx>(tcx: TyCtxt<'a, 'tcx, 'tcx>,
477                           param_env: ty::ParamEnv<'tcx>,
478                           ty: ty::Ty<'tcx>) -> Option<u64> {
479     tcx.layout_of(param_env.and(ty)).ok().map(|layout| layout.size.bytes())
480 }
481
482 struct CanConstProp {
483     can_const_prop: IndexVec<Local, bool>,
484     // false at the beginning, once set, there are not allowed to be any more assignments
485     found_assignment: IndexVec<Local, bool>,
486 }
487
488 impl CanConstProp {
489     /// returns true if `local` can be propagated
490     fn check(mir: &Mir<'_>) -> IndexVec<Local, bool> {
491         let mut cpv = CanConstProp {
492             can_const_prop: IndexVec::from_elem(true, &mir.local_decls),
493             found_assignment: IndexVec::from_elem(false, &mir.local_decls),
494         };
495         for (local, val) in cpv.can_const_prop.iter_enumerated_mut() {
496             // cannot use args at all
497             // cannot use locals because if x < y { y - x } else { x - y } would
498             //        lint for x != y
499             // FIXME(oli-obk): lint variables until they are used in a condition
500             // FIXME(oli-obk): lint if return value is constant
501             *val = mir.local_kind(local) == LocalKind::Temp;
502         }
503         cpv.visit_mir(mir);
504         cpv.can_const_prop
505     }
506 }
507
508 impl<'tcx> Visitor<'tcx> for CanConstProp {
509     fn visit_local(
510         &mut self,
511         &local: &Local,
512         context: PlaceContext<'tcx>,
513         _: Location,
514     ) {
515         use rustc::mir::visit::PlaceContext::*;
516         match context {
517             // Constants must have at most one write
518             // FIXME(oli-obk): we could be more powerful here, if the multiple writes
519             // only occur in independent execution paths
520             MutatingUse(MutatingUseContext::Store) => if self.found_assignment[local] {
521                 self.can_const_prop[local] = false;
522             } else {
523                 self.found_assignment[local] = true
524             },
525             // Reading constants is allowed an arbitrary number of times
526             NonMutatingUse(NonMutatingUseContext::Copy) |
527             NonMutatingUse(NonMutatingUseContext::Move) |
528             NonMutatingUse(NonMutatingUseContext::Inspect) |
529             NonMutatingUse(NonMutatingUseContext::Projection) |
530             MutatingUse(MutatingUseContext::Projection) |
531             NonUse(_) => {},
532             _ => self.can_const_prop[local] = false,
533         }
534     }
535 }
536
537 impl<'b, 'a, 'tcx> Visitor<'tcx> for ConstPropagator<'b, 'a, 'tcx> {
538     fn visit_constant(
539         &mut self,
540         constant: &Constant<'tcx>,
541         location: Location,
542     ) {
543         trace!("visit_constant: {:?}", constant);
544         self.super_constant(constant, location);
545         let source_info = *self.mir.source_info(location);
546         self.eval_constant(constant, source_info);
547     }
548
549     fn visit_statement(
550         &mut self,
551         block: BasicBlock,
552         statement: &Statement<'tcx>,
553         location: Location,
554     ) {
555         trace!("visit_statement: {:?}", statement);
556         if let StatementKind::Assign(ref place, ref rval) = statement.kind {
557             let place_ty: ty::Ty<'tcx> = place
558                 .ty(&self.mir.local_decls, self.tcx)
559                 .to_ty(self.tcx);
560             if let Ok(place_layout) = self.tcx.layout_of(self.param_env.and(place_ty)) {
561                 if let Some(value) = self.const_prop(rval, place_layout, statement.source_info) {
562                     if let Place::Base(PlaceBase::Local(local)) = *place {
563                         trace!("checking whether {:?} can be stored to {:?}", value, local);
564                         if self.can_const_prop[local] {
565                             trace!("storing {:?} to {:?}", value, local);
566                             assert!(self.places[local].is_none());
567                             self.places[local] = Some(value);
568                         }
569                     }
570                 }
571             }
572         }
573         self.super_statement(block, statement, location);
574     }
575
576     fn visit_terminator_kind(
577         &mut self,
578         block: BasicBlock,
579         kind: &TerminatorKind<'tcx>,
580         location: Location,
581     ) {
582         self.super_terminator_kind(block, kind, location);
583         let source_info = *self.mir.source_info(location);
584         if let TerminatorKind::Assert { expected, msg, cond, .. } = kind {
585             if let Some(value) = self.eval_operand(cond, source_info) {
586                 trace!("assertion on {:?} should be {:?}", value, expected);
587                 let expected = ScalarMaybeUndef::from(Scalar::from_bool(*expected));
588                 if expected != self.ecx.read_scalar(value.0).unwrap() {
589                     // poison all places this operand references so that further code
590                     // doesn't use the invalid value
591                     match cond {
592                         Operand::Move(ref place) | Operand::Copy(ref place) => {
593                             let mut place = place;
594                             while let Place::Projection(ref proj) = *place {
595                                 place = &proj.base;
596                             }
597                             if let Place::Base(PlaceBase::Local(local)) = *place {
598                                 self.places[local] = None;
599                             }
600                         },
601                         Operand::Constant(_) => {}
602                     }
603                     let span = self.mir[block]
604                         .terminator
605                         .as_ref()
606                         .unwrap()
607                         .source_info
608                         .span;
609                     let hir_id = self
610                         .tcx
611                         .hir()
612                         .as_local_hir_id(self.source.def_id())
613                         .expect("some part of a failing const eval must be local");
614                     use rustc::mir::interpret::EvalErrorKind::*;
615                     let msg = match msg {
616                         Overflow(_) |
617                         OverflowNeg |
618                         DivisionByZero |
619                         RemainderByZero => msg.description().to_owned(),
620                         BoundsCheck { ref len, ref index } => {
621                             let len = self
622                                 .eval_operand(len, source_info)
623                                 .expect("len must be const");
624                             let len = match self.ecx.read_scalar(len.0) {
625                                 Ok(ScalarMaybeUndef::Scalar(Scalar::Bits {
626                                     bits, ..
627                                 })) => bits,
628                                 other => bug!("const len not primitive: {:?}", other),
629                             };
630                             let index = self
631                                 .eval_operand(index, source_info)
632                                 .expect("index must be const");
633                             let index = match self.ecx.read_scalar(index.0) {
634                                 Ok(ScalarMaybeUndef::Scalar(Scalar::Bits {
635                                     bits, ..
636                                 })) => bits,
637                                 other => bug!("const index not primitive: {:?}", other),
638                             };
639                             format!(
640                                 "index out of bounds: \
641                                 the len is {} but the index is {}",
642                                 len,
643                                 index,
644                             )
645                         },
646                         // Need proper const propagator for these
647                         _ => return,
648                     };
649                     self.tcx.lint_hir(
650                         ::rustc::lint::builtin::CONST_ERR,
651                         hir_id,
652                         span,
653                         &msg,
654                     );
655                 }
656             }
657         }
658     }
659 }