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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to