1 use rustc_errors::struct_span_err;
3 use rustc_hir::def_id::{CrateNum, DefId, LOCAL_CRATE};
4 use rustc_hir::itemlikevisit::ItemLikeVisitor;
5 use rustc_middle::ty::TyCtxt;
6 use rustc_trait_selection::traits::{self, SkipLeakCheck};
8 pub fn crate_inherent_impls_overlap_check(tcx: TyCtxt<'_>, crate_num: CrateNum) {
9 assert_eq!(crate_num, LOCAL_CRATE);
10 let krate = tcx.hir().krate();
11 krate.visit_all_item_likes(&mut InherentOverlapChecker { tcx });
14 struct InherentOverlapChecker<'tcx> {
18 impl InherentOverlapChecker<'tcx> {
19 /// Checks whether any associated items in impls 1 and 2 share the same identifier and
21 fn impls_have_common_items(&self, impl1: DefId, impl2: DefId) -> bool {
22 let impl_items1 = self.tcx.associated_items(impl1);
23 let impl_items2 = self.tcx.associated_items(impl2);
25 for item1 in impl_items1.in_definition_order() {
26 let collision = impl_items2.filter_by_name_unhygienic(item1.ident.name).any(|item2| {
27 // Symbols and namespace match, compare hygienically.
28 item1.kind.namespace() == item2.kind.namespace()
29 && item1.ident.normalize_to_macros_2_0()
30 == item2.ident.normalize_to_macros_2_0()
41 fn check_for_common_items_in_impls(
45 overlap: traits::OverlapResult<'_>,
47 let impl_items1 = self.tcx.associated_items(impl1);
48 let impl_items2 = self.tcx.associated_items(impl2);
50 for item1 in impl_items1.in_definition_order() {
51 let collision = impl_items2.filter_by_name_unhygienic(item1.ident.name).find(|item2| {
52 // Symbols and namespace match, compare hygienically.
53 item1.kind.namespace() == item2.kind.namespace()
54 && item1.ident.normalize_to_macros_2_0()
55 == item2.ident.normalize_to_macros_2_0()
58 if let Some(item2) = collision {
59 let name = item1.ident.normalize_to_macros_2_0();
60 let mut err = struct_span_err!(
62 self.tcx.span_of_impl(item1.def_id).unwrap(),
64 "duplicate definitions with name `{}`",
68 self.tcx.span_of_impl(item1.def_id).unwrap(),
69 format!("duplicate definitions for `{}`", name),
72 self.tcx.span_of_impl(item2.def_id).unwrap(),
73 format!("other definition for `{}`", name),
76 for cause in &overlap.intercrate_ambiguity_causes {
77 cause.add_intercrate_ambiguity_hint(&mut err);
80 if overlap.involves_placeholder {
81 traits::add_placeholder_note(&mut err);
89 fn check_for_overlapping_inherent_impls(&self, impl1_def_id: DefId, impl2_def_id: DefId) {
90 traits::overlapping_impls(
94 // We go ahead and just skip the leak check for
95 // inherent impls without warning.
98 self.check_for_common_items_in_impls(impl1_def_id, impl2_def_id, overlap);
106 impl ItemLikeVisitor<'v> for InherentOverlapChecker<'tcx> {
107 fn visit_item(&mut self, item: &'v hir::Item<'v>) {
109 hir::ItemKind::Enum(..)
110 | hir::ItemKind::Struct(..)
111 | hir::ItemKind::Trait(..)
112 | hir::ItemKind::Union(..) => {
113 let ty_def_id = self.tcx.hir().local_def_id(item.hir_id);
114 let impls = self.tcx.inherent_impls(ty_def_id);
116 for (i, &impl1_def_id) in impls.iter().enumerate() {
117 for &impl2_def_id in &impls[(i + 1)..] {
118 if self.impls_have_common_items(impl1_def_id, impl2_def_id) {
119 self.check_for_overlapping_inherent_impls(impl1_def_id, impl2_def_id);
128 fn visit_trait_item(&mut self, _trait_item: &hir::TraitItem<'v>) {}
130 fn visit_impl_item(&mut self, _impl_item: &hir::ImplItem<'v>) {}