[PATCH] Add partial support for the hasDeclaration() matcher in the dynamic layer.

Samuel Benzaquen sbenza at google.com
Thu Nov 14 08:24:33 PST 2013



================
Comment at: include/clang/ASTMatchers/ASTMatchers.h:1700-1702
@@ +1699,5 @@
+    internal::HasDeclarationMatcher, internal::Matcher<Decl>,
+    AST_POLYMORPHIC_SUPPORTED_TYPES_6(QualType, CallExpr, CXXConstructExpr,
+                                      MemberExpr, TypedefType,
+                                      TemplateSpecializationType),
+    internal::CTC_Disabled>
----------------
Manuel Klimek wrote:
> So why do we specify supported types at all, if they're not checked?
This is the list used to instantiate the dynamic matchers.
We can't check it because it is not complete. ie it is missing the ones that go through the template implementation.
If I enable the check, it fails on those.


http://llvm-reviews.chandlerc.com/D1889

BRANCH
  has_decl

ARCANIST PROJECT
  clang



More information about the cfe-commits mailing list