[PATCH] D36308: Add handling for DeducedType to HasDeclarationMatcher

Manuel Klimek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 4 06:45:34 PDT 2017


klimek accepted this revision.
klimek added a comment.
This revision is now accepted and ready to land.

Thanks! LG after comment change.

Also, should we add some tests to clang-tidy? :)



================
Comment at: include/clang/ASTMatchers/ASTMatchersInternal.h:745
+
+    // For deduced types, use the deduced type
+    const Type *EffectiveType = &Node;
----------------
I think the behavior doesn't need a comment, but the reason :)

  // DeducedType dos not have declarations of its own, so
  // match the deduced type instead.


https://reviews.llvm.org/D36308





More information about the cfe-commits mailing list