sammccall added a comment. Yeah, this is a thorny question. The language hasn't chosen clear names and neither has the AST in many cases. (Just yesterday I lost a lot of time to CXXRecordDecl->getBodyRBrace()...)
I think `anyRedeclaration(functionDecl(isStaticStorageClass))`is too difficult to get right and too easy to get wrong for such a common/simple concept. I know what a static method is, but I would have made the same mistake here. In general asking people to describe syntax to answer a semantic question invites this. Here's a hack: CXXMethodDecl has both `isStatic()` and `isInstance()`. I agree that `isStatic` is a risky name, but I don't think `isInstance` is. Can we just spell the matcher you want `not(isInstance())`? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115106/new/ https://reviews.llvm.org/D115106 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
