Detect pub structs never constructed even though they impl pub trait with assoc constants

Author: mu001999

compiler/rustc_passes/src/dead.rs

@@ -472,7 +472,7 @@ impl<'tcx> MarkSymbolVisitor<'tcx> {
  && let ItemKind::Impl(impl_ref) =
  self.tcx.hir().expect_item(local_impl_id).kind
  {
- if matches!(trait_item.kind, hir::TraitItemKind::Fn(..))
+ if !matches!(trait_item.kind, hir::TraitItemKind::Type(..))
  && !ty_ref_to_pub_struct(self.tcx, impl_ref.self_ty)
  .ty_and_all_fields_are_public
  {
@@ -802,7 +802,7 @@ fn check_item<'tcx>(
  // And we access the Map here to get HirId from LocalDefId
  for local_def_id in local_def_ids {
  // check the function may construct Self
- let mut may_construct_self = true;
+ let mut may_construct_self = false;
  if let Some(fn_sig) =
  tcx.hir().fn_sig_by_hir_id(tcx.local_def_id_to_hir_id(local_def_id))
  {

tests/ui/lint/dead-code/unused-adt-impl-pub-trait-with-assoc-const.rs

@@ -0,0 +1,32 @@
+#![deny(dead_code)]
+
+struct T1; //~ ERROR struct `T1` is never constructed
+pub struct T2(i32); //~ ERROR struct `T2` is never constructed
+
+trait Trait1 { //~ ERROR trait `Trait1` is never used
+ const UNUSED: i32;
+ fn unused(&self) {}
+}
+
+pub trait Trait2 {
+ const MAY_USED: i32;
+ fn may_used(&self) {}
+}
+
+impl Trait1 for T1 {
+ const UNUSED: i32 = 0;
+}
+
+impl Trait1 for T2 {
+ const UNUSED: i32 = 0;
+}
+
+impl Trait2 for T1 {
+ const MAY_USED: i32 = 0;
+}
+
+impl Trait2 for T2 {
+ const MAY_USED: i32 = 0;
+}
+
+fn main() {}

tests/ui/lint/dead-code/unused-adt-impl-pub-trait-with-assoc-const.stderr

@@ -0,0 +1,26 @@
+error: struct `T1` is never constructed
+ --> $DIR/unused-adt-impl-pub-trait-with-assoc-const.rs:3:8
+ |
+LL | struct T1;
+ | ^^
+ |
+note: the lint level is defined here
+ --> $DIR/unused-adt-impl-pub-trait-with-assoc-const.rs:1:9
+ |
+LL | #![deny(dead_code)]
+ | ^^^^^^^^^
+
+error: struct `T2` is never constructed
+ --> $DIR/unused-adt-impl-pub-trait-with-assoc-const.rs:4:12
+ |
+LL | pub struct T2(i32);
+ | ^^
+
+error: trait `Trait1` is never used
+ --> $DIR/unused-adt-impl-pub-trait-with-assoc-const.rs:6:7
+ |
+LL | trait Trait1 {
+ | ^^^^^^
+
+error: aborting due to 3 previous errors
+