[PATCH] D147411: [clang-tidy] Fix readability-static-accessed-through-instance check for anonymous structs

André Schackier via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Apr 2 04:11:02 PDT 2023


AMS21 added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/readability/StaticAccessedThroughInstanceCheck.cpp:78
+  // Ignore anonymous structs/classes
+  if (StringRef(BaseTypeName).contains("(unnamed "))
+    return;
----------------
I wonder if there's a cleaner and more performant way to check this.
I've tried `isAnonymousStructOrUnion()` from the `RecordDecl` but that seems to be about something else.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D147411/new/

https://reviews.llvm.org/D147411



More information about the cfe-commits mailing list