1 //! See The Book chapter on the borrow checker for more details.
3 #![allow(non_camel_case_types)]
5 pub use LoanPathKind::*;
6 pub use LoanPathElem::*;
7 pub use bckerr_code::*;
8 pub use AliasableViolationKind::*;
9 pub use MovedValueUseKind::*;
13 use rustc::hir::HirId;
15 use rustc::hir::map::blocks::FnLikeNode;
17 use rustc::middle::borrowck::{BorrowCheckResult, SignalledError};
18 use rustc::hir::def_id::{DefId, LocalDefId};
19 use rustc::middle::expr_use_visitor as euv;
20 use rustc::middle::mem_categorization as mc;
21 use rustc::middle::mem_categorization::Categorization;
22 use rustc::middle::mem_categorization::ImmutabilityBlame;
23 use rustc::middle::region;
24 use rustc::middle::free_region::RegionRelations;
25 use rustc::ty::{self, Ty, TyCtxt};
26 use rustc::ty::query::Providers;
27 use rustc_mir::util::borrowck_errors::{BorrowckErrors, Origin};
28 use rustc_mir::util::suggest_ref_mut;
29 use rustc::util::nodemap::FxHashSet;
32 use std::cell::{Cell, RefCell};
35 use std::hash::{Hash, Hasher};
36 use syntax::source_map::CompilerDesugaringKind;
37 use syntax_pos::{MultiSpan, Span};
38 use errors::{Applicability, DiagnosticBuilder, DiagnosticId};
43 use crate::dataflow::{DataFlowContext, BitwiseOperator, DataFlowOperator, KillFrom};
51 #[derive(Clone, Copy)]
52 pub struct LoanDataFlowOperator;
54 pub type LoanDataFlow<'tcx> = DataFlowContext<'tcx, LoanDataFlowOperator>;
56 pub fn check_crate<'tcx>(tcx: TyCtxt<'tcx>) {
57 tcx.par_body_owners(|body_owner_def_id| {
58 tcx.ensure().borrowck(body_owner_def_id);
62 pub fn provide(providers: &mut Providers<'_>) {
63 *providers = Providers {
69 /// Collection of conclusions determined via borrow checker analyses.
70 pub struct AnalysisData<'tcx> {
71 pub all_loans: Vec<Loan<'tcx>>,
72 pub loans: DataFlowContext<'tcx, LoanDataFlowOperator>,
73 pub move_data: move_data::FlowedMoveData<'tcx>,
76 fn borrowck<'tcx>(tcx: TyCtxt<'tcx>, owner_def_id: DefId) -> &'tcx BorrowCheckResult {
77 assert!(tcx.use_ast_borrowck() || tcx.migrate_borrowck());
79 debug!("borrowck(body_owner_def_id={:?})", owner_def_id);
81 let owner_id = tcx.hir().as_local_hir_id(owner_def_id).unwrap();
83 match tcx.hir().get(owner_id) {
85 // We get invoked with anything that has MIR, but some of
86 // those things (notably the synthesized constructors from
87 // tuple structs/variants) do not have an associated body
88 // and do not need borrowchecking.
89 return tcx.arena.alloc(BorrowCheckResult {
90 used_mut_nodes: Default::default(),
91 signalled_any_error: SignalledError::NoErrorsSeen,
97 let body_id = tcx.hir().body_owned_by(owner_id);
98 let tables = tcx.typeck_tables_of(owner_def_id);
99 let region_scope_tree = tcx.region_scope_tree(owner_def_id);
100 let body = tcx.hir().body(body_id);
101 let mut bccx = BorrowckCtxt {
107 used_mut_nodes: Default::default(),
108 signalled_any_error: Cell::new(SignalledError::NoErrorsSeen),
111 // Eventually, borrowck will always read the MIR, but at the
112 // moment we do not. So, for now, we always force MIR to be
113 // constructed for a given fn, since this may result in errors
114 // being reported and we want that to happen.
116 // Note that `mir_validated` is a "stealable" result; the
117 // thief, `optimized_mir()`, forces borrowck, so we know that
118 // is not yet stolen.
119 tcx.ensure().mir_validated(owner_def_id);
121 // option dance because you can't capture an uninitialized variable
124 if let Some(AnalysisData { all_loans,
126 move_data: flowed_moves }) =
127 build_borrowck_dataflow_data(&mut bccx, false, body_id,
129 cfg = Some(cfg::CFG::new(bccx.tcx, &body));
130 cfg.as_mut().unwrap()
133 check_loans::check_loans(&mut bccx, &loan_dfcx, &flowed_moves, &all_loans, body);
136 tcx.arena.alloc(BorrowCheckResult {
137 used_mut_nodes: bccx.used_mut_nodes.into_inner(),
138 signalled_any_error: bccx.signalled_any_error.into_inner(),
142 fn build_borrowck_dataflow_data<'a, 'c, 'tcx, F>(
143 this: &mut BorrowckCtxt<'a, 'tcx>,
144 force_analysis: bool,
145 body_id: hir::BodyId,
147 ) -> Option<AnalysisData<'tcx>>
149 F: FnOnce(&mut BorrowckCtxt<'a, 'tcx>) -> &'c cfg::CFG,
151 // Check the body of fn items.
152 let (all_loans, move_data) =
153 gather_loans::gather_loans_in_fn(this, body_id);
155 if !force_analysis && move_data.is_empty() && all_loans.is_empty() {
156 // large arrays of data inserted as constants can take a lot of
157 // time and memory to borrow-check - see issue #36799. However,
158 // they don't have places, so no borrow-check is actually needed.
159 // Recognize that case and skip borrow-checking.
160 debug!("skipping loan propagation for {:?} because of no loans", body_id);
163 debug!("propagating loans in {:?}", body_id);
166 let cfg = get_cfg(this);
168 DataFlowContext::new(this.tcx,
172 LoanDataFlowOperator,
174 for (loan_idx, loan) in all_loans.iter().enumerate() {
175 loan_dfcx.add_gen(loan.gen_scope.item_local_id(), loan_idx);
176 loan_dfcx.add_kill(KillFrom::ScopeEnd,
177 loan.kill_scope.item_local_id(),
180 loan_dfcx.add_kills_from_flow_exits(cfg);
181 loan_dfcx.propagate(cfg, this.body);
183 let flowed_moves = move_data::FlowedMoveData::new(move_data,
188 Some(AnalysisData { all_loans,
190 move_data:flowed_moves })
193 /// Accessor for introspective clients inspecting `AnalysisData` and
194 /// the `BorrowckCtxt` itself , e.g., the flowgraph visualizer.
195 pub fn build_borrowck_dataflow_data_for_fn<'a, 'tcx>(
197 body_id: hir::BodyId,
199 ) -> (BorrowckCtxt<'a, 'tcx>, AnalysisData<'tcx>) {
200 let owner_id = tcx.hir().body_owner(body_id);
201 let owner_def_id = tcx.hir().local_def_id_from_hir_id(owner_id);
202 let tables = tcx.typeck_tables_of(owner_def_id);
203 let region_scope_tree = tcx.region_scope_tree(owner_def_id);
204 let body = tcx.hir().body(body_id);
205 let mut bccx = BorrowckCtxt {
211 used_mut_nodes: Default::default(),
212 signalled_any_error: Cell::new(SignalledError::NoErrorsSeen),
215 let dataflow_data = build_borrowck_dataflow_data(&mut bccx, true, body_id, |_| cfg);
216 (bccx, dataflow_data.unwrap())
219 // ----------------------------------------------------------------------
222 pub struct BorrowckCtxt<'a, 'tcx> {
225 // tables for the current thing we are checking; set to
226 // Some in `borrowck_fn` and cleared later
227 tables: &'a ty::TypeckTables<'tcx>,
229 region_scope_tree: &'tcx region::ScopeTree,
233 body: &'tcx hir::Body,
235 used_mut_nodes: RefCell<FxHashSet<HirId>>,
237 signalled_any_error: Cell<SignalledError>,
240 impl BorrowckCtxt<'_, 'tcx> {
241 fn signal_error(&self) {
242 self.signalled_any_error.set(SignalledError::SawSomeError);
246 impl BorrowckErrors<'a> for &'a BorrowckCtxt<'_, 'tcx> {
247 fn struct_span_err_with_code<S: Into<MultiSpan>>(self,
251 -> DiagnosticBuilder<'a>
253 self.tcx.sess.struct_span_err_with_code(sp, msg, code)
256 fn struct_span_err<S: Into<MultiSpan>>(self,
259 -> DiagnosticBuilder<'a>
261 self.tcx.sess.struct_span_err(sp, msg)
264 fn cancel_if_wrong_origin(self,
265 mut diag: DiagnosticBuilder<'a>,
267 -> DiagnosticBuilder<'a>
269 if !o.should_emit_errors(self.tcx.borrowck_mode()) {
270 self.tcx.sess.diagnostic().cancel(&mut diag);
276 ///////////////////////////////////////////////////////////////////////////
277 // Loans and loan paths
279 /// Record of a loan that was issued.
280 pub struct Loan<'tcx> {
282 loan_path: Rc<LoanPath<'tcx>>,
283 kind: ty::BorrowKind,
284 restricted_paths: Vec<Rc<LoanPath<'tcx>>>,
286 /// gen_scope indicates where loan is introduced. Typically the
287 /// loan is introduced at the point of the borrow, but in some
288 /// cases, notably method arguments, the loan may be introduced
289 /// only later, once it comes into scope. See also
290 /// `GatherLoanCtxt::compute_gen_scope`.
291 gen_scope: region::Scope,
293 /// kill_scope indicates when the loan goes out of scope. This is
294 /// either when the lifetime expires or when the local variable
295 /// which roots the loan-path goes out of scope, whichever happens
296 /// faster. See also `GatherLoanCtxt::compute_kill_scope`.
297 kill_scope: region::Scope,
299 cause: euv::LoanCause,
302 impl<'tcx> Loan<'tcx> {
303 pub fn loan_path(&self) -> Rc<LoanPath<'tcx>> {
304 self.loan_path.clone()
309 pub struct LoanPath<'tcx> {
310 kind: LoanPathKind<'tcx>,
314 impl<'tcx> PartialEq for LoanPath<'tcx> {
315 fn eq(&self, that: &LoanPath<'tcx>) -> bool {
316 self.kind == that.kind
320 impl<'tcx> Hash for LoanPath<'tcx> {
321 fn hash<H: Hasher>(&self, state: &mut H) {
322 self.kind.hash(state);
326 #[derive(PartialEq, Eq, Hash, Debug)]
327 pub enum LoanPathKind<'tcx> {
328 LpVar(hir::HirId), // `x` in README.md
329 LpUpvar(ty::UpvarId), // `x` captured by-value into closure
330 LpDowncast(Rc<LoanPath<'tcx>>, DefId), // `x` downcast to particular enum variant
331 LpExtend(Rc<LoanPath<'tcx>>, mc::MutabilityCategory, LoanPathElem<'tcx>)
334 impl<'tcx> LoanPath<'tcx> {
335 fn new(kind: LoanPathKind<'tcx>, ty: Ty<'tcx>) -> LoanPath<'tcx> {
336 LoanPath { kind: kind, ty: ty }
339 fn to_type(&self) -> Ty<'tcx> { self.ty }
341 fn has_downcast(&self) -> bool {
343 LpDowncast(_, _) => true,
344 LpExtend(ref lp, _, LpInterior(_, _)) => {
352 // FIXME (pnkfelix): See discussion here
353 // https://github.com/pnkfelix/rust/commit/
354 // b2b39e8700e37ad32b486b9a8409b50a8a53aa51#commitcomment-7892003
355 const DOWNCAST_PRINTED_OPERATOR: &'static str = " as ";
357 // A local, "cleaned" version of `mc::InteriorKind` that drops
358 // information that is not relevant to loan-path analysis. (In
359 // particular, the distinction between how precisely an array-element
360 // is tracked is irrelevant here.)
361 #[derive(Clone, Copy, PartialEq, Eq, Hash)]
362 pub enum InteriorKind {
363 InteriorField(mc::FieldIndex),
367 trait ToInteriorKind { fn cleaned(self) -> InteriorKind; }
368 impl ToInteriorKind for mc::InteriorKind {
369 fn cleaned(self) -> InteriorKind {
371 mc::InteriorField(name) => InteriorField(name),
372 mc::InteriorElement(_) => InteriorElement,
378 // - a pointer dereference (`*P` in README.md)
379 // - a field reference, with an optional definition of the containing
380 // enum variant (`P.f` in README.md)
381 // `DefId` is present when the field is part of struct that is in
382 // a variant of an enum. For instance in:
383 // `enum E { X { foo: u32 }, Y { foo: u32 }}`
384 // each `foo` is qualified by the definitition id of the variant (`X` or `Y`).
385 #[derive(Copy, Clone, PartialEq, Eq, Hash, Debug)]
386 pub enum LoanPathElem<'tcx> {
387 LpDeref(mc::PointerKind<'tcx>),
388 LpInterior(Option<DefId>, InteriorKind),
391 fn closure_to_block(closure_id: LocalDefId, tcx: TyCtxt<'_>) -> HirId {
392 let closure_id = tcx.hir().local_def_id_to_hir_id(closure_id);
393 match tcx.hir().get(closure_id) {
394 Node::Expr(expr) => match expr.node {
395 hir::ExprKind::Closure(.., body_id, _, _) => {
399 bug!("encountered non-closure id: {}", closure_id)
402 _ => bug!("encountered non-expr id: {}", closure_id)
406 impl LoanPath<'tcx> {
407 pub fn kill_scope(&self, bccx: &BorrowckCtxt<'_, 'tcx>) -> region::Scope {
410 bccx.region_scope_tree.var_scope(hir_id.local_id)
412 LpUpvar(upvar_id) => {
413 let block_id = closure_to_block(upvar_id.closure_expr_id, bccx.tcx);
414 region::Scope { id: block_id.local_id, data: region::ScopeData::Node }
416 LpDowncast(ref base, _) |
417 LpExtend(ref base, ..) => base.kill_scope(bccx),
421 fn has_fork(&self, other: &LoanPath<'tcx>) -> bool {
422 match (&self.kind, &other.kind) {
423 (&LpExtend(ref base, _, LpInterior(opt_variant_id, id)),
424 &LpExtend(ref base2, _, LpInterior(opt_variant_id2, id2))) =>
425 if id == id2 && opt_variant_id == opt_variant_id2 {
426 base.has_fork(&base2)
430 (&LpExtend(ref base, _, LpDeref(_)), _) => base.has_fork(other),
431 (_, &LpExtend(ref base, _, LpDeref(_))) => self.has_fork(&base),
436 fn depth(&self) -> usize {
438 LpExtend(ref base, _, LpDeref(_)) => base.depth(),
439 LpExtend(ref base, _, LpInterior(..)) => base.depth() + 1,
444 fn common(&self, other: &LoanPath<'tcx>) -> Option<LoanPath<'tcx>> {
445 match (&self.kind, &other.kind) {
446 (&LpExtend(ref base, a, LpInterior(opt_variant_id, id)),
447 &LpExtend(ref base2, _, LpInterior(opt_variant_id2, id2))) => {
448 if id == id2 && opt_variant_id == opt_variant_id2 {
449 base.common(&base2).map(|x| {
451 if base.depth() == xd && base2.depth() == xd {
453 kind: LpExtend(Rc::new(x), a, LpInterior(opt_variant_id, id)),
464 (&LpExtend(ref base, _, LpDeref(_)), _) => base.common(other),
465 (_, &LpExtend(ref other, _, LpDeref(_))) => self.common(&other),
466 (&LpVar(id), &LpVar(id2)) => {
468 Some(LoanPath { kind: LpVar(id), ty: self.ty })
473 (&LpUpvar(id), &LpUpvar(id2)) => {
475 Some(LoanPath { kind: LpUpvar(id), ty: self.ty })
485 // Avoid "cannot borrow immutable field `self.x` as mutable" as that implies that a field *can* be
486 // mutable independently of the struct it belongs to. (#35937)
487 pub fn opt_loan_path_is_field<'tcx>(cmt: &mc::cmt_<'tcx>) -> (Option<Rc<LoanPath<'tcx>>>, bool) {
488 let new_lp = |v: LoanPathKind<'tcx>| Rc::new(LoanPath::new(v, cmt.ty));
491 Categorization::Rvalue(..) |
492 Categorization::ThreadLocal(..) |
493 Categorization::StaticItem => {
497 Categorization::Local(id) => {
498 (Some(new_lp(LpVar(id))), false)
501 Categorization::Upvar(mc::Upvar { id, .. }) => {
502 (Some(new_lp(LpUpvar(id))), false)
505 Categorization::Deref(ref cmt_base, pk) => {
506 let lp = opt_loan_path_is_field(cmt_base);
508 new_lp(LpExtend(lp, cmt.mutbl, LpDeref(pk)))
512 Categorization::Interior(ref cmt_base, ik) => {
513 (opt_loan_path(cmt_base).map(|lp| {
514 let opt_variant_id = match cmt_base.cat {
515 Categorization::Downcast(_, did) => Some(did),
518 new_lp(LpExtend(lp, cmt.mutbl, LpInterior(opt_variant_id, ik.cleaned())))
522 Categorization::Downcast(ref cmt_base, variant_def_id) => {
523 let lp = opt_loan_path_is_field(cmt_base);
525 new_lp(LpDowncast(lp, variant_def_id))
531 /// Computes the `LoanPath` (if any) for a `cmt`.
532 /// Note that this logic is somewhat duplicated in
533 /// the method `compute()` found in `gather_loans::restrictions`,
534 /// which allows it to share common loan path pieces as it
535 /// traverses the CMT.
536 pub fn opt_loan_path<'tcx>(cmt: &mc::cmt_<'tcx>) -> Option<Rc<LoanPath<'tcx>>> {
537 opt_loan_path_is_field(cmt).0
540 ///////////////////////////////////////////////////////////////////////////
543 // Errors that can occur
544 #[derive(Debug, PartialEq)]
545 pub enum bckerr_code<'tcx> {
547 /// superscope, subscope, loan cause
548 err_out_of_scope(ty::Region<'tcx>, ty::Region<'tcx>, euv::LoanCause),
549 err_borrowed_pointer_too_short(ty::Region<'tcx>, ty::Region<'tcx>), // loan, ptr
552 // Combination of an error code and the categorization of the expression
554 #[derive(Debug, PartialEq)]
555 pub struct BckError<'c, 'tcx> {
557 cause: AliasableViolationKind,
558 cmt: &'c mc::cmt_<'tcx>,
559 code: bckerr_code<'tcx>
562 #[derive(Copy, Clone, Debug, PartialEq)]
563 pub enum AliasableViolationKind {
565 BorrowViolation(euv::LoanCause)
568 #[derive(Copy, Clone, Debug)]
569 pub enum MovedValueUseKind {
574 ///////////////////////////////////////////////////////////////////////////
577 impl BorrowckCtxt<'_, 'tcx> {
578 pub fn is_subregion_of(&self,
579 r_sub: ty::Region<'tcx>,
580 r_sup: ty::Region<'tcx>)
583 let region_rels = RegionRelations::new(self.tcx,
585 &self.region_scope_tree,
586 &self.tables.free_region_map);
587 region_rels.is_subregion_of(r_sub, r_sup)
590 pub fn report(&self, err: BckError<'a, 'tcx>) {
591 // Catch and handle some particular cases.
592 match (&err.code, &err.cause) {
593 (&err_out_of_scope(&ty::ReScope(_), &ty::ReStatic, _),
594 &BorrowViolation(euv::ClosureCapture(span))) |
595 (&err_out_of_scope(&ty::ReScope(_), &ty::ReEarlyBound(..), _),
596 &BorrowViolation(euv::ClosureCapture(span))) |
597 (&err_out_of_scope(&ty::ReScope(_), &ty::ReFree(..), _),
598 &BorrowViolation(euv::ClosureCapture(span))) => {
599 return self.report_out_of_scope_escaping_closure_capture(&err, span);
604 self.report_bckerr(&err);
607 pub fn report_use_of_moved_value(&self,
609 use_kind: MovedValueUseKind,
611 the_move: &move_data::Move,
612 moved_lp: &LoanPath<'tcx>) {
613 let (verb, verb_participle) = match use_kind {
614 MovedInUse => ("use", "used"),
615 MovedInCapture => ("capture", "captured"),
618 let (_ol, _moved_lp_msg, mut err, need_note) = match the_move.kind {
619 move_data::Declared => {
620 // If this is an uninitialized variable, just emit a simple warning
622 self.cannot_act_on_uninitialized_variable(use_span,
624 &self.loan_path_to_string(lp),
626 .span_label(use_span, format!("use of possibly uninitialized `{}`",
627 self.loan_path_to_string(lp)))
633 // If moved_lp is something like `x.a`, and lp is something like `x.b`, we would
634 // normally generate a rather confusing message:
636 // error: use of moved value: `x.b`
637 // note: `x.a` moved here...
639 // What we want to do instead is get the 'common ancestor' of the two moves and
640 // use that for most of the message instead, giving is something like this:
642 // error: use of moved value: `x`
643 // note: `x` moved here (through moving `x.a`)...
645 let common = moved_lp.common(lp);
646 let has_common = common.is_some();
647 let has_fork = moved_lp.has_fork(lp);
648 let (nl, ol, moved_lp_msg) =
649 if has_fork && has_common {
650 let nl = self.loan_path_to_string(&common.unwrap());
652 let moved_lp_msg = format!(" (through moving `{}`)",
653 self.loan_path_to_string(moved_lp));
654 (nl, ol, moved_lp_msg)
656 (self.loan_path_to_string(lp),
657 self.loan_path_to_string(moved_lp),
661 let partial = moved_lp.depth() > lp.depth();
662 let msg = if !has_fork && partial { "partially " }
663 else if has_fork && !has_common { "collaterally "}
665 let mut err = self.cannot_act_on_moved_value(use_span,
670 let need_note = match lp.ty.sty {
671 ty::Closure(id, _) => {
672 let hir_id = self.tcx.hir().as_local_hir_id(id).unwrap();
673 if let Some((span, name)) = self.tables.closure_kind_origins().get(hir_id) {
674 err.span_note(*span, &format!(
675 "closure cannot be invoked more than once because \
676 it moves the variable `{}` out of its environment",
686 (ol, moved_lp_msg, err, need_note)
690 // Get type of value and span where it was previously
692 let hir_id = hir::HirId {
693 owner: self.body.value.hir_id.owner,
694 local_id: the_move.id
696 let (move_span, move_note) = match the_move.kind {
697 move_data::Declared => {
701 move_data::MoveExpr |
702 move_data::MovePat => (self.tcx.hir().span(hir_id), ""),
704 move_data::Captured =>
705 (match self.tcx.hir().expect_expr(hir_id).node {
706 hir::ExprKind::Closure(.., fn_decl_span, _) => fn_decl_span,
707 ref r => bug!("Captured({:?}) maps to non-closure: {:?}",
709 }, " (into closure)"),
712 // Annotate the use and the move in the span. Watch out for
713 // the case where the use and the move are the same. This
714 // means the use is in a loop.
715 err = if use_span == move_span {
718 format!("value moved{} here in previous iteration of loop",
722 err.span_label(use_span, format!("value {} here after move", verb_participle));
723 err.span_label(move_span, format!("value moved{} here", move_note));
729 "move occurs because {} has type `{}`, which does not implement the `Copy` trait",
730 if moved_lp.has_downcast() {
731 "the value".to_string()
733 format!("`{}`", self.loan_path_to_string(moved_lp))
737 if let (Some(CompilerDesugaringKind::ForLoop), Ok(snippet)) = (
738 move_span.compiler_desugaring_kind(),
739 self.tcx.sess.source_map().span_to_snippet(move_span),
741 if !snippet.starts_with("&") {
744 "consider borrowing this to avoid moving it into the for loop",
745 format!("&{}", snippet),
746 Applicability::MaybeIncorrect,
751 // Note: we used to suggest adding a `ref binding` or calling
752 // `clone` but those suggestions have been removed because
753 // they are often not what you actually want to do, and were
754 // not considered particularly helpful.
760 pub fn report_partial_reinitialization_of_uninitialized_structure(
763 lp: &LoanPath<'tcx>) {
764 self.cannot_partially_reinit_an_uninit_struct(span,
765 &self.loan_path_to_string(lp),
771 pub fn report_reassigned_immutable_variable(&self,
775 &move_data::Assignment) {
776 let mut err = self.cannot_reassign_immutable(span,
777 &self.loan_path_to_string(lp),
780 err.span_label(span, "cannot assign twice to immutable variable");
781 if span != assign.span {
782 err.span_label(assign.span, format!("first assignment to `{}`",
783 self.loan_path_to_string(lp)));
789 fn report_bckerr(&self, err: &BckError<'a, 'tcx>) {
790 let error_span = err.span.clone();
794 let descr: Cow<'static, str> = match err.cmt.note {
795 mc::NoteClosureEnv(_) | mc::NoteUpvarRef(_) => {
796 self.cmt_to_cow_str(&err.cmt)
798 _ => match opt_loan_path_is_field(&err.cmt) {
800 format!("{} of {} binding",
801 self.cmt_to_cow_str(&err.cmt),
802 err.cmt.mutbl.to_user_str()).into()
807 err.cmt.mutbl.to_user_str(),
808 self.cmt_to_cow_str(&err.cmt)).into()
811 (Some(lp), true) => {
812 format!("{} `{}` of {} binding",
813 self.cmt_to_cow_str(&err.cmt),
814 self.loan_path_to_string(&lp),
815 err.cmt.mutbl.to_user_str()).into()
817 (Some(lp), false) => {
818 format!("{} {} `{}`",
819 err.cmt.mutbl.to_user_str(),
820 self.cmt_to_cow_str(&err.cmt),
821 self.loan_path_to_string(&lp)).into()
826 let mut db = match err.cause {
827 MutabilityViolation => {
828 let mut db = self.cannot_assign(error_span, &descr, Origin::Ast);
829 if let mc::NoteClosureEnv(upvar_id) = err.cmt.note {
830 let hir_id = upvar_id.var_path.hir_id;
831 let sp = self.tcx.hir().span(hir_id);
832 let fn_closure_msg = "`Fn` closures cannot capture their enclosing \
833 environment for modifications";
834 match (self.tcx.sess.source_map().span_to_snippet(sp), &err.cmt.cat) {
835 (_, &Categorization::Upvar(mc::Upvar {
836 kind: ty::ClosureKind::Fn, ..
838 db.note(fn_closure_msg);
839 // we should point at the cause for this closure being
840 // identified as `Fn` (like in signature of method this
841 // closure was passed into)
843 (Ok(ref snippet), ref cat) => {
844 let msg = &format!("consider making `{}` mutable", snippet);
845 let suggestion = format!("mut {}", snippet);
847 if let &Categorization::Deref(ref cmt, _) = cat {
848 if let Categorization::Upvar(mc::Upvar {
849 kind: ty::ClosureKind::Fn, ..
851 db.note(fn_closure_msg);
857 Applicability::Unspecified,
865 Applicability::Unspecified,
870 db.span_help(sp, "consider making this binding mutable");
877 BorrowViolation(euv::ClosureCapture(_)) => {
878 self.closure_cannot_assign_to_borrowed(error_span, &descr, Origin::Ast)
880 BorrowViolation(euv::OverloadedOperator) |
881 BorrowViolation(euv::AddrOf) |
882 BorrowViolation(euv::RefBinding) |
883 BorrowViolation(euv::AutoRef) |
884 BorrowViolation(euv::AutoUnsafe) |
885 BorrowViolation(euv::ForLoop) |
886 BorrowViolation(euv::MatchDiscriminant) => {
887 self.cannot_borrow_path_as_mutable(error_span, &descr, Origin::Ast)
889 BorrowViolation(euv::ClosureInvocation) => {
890 span_bug!(err.span, "err_mutbl with a closure invocation");
894 // We add a special note about `IndexMut`, if the source of this error
895 // is the fact that `Index` is implemented, but `IndexMut` is not. Needing
896 // to implement two traits for "one operator" is not very intuitive for
898 if err.cmt.note == mc::NoteIndex {
899 let node = self.tcx.hir().get(err.cmt.hir_id);
901 // This pattern probably always matches.
903 hir::Expr { node: hir::ExprKind::Index(lhs, _), ..}
905 let ty = self.tables.expr_ty(lhs);
908 "trait `IndexMut` is required to modify indexed content, but \
909 it is not implemented for `{}`",
915 self.note_and_explain_mutbl_error(&mut db, &err, &error_span);
916 self.note_immutability_blame(
918 err.cmt.immutability_blame(),
924 err_out_of_scope(super_scope, sub_scope, cause) => {
925 let msg = match opt_loan_path(&err.cmt) {
926 None => "borrowed value".to_string(),
928 format!("`{}`", self.loan_path_to_string(&lp))
932 let mut db = self.path_does_not_live_long_enough(error_span, &msg, Origin::Ast);
933 let value_kind = match err.cmt.cat {
934 mc::Categorization::Rvalue(..) => "temporary value",
935 _ => "borrowed value",
938 let is_closure = match cause {
939 euv::ClosureCapture(s) => {
940 // The primary span starts out as the closure creation point.
941 // Change the primary span here to highlight the use of the variable
942 // in the closure, because it seems more natural. Highlight
943 // closure creation point as a secondary span.
944 match db.span.primary_span() {
946 db.span = MultiSpan::from_span(s);
947 db.span_label(primary, "capture occurs here");
948 db.span_label(s, format!("{} does not live long enough",
956 db.span_label(error_span, format!("{} does not live long enough",
962 let sub_span = self.region_end_span(sub_scope);
963 let super_span = self.region_end_span(super_scope);
965 match (sub_span, super_span) {
966 (Some(s1), Some(s2)) if s1 == s2 => {
968 let msg = match opt_loan_path(&err.cmt) {
969 None => value_kind.to_string(),
971 format!("`{}`", self.loan_path_to_string(&lp))
975 format!("{} dropped here while still borrowed", msg));
977 db.span_label(s1, format!("{} dropped before borrower", value_kind));
979 db.note("values in a scope are dropped in the opposite order \
982 (Some(s1), Some(s2)) if !is_closure => {
983 let msg = match opt_loan_path(&err.cmt) {
984 None => value_kind.to_string(),
986 format!("`{}`", self.loan_path_to_string(&lp))
989 db.span_label(s2, format!("{} dropped here while still borrowed", msg));
990 db.span_label(s1, format!("{} needs to live until here", value_kind));
995 db.span_label(s, format!("{} needs to live until here",
999 self.tcx.note_and_explain_region(
1000 &self.region_scope_tree,
1002 "borrowed value must be valid for ",
1009 db.span_label(s, format!("{} only lives until here", value_kind));
1012 self.tcx.note_and_explain_region(
1013 &self.region_scope_tree,
1015 "...but borrowed value is only valid for ",
1023 if let ty::ReScope(scope) = *super_scope {
1024 let hir_id = scope.hir_id(&self.region_scope_tree);
1025 match self.tcx.hir().find(hir_id) {
1026 Some(Node::Stmt(_)) => {
1027 if *sub_scope != ty::ReStatic {
1028 db.note("consider using a `let` binding to increase its lifetime");
1037 self.signal_error();
1039 err_borrowed_pointer_too_short(loan_scope, ptr_scope) => {
1040 let descr = self.cmt_to_path_or_string(err.cmt);
1041 let mut db = self.lifetime_too_short_for_reborrow(error_span, &descr, Origin::Ast);
1042 let descr: Cow<'static, str> = match opt_loan_path(&err.cmt) {
1044 format!("`{}`", self.loan_path_to_string(&lp)).into()
1046 None => self.cmt_to_cow_str(&err.cmt)
1048 self.tcx.note_and_explain_region(
1049 &self.region_scope_tree,
1051 &format!("{} would have to be valid for ",
1055 self.tcx.note_and_explain_region(
1056 &self.region_scope_tree,
1058 &format!("...but {} is only valid for ", descr),
1063 self.signal_error();
1068 pub fn report_aliasability_violation(&self,
1070 kind: AliasableViolationKind,
1071 cause: mc::AliasableReason,
1072 cmt: &mc::cmt_<'tcx>) {
1073 let mut is_closure = false;
1074 let prefix = match kind {
1075 MutabilityViolation => {
1076 "cannot assign to data"
1078 BorrowViolation(euv::ClosureCapture(_)) |
1079 BorrowViolation(euv::OverloadedOperator) |
1080 BorrowViolation(euv::AddrOf) |
1081 BorrowViolation(euv::AutoRef) |
1082 BorrowViolation(euv::AutoUnsafe) |
1083 BorrowViolation(euv::RefBinding) |
1084 BorrowViolation(euv::MatchDiscriminant) => {
1085 "cannot borrow data mutably"
1087 BorrowViolation(euv::ClosureInvocation) => {
1089 "closure invocation"
1092 BorrowViolation(euv::ForLoop) => {
1098 mc::AliasableStaticMut => {
1099 // This path cannot occur. `static mut X` is not checked
1100 // for aliasability violations.
1101 span_bug!(span, "aliasability violation for static mut `{}`", prefix)
1103 mc::AliasableStatic | mc::AliasableBorrowed => {}
1105 let blame = cmt.immutability_blame();
1106 let mut err = match blame {
1107 Some(ImmutabilityBlame::ClosureEnv(id)) => {
1108 // FIXME: the distinction between these 2 messages looks wrong.
1109 let help_msg = if let BorrowViolation(euv::ClosureCapture(_)) = kind {
1110 // The aliasability violation with closure captures can
1111 // happen for nested closures, so we know the enclosing
1112 // closure incorrectly accepts an `Fn` while it needs to
1114 "consider changing this to accept closures that implement `FnMut`"
1117 "consider changing this closure to take self by mutable reference"
1119 let hir_id = self.tcx.hir().local_def_id_to_hir_id(id);
1120 let help_span = self.tcx.hir().span(hir_id);
1121 self.cannot_act_on_capture_in_sharable_fn(span,
1123 (help_span, help_msg),
1127 self.cannot_assign_into_immutable_reference(span, prefix,
1131 self.note_immutability_blame(
1138 err.help("closures behind references must be called via `&mut`");
1141 self.signal_error();
1144 /// Given a type, if it is an immutable reference, return a suggestion to make it mutable
1145 fn suggest_mut_for_immutable(&self, pty: &hir::Ty, is_implicit_self: bool) -> Option<String> {
1146 // Check whether the argument is an immutable reference
1147 debug!("suggest_mut_for_immutable({:?}, {:?})", pty, is_implicit_self);
1148 if let hir::TyKind::Rptr(lifetime, hir::MutTy {
1149 mutbl: hir::Mutability::MutImmutable,
1152 // Account for existing lifetimes when generating the message
1153 let pointee_snippet = match self.tcx.sess.source_map().span_to_snippet(ty.span) {
1154 Ok(snippet) => snippet,
1158 let lifetime_snippet = if !lifetime.is_elided() {
1159 format!("{} ", match self.tcx.sess.source_map().span_to_snippet(lifetime.span) {
1160 Ok(lifetime_snippet) => lifetime_snippet,
1166 Some(format!("use `&{}mut {}` here to make mutable",
1168 if is_implicit_self { "self" } else { &*pointee_snippet }))
1174 fn local_binding_mode(&self, hir_id: hir::HirId) -> ty::BindingMode {
1175 let pat = match self.tcx.hir().get(hir_id) {
1176 Node::Binding(pat) => pat,
1177 node => bug!("bad node for local: {:?}", node)
1181 hir::PatKind::Binding(..) => {
1183 .pat_binding_modes()
1185 .expect("missing binding mode")
1187 _ => bug!("local is not a binding: {:?}", pat)
1191 fn local_ty(&self, hir_id: hir::HirId) -> (Option<&hir::Ty>, bool) {
1192 let parent = self.tcx.hir().get_parent_node(hir_id);
1193 let parent_node = self.tcx.hir().get(parent);
1195 // The parent node is like a fn
1196 if let Some(fn_like) = FnLikeNode::from_node(parent_node) {
1197 // `nid`'s parent's `Body`
1198 let fn_body = self.tcx.hir().body(fn_like.body());
1199 // Get the position of `node_id` in the arguments list
1200 let arg_pos = fn_body.arguments.iter().position(|arg| arg.pat.hir_id == hir_id);
1201 if let Some(i) = arg_pos {
1202 // The argument's `Ty`
1203 (Some(&fn_like.decl().inputs[i]),
1204 i == 0 && fn_like.decl().implicit_self.has_implicit_self())
1213 fn note_immutability_blame(&self,
1214 db: &mut DiagnosticBuilder<'_>,
1215 blame: Option<ImmutabilityBlame<'_>>,
1216 error_hir_id: hir::HirId) {
1219 Some(ImmutabilityBlame::ClosureEnv(_)) => {}
1220 Some(ImmutabilityBlame::ImmLocal(hir_id)) => {
1221 self.note_immutable_local(db, error_hir_id, hir_id)
1223 Some(ImmutabilityBlame::LocalDeref(hir_id)) => {
1224 match self.local_binding_mode(hir_id) {
1225 ty::BindByReference(..) => {
1226 let let_span = self.tcx.hir().span(hir_id);
1227 let suggestion = suggest_ref_mut(self.tcx, let_span);
1228 if let Some(replace_str) = suggestion {
1231 "use a mutable reference instead",
1233 // I believe this can be machine applicable,
1234 // but if there are multiple attempted uses of an immutable
1235 // reference, I don't know how rustfix handles it, it might
1236 // attempt fixing them multiple times.
1238 Applicability::Unspecified,
1242 ty::BindByValue(..) => {
1243 if let (Some(local_ty), is_implicit_self) = self.local_ty(hir_id) {
1245 self.suggest_mut_for_immutable(local_ty, is_implicit_self) {
1246 db.span_label(local_ty.span, msg);
1252 Some(ImmutabilityBlame::AdtFieldDeref(_, field)) => {
1253 let hir_id = match self.tcx.hir().as_local_hir_id(field.did) {
1254 Some(hir_id) => hir_id,
1258 if let Node::Field(ref field) = self.tcx.hir().get(hir_id) {
1259 if let Some(msg) = self.suggest_mut_for_immutable(&field.ty, false) {
1260 db.span_label(field.ty.span, msg);
1267 // Suggest a fix when trying to mutably borrow an immutable local
1268 // binding: either to make the binding mutable (if its type is
1269 // not a mutable reference) or to avoid borrowing altogether
1270 fn note_immutable_local(&self,
1271 db: &mut DiagnosticBuilder<'_>,
1272 borrowed_hir_id: hir::HirId,
1273 binding_hir_id: hir::HirId) {
1274 let let_span = self.tcx.hir().span(binding_hir_id);
1275 if let ty::BindByValue(..) = self.local_binding_mode(binding_hir_id) {
1276 if let Ok(snippet) = self.tcx.sess.source_map().span_to_snippet(let_span) {
1277 let (ty, is_implicit_self) = self.local_ty(binding_hir_id);
1278 if is_implicit_self && snippet != "self" {
1279 // avoid suggesting `mut &self`.
1282 if let Some(&hir::TyKind::Rptr(
1285 mutbl: hir::MutMutable,
1288 )) = ty.map(|t| &t.node)
1290 let borrow_expr_id = self.tcx.hir().get_parent_node(borrowed_hir_id);
1292 self.tcx.hir().span(borrow_expr_id),
1293 "consider removing the `&mut`, as it is an \
1294 immutable binding to a mutable reference",
1296 Applicability::MachineApplicable,
1301 "make this binding mutable",
1302 format!("mut {}", snippet),
1303 Applicability::MachineApplicable,
1310 fn report_out_of_scope_escaping_closure_capture(&self,
1311 err: &BckError<'a, 'tcx>,
1314 let cmt_path_or_string = self.cmt_to_path_or_string(&err.cmt);
1317 match self.tcx.sess.source_map().span_to_snippet(err.span) {
1318 Ok(string) => format!("move {}", string),
1319 Err(_) => "move |<args>| <body>".to_string()
1322 self.cannot_capture_in_long_lived_closure(err.span,
1323 &cmt_path_or_string,
1328 &format!("to force the closure to take ownership of {} \
1329 (and any other referenced variables), \
1330 use the `move` keyword",
1331 cmt_path_or_string),
1333 Applicability::MachineApplicable,
1336 self.signal_error();
1339 fn region_end_span(&self, region: ty::Region<'tcx>) -> Option<Span> {
1341 ty::ReScope(scope) => {
1342 Some(self.tcx.sess.source_map().end_point(
1343 scope.span(self.tcx, &self.region_scope_tree)))
1349 fn note_and_explain_mutbl_error(&self, db: &mut DiagnosticBuilder<'_>, err: &BckError<'a, 'tcx>,
1350 error_span: &Span) {
1351 match err.cmt.note {
1352 mc::NoteClosureEnv(upvar_id) | mc::NoteUpvarRef(upvar_id) => {
1353 // If this is an `Fn` closure, it simply can't mutate upvars.
1354 // If it's an `FnMut` closure, the original variable was declared immutable.
1355 // We need to determine which is the case here.
1356 let kind = match err.cmt.upvar_cat().unwrap() {
1357 Categorization::Upvar(mc::Upvar { kind, .. }) => kind,
1360 if *kind == ty::ClosureKind::Fn {
1361 let closure_hir_id =
1362 self.tcx.hir().local_def_id_to_hir_id(upvar_id.closure_expr_id);
1363 db.span_help(self.tcx.hir().span(closure_hir_id),
1364 "consider changing this closure to take \
1365 self by mutable reference");
1369 if let Categorization::Deref(..) = err.cmt.cat {
1370 db.span_label(*error_span, "cannot borrow as mutable");
1371 } else if let Categorization::Local(local_id) = err.cmt.cat {
1372 let span = self.tcx.hir().span(local_id);
1373 if let Ok(snippet) = self.tcx.sess.source_map().span_to_snippet(span) {
1374 if snippet.starts_with("ref mut ") || snippet.starts_with("&mut ") {
1375 db.span_label(*error_span, "cannot reborrow mutably");
1376 db.span_label(*error_span, "try removing `&mut` here");
1378 db.span_label(*error_span, "cannot borrow mutably");
1381 db.span_label(*error_span, "cannot borrow mutably");
1383 } else if let Categorization::Interior(ref cmt, _) = err.cmt.cat {
1384 if let mc::MutabilityCategory::McImmutable = cmt.mutbl {
1385 db.span_label(*error_span,
1386 "cannot mutably borrow field of immutable binding");
1392 pub fn append_loan_path_to_string(&self,
1393 loan_path: &LoanPath<'tcx>,
1395 match loan_path.kind {
1396 LpUpvar(ty::UpvarId { var_path: ty::UpvarPath { hir_id: id }, closure_expr_id: _ }) => {
1397 out.push_str(&self.tcx.hir().name(id).as_str());
1400 out.push_str(&self.tcx.hir().name(id).as_str());
1403 LpDowncast(ref lp_base, variant_def_id) => {
1405 self.append_loan_path_to_string(&lp_base, out);
1406 out.push_str(DOWNCAST_PRINTED_OPERATOR);
1407 out.push_str(&self.tcx.def_path_str(variant_def_id));
1411 LpExtend(ref lp_base, _, LpInterior(_, InteriorField(mc::FieldIndex(_, info)))) => {
1412 self.append_autoderefd_loan_path_to_string(&lp_base, out);
1414 out.push_str(&info.as_str());
1417 LpExtend(ref lp_base, _, LpInterior(_, InteriorElement)) => {
1418 self.append_autoderefd_loan_path_to_string(&lp_base, out);
1419 out.push_str("[..]");
1422 LpExtend(ref lp_base, _, LpDeref(_)) => {
1424 self.append_loan_path_to_string(&lp_base, out);
1429 pub fn append_autoderefd_loan_path_to_string(&self,
1430 loan_path: &LoanPath<'tcx>,
1432 match loan_path.kind {
1433 LpExtend(ref lp_base, _, LpDeref(_)) => {
1434 // For a path like `(*x).f` or `(*x)[3]`, autoderef
1435 // rules would normally allow users to omit the `*x`.
1436 // So just serialize such paths to `x.f` or x[3]` respectively.
1437 self.append_autoderefd_loan_path_to_string(&lp_base, out)
1440 LpDowncast(ref lp_base, variant_def_id) => {
1442 self.append_autoderefd_loan_path_to_string(&lp_base, out);
1443 out.push_str(DOWNCAST_PRINTED_OPERATOR);
1444 out.push_str(&self.tcx.def_path_str(variant_def_id));
1448 LpVar(..) | LpUpvar(..) | LpExtend(.., LpInterior(..)) => {
1449 self.append_loan_path_to_string(loan_path, out)
1454 pub fn loan_path_to_string(&self, loan_path: &LoanPath<'tcx>) -> String {
1455 let mut result = String::new();
1456 self.append_loan_path_to_string(loan_path, &mut result);
1460 pub fn cmt_to_cow_str(&self, cmt: &mc::cmt_<'tcx>) -> Cow<'static, str> {
1461 cmt.descriptive_string(self.tcx)
1464 pub fn cmt_to_path_or_string(&self, cmt: &mc::cmt_<'tcx>) -> String {
1465 match opt_loan_path(cmt) {
1466 Some(lp) => format!("`{}`", self.loan_path_to_string(&lp)),
1467 None => self.cmt_to_cow_str(cmt).into_owned(),
1472 impl BitwiseOperator for LoanDataFlowOperator {
1474 fn join(&self, succ: usize, pred: usize) -> usize {
1475 succ | pred // loans from both preds are in scope
1479 impl DataFlowOperator for LoanDataFlowOperator {
1481 fn initial_value(&self) -> bool {
1482 false // no loans in scope by default
1486 impl fmt::Debug for InteriorKind {
1487 fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
1489 InteriorField(mc::FieldIndex(_, info)) => write!(f, "{}", info),
1490 InteriorElement => write!(f, "[]"),
1495 impl<'tcx> fmt::Debug for Loan<'tcx> {
1496 fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
1497 write!(f, "Loan_{}({:?}, {:?}, {:?}-{:?}, {:?})",
1503 self.restricted_paths)
1507 impl<'tcx> fmt::Debug for LoanPath<'tcx> {
1508 fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
1511 write!(f, "$({})", ty::tls::with(|tcx| tcx.hir().node_to_string(id)))
1514 LpUpvar(ty::UpvarId{ var_path: ty::UpvarPath {hir_id: var_id}, closure_expr_id }) => {
1515 let s = ty::tls::with(|tcx| {
1516 tcx.hir().node_to_string(var_id)
1518 write!(f, "$({} captured by id={:?})", s, closure_expr_id)
1521 LpDowncast(ref lp, variant_def_id) => {
1522 let variant_str = if variant_def_id.is_local() {
1523 ty::tls::with(|tcx| tcx.def_path_str(variant_def_id))
1525 format!("{:?}", variant_def_id)
1527 write!(f, "({:?}{}{})", lp, DOWNCAST_PRINTED_OPERATOR, variant_str)
1530 LpExtend(ref lp, _, LpDeref(_)) => {
1531 write!(f, "{:?}.*", lp)
1534 LpExtend(ref lp, _, LpInterior(_, ref interior)) => {
1535 write!(f, "{:?}.{:?}", lp, interior)
1541 impl<'tcx> fmt::Display for LoanPath<'tcx> {
1542 fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
1545 write!(f, "$({})", ty::tls::with(|tcx| tcx.hir().hir_to_user_string(id)))
1548 LpUpvar(ty::UpvarId{ var_path: ty::UpvarPath { hir_id }, closure_expr_id: _ }) => {
1549 let s = ty::tls::with(|tcx| {
1550 tcx.hir().node_to_string(hir_id)
1552 write!(f, "$({} captured by closure)", s)
1555 LpDowncast(ref lp, variant_def_id) => {
1556 let variant_str = if variant_def_id.is_local() {
1557 ty::tls::with(|tcx| tcx.def_path_str(variant_def_id))
1559 format!("{:?}", variant_def_id)
1561 write!(f, "({}{}{})", lp, DOWNCAST_PRINTED_OPERATOR, variant_str)
1564 LpExtend(ref lp, _, LpDeref(_)) => {
1565 write!(f, "{}.*", lp)
1568 LpExtend(ref lp, _, LpInterior(_, ref interior)) => {
1569 write!(f, "{}.{:?}", lp, interior)