[PATCH] D69218: [ASTMatchers] Add `cxxBaseSpecifier` matcher (non-top-level)
    Nikita Kniazev via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Sun Mar 28 11:34:37 PDT 2021
    
    
  
nick added inline comments.
================
Comment at: clang/unittests/ASTMatchers/Dynamic/RegistryTest.cpp:301
+TEST_F(RegistryTest, CXXBaseSpecifier) {
+  // TODO: rewrite with top-level cxxBaseSpecifier matcher when available
+  DeclarationMatcher ClassHasAnyDirectBase =
----------------
steveire wrote:
> @nick Is this implemented in another MR? I don't see anything in your list of revisions. I think this is reasonable as is, but wondering if you intend to implement the top-level support too.
The patch had some of the top-level matcher parts, but it cut them off when rebased. I have no need in it myself and I am not sure there is any kind of use for it.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69218/new/
https://reviews.llvm.org/D69218
    
    
More information about the cfe-commits
mailing list