[PATCH] D61552: [clang] Adapt ASTMatcher to explicit(bool) specifier
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jun 19 08:52:57 PDT 2019
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM aside from a documentation nit.
================
Comment at: clang/include/clang/ASTMatchers/ASTMatchers.h:6190
+/// cxxDeductionGuideDecl(isExplicit()) will match #6, but not #5.
+/// cxxConstructorDecl(isExplicit()) will match #8, but not #7 or #9.
+AST_POLYMORPHIC_MATCHER(isExplicit, AST_POLYMORPHIC_SUPPORTED_TYPES(
----------------
Tyker wrote:
> aaron.ballman wrote:
> > Why won't it match #2?
> #2 matches.
I figured it would -- the comment should be updated to correct that (and then generate the docs again).
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61552/new/
https://reviews.llvm.org/D61552
More information about the cfe-commits
mailing list