]> git.lizzy.rs Git - rust.git/blob - src/tools/clippy/clippy_lints/src/utils/hir_utils.rs
Auto merge of #79780 - camelid:use-summary_opts, r=GuillaumeGomez
[rust.git] / src / tools / clippy / clippy_lints / src / utils / hir_utils.rs
1 use crate::consts::{constant_context, constant_simple};
2 use crate::utils::differing_macro_contexts;
3 use rustc_ast::ast::InlineAsmTemplatePiece;
4 use rustc_data_structures::stable_hasher::{HashStable, StableHasher};
5 use rustc_hir::{
6     BinOpKind, Block, BlockCheckMode, BodyId, BorrowKind, CaptureBy, Expr, ExprKind, Field, FieldPat, FnRetTy,
7     GenericArg, GenericArgs, Guard, InlineAsmOperand, Lifetime, LifetimeName, ParamName, Pat, PatKind, Path,
8     PathSegment, QPath, Stmt, StmtKind, Ty, TyKind, TypeBinding,
9 };
10 use rustc_lint::LateContext;
11 use rustc_middle::ich::StableHashingContextProvider;
12 use rustc_middle::ty::TypeckResults;
13 use rustc_span::Symbol;
14 use std::hash::Hash;
15
16 /// Type used to check whether two ast are the same. This is different from the
17 /// operator
18 /// `==` on ast types as this operator would compare true equality with ID and
19 /// span.
20 ///
21 /// Note that some expressions kinds are not considered but could be added.
22 pub struct SpanlessEq<'a, 'tcx> {
23     /// Context used to evaluate constant expressions.
24     cx: &'a LateContext<'tcx>,
25     maybe_typeck_results: Option<&'tcx TypeckResults<'tcx>>,
26     allow_side_effects: bool,
27 }
28
29 impl<'a, 'tcx> SpanlessEq<'a, 'tcx> {
30     pub fn new(cx: &'a LateContext<'tcx>) -> Self {
31         Self {
32             cx,
33             maybe_typeck_results: cx.maybe_typeck_results(),
34             allow_side_effects: true,
35         }
36     }
37
38     /// Consider expressions containing potential side effects as not equal.
39     pub fn deny_side_effects(self) -> Self {
40         Self {
41             allow_side_effects: false,
42             ..self
43         }
44     }
45
46     /// Checks whether two statements are the same.
47     pub fn eq_stmt(&mut self, left: &Stmt<'_>, right: &Stmt<'_>) -> bool {
48         match (&left.kind, &right.kind) {
49             (&StmtKind::Local(ref l), &StmtKind::Local(ref r)) => {
50                 self.eq_pat(&l.pat, &r.pat)
51                     && both(&l.ty, &r.ty, |l, r| self.eq_ty(l, r))
52                     && both(&l.init, &r.init, |l, r| self.eq_expr(l, r))
53             },
54             (&StmtKind::Expr(ref l), &StmtKind::Expr(ref r)) | (&StmtKind::Semi(ref l), &StmtKind::Semi(ref r)) => {
55                 self.eq_expr(l, r)
56             },
57             _ => false,
58         }
59     }
60
61     /// Checks whether two blocks are the same.
62     pub fn eq_block(&mut self, left: &Block<'_>, right: &Block<'_>) -> bool {
63         over(&left.stmts, &right.stmts, |l, r| self.eq_stmt(l, r))
64             && both(&left.expr, &right.expr, |l, r| self.eq_expr(l, r))
65     }
66
67     #[allow(clippy::similar_names)]
68     pub fn eq_expr(&mut self, left: &Expr<'_>, right: &Expr<'_>) -> bool {
69         if !self.allow_side_effects && differing_macro_contexts(left.span, right.span) {
70             return false;
71         }
72
73         if let Some(typeck_results) = self.maybe_typeck_results {
74             if let (Some(l), Some(r)) = (
75                 constant_simple(self.cx, typeck_results, left),
76                 constant_simple(self.cx, typeck_results, right),
77             ) {
78                 if l == r {
79                     return true;
80                 }
81             }
82         }
83
84         match (&reduce_exprkind(&left.kind), &reduce_exprkind(&right.kind)) {
85             (&ExprKind::AddrOf(lb, l_mut, ref le), &ExprKind::AddrOf(rb, r_mut, ref re)) => {
86                 lb == rb && l_mut == r_mut && self.eq_expr(le, re)
87             },
88             (&ExprKind::Continue(li), &ExprKind::Continue(ri)) => {
89                 both(&li.label, &ri.label, |l, r| l.ident.as_str() == r.ident.as_str())
90             },
91             (&ExprKind::Assign(ref ll, ref lr, _), &ExprKind::Assign(ref rl, ref rr, _)) => {
92                 self.allow_side_effects && self.eq_expr(ll, rl) && self.eq_expr(lr, rr)
93             },
94             (&ExprKind::AssignOp(ref lo, ref ll, ref lr), &ExprKind::AssignOp(ref ro, ref rl, ref rr)) => {
95                 self.allow_side_effects && lo.node == ro.node && self.eq_expr(ll, rl) && self.eq_expr(lr, rr)
96             },
97             (&ExprKind::Block(ref l, _), &ExprKind::Block(ref r, _)) => self.eq_block(l, r),
98             (&ExprKind::Binary(l_op, ref ll, ref lr), &ExprKind::Binary(r_op, ref rl, ref rr)) => {
99                 l_op.node == r_op.node && self.eq_expr(ll, rl) && self.eq_expr(lr, rr)
100                     || swap_binop(l_op.node, ll, lr).map_or(false, |(l_op, ll, lr)| {
101                         l_op == r_op.node && self.eq_expr(ll, rl) && self.eq_expr(lr, rr)
102                     })
103             },
104             (&ExprKind::Break(li, ref le), &ExprKind::Break(ri, ref re)) => {
105                 both(&li.label, &ri.label, |l, r| l.ident.as_str() == r.ident.as_str())
106                     && both(le, re, |l, r| self.eq_expr(l, r))
107             },
108             (&ExprKind::Box(ref l), &ExprKind::Box(ref r)) => self.eq_expr(l, r),
109             (&ExprKind::Call(l_fun, l_args), &ExprKind::Call(r_fun, r_args)) => {
110                 self.allow_side_effects && self.eq_expr(l_fun, r_fun) && self.eq_exprs(l_args, r_args)
111             },
112             (&ExprKind::Cast(ref lx, ref lt), &ExprKind::Cast(ref rx, ref rt))
113             | (&ExprKind::Type(ref lx, ref lt), &ExprKind::Type(ref rx, ref rt)) => {
114                 self.eq_expr(lx, rx) && self.eq_ty(lt, rt)
115             },
116             (&ExprKind::Field(ref l_f_exp, ref l_f_ident), &ExprKind::Field(ref r_f_exp, ref r_f_ident)) => {
117                 l_f_ident.name == r_f_ident.name && self.eq_expr(l_f_exp, r_f_exp)
118             },
119             (&ExprKind::Index(ref la, ref li), &ExprKind::Index(ref ra, ref ri)) => {
120                 self.eq_expr(la, ra) && self.eq_expr(li, ri)
121             },
122             (&ExprKind::Lit(ref l), &ExprKind::Lit(ref r)) => l.node == r.node,
123             (&ExprKind::Loop(ref lb, ref ll, ref lls), &ExprKind::Loop(ref rb, ref rl, ref rls)) => {
124                 lls == rls && self.eq_block(lb, rb) && both(ll, rl, |l, r| l.ident.as_str() == r.ident.as_str())
125             },
126             (&ExprKind::Match(ref le, ref la, ref ls), &ExprKind::Match(ref re, ref ra, ref rs)) => {
127                 ls == rs
128                     && self.eq_expr(le, re)
129                     && over(la, ra, |l, r| {
130                         self.eq_expr(&l.body, &r.body)
131                             && both(&l.guard, &r.guard, |l, r| self.eq_guard(l, r))
132                             && self.eq_pat(&l.pat, &r.pat)
133                     })
134             },
135             (&ExprKind::MethodCall(l_path, _, l_args, _), &ExprKind::MethodCall(r_path, _, r_args, _)) => {
136                 self.allow_side_effects && self.eq_path_segment(l_path, r_path) && self.eq_exprs(l_args, r_args)
137             },
138             (&ExprKind::Repeat(ref le, ref ll_id), &ExprKind::Repeat(ref re, ref rl_id)) => {
139                 let mut celcx = constant_context(self.cx, self.cx.tcx.typeck_body(ll_id.body));
140                 let ll = celcx.expr(&self.cx.tcx.hir().body(ll_id.body).value);
141                 let mut celcx = constant_context(self.cx, self.cx.tcx.typeck_body(rl_id.body));
142                 let rl = celcx.expr(&self.cx.tcx.hir().body(rl_id.body).value);
143
144                 self.eq_expr(le, re) && ll == rl
145             },
146             (&ExprKind::Ret(ref l), &ExprKind::Ret(ref r)) => both(l, r, |l, r| self.eq_expr(l, r)),
147             (&ExprKind::Path(ref l), &ExprKind::Path(ref r)) => self.eq_qpath(l, r),
148             (&ExprKind::Struct(ref l_path, ref lf, ref lo), &ExprKind::Struct(ref r_path, ref rf, ref ro)) => {
149                 self.eq_qpath(l_path, r_path)
150                     && both(lo, ro, |l, r| self.eq_expr(l, r))
151                     && over(lf, rf, |l, r| self.eq_field(l, r))
152             },
153             (&ExprKind::Tup(l_tup), &ExprKind::Tup(r_tup)) => self.eq_exprs(l_tup, r_tup),
154             (&ExprKind::Unary(l_op, ref le), &ExprKind::Unary(r_op, ref re)) => l_op == r_op && self.eq_expr(le, re),
155             (&ExprKind::Array(l), &ExprKind::Array(r)) => self.eq_exprs(l, r),
156             (&ExprKind::DropTemps(ref le), &ExprKind::DropTemps(ref re)) => self.eq_expr(le, re),
157             _ => false,
158         }
159     }
160
161     fn eq_exprs(&mut self, left: &[Expr<'_>], right: &[Expr<'_>]) -> bool {
162         over(left, right, |l, r| self.eq_expr(l, r))
163     }
164
165     fn eq_field(&mut self, left: &Field<'_>, right: &Field<'_>) -> bool {
166         left.ident.name == right.ident.name && self.eq_expr(&left.expr, &right.expr)
167     }
168
169     fn eq_guard(&mut self, left: &Guard<'_>, right: &Guard<'_>) -> bool {
170         match (left, right) {
171             (Guard::If(l), Guard::If(r)) => self.eq_expr(l, r),
172         }
173     }
174
175     fn eq_generic_arg(&mut self, left: &GenericArg<'_>, right: &GenericArg<'_>) -> bool {
176         match (left, right) {
177             (GenericArg::Lifetime(l_lt), GenericArg::Lifetime(r_lt)) => Self::eq_lifetime(l_lt, r_lt),
178             (GenericArg::Type(l_ty), GenericArg::Type(r_ty)) => self.eq_ty(l_ty, r_ty),
179             _ => false,
180         }
181     }
182
183     fn eq_lifetime(left: &Lifetime, right: &Lifetime) -> bool {
184         left.name == right.name
185     }
186
187     pub fn eq_fieldpat(&mut self, left: &FieldPat<'_>, right: &FieldPat<'_>) -> bool {
188         let (FieldPat { ident: li, pat: lp, .. }, FieldPat { ident: ri, pat: rp, .. }) = (&left, &right);
189         li.name.as_str() == ri.name.as_str() && self.eq_pat(lp, rp)
190     }
191
192     /// Checks whether two patterns are the same.
193     pub fn eq_pat(&mut self, left: &Pat<'_>, right: &Pat<'_>) -> bool {
194         match (&left.kind, &right.kind) {
195             (&PatKind::Box(ref l), &PatKind::Box(ref r)) => self.eq_pat(l, r),
196             (&PatKind::Struct(ref lp, ref la, ..), &PatKind::Struct(ref rp, ref ra, ..)) => {
197                 self.eq_qpath(lp, rp) && over(la, ra, |l, r| self.eq_fieldpat(l, r))
198             },
199             (&PatKind::TupleStruct(ref lp, ref la, ls), &PatKind::TupleStruct(ref rp, ref ra, rs)) => {
200                 self.eq_qpath(lp, rp) && over(la, ra, |l, r| self.eq_pat(l, r)) && ls == rs
201             },
202             (&PatKind::Binding(ref lb, .., ref li, ref lp), &PatKind::Binding(ref rb, .., ref ri, ref rp)) => {
203                 lb == rb && li.name.as_str() == ri.name.as_str() && both(lp, rp, |l, r| self.eq_pat(l, r))
204             },
205             (&PatKind::Path(ref l), &PatKind::Path(ref r)) => self.eq_qpath(l, r),
206             (&PatKind::Lit(ref l), &PatKind::Lit(ref r)) => self.eq_expr(l, r),
207             (&PatKind::Tuple(ref l, ls), &PatKind::Tuple(ref r, rs)) => {
208                 ls == rs && over(l, r, |l, r| self.eq_pat(l, r))
209             },
210             (&PatKind::Range(ref ls, ref le, li), &PatKind::Range(ref rs, ref re, ri)) => {
211                 both(ls, rs, |a, b| self.eq_expr(a, b)) && both(le, re, |a, b| self.eq_expr(a, b)) && (li == ri)
212             },
213             (&PatKind::Ref(ref le, ref lm), &PatKind::Ref(ref re, ref rm)) => lm == rm && self.eq_pat(le, re),
214             (&PatKind::Slice(ref ls, ref li, ref le), &PatKind::Slice(ref rs, ref ri, ref re)) => {
215                 over(ls, rs, |l, r| self.eq_pat(l, r))
216                     && over(le, re, |l, r| self.eq_pat(l, r))
217                     && both(li, ri, |l, r| self.eq_pat(l, r))
218             },
219             (&PatKind::Wild, &PatKind::Wild) => true,
220             _ => false,
221         }
222     }
223
224     #[allow(clippy::similar_names)]
225     fn eq_qpath(&mut self, left: &QPath<'_>, right: &QPath<'_>) -> bool {
226         match (left, right) {
227             (&QPath::Resolved(ref lty, ref lpath), &QPath::Resolved(ref rty, ref rpath)) => {
228                 both(lty, rty, |l, r| self.eq_ty(l, r)) && self.eq_path(lpath, rpath)
229             },
230             (&QPath::TypeRelative(ref lty, ref lseg), &QPath::TypeRelative(ref rty, ref rseg)) => {
231                 self.eq_ty(lty, rty) && self.eq_path_segment(lseg, rseg)
232             },
233             (&QPath::LangItem(llang_item, _), &QPath::LangItem(rlang_item, _)) => llang_item == rlang_item,
234             _ => false,
235         }
236     }
237
238     fn eq_path(&mut self, left: &Path<'_>, right: &Path<'_>) -> bool {
239         left.is_global() == right.is_global()
240             && over(&left.segments, &right.segments, |l, r| self.eq_path_segment(l, r))
241     }
242
243     fn eq_path_parameters(&mut self, left: &GenericArgs<'_>, right: &GenericArgs<'_>) -> bool {
244         if !(left.parenthesized || right.parenthesized) {
245             over(&left.args, &right.args, |l, r| self.eq_generic_arg(l, r)) // FIXME(flip1995): may not work
246                 && over(&left.bindings, &right.bindings, |l, r| self.eq_type_binding(l, r))
247         } else if left.parenthesized && right.parenthesized {
248             over(left.inputs(), right.inputs(), |l, r| self.eq_ty(l, r))
249                 && both(&Some(&left.bindings[0].ty()), &Some(&right.bindings[0].ty()), |l, r| {
250                     self.eq_ty(l, r)
251                 })
252         } else {
253             false
254         }
255     }
256
257     pub fn eq_path_segments(&mut self, left: &[PathSegment<'_>], right: &[PathSegment<'_>]) -> bool {
258         left.len() == right.len() && left.iter().zip(right).all(|(l, r)| self.eq_path_segment(l, r))
259     }
260
261     pub fn eq_path_segment(&mut self, left: &PathSegment<'_>, right: &PathSegment<'_>) -> bool {
262         // The == of idents doesn't work with different contexts,
263         // we have to be explicit about hygiene
264         left.ident.as_str() == right.ident.as_str()
265             && both(&left.args, &right.args, |l, r| self.eq_path_parameters(l, r))
266     }
267
268     pub fn eq_ty(&mut self, left: &Ty<'_>, right: &Ty<'_>) -> bool {
269         self.eq_ty_kind(&left.kind, &right.kind)
270     }
271
272     #[allow(clippy::similar_names)]
273     pub fn eq_ty_kind(&mut self, left: &TyKind<'_>, right: &TyKind<'_>) -> bool {
274         match (left, right) {
275             (&TyKind::Slice(ref l_vec), &TyKind::Slice(ref r_vec)) => self.eq_ty(l_vec, r_vec),
276             (&TyKind::Array(ref lt, ref ll_id), &TyKind::Array(ref rt, ref rl_id)) => {
277                 let old_maybe_typeck_results = self.maybe_typeck_results;
278
279                 let mut celcx = constant_context(self.cx, self.cx.tcx.typeck_body(ll_id.body));
280                 self.maybe_typeck_results = Some(self.cx.tcx.typeck_body(ll_id.body));
281                 let ll = celcx.expr(&self.cx.tcx.hir().body(ll_id.body).value);
282
283                 let mut celcx = constant_context(self.cx, self.cx.tcx.typeck_body(rl_id.body));
284                 self.maybe_typeck_results = Some(self.cx.tcx.typeck_body(rl_id.body));
285                 let rl = celcx.expr(&self.cx.tcx.hir().body(rl_id.body).value);
286
287                 let eq_ty = self.eq_ty(lt, rt);
288                 self.maybe_typeck_results = old_maybe_typeck_results;
289                 eq_ty && ll == rl
290             },
291             (&TyKind::Ptr(ref l_mut), &TyKind::Ptr(ref r_mut)) => {
292                 l_mut.mutbl == r_mut.mutbl && self.eq_ty(&*l_mut.ty, &*r_mut.ty)
293             },
294             (&TyKind::Rptr(_, ref l_rmut), &TyKind::Rptr(_, ref r_rmut)) => {
295                 l_rmut.mutbl == r_rmut.mutbl && self.eq_ty(&*l_rmut.ty, &*r_rmut.ty)
296             },
297             (&TyKind::Path(ref l), &TyKind::Path(ref r)) => self.eq_qpath(l, r),
298             (&TyKind::Tup(ref l), &TyKind::Tup(ref r)) => over(l, r, |l, r| self.eq_ty(l, r)),
299             (&TyKind::Infer, &TyKind::Infer) => true,
300             _ => false,
301         }
302     }
303
304     fn eq_type_binding(&mut self, left: &TypeBinding<'_>, right: &TypeBinding<'_>) -> bool {
305         left.ident.name == right.ident.name && self.eq_ty(&left.ty(), &right.ty())
306     }
307 }
308
309 /// Some simple reductions like `{ return }` => `return`
310 fn reduce_exprkind<'hir>(kind: &'hir ExprKind<'hir>) -> &ExprKind<'hir> {
311     if let ExprKind::Block(block, _) = kind {
312         match (block.stmts, block.expr) {
313             // `{}` => `()`
314             ([], None) => &ExprKind::Tup(&[]),
315             ([], Some(expr)) => match expr.kind {
316                 // `{ return .. }` => `return ..`
317                 ExprKind::Ret(..) => &expr.kind,
318                 _ => kind,
319             },
320             ([stmt], None) => match stmt.kind {
321                 StmtKind::Expr(expr) | StmtKind::Semi(expr) => match expr.kind {
322                     // `{ return ..; }` => `return ..`
323                     ExprKind::Ret(..) => &expr.kind,
324                     _ => kind,
325                 },
326                 _ => kind,
327             },
328             _ => kind,
329         }
330     } else {
331         kind
332     }
333 }
334
335 fn swap_binop<'a>(
336     binop: BinOpKind,
337     lhs: &'a Expr<'a>,
338     rhs: &'a Expr<'a>,
339 ) -> Option<(BinOpKind, &'a Expr<'a>, &'a Expr<'a>)> {
340     match binop {
341         BinOpKind::Add | BinOpKind::Eq | BinOpKind::Ne | BinOpKind::BitAnd | BinOpKind::BitXor | BinOpKind::BitOr => {
342             Some((binop, rhs, lhs))
343         },
344         BinOpKind::Lt => Some((BinOpKind::Gt, rhs, lhs)),
345         BinOpKind::Le => Some((BinOpKind::Ge, rhs, lhs)),
346         BinOpKind::Ge => Some((BinOpKind::Le, rhs, lhs)),
347         BinOpKind::Gt => Some((BinOpKind::Lt, rhs, lhs)),
348         BinOpKind::Mul // Not always commutative, e.g. with matrices. See issue #5698
349         | BinOpKind::Shl
350         | BinOpKind::Shr
351         | BinOpKind::Rem
352         | BinOpKind::Sub
353         | BinOpKind::Div
354         | BinOpKind::And
355         | BinOpKind::Or => None,
356     }
357 }
358
359 /// Checks if the two `Option`s are both `None` or some equal values as per
360 /// `eq_fn`.
361 pub fn both<X>(l: &Option<X>, r: &Option<X>, mut eq_fn: impl FnMut(&X, &X) -> bool) -> bool {
362     l.as_ref()
363         .map_or_else(|| r.is_none(), |x| r.as_ref().map_or(false, |y| eq_fn(x, y)))
364 }
365
366 /// Checks if two slices are equal as per `eq_fn`.
367 pub fn over<X>(left: &[X], right: &[X], mut eq_fn: impl FnMut(&X, &X) -> bool) -> bool {
368     left.len() == right.len() && left.iter().zip(right).all(|(x, y)| eq_fn(x, y))
369 }
370
371 /// Checks if two expressions evaluate to the same value, and don't contain any side effects.
372 pub fn eq_expr_value(cx: &LateContext<'_>, left: &Expr<'_>, right: &Expr<'_>) -> bool {
373     SpanlessEq::new(cx).deny_side_effects().eq_expr(left, right)
374 }
375
376 /// Type used to hash an ast element. This is different from the `Hash` trait
377 /// on ast types as this
378 /// trait would consider IDs and spans.
379 ///
380 /// All expressions kind are hashed, but some might have a weaker hash.
381 pub struct SpanlessHash<'a, 'tcx> {
382     /// Context used to evaluate constant expressions.
383     cx: &'a LateContext<'tcx>,
384     maybe_typeck_results: Option<&'tcx TypeckResults<'tcx>>,
385     s: StableHasher,
386 }
387
388 impl<'a, 'tcx> SpanlessHash<'a, 'tcx> {
389     pub fn new(cx: &'a LateContext<'tcx>) -> Self {
390         Self {
391             cx,
392             maybe_typeck_results: cx.maybe_typeck_results(),
393             s: StableHasher::new(),
394         }
395     }
396
397     pub fn finish(self) -> u64 {
398         self.s.finish()
399     }
400
401     pub fn hash_block(&mut self, b: &Block<'_>) {
402         for s in b.stmts {
403             self.hash_stmt(s);
404         }
405
406         if let Some(ref e) = b.expr {
407             self.hash_expr(e);
408         }
409
410         match b.rules {
411             BlockCheckMode::DefaultBlock => 0,
412             BlockCheckMode::UnsafeBlock(_) => 1,
413             BlockCheckMode::PushUnsafeBlock(_) => 2,
414             BlockCheckMode::PopUnsafeBlock(_) => 3,
415         }
416         .hash(&mut self.s);
417     }
418
419     #[allow(clippy::many_single_char_names, clippy::too_many_lines)]
420     pub fn hash_expr(&mut self, e: &Expr<'_>) {
421         let simple_const = self
422             .maybe_typeck_results
423             .and_then(|typeck_results| constant_simple(self.cx, typeck_results, e));
424
425         // const hashing may result in the same hash as some unrelated node, so add a sort of
426         // discriminant depending on which path we're choosing next
427         simple_const.is_some().hash(&mut self.s);
428
429         if let Some(e) = simple_const {
430             return e.hash(&mut self.s);
431         }
432
433         std::mem::discriminant(&e.kind).hash(&mut self.s);
434
435         match e.kind {
436             ExprKind::AddrOf(kind, m, ref e) => {
437                 match kind {
438                     BorrowKind::Ref => 0,
439                     BorrowKind::Raw => 1,
440                 }
441                 .hash(&mut self.s);
442                 m.hash(&mut self.s);
443                 self.hash_expr(e);
444             },
445             ExprKind::Continue(i) => {
446                 if let Some(i) = i.label {
447                     self.hash_name(i.ident.name);
448                 }
449             },
450             ExprKind::Assign(ref l, ref r, _) => {
451                 self.hash_expr(l);
452                 self.hash_expr(r);
453             },
454             ExprKind::AssignOp(ref o, ref l, ref r) => {
455                 o.node
456                     .hash_stable(&mut self.cx.tcx.get_stable_hashing_context(), &mut self.s);
457                 self.hash_expr(l);
458                 self.hash_expr(r);
459             },
460             ExprKind::Block(ref b, _) => {
461                 self.hash_block(b);
462             },
463             ExprKind::Binary(op, ref l, ref r) => {
464                 op.node
465                     .hash_stable(&mut self.cx.tcx.get_stable_hashing_context(), &mut self.s);
466                 self.hash_expr(l);
467                 self.hash_expr(r);
468             },
469             ExprKind::Break(i, ref j) => {
470                 if let Some(i) = i.label {
471                     self.hash_name(i.ident.name);
472                 }
473                 if let Some(ref j) = *j {
474                     self.hash_expr(&*j);
475                 }
476             },
477             ExprKind::Box(ref e) | ExprKind::DropTemps(ref e) | ExprKind::Yield(ref e, _) => {
478                 self.hash_expr(e);
479             },
480             ExprKind::Call(ref fun, args) => {
481                 self.hash_expr(fun);
482                 self.hash_exprs(args);
483             },
484             ExprKind::Cast(ref e, ref ty) | ExprKind::Type(ref e, ref ty) => {
485                 self.hash_expr(e);
486                 self.hash_ty(ty);
487             },
488             ExprKind::Closure(cap, _, eid, _, _) => {
489                 match cap {
490                     CaptureBy::Value => 0,
491                     CaptureBy::Ref => 1,
492                 }
493                 .hash(&mut self.s);
494                 // closures inherit TypeckResults
495                 self.hash_expr(&self.cx.tcx.hir().body(eid).value);
496             },
497             ExprKind::Field(ref e, ref f) => {
498                 self.hash_expr(e);
499                 self.hash_name(f.name);
500             },
501             ExprKind::Index(ref a, ref i) => {
502                 self.hash_expr(a);
503                 self.hash_expr(i);
504             },
505             ExprKind::InlineAsm(ref asm) => {
506                 for piece in asm.template {
507                     match piece {
508                         InlineAsmTemplatePiece::String(s) => s.hash(&mut self.s),
509                         InlineAsmTemplatePiece::Placeholder {
510                             operand_idx,
511                             modifier,
512                             span: _,
513                         } => {
514                             operand_idx.hash(&mut self.s);
515                             modifier.hash(&mut self.s);
516                         },
517                     }
518                 }
519                 asm.options.hash(&mut self.s);
520                 for (op, _op_sp) in asm.operands {
521                     match op {
522                         InlineAsmOperand::In { reg, expr } => {
523                             reg.hash(&mut self.s);
524                             self.hash_expr(expr);
525                         },
526                         InlineAsmOperand::Out { reg, late, expr } => {
527                             reg.hash(&mut self.s);
528                             late.hash(&mut self.s);
529                             if let Some(expr) = expr {
530                                 self.hash_expr(expr);
531                             }
532                         },
533                         InlineAsmOperand::InOut { reg, late, expr } => {
534                             reg.hash(&mut self.s);
535                             late.hash(&mut self.s);
536                             self.hash_expr(expr);
537                         },
538                         InlineAsmOperand::SplitInOut {
539                             reg,
540                             late,
541                             in_expr,
542                             out_expr,
543                         } => {
544                             reg.hash(&mut self.s);
545                             late.hash(&mut self.s);
546                             self.hash_expr(in_expr);
547                             if let Some(out_expr) = out_expr {
548                                 self.hash_expr(out_expr);
549                             }
550                         },
551                         InlineAsmOperand::Const { expr } | InlineAsmOperand::Sym { expr } => self.hash_expr(expr),
552                     }
553                 }
554             },
555             ExprKind::LlvmInlineAsm(..) | ExprKind::Err => {},
556             ExprKind::Lit(ref l) => {
557                 l.node.hash(&mut self.s);
558             },
559             ExprKind::Loop(ref b, ref i, _) => {
560                 self.hash_block(b);
561                 if let Some(i) = *i {
562                     self.hash_name(i.ident.name);
563                 }
564             },
565             ExprKind::Match(ref e, arms, ref s) => {
566                 self.hash_expr(e);
567
568                 for arm in arms {
569                     // TODO: arm.pat?
570                     if let Some(ref e) = arm.guard {
571                         self.hash_guard(e);
572                     }
573                     self.hash_expr(&arm.body);
574                 }
575
576                 s.hash(&mut self.s);
577             },
578             ExprKind::MethodCall(ref path, ref _tys, args, ref _fn_span) => {
579                 self.hash_name(path.ident.name);
580                 self.hash_exprs(args);
581             },
582             ExprKind::ConstBlock(ref l_id) => {
583                 self.hash_body(l_id.body);
584             },
585             ExprKind::Repeat(ref e, ref l_id) => {
586                 self.hash_expr(e);
587                 self.hash_body(l_id.body);
588             },
589             ExprKind::Ret(ref e) => {
590                 if let Some(ref e) = *e {
591                     self.hash_expr(e);
592                 }
593             },
594             ExprKind::Path(ref qpath) => {
595                 self.hash_qpath(qpath);
596             },
597             ExprKind::Struct(ref path, fields, ref expr) => {
598                 self.hash_qpath(path);
599
600                 for f in fields {
601                     self.hash_name(f.ident.name);
602                     self.hash_expr(&f.expr);
603                 }
604
605                 if let Some(ref e) = *expr {
606                     self.hash_expr(e);
607                 }
608             },
609             ExprKind::Tup(tup) => {
610                 self.hash_exprs(tup);
611             },
612             ExprKind::Array(v) => {
613                 self.hash_exprs(v);
614             },
615             ExprKind::Unary(lop, ref le) => {
616                 lop.hash_stable(&mut self.cx.tcx.get_stable_hashing_context(), &mut self.s);
617                 self.hash_expr(le);
618             },
619         }
620     }
621
622     pub fn hash_exprs(&mut self, e: &[Expr<'_>]) {
623         for e in e {
624             self.hash_expr(e);
625         }
626     }
627
628     pub fn hash_name(&mut self, n: Symbol) {
629         n.as_str().hash(&mut self.s);
630     }
631
632     pub fn hash_qpath(&mut self, p: &QPath<'_>) {
633         match *p {
634             QPath::Resolved(_, ref path) => {
635                 self.hash_path(path);
636             },
637             QPath::TypeRelative(_, ref path) => {
638                 self.hash_name(path.ident.name);
639             },
640             QPath::LangItem(lang_item, ..) => {
641                 lang_item.hash_stable(&mut self.cx.tcx.get_stable_hashing_context(), &mut self.s);
642             },
643         }
644         // self.maybe_typeck_results.unwrap().qpath_res(p, id).hash(&mut self.s);
645     }
646
647     pub fn hash_path(&mut self, p: &Path<'_>) {
648         p.is_global().hash(&mut self.s);
649         for p in p.segments {
650             self.hash_name(p.ident.name);
651         }
652     }
653
654     pub fn hash_stmt(&mut self, b: &Stmt<'_>) {
655         std::mem::discriminant(&b.kind).hash(&mut self.s);
656
657         match &b.kind {
658             StmtKind::Local(local) => {
659                 if let Some(ref init) = local.init {
660                     self.hash_expr(init);
661                 }
662             },
663             StmtKind::Item(..) => {},
664             StmtKind::Expr(expr) | StmtKind::Semi(expr) => {
665                 self.hash_expr(expr);
666             },
667         }
668     }
669
670     pub fn hash_guard(&mut self, g: &Guard<'_>) {
671         match g {
672             Guard::If(ref expr) => {
673                 self.hash_expr(expr);
674             },
675         }
676     }
677
678     pub fn hash_lifetime(&mut self, lifetime: &Lifetime) {
679         std::mem::discriminant(&lifetime.name).hash(&mut self.s);
680         if let LifetimeName::Param(ref name) = lifetime.name {
681             std::mem::discriminant(name).hash(&mut self.s);
682             match name {
683                 ParamName::Plain(ref ident) => {
684                     ident.name.hash(&mut self.s);
685                 },
686                 ParamName::Fresh(ref size) => {
687                     size.hash(&mut self.s);
688                 },
689                 ParamName::Error => {},
690             }
691         }
692     }
693
694     pub fn hash_ty(&mut self, ty: &Ty<'_>) {
695         self.hash_tykind(&ty.kind);
696     }
697
698     pub fn hash_tykind(&mut self, ty: &TyKind<'_>) {
699         std::mem::discriminant(ty).hash(&mut self.s);
700         match ty {
701             TyKind::Slice(ty) => {
702                 self.hash_ty(ty);
703             },
704             TyKind::Array(ty, anon_const) => {
705                 self.hash_ty(ty);
706                 self.hash_body(anon_const.body);
707             },
708             TyKind::Ptr(mut_ty) => {
709                 self.hash_ty(&mut_ty.ty);
710                 mut_ty.mutbl.hash(&mut self.s);
711             },
712             TyKind::Rptr(lifetime, mut_ty) => {
713                 self.hash_lifetime(lifetime);
714                 self.hash_ty(&mut_ty.ty);
715                 mut_ty.mutbl.hash(&mut self.s);
716             },
717             TyKind::BareFn(bfn) => {
718                 bfn.unsafety.hash(&mut self.s);
719                 bfn.abi.hash(&mut self.s);
720                 for arg in bfn.decl.inputs {
721                     self.hash_ty(&arg);
722                 }
723                 match bfn.decl.output {
724                     FnRetTy::DefaultReturn(_) => {
725                         ().hash(&mut self.s);
726                     },
727                     FnRetTy::Return(ref ty) => {
728                         self.hash_ty(ty);
729                     },
730                 }
731                 bfn.decl.c_variadic.hash(&mut self.s);
732             },
733             TyKind::Tup(ty_list) => {
734                 for ty in *ty_list {
735                     self.hash_ty(ty);
736                 }
737             },
738             TyKind::Path(qpath) => match qpath {
739                 QPath::Resolved(ref maybe_ty, ref path) => {
740                     if let Some(ref ty) = maybe_ty {
741                         self.hash_ty(ty);
742                     }
743                     for segment in path.segments {
744                         segment.ident.name.hash(&mut self.s);
745                         self.hash_generic_args(segment.generic_args().args);
746                     }
747                 },
748                 QPath::TypeRelative(ref ty, ref segment) => {
749                     self.hash_ty(ty);
750                     segment.ident.name.hash(&mut self.s);
751                 },
752                 QPath::LangItem(lang_item, ..) => {
753                     lang_item.hash(&mut self.s);
754                 },
755             },
756             TyKind::OpaqueDef(_, arg_list) => {
757                 self.hash_generic_args(arg_list);
758             },
759             TyKind::TraitObject(_, lifetime) => {
760                 self.hash_lifetime(lifetime);
761             },
762             TyKind::Typeof(anon_const) => {
763                 self.hash_body(anon_const.body);
764             },
765             TyKind::Err | TyKind::Infer | TyKind::Never => {},
766         }
767     }
768
769     pub fn hash_body(&mut self, body_id: BodyId) {
770         // swap out TypeckResults when hashing a body
771         let old_maybe_typeck_results = self.maybe_typeck_results.replace(self.cx.tcx.typeck_body(body_id));
772         self.hash_expr(&self.cx.tcx.hir().body(body_id).value);
773         self.maybe_typeck_results = old_maybe_typeck_results;
774     }
775
776     fn hash_generic_args(&mut self, arg_list: &[GenericArg<'_>]) {
777         for arg in arg_list {
778             match arg {
779                 GenericArg::Lifetime(ref l) => self.hash_lifetime(l),
780                 GenericArg::Type(ref ty) => self.hash_ty(&ty),
781                 GenericArg::Const(ref ca) => self.hash_body(ca.value.body),
782             }
783         }
784     }
785 }