[clang] [Clang] handle [[warn_unused]] attribute for unused private fields (PR #120734)

Erich Keane via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 20 19:21:30 PST 2024


================
@@ -3307,6 +3307,29 @@ void Sema::CheckShadowInheritedFields(const SourceLocation &Loc,
   }
 }
 
+template <typename AttrType>
+inline static bool HasAttribute(const QualType &T) {
+  if (const TagDecl *TD = T->getAsTagDecl())
+    return TD->hasAttr<AttrType>();
+  if (const TypedefType *TDT = T->getAs<TypedefType>())
+    return TDT->getDecl()->hasAttr<AttrType>();
+  return false;
+}
+
+inline static bool IsUnusedPrivateField(FieldDecl *FD) {
----------------
erichkeane wrote:

I again am not sure I get what you're suggesting?  I'm sorry, I don't get the proposal.  Something like `IsPotentiallyUnusedPrivateField` or something might work, but the fact that it is not actually checking if it is used is problematic.

https://github.com/llvm/llvm-project/pull/120734


More information about the cfe-commits mailing list