]>
Commit | Line | Data |
---|---|---|
c295e0f8 | 1 | use rustc_data_structures::sync::Lock; |
dfeec247 | 2 | use rustc_hir as hir; |
487cf647 | 3 | use rustc_hir::def_id::LocalDefId; |
dfeec247 | 4 | use rustc_hir::intravisit; |
dfeec247 | 5 | use rustc_hir::{HirId, ItemLocalId}; |
064997fb | 6 | use rustc_index::bit_set::GrowableBitSet; |
5099ac24 | 7 | use rustc_middle::hir::nested_filter; |
353b0b11 | 8 | use rustc_middle::ty::TyCtxt; |
cc61c64b | 9 | |
ba9703b0 | 10 | pub fn check_crate(tcx: TyCtxt<'_>) { |
064997fb | 11 | if tcx.sess.opts.unstable_opts.hir_stats { |
c295e0f8 XL |
12 | crate::hir_stats::print_hir_stats(tcx); |
13 | } | |
14 | ||
064997fb FG |
15 | #[cfg(debug_assertions)] |
16 | { | |
17 | let errors = Lock::new(Vec::new()); | |
0731742a | 18 | |
487cf647 | 19 | tcx.hir().par_for_each_module(|module_id| { |
064997fb | 20 | let mut v = HirIdValidator { |
487cf647 | 21 | tcx, |
064997fb FG |
22 | owner: None, |
23 | hir_ids_seen: Default::default(), | |
24 | errors: &errors, | |
25 | }; | |
923072b8 | 26 | |
064997fb FG |
27 | tcx.hir().visit_item_likes_in_module(module_id, &mut v); |
28 | }); | |
0731742a | 29 | |
064997fb | 30 | let errors = errors.into_inner(); |
0731742a | 31 | |
064997fb FG |
32 | if !errors.is_empty() { |
33 | let message = errors.iter().fold(String::new(), |s1, s2| s1 + "\n" + s2); | |
34 | tcx.sess.delay_span_bug(rustc_span::DUMMY_SP, &message); | |
35 | } | |
2c00a5a8 | 36 | } |
cc61c64b XL |
37 | } |
38 | ||
dc9dc135 | 39 | struct HirIdValidator<'a, 'hir> { |
487cf647 | 40 | tcx: TyCtxt<'hir>, |
2b03887a | 41 | owner: Option<hir::OwnerId>, |
064997fb | 42 | hir_ids_seen: GrowableBitSet<ItemLocalId>, |
0731742a | 43 | errors: &'a Lock<Vec<String>>, |
cc61c64b XL |
44 | } |
45 | ||
923072b8 | 46 | impl<'a, 'hir> HirIdValidator<'a, 'hir> { |
487cf647 FG |
47 | fn new_visitor(&self, tcx: TyCtxt<'hir>) -> HirIdValidator<'a, 'hir> { |
48 | HirIdValidator { tcx, owner: None, hir_ids_seen: Default::default(), errors: self.errors } | |
cc61c64b | 49 | } |
cc61c64b | 50 | |
0731742a XL |
51 | #[cold] |
52 | #[inline(never)] | |
53 | fn error(&self, f: impl FnOnce() -> String) { | |
54 | self.errors.lock().push(f()); | |
55 | } | |
cc61c64b | 56 | |
2b03887a | 57 | fn check<F: FnOnce(&mut HirIdValidator<'a, 'hir>)>(&mut self, owner: hir::OwnerId, walk: F) { |
ba9703b0 | 58 | assert!(self.owner.is_none()); |
ba9703b0 | 59 | self.owner = Some(owner); |
cc61c64b XL |
60 | walk(self); |
61 | ||
2b03887a | 62 | if owner == hir::CRATE_OWNER_ID { |
13cf67c4 | 63 | return; |
cc61c64b XL |
64 | } |
65 | ||
66 | // There's always at least one entry for the owning item itself | |
60c5eb7d XL |
67 | let max = self |
68 | .hir_ids_seen | |
69 | .iter() | |
70 | .map(|local_id| local_id.as_usize()) | |
71 | .max() | |
72 | .expect("owning item has no entry"); | |
cc61c64b XL |
73 | |
74 | if max != self.hir_ids_seen.len() - 1 { | |
9ffffee4 FG |
75 | let hir = self.tcx.hir(); |
76 | let pretty_owner = hir.def_path(owner.def_id).to_string_no_crate_verbose(); | |
cc61c64b | 77 | |
9ffffee4 FG |
78 | let missing_items: Vec<_> = (0..=max as u32) |
79 | .map(|i| ItemLocalId::from_u32(i)) | |
80 | .filter(|&local_id| !self.hir_ids_seen.contains(local_id)) | |
81 | .map(|local_id| hir.node_to_string(HirId { owner, local_id })) | |
82 | .collect(); | |
cc61c64b | 83 | |
9ffffee4 FG |
84 | let seen_items: Vec<_> = self |
85 | .hir_ids_seen | |
86 | .iter() | |
87 | .map(|local_id| hir.node_to_string(HirId { owner, local_id })) | |
88 | .collect(); | |
94b46f34 | 89 | |
60c5eb7d XL |
90 | self.error(|| { |
91 | format!( | |
92 | "ItemLocalIds not assigned densely in {}. \ | |
9ffffee4 FG |
93 | Max ItemLocalId = {}, missing IDs = {:#?}; seen IDs = {:#?}", |
94 | pretty_owner, max, missing_items, seen_items | |
60c5eb7d XL |
95 | ) |
96 | }); | |
cc61c64b XL |
97 | } |
98 | } | |
487cf647 FG |
99 | |
100 | fn check_nested_id(&mut self, id: LocalDefId) { | |
101 | let Some(owner) = self.owner else { return }; | |
102 | let def_parent = self.tcx.local_parent(id); | |
103 | let def_parent_hir_id = self.tcx.local_def_id_to_hir_id(def_parent); | |
104 | if def_parent_hir_id.owner != owner { | |
105 | self.error(|| { | |
106 | format!( | |
107 | "inconsistent Def parent at `{:?}` for `{:?}`:\nexpected={:?}\nfound={:?}", | |
108 | self.tcx.def_span(id), | |
109 | id, | |
110 | owner, | |
111 | def_parent_hir_id | |
112 | ) | |
113 | }); | |
114 | } | |
115 | } | |
cc61c64b XL |
116 | } |
117 | ||
dc9dc135 | 118 | impl<'a, 'hir> intravisit::Visitor<'hir> for HirIdValidator<'a, 'hir> { |
5099ac24 | 119 | type NestedFilter = nested_filter::OnlyBodies; |
dfeec247 | 120 | |
5099ac24 | 121 | fn nested_visit_map(&mut self) -> Self::Map { |
487cf647 FG |
122 | self.tcx.hir() |
123 | } | |
124 | ||
125 | fn visit_nested_item(&mut self, id: hir::ItemId) { | |
126 | self.check_nested_id(id.owner_id.def_id); | |
127 | } | |
128 | ||
129 | fn visit_nested_trait_item(&mut self, id: hir::TraitItemId) { | |
130 | self.check_nested_id(id.owner_id.def_id); | |
131 | } | |
132 | ||
133 | fn visit_nested_impl_item(&mut self, id: hir::ImplItemId) { | |
134 | self.check_nested_id(id.owner_id.def_id); | |
135 | } | |
136 | ||
137 | fn visit_nested_foreign_item(&mut self, id: hir::ForeignItemId) { | |
138 | self.check_nested_id(id.owner_id.def_id); | |
cc61c64b XL |
139 | } |
140 | ||
923072b8 | 141 | fn visit_item(&mut self, i: &'hir hir::Item<'hir>) { |
487cf647 | 142 | let mut inner_visitor = self.new_visitor(self.tcx); |
2b03887a | 143 | inner_visitor.check(i.owner_id, |this| intravisit::walk_item(this, i)); |
923072b8 FG |
144 | } |
145 | ||
9fa01778 | 146 | fn visit_id(&mut self, hir_id: HirId) { |
ba9703b0 | 147 | let owner = self.owner.expect("no owner"); |
cc61c64b | 148 | |
9fa01778 | 149 | if owner != hir_id.owner { |
60c5eb7d XL |
150 | self.error(|| { |
151 | format!( | |
152 | "HirIdValidator: The recorded owner of {} is {} instead of {}", | |
487cf647 FG |
153 | self.tcx.hir().node_to_string(hir_id), |
154 | self.tcx.hir().def_path(hir_id.owner.def_id).to_string_no_crate_verbose(), | |
155 | self.tcx.hir().def_path(owner.def_id).to_string_no_crate_verbose() | |
60c5eb7d XL |
156 | ) |
157 | }); | |
cc61c64b XL |
158 | } |
159 | ||
9fa01778 | 160 | self.hir_ids_seen.insert(hir_id.local_id); |
cc61c64b XL |
161 | } |
162 | ||
923072b8 | 163 | fn visit_foreign_item(&mut self, i: &'hir hir::ForeignItem<'hir>) { |
487cf647 | 164 | let mut inner_visitor = self.new_visitor(self.tcx); |
2b03887a | 165 | inner_visitor.check(i.owner_id, |this| intravisit::walk_foreign_item(this, i)); |
923072b8 FG |
166 | } |
167 | ||
168 | fn visit_trait_item(&mut self, i: &'hir hir::TraitItem<'hir>) { | |
487cf647 | 169 | let mut inner_visitor = self.new_visitor(self.tcx); |
2b03887a | 170 | inner_visitor.check(i.owner_id, |this| intravisit::walk_trait_item(this, i)); |
cc61c64b | 171 | } |
29967ef6 | 172 | |
923072b8 | 173 | fn visit_impl_item(&mut self, i: &'hir hir::ImplItem<'hir>) { |
487cf647 | 174 | let mut inner_visitor = self.new_visitor(self.tcx); |
2b03887a | 175 | inner_visitor.check(i.owner_id, |this| intravisit::walk_impl_item(this, i)); |
fc512014 | 176 | } |
cc61c64b | 177 | } |