Make Const::val() return a ref, to avoid copies.

nnethercote · nnethercote · commit a9d3cd1ded19 · 2022-02-21T17:42:20.000+11:00
The idea comes from the last commit of #90951.
diff --git a/compiler/rustc_borrowck/src/type_check/mod.rs b/compiler/rustc_borrowck/src/type_check/mod.rs
@@ -384,7 +384,7 @@ impl<'a, 'b, 'tcx> Visitor<'tcx> for TypeVerifier<'a, 'b, 'tcx> {
             let tcx = self.tcx();
             let maybe_uneval = match constant.literal {
                 ConstantKind::Ty(ct) => match ct.val() {
-                    ty::ConstKind::Unevaluated(uv) => Some(uv),
+                    &ty::ConstKind::Unevaluated(uv) => Some(uv),
                     _ => None,
                 },
                 _ => None,
@@ -1956,7 +1956,7 @@ impl<'a, 'tcx> TypeChecker<'a, 'tcx> {
         if let Operand::Constant(constant) = op {
             let maybe_uneval = match constant.literal {
                 ConstantKind::Ty(ct) => match ct.val() {
-                    ty::ConstKind::Unevaluated(uv) => Some(uv),
+                    &ty::ConstKind::Unevaluated(uv) => Some(uv),
                     _ => None,
                 },
                 _ => None,
diff --git a/compiler/rustc_codegen_cranelift/src/constant.rs b/compiler/rustc_codegen_cranelift/src/constant.rs
@@ -48,7 +48,7 @@ pub(crate) fn check_constants(fx: &mut FunctionCx<'_, '_, '_>) -> bool {
         };
         match const_.val() {
             ConstKind::Value(_) => {}
-            ConstKind::Unevaluated(unevaluated) => {
+            &ConstKind::Unevaluated(unevaluated) => {
                 if let Err(err) =
                     fx.tcx.const_eval_resolve(ParamEnv::reveal_all(), unevaluated, None)
                 {
@@ -128,7 +128,7 @@ pub(crate) fn codegen_constant<'tcx>(
         ConstantKind::Val(val, ty) => return codegen_const_value(fx, val, ty),
     };
     let const_val = match const_.val() {
-        ConstKind::Value(const_val) => const_val,
+        &ConstKind::Value(const_val) => const_val,
         ConstKind::Unevaluated(ty::Unevaluated { def, substs, promoted })
             if fx.tcx.is_static(def.did) =>
         {
@@ -137,7 +137,7 @@ pub(crate) fn codegen_constant<'tcx>(
 
             return codegen_static_ref(fx, def.did, fx.layout_of(const_.ty())).to_cvalue(fx);
         }
-        ConstKind::Unevaluated(unevaluated) => {
+        &ConstKind::Unevaluated(unevaluated) => {
             match fx.tcx.const_eval_resolve(ParamEnv::reveal_all(), unevaluated, None) {
                 Ok(const_val) => const_val,
                 Err(_) => {
diff --git a/compiler/rustc_codegen_ssa/src/mir/constant.rs b/compiler/rustc_codegen_ssa/src/mir/constant.rs
@@ -30,15 +30,15 @@ impl<'a, 'tcx, Bx: BuilderMethods<'a, 'tcx>> FunctionCx<'a, 'tcx, Bx> {
             mir::ConstantKind::Val(val, _) => return Ok(val),
         };
         match ct.val() {
-            ty::ConstKind::Unevaluated(ct) => self
+            &ty::ConstKind::Unevaluated(ct) => self
                 .cx
                 .tcx()
                 .const_eval_resolve(ty::ParamEnv::reveal_all(), ct, None)
                 .map_err(|err| {
                     self.cx.tcx().sess.span_err(constant.span, "erroneous constant encountered");
                     err
                 }),
-            ty::ConstKind::Value(value) => Ok(value),
+            &ty::ConstKind::Value(value) => Ok(value),
             err => span_bug!(
                 constant.span,
                 "encountered bad ConstKind after monomorphizing: {:?}",
diff --git a/compiler/rustc_const_eval/src/interpret/operand.rs b/compiler/rustc_const_eval/src/interpret/operand.rs
@@ -574,7 +574,7 @@ impl<'mir, 'tcx: 'mir, M: Machine<'mir, 'tcx>> InterpCx<'mir, 'tcx, M> {
             ty::ConstKind::Infer(..) | ty::ConstKind::Placeholder(..) => {
                 span_bug!(self.cur_span(), "const_to_op: Unexpected ConstKind {:?}", val)
             }
-            ty::ConstKind::Value(val_val) => self.const_val_to_op(val_val, val.ty(), layout),
+            &ty::ConstKind::Value(val_val) => self.const_val_to_op(val_val, val.ty(), layout),
         }
     }
 
diff --git a/compiler/rustc_infer/src/infer/canonical/canonicalizer.rs b/compiler/rustc_infer/src/infer/canonical/canonicalizer.rs
@@ -477,7 +477,7 @@ impl<'cx, 'tcx> TypeFolder<'tcx> for Canonicalizer<'cx, 'tcx> {
 
     fn fold_const(&mut self, ct: ty::Const<'tcx>) -> ty::Const<'tcx> {
         match ct.val() {
-            ty::ConstKind::Infer(InferConst::Var(vid)) => {
+            &ty::ConstKind::Infer(InferConst::Var(vid)) => {
                 debug!("canonical: const var found with vid {:?}", vid);
                 match self.infcx.probe_const_var(vid) {
                     Ok(c) => {
@@ -502,14 +502,14 @@ impl<'cx, 'tcx> TypeFolder<'tcx> for Canonicalizer<'cx, 'tcx> {
             ty::ConstKind::Infer(InferConst::Fresh(_)) => {
                 bug!("encountered a fresh const during canonicalization")
             }
-            ty::ConstKind::Bound(debruijn, _) => {
+            &ty::ConstKind::Bound(debruijn, _) => {
                 if debruijn >= self.binder_index {
                     bug!("escaping bound type during canonicalization")
                 } else {
                     return ct;
                 }
             }
-            ty::ConstKind::Placeholder(placeholder) => {
+            &ty::ConstKind::Placeholder(placeholder) => {
                 return self.canonicalize_const_var(
                     CanonicalVarInfo { kind: CanonicalVarKind::PlaceholderConst(placeholder) },
                     ct,
diff --git a/compiler/rustc_infer/src/infer/canonical/query_response.rs b/compiler/rustc_infer/src/infer/canonical/query_response.rs
@@ -437,7 +437,7 @@ impl<'cx, 'tcx> InferCtxt<'cx, 'tcx> {
                     }
                 }
                 GenericArgKind::Const(result_value) => {
-                    if let ty::ConstKind::Bound(debrujin, b) = result_value.val() {
+                    if let &ty::ConstKind::Bound(debrujin, b) = result_value.val() {
                         // ...in which case we would set `canonical_vars[0]` to `Some(const X)`.
 
                         // We only allow a `ty::INNERMOST` index in substitutions.
diff --git a/compiler/rustc_infer/src/infer/combine.rs b/compiler/rustc_infer/src/infer/combine.rs
@@ -141,8 +141,8 @@ impl<'infcx, 'tcx> InferCtxt<'infcx, 'tcx> {
 
         match (a.val(), b.val()) {
             (
-                ty::ConstKind::Infer(InferConst::Var(a_vid)),
-                ty::ConstKind::Infer(InferConst::Var(b_vid)),
+                &ty::ConstKind::Infer(InferConst::Var(a_vid)),
+                &ty::ConstKind::Infer(InferConst::Var(b_vid)),
             ) => {
                 self.inner
                     .borrow_mut()
@@ -158,11 +158,11 @@ impl<'infcx, 'tcx> InferCtxt<'infcx, 'tcx> {
                 bug!("tried to combine ConstKind::Infer/ConstKind::Infer(InferConst::Var)")
             }
 
-            (ty::ConstKind::Infer(InferConst::Var(vid)), _) => {
+            (&ty::ConstKind::Infer(InferConst::Var(vid)), _) => {
                 return self.unify_const_variable(relation.param_env(), vid, b, a_is_expected);
             }
 
-            (_, ty::ConstKind::Infer(InferConst::Var(vid))) => {
+            (_, &ty::ConstKind::Infer(InferConst::Var(vid))) => {
                 return self.unify_const_variable(relation.param_env(), vid, a, !a_is_expected);
             }
             (ty::ConstKind::Unevaluated(..), _) if self.tcx.lazy_normalization() => {
@@ -722,7 +722,7 @@ impl<'tcx> TypeRelation<'tcx> for Generalizer<'_, 'tcx> {
         assert_eq!(c, c2); // we are abusing TypeRelation here; both LHS and RHS ought to be ==
 
         match c.val() {
-            ty::ConstKind::Infer(InferConst::Var(vid)) => {
+            &ty::ConstKind::Infer(InferConst::Var(vid)) => {
                 let mut inner = self.infcx.inner.borrow_mut();
                 let variable_table = &mut inner.const_unification_table();
                 let var_value = variable_table.probe_value(vid);
@@ -744,7 +744,7 @@ impl<'tcx> TypeRelation<'tcx> for Generalizer<'_, 'tcx> {
                     }
                 }
             }
-            ty::ConstKind::Unevaluated(ty::Unevaluated { def, substs, promoted })
+            &ty::ConstKind::Unevaluated(ty::Unevaluated { def, substs, promoted })
                 if self.tcx().lazy_normalization() =>
             {
                 assert_eq!(promoted, None);
@@ -952,7 +952,7 @@ impl<'tcx> TypeRelation<'tcx> for ConstInferUnifier<'_, 'tcx> {
         debug!("ConstInferUnifier: c={:?}", c);
 
         match c.val() {
-            ty::ConstKind::Infer(InferConst::Var(vid)) => {
+            &ty::ConstKind::Infer(InferConst::Var(vid)) => {
                 // Check if the current unification would end up
                 // unifying `target_vid` with a const which contains
                 // an inference variable which is unioned with `target_vid`.
@@ -990,7 +990,7 @@ impl<'tcx> TypeRelation<'tcx> for ConstInferUnifier<'_, 'tcx> {
                     }
                 }
             }
-            ty::ConstKind::Unevaluated(ty::Unevaluated { def, substs, promoted })
+            &ty::ConstKind::Unevaluated(ty::Unevaluated { def, substs, promoted })
                 if self.tcx().lazy_normalization() =>
             {
                 assert_eq!(promoted, None);
diff --git a/compiler/rustc_infer/src/infer/error_reporting/need_type_info.rs b/compiler/rustc_infer/src/infer/error_reporting/need_type_info.rs
@@ -410,7 +410,7 @@ impl<'a, 'tcx> InferCtxt<'a, 'tcx> {
             }
             GenericArgKind::Const(ct) => {
                 match ct.val() {
-                    ty::ConstKind::Infer(InferConst::Var(vid)) => {
+                    &ty::ConstKind::Infer(InferConst::Var(vid)) => {
                         let origin = self
                             .inner
                             .borrow_mut()
diff --git a/compiler/rustc_infer/src/infer/freshen.rs b/compiler/rustc_infer/src/infer/freshen.rs
@@ -223,7 +223,7 @@ impl<'a, 'tcx> TypeFolder<'tcx> for TypeFreshener<'a, 'tcx> {
 
     fn fold_const(&mut self, ct: ty::Const<'tcx>) -> ty::Const<'tcx> {
         match ct.val() {
-            ty::ConstKind::Infer(ty::InferConst::Var(v)) => {
+            &ty::ConstKind::Infer(ty::InferConst::Var(v)) => {
                 let opt_ct = self
                     .infcx
                     .inner
@@ -239,7 +239,7 @@ impl<'a, 'tcx> TypeFolder<'tcx> for TypeFreshener<'a, 'tcx> {
                     ct.ty(),
                 );
             }
-            ty::ConstKind::Infer(ty::InferConst::Fresh(i)) => {
+            &ty::ConstKind::Infer(ty::InferConst::Fresh(i)) => {
                 if i >= self.const_freshen_count {
                     bug!(
                         "Encountered a freshend const with id {} \
diff --git a/compiler/rustc_infer/src/infer/mod.rs b/compiler/rustc_infer/src/infer/mod.rs
@@ -1761,7 +1761,7 @@ impl<'tcx> TyOrConstInferVar<'tcx> {
     /// for constants other than `ty::ConstKind::Infer(_)` (or `InferConst::Fresh`).
     pub fn maybe_from_const(ct: ty::Const<'tcx>) -> Option<Self> {
         match ct.val() {
-            ty::ConstKind::Infer(InferConst::Var(v)) => Some(TyOrConstInferVar::Const(v)),
+            &ty::ConstKind::Infer(InferConst::Var(v)) => Some(TyOrConstInferVar::Const(v)),
             _ => None,
         }
     }
@@ -1781,7 +1781,7 @@ impl<'a, 'tcx> TypeFolder<'tcx> for ShallowResolver<'a, 'tcx> {
     }
 
     fn fold_const(&mut self, ct: ty::Const<'tcx>) -> ty::Const<'tcx> {
-        if let ty::ConstKind::Infer(InferConst::Var(vid)) = ct.val() {
+        if let &ty::ConstKind::Infer(InferConst::Var(vid)) = ct.val() {
             self.infcx
                 .inner
                 .borrow_mut()
diff --git a/compiler/rustc_infer/src/infer/nll_relate/mod.rs b/compiler/rustc_infer/src/infer/nll_relate/mod.rs
@@ -999,7 +999,7 @@ where
             ty::ConstKind::Infer(InferConst::Var(_)) if D::forbid_inference_vars() => {
                 bug!("unexpected inference variable encountered in NLL generalization: {:?}", a);
             }
-            ty::ConstKind::Infer(InferConst::Var(vid)) => {
+            &ty::ConstKind::Infer(InferConst::Var(vid)) => {
                 let mut inner = self.infcx.inner.borrow_mut();
                 let variable_table = &mut inner.const_unification_table();
                 let var_value = variable_table.probe_value(vid);
diff --git a/compiler/rustc_infer/src/infer/resolve.rs b/compiler/rustc_infer/src/infer/resolve.rs
@@ -224,7 +224,7 @@ impl<'a, 'tcx> FallibleTypeFolder<'tcx> for FullTypeResolver<'a, 'tcx> {
         } else {
             let c = self.infcx.shallow_resolve(c);
             match c.val() {
-                ty::ConstKind::Infer(InferConst::Var(vid)) => {
+                &ty::ConstKind::Infer(InferConst::Var(vid)) => {
                     return Err(FixupError::UnresolvedConst(vid));
                 }
                 ty::ConstKind::Infer(InferConst::Fresh(_)) => {
diff --git a/compiler/rustc_middle/src/infer/unify_key.rs b/compiler/rustc_middle/src/infer/unify_key.rs
@@ -176,7 +176,7 @@ where
     V: snapshot_vec::VecLike<unify::Delegate<ty::ConstVid<'tcx>>>,
     L: UndoLogs<snapshot_vec::UndoLog<unify::Delegate<ty::ConstVid<'tcx>>>>,
 {
-    if let ty::ConstKind::Infer(InferConst::Var(vid)) = c.val() {
+    if let &ty::ConstKind::Infer(InferConst::Var(vid)) = c.val() {
         match table.probe_value(vid).val.known() {
             Some(c) => c,
             None => c,
diff --git a/compiler/rustc_middle/src/mir/pretty.rs b/compiler/rustc_middle/src/mir/pretty.rs
@@ -682,7 +682,7 @@ pub fn write_allocations<'tcx>(
 
     impl<'tcx> Visitor<'tcx> for CollectAllocIds {
         fn visit_const(&mut self, c: ty::Const<'tcx>, _loc: Location) {
-            if let ty::ConstKind::Value(val) = c.val() {
+            if let &ty::ConstKind::Value(val) = c.val() {
                 self.0.extend(alloc_ids_from_const(val));
             }
         }
diff --git a/compiler/rustc_middle/src/ty/consts.rs b/compiler/rustc_middle/src/ty/consts.rs
@@ -21,7 +21,6 @@ pub use valtree::*;
 
 /// Use this rather than `ConstS`, whenever possible.
 #[derive(Copy, Clone, PartialEq, Eq, PartialOrd, Ord, Hash, HashStable)]
-#[cfg_attr(not(bootstrap), rustc_pass_by_value)]
 pub struct Const<'tcx>(pub Interned<'tcx, ConstS<'tcx>>);
 
 impl<'tcx> fmt::Debug for Const<'tcx> {
@@ -48,8 +47,8 @@ impl<'tcx> Const<'tcx> {
         self.0.ty
     }
 
-    pub fn val(self) -> ConstKind<'tcx> {
-        self.0.val
+    pub fn val(&self) -> &ConstKind<'tcx> {
+        &self.0.val
     }
 
     /// Literals and const generic parameters are eagerly converted to a constant, everything else
diff --git a/compiler/rustc_middle/src/ty/context.rs b/compiler/rustc_middle/src/ty/context.rs
@@ -870,7 +870,7 @@ impl<'tcx> CanonicalUserType<'tcx> {
                         },
 
                         GenericArgKind::Const(ct) => match ct.val() {
-                            ty::ConstKind::Bound(debruijn, b) => {
+                            &ty::ConstKind::Bound(debruijn, b) => {
                                 // We only allow a `ty::INNERMOST` index in substitutions.
                                 assert_eq!(debruijn, ty::INNERMOST);
                                 cvar == b
diff --git a/compiler/rustc_middle/src/ty/diagnostics.rs b/compiler/rustc_middle/src/ty/diagnostics.rs
@@ -76,7 +76,7 @@ impl<'tcx> Ty<'tcx> {
             }
         }
 
-        fn const_is_suggestable(kind: ConstKind<'_>) -> bool {
+        fn const_is_suggestable(kind: &ConstKind<'_>) -> bool {
             match kind {
                 ConstKind::Infer(..)
                 | ConstKind::Bound(..)
diff --git a/compiler/rustc_middle/src/ty/flags.rs b/compiler/rustc_middle/src/ty/flags.rs
@@ -289,15 +289,15 @@ impl FlagComputation {
     fn add_const(&mut self, c: ty::Const<'_>) {
         self.add_ty(c.ty());
         match c.val() {
-            ty::ConstKind::Unevaluated(unevaluated) => self.add_unevaluated_const(unevaluated),
+            &ty::ConstKind::Unevaluated(unevaluated) => self.add_unevaluated_const(unevaluated),
             ty::ConstKind::Infer(infer) => {
                 self.add_flags(TypeFlags::STILL_FURTHER_SPECIALIZABLE);
                 match infer {
                     InferConst::Fresh(_) => self.add_flags(TypeFlags::HAS_CT_FRESH),
                     InferConst::Var(_) => self.add_flags(TypeFlags::HAS_CT_INFER),
                 }
             }
-            ty::ConstKind::Bound(debruijn, _) => {
+            &ty::ConstKind::Bound(debruijn, _) => {
                 self.add_bound_var(debruijn);
             }
             ty::ConstKind::Param(_) => {
diff --git a/compiler/rustc_middle/src/ty/fold.rs b/compiler/rustc_middle/src/ty/fold.rs
@@ -689,7 +689,7 @@ impl<'a, 'tcx> TypeFolder<'tcx> for BoundVarReplacer<'a, 'tcx> {
 
     fn fold_const(&mut self, ct: ty::Const<'tcx>) -> ty::Const<'tcx> {
         match ct.val() {
-            ty::ConstKind::Bound(debruijn, bound_const) if debruijn == self.current_index => {
+            &ty::ConstKind::Bound(debruijn, bound_const) if debruijn == self.current_index => {
                 if let Some(fld_c) = self.fld_c.as_mut() {
                     let ct = fld_c(bound_const, ct.ty());
                     return ty::fold::shift_vars(self.tcx, ct, self.current_index.as_u32());
@@ -1083,7 +1083,7 @@ impl<'tcx> TypeFolder<'tcx> for Shifter<'tcx> {
     }
 
     fn fold_const(&mut self, ct: ty::Const<'tcx>) -> ty::Const<'tcx> {
-        if let ty::ConstKind::Bound(debruijn, bound_ct) = ct.val() {
+        if let &ty::ConstKind::Bound(debruijn, bound_ct) = ct.val() {
             if self.amount == 0 || debruijn < self.current_index {
                 ct
             } else {
@@ -1200,7 +1200,7 @@ impl<'tcx> TypeVisitor<'tcx> for HasEscapingVarsVisitor {
         // const, as it has types/regions embedded in a lot of other
         // places.
         match ct.val() {
-            ty::ConstKind::Bound(debruijn, _) if debruijn >= self.outer_index => {
+            &ty::ConstKind::Bound(debruijn, _) if debruijn >= self.outer_index => {
                 ControlFlow::Break(FoundEscapingVars)
             }
             _ => ct.super_visit_with(self),
diff --git a/compiler/rustc_middle/src/ty/print/pretty.rs b/compiler/rustc_middle/src/ty/print/pretty.rs
@@ -1199,7 +1199,7 @@ pub trait PrettyPrinter<'tcx>:
                     }
                 }
             }
-            ty::ConstKind::Infer(infer_ct) => {
+            &ty::ConstKind::Infer(infer_ct) => {
                 match infer_ct {
                     ty::InferConst::Var(ct_vid)
                         if let Some(name) = self.const_infer_name(ct_vid) =>
@@ -1208,11 +1208,11 @@ pub trait PrettyPrinter<'tcx>:
                 }
             }
             ty::ConstKind::Param(ParamConst { name, .. }) => p!(write("{}", name)),
-            ty::ConstKind::Value(value) => {
+            &ty::ConstKind::Value(value) => {
                 return self.pretty_print_const_value(value, ct.ty(), print_ty);
             }
 
-            ty::ConstKind::Bound(debruijn, bound_var) => {
+            &ty::ConstKind::Bound(debruijn, bound_var) => {
                 self.pretty_print_bound_var(debruijn, bound_var)?
             }
             ty::ConstKind::Placeholder(placeholder) => p!(write("Placeholder({:?})", placeholder)),
diff --git a/compiler/rustc_middle/src/ty/relate.rs b/compiler/rustc_middle/src/ty/relate.rs
@@ -580,7 +580,7 @@ pub fn super_relate_consts<'tcx, R: TypeRelation<'tcx>>(
 
         (ty::ConstKind::Param(a_p), ty::ConstKind::Param(b_p)) => a_p.index == b_p.index,
         (ty::ConstKind::Placeholder(p1), ty::ConstKind::Placeholder(p2)) => p1 == p2,
-        (ty::ConstKind::Value(a_val), ty::ConstKind::Value(b_val)) => {
+        (&ty::ConstKind::Value(a_val), &ty::ConstKind::Value(b_val)) => {
             check_const_value_eq(relation, a_val, b_val, a, b)?
         }
 
diff --git a/compiler/rustc_middle/src/ty/structural_impls.rs b/compiler/rustc_middle/src/ty/structural_impls.rs
@@ -1159,7 +1159,7 @@ impl<'tcx> TypeFoldable<'tcx> for ty::Const<'tcx> {
     ) -> Result<Self, F::Error> {
         let ty = self.ty().try_fold_with(folder)?;
         let val = self.val().try_fold_with(folder)?;
-        if ty != self.ty() || val != self.val() {
+        if ty != self.ty() || val != *self.val() {
             Ok(folder.tcx().mk_const(ty::ConstS { ty, val }))
         } else {
             Ok(self)
diff --git a/compiler/rustc_middle/src/ty/subst.rs b/compiler/rustc_middle/src/ty/subst.rs
@@ -521,7 +521,7 @@ impl<'a, 'tcx> TypeFolder<'tcx> for SubstFolder<'a, 'tcx> {
     }
 
     fn fold_const(&mut self, c: ty::Const<'tcx>) -> ty::Const<'tcx> {
-        if let ty::ConstKind::Param(p) = c.val() {
+        if let &ty::ConstKind::Param(p) = c.val() {
             self.const_for_param(p, c)
         } else {
             c.super_fold_with(self)
diff --git a/compiler/rustc_mir_build/src/thir/pattern/mod.rs b/compiler/rustc_mir_build/src/thir/pattern/mod.rs
@@ -792,8 +792,8 @@ crate fn compare_const_vals<'tcx>(
 
     if let ty::Str = ty.kind() {
         if let (
-            ty::ConstKind::Value(a_val @ ConstValue::Slice { .. }),
-            ty::ConstKind::Value(b_val @ ConstValue::Slice { .. }),
+            &ty::ConstKind::Value(a_val @ ConstValue::Slice { .. }),
+            &ty::ConstKind::Value(b_val @ ConstValue::Slice { .. }),
         ) = (a.val(), b.val())
         {
             let a_bytes = get_slice_bytes(&tcx, a_val);
diff --git a/compiler/rustc_monomorphize/src/collector.rs b/compiler/rustc_monomorphize/src/collector.rs
diff --git a/compiler/rustc_monomorphize/src/polymorphize.rs b/compiler/rustc_monomorphize/src/polymorphize.rs
diff --git a/compiler/rustc_symbol_mangling/src/v0.rs b/compiler/rustc_symbol_mangling/src/v0.rs
diff --git a/compiler/rustc_trait_selection/src/traits/auto_trait.rs b/compiler/rustc_trait_selection/src/traits/auto_trait.rs
diff --git a/compiler/rustc_trait_selection/src/traits/fulfill.rs b/compiler/rustc_trait_selection/src/traits/fulfill.rs
diff --git a/compiler/rustc_trait_selection/src/traits/project.rs b/compiler/rustc_trait_selection/src/traits/project.rs
diff --git a/compiler/rustc_trait_selection/src/traits/query/normalize.rs b/compiler/rustc_trait_selection/src/traits/query/normalize.rs
diff --git a/compiler/rustc_trait_selection/src/traits/select/mod.rs b/compiler/rustc_trait_selection/src/traits/select/mod.rs
diff --git a/compiler/rustc_trait_selection/src/traits/wf.rs b/compiler/rustc_trait_selection/src/traits/wf.rs
diff --git a/compiler/rustc_traits/src/chalk/lowering.rs b/compiler/rustc_traits/src/chalk/lowering.rs
diff --git a/compiler/rustc_typeck/src/constrained_generic_params.rs b/compiler/rustc_typeck/src/constrained_generic_params.rs
diff --git a/src/tools/clippy/clippy_utils/src/consts.rs b/src/tools/clippy/clippy_utils/src/consts.rs

Original file line number	Diff line number	Diff line change
`@@ -574,7 +574,7 @@ impl<'mir, 'tcx: 'mir, M: Machine<'mir, 'tcx>> InterpCx<'mir, 'tcx, M> {`
`574`	`574`	`ty::ConstKind::Infer(..) \| ty::ConstKind::Placeholder(..) => {`
`575`	`575`	`span_bug!(self.cur_span(), "const_to_op: Unexpected ConstKind {:?}", val)`
`576`	`576`	`}`
`577`		`- ty::ConstKind::Value(val_val) => self.const_val_to_op(val_val, val.ty(), layout),`
	`577`	`+ &ty::ConstKind::Value(val_val) => self.const_val_to_op(val_val, val.ty(), layout),`
`578`	`578`	`}`
`579`	`579`	`}`
`580`	`580`
Original file line number	Diff line number	Diff line change
`@@ -437,7 +437,7 @@ impl<'cx, 'tcx> InferCtxt<'cx, 'tcx> {`
`437`	`437`	`}`
`438`	`438`	`}`
`439`	`439`	`GenericArgKind::Const(result_value) => {`
`440`		`- if let ty::ConstKind::Bound(debrujin, b) = result_value.val() {`
	`440`	`+ if let &ty::ConstKind::Bound(debrujin, b) = result_value.val() {`
`441`	`441`	// ...in which case we would set `canonical_vars[0]` to `Some(const X)`.
`442`	`442`
`443`	`443`	// We only allow a `ty::INNERMOST` index in substitutions.
Original file line number	Diff line number	Diff line change
`@@ -141,8 +141,8 @@ impl<'infcx, 'tcx> InferCtxt<'infcx, 'tcx> {`
`141`	`141`
`142`	`142`	`match (a.val(), b.val()) {`
`143`	`143`	`(`
`144`		`- ty::ConstKind::Infer(InferConst::Var(a_vid)),`
`145`		`- ty::ConstKind::Infer(InferConst::Var(b_vid)),`
	`144`	`+ &ty::ConstKind::Infer(InferConst::Var(a_vid)),`
	`145`	`+ &ty::ConstKind::Infer(InferConst::Var(b_vid)),`
`146`	`146`	`) => {`
`147`	`147`	`self.inner`
`148`	`148`	`.borrow_mut()`
`@@ -158,11 +158,11 @@ impl<'infcx, 'tcx> InferCtxt<'infcx, 'tcx> {`
`158`	`158`	`bug!("tried to combine ConstKind::Infer/ConstKind::Infer(InferConst::Var)")`
`159`	`159`	`}`
`160`	`160`
`161`		`- (ty::ConstKind::Infer(InferConst::Var(vid)), _) => {`
	`161`	`+ (&ty::ConstKind::Infer(InferConst::Var(vid)), _) => {`
`162`	`162`	`return self.unify_const_variable(relation.param_env(), vid, b, a_is_expected);`
`163`	`163`	`}`
`164`	`164`
`165`		`- (_, ty::ConstKind::Infer(InferConst::Var(vid))) => {`
	`165`	`+ (_, &ty::ConstKind::Infer(InferConst::Var(vid))) => {`
`166`	`166`	`return self.unify_const_variable(relation.param_env(), vid, a, !a_is_expected);`
`167`	`167`	`}`
`168`	`168`	`(ty::ConstKind::Unevaluated(..), _) if self.tcx.lazy_normalization() => {`
`@@ -722,7 +722,7 @@ impl<'tcx> TypeRelation<'tcx> for Generalizer<'_, 'tcx> {`
`722`	`722`	`assert_eq!(c, c2); // we are abusing TypeRelation here; both LHS and RHS ought to be ==`
`723`	`723`
`724`	`724`	`match c.val() {`
`725`		`- ty::ConstKind::Infer(InferConst::Var(vid)) => {`
	`725`	`+ &ty::ConstKind::Infer(InferConst::Var(vid)) => {`
`726`	`726`	`let mut inner = self.infcx.inner.borrow_mut();`
`727`	`727`	`let variable_table = &mut inner.const_unification_table();`
`728`	`728`	`let var_value = variable_table.probe_value(vid);`
`@@ -744,7 +744,7 @@ impl<'tcx> TypeRelation<'tcx> for Generalizer<'_, 'tcx> {`
`744`	`744`	`}`
`745`	`745`	`}`
`746`	`746`	`}`
`747`		`- ty::ConstKind::Unevaluated(ty::Unevaluated { def, substs, promoted })`
	`747`	`+ &ty::ConstKind::Unevaluated(ty::Unevaluated { def, substs, promoted })`
`748`	`748`	`if self.tcx().lazy_normalization() =>`
`749`	`749`	`{`
`750`	`750`	`assert_eq!(promoted, None);`
`@@ -952,7 +952,7 @@ impl<'tcx> TypeRelation<'tcx> for ConstInferUnifier<'_, 'tcx> {`
`952`	`952`	`debug!("ConstInferUnifier: c={:?}", c);`
`953`	`953`
`954`	`954`	`match c.val() {`
`955`		`- ty::ConstKind::Infer(InferConst::Var(vid)) => {`
	`955`	`+ &ty::ConstKind::Infer(InferConst::Var(vid)) => {`
`956`	`956`	`// Check if the current unification would end up`
`957`	`957`	// unifying `target_vid` with a const which contains
`958`	`958`	// an inference variable which is unioned with `target_vid`.
`@@ -990,7 +990,7 @@ impl<'tcx> TypeRelation<'tcx> for ConstInferUnifier<'_, 'tcx> {`
`990`	`990`	`}`
`991`	`991`	`}`
`992`	`992`	`}`
`993`		`- ty::ConstKind::Unevaluated(ty::Unevaluated { def, substs, promoted })`
	`993`	`+ &ty::ConstKind::Unevaluated(ty::Unevaluated { def, substs, promoted })`
`994`	`994`	`if self.tcx().lazy_normalization() =>`
`995`	`995`	`{`
`996`	`996`	`assert_eq!(promoted, None);`
Original file line number	Diff line number	Diff line change
`@@ -410,7 +410,7 @@ impl<'a, 'tcx> InferCtxt<'a, 'tcx> {`
`410`	`410`	`}`
`411`	`411`	`GenericArgKind::Const(ct) => {`
`412`	`412`	`match ct.val() {`
`413`		`- ty::ConstKind::Infer(InferConst::Var(vid)) => {`
	`413`	`+ &ty::ConstKind::Infer(InferConst::Var(vid)) => {`
`414`	`414`	`let origin = self`
`415`	`415`	`.inner`
`416`	`416`	`.borrow_mut()`
Original file line number	Diff line number	Diff line change
`@@ -1761,7 +1761,7 @@ impl<'tcx> TyOrConstInferVar<'tcx> {`
`1761`	`1761`	/// for constants other than `ty::ConstKind::Infer(_)` (or `InferConst::Fresh`).
`1762`	`1762`	`pub fn maybe_from_const(ct: ty::Const<'tcx>) -> Option<Self> {`
`1763`	`1763`	`match ct.val() {`
`1764`		`- ty::ConstKind::Infer(InferConst::Var(v)) => Some(TyOrConstInferVar::Const(v)),`
	`1764`	`+ &ty::ConstKind::Infer(InferConst::Var(v)) => Some(TyOrConstInferVar::Const(v)),`
`1765`	`1765`	`_ => None,`
`1766`	`1766`	`}`
`1767`	`1767`	`}`
`@@ -1781,7 +1781,7 @@ impl<'a, 'tcx> TypeFolder<'tcx> for ShallowResolver<'a, 'tcx> {`
`1781`	`1781`	`}`
`1782`	`1782`
`1783`	`1783`	`fn fold_const(&mut self, ct: ty::Const<'tcx>) -> ty::Const<'tcx> {`
`1784`		`- if let ty::ConstKind::Infer(InferConst::Var(vid)) = ct.val() {`
	`1784`	`+ if let &ty::ConstKind::Infer(InferConst::Var(vid)) = ct.val() {`
`1785`	`1785`	`self.infcx`
`1786`	`1786`	`.inner`
`1787`	`1787`	`.borrow_mut()`