[clang-tools-extra] [include-cleaner] Report refs for enum constants used through namespace aliases (PR #106706)

Ilya Biryukov via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 30 07:04:47 PDT 2024


================
@@ -146,9 +148,24 @@ class ASTWalker : public RecursiveASTVisitor<ASTWalker> {
     //
     // If it's an enum constant, it must be due to prior decl. Report references
     // to it when qualifier isn't a type.
-    if (llvm::isa<EnumConstantDecl>(FD)) {
-      if (!DRE->getQualifier() || DRE->getQualifier()->getAsNamespace())
-        report(DRE->getLocation(), FD);
+    auto QualifierIsNamepsaceOrNone = [&DRE]() {
+      const auto *Qual = DRE->getQualifier();
+      if (!Qual)
+        return true;
+      switch (Qual->getKind()) {
+      case NestedNameSpecifier::Namespace:
+      case NestedNameSpecifier::NamespaceAlias:
+      case NestedNameSpecifier::Global:
+        return true;
+      case NestedNameSpecifier::TypeSpec:
+      case NestedNameSpecifier::TypeSpecWithTemplate:
+      case NestedNameSpecifier::Super:
+      case NestedNameSpecifier::Identifier:
+        return false;
+      }
----------------
ilya-biryukov wrote:

NIT: maybe add `llvm_unreachable` to make sure we get some error messages if someone adds a new kind of nested-specifier in the future?

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


More information about the cfe-commits mailing list