]> git.proxmox.com Git - rustc.git/blame - compiler/rustc_passes/src/hir_id_validator.rs
New upstream version 1.70.0+dfsg1
[rustc.git] / compiler / rustc_passes / src / hir_id_validator.rs
CommitLineData
c295e0f8 1use rustc_data_structures::sync::Lock;
dfeec247 2use rustc_hir as hir;
487cf647 3use rustc_hir::def_id::LocalDefId;
dfeec247 4use rustc_hir::intravisit;
dfeec247 5use rustc_hir::{HirId, ItemLocalId};
064997fb 6use rustc_index::bit_set::GrowableBitSet;
5099ac24 7use rustc_middle::hir::nested_filter;
353b0b11 8use rustc_middle::ty::TyCtxt;
cc61c64b 9
ba9703b0 10pub 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 39struct 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 46impl<'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 118impl<'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}