[PATCH] ASTMatcher for Attr

Jacques Pienaar jpienaar at google.com
Thu Aug 21 08:50:42 PDT 2014


================
Comment at: include/clang/ASTMatchers/ASTMatchers.h:3669-3672
@@ +3668,6 @@
+/// f.
+template <typename Attr>
+inline internal::Matcher<Decl> hasAttr() {
+  return internal::HasAttrMatcher<Decl, Attr>();
+}
+
----------------
Manuel Klimek wrote:
> If we have this matcher, do the specialized ones still provide enough value? I was expecting us to either go for the specialized ones, or provide a generic one. Having both in the interface seems odd...
Agreed. I'll remove the specialized ones.

http://reviews.llvm.org/D4996






More information about the cfe-commits mailing list