steveire added a comment.
@nick Sorry that getting these changes merged takes so long.
@njames93 If you have an alternative way forward, please let us know what it is.
Otherwise, this LGTM too and we should merge it soon unless there are
objections which haven't been addressed.
================
Comment at: clang/unittests/ASTMatchers/Dynamic/RegistryTest.cpp:301
+TEST_F(RegistryTest, CXXBaseSpecifier) {
+ // TODO: rewrite with top-level cxxBaseSpecifier matcher when available
+ DeclarationMatcher ClassHasAnyDirectBase =
----------------
@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.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69218/new/
https://reviews.llvm.org/D69218
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
