[PATCH] D69218: [clang][AST] Add `CXXBaseSpecifier` matcher support

2021-03-17 Thread Stephen Kelly via Phabricator via cfe-commits
steveire added reviewers: steveire, njames93. steveire added a comment. Could you rebase this? Parts of it were common with another patch which was merged. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69218/new/ https://reviews.llvm.org/D69218 _

[PATCH] D69218: [clang][AST] Add `CXXBaseSpecifier` matcher support

2019-10-20 Thread Nikita Kniazev via Phabricator via cfe-commits
nick updated this revision to Diff 225783. nick added a comment. Fixed typo. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69218/new/ https://reviews.llvm.org/D69218 Files: clang/docs/LibASTMatchersReference.html clang/include/clang/AST/ASTTyp

[PATCH] D69218: [clang][AST] Add `CXXBaseSpecifier` matcher support

2019-10-19 Thread Nikita Kniazev via Phabricator via cfe-commits
nick updated this revision to Diff 225770. nick added a comment. Regenerated the docs, added more tests. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69218/new/ https://reviews.llvm.org/D69218 Files: clang/docs/LibASTMatchersReference.html cl

[PATCH] D69218: [clang][AST] Add `CXXBaseSpecifier` matcher support

2019-10-19 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri added a comment. Needs more tests - what happens with multiple inheritance, what about inheriting from inherited? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69218/new/ https://reviews.llvm.org/D69218 __

[PATCH] D69218: [clang][AST] Add `CXXBaseSpecifier` matcher support

2019-10-19 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri added a comment. Missing docs regeneration (`clang/docs/tools/dump_ast_matchers.py` i think) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69218/new/ https://reviews.llvm.org/D69218 ___ cfe-

[PATCH] D69218: [clang][AST] Add `CXXBaseSpecifier` matcher support

2019-10-19 Thread Nikita Kniazev via Phabricator via cfe-commits
nick created this revision. nick added a reviewer: klimek. nick added a project: clang. Herald added a subscriber: cfe-commits. Required for capturing base specifier in matchers: `cxxRecordDecl(hasBase(cxxBaseSpecifier().bind("base")))` Will make implementation of D69000