expand: Add error check if derive has wrong item

The DeriveVisitor previously dispatched calls to specific handlers (like
DeriveClone) without validating the input item type. This caused internal
compiler errors (ICEs) when users attempted to derive traits on invalid items
like functions or traits, as the handlers assumed the input was a struct,
enum, or union.

This patch adds a validation predicate `validate_item` to the DeriveVisitor
to reject invalid items gracefully with a diagnostic error before dispatching.

Fixes #3875

gcc/rust/ChangeLog:

	* expand/rust-derive.h (DeriveVisitor::validate_item): Add prototype.
	* expand/rust-derive.cc (DeriveVisitor::derive): Use visitor validation.
	(DeriveVisitor::validate_item): New function to check item kind.

gcc/testsuite/ChangeLog:

	* rust/compile/issue-3875.rs: New test.

Signed-off-by: Jayant Chauhan <0001jayant@gmail.com>

Author: Pasta-coder

gcc/rust/expand/rust-derive.cc

@@ -40,14 +40,9 @@ DeriveVisitor::derive (Item &item, const Attribute &attr,
 {
   auto loc = attr.get_locus ();
 
-  using Kind = AST::Item::Kind;
-  auto item_kind = item.get_item_kind ();
-  if (item_kind != Kind::Enum && item_kind != Kind::Struct
-      && item_kind != Kind::Union)
+  if (!DeriveVisitor::validate_item (item, attr))
     {
-      rust_error_at (loc,
-		     "derive may only be applied to structs, enums and unions");
-      return {};
+      return std::vector<std::unique_ptr<Item>> ();
     }
 
   switch (to_derive)
@@ -159,5 +154,20 @@ DeriveVisitor::setup_impl_generics (
   return ImplGenerics{std::move (self_type_path), std::move (impl_generics)};
 }
 
+bool
+DeriveVisitor::validate_item (const Item &item, const Attribute &attr)
+{
+  using Kind = AST::Item::Kind;
+  auto item_kind = item.get_item_kind ();
+
+  if (item_kind == Kind::Enum || item_kind == Kind::Struct
+      || item_kind == Kind::Union || item_kind == Kind::TupleStruct)
+    return true;
+
+  rust_error_at (attr.get_locus (),
+		 "derive may only be applied to structs, enums and unions");
+  return false;
+}
+
 } // namespace AST
 } // namespace Rust

gcc/rust/expand/rust-derive.h

@@ -71,6 +71,7 @@ class DeriveVisitor : public AST::ASTVisitor
     = tl::nullopt) const;
 
 private:
+  static bool validate_item (const AST::Item &item, const Attribute &attr);
   // the 4 "allowed" visitors, which a derive-visitor can specify and override
   virtual void visit_struct (StructStruct &struct_item) = 0;
   virtual void visit_tuple (TupleStruct &tuple_item) = 0;

gcc/testsuite/rust/compile/issue-3875.rs

@@ -0,0 +1,12 @@
+// { dg-options "-w" }
+
+#[derive(Clone)] // { dg-error "derive may only be applied to structs, enums and unions" }
+fn foo() {}      // { dg-excess-errors "" }
+
+#[derive(Debug)] // { dg-error "derive may only be applied to structs, enums and unions" }
+trait Bar {}     // { dg-excess-errors "" }
+
+#[derive(Copy, Clone)]
+struct TupleStruct(i32); // This should NOT error
+
+fn main() {}