etienneb marked 3 inline comments as done. etienneb added a comment. This patch has been modified to support dynamic matchers. It rely on : http://reviews.llvm.org/D20207
Comments? ================ Comment at: lib/ASTMatchers/Dynamic/Marshallers.h:102 @@ +101,3 @@ + static clang::CastKind getCastKind(llvm::StringRef AttrKind) { + return llvm::StringSwitch<clang::CastKind>(AttrKind) +#define CAST_OPERATION(Name) .Case( #Name, CK_##Name) ---------------- moved to that solution. (let try it) ================ Comment at: lib/ASTMatchers/Dynamic/Registry.cpp:78 @@ -77,2 +77,3 @@ // Need Variant/Parser fixes: + // hasCastKind // ofKind ---------------- sbenza wrote: > Is it registered or not? > You add this comment, but also add the matcher in the registry below. bad revert. I'm making a patch to allow dynamic matcher, and I'll come back fixing this CL. http://reviews.llvm.org/D19871 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits