alexfh added a comment.

In http://reviews.llvm.org/D19871#419947, @aaron.ballman wrote:
> Is this required for some purpose?
>
> I'm not keen on adding new AST matchers that we cannot expose via the dynamic 
> API, so I would prefer to solve that problem if we want to add this matcher.


I agree that when possible, matchers should be available via the dynamic 
matchers API. It doesn't seem overly complicated to add this support here. As 
far as I understand, we just need to register the matcher in 
lib/ASTMatchers/Dynamic/Registry.cpp and add support for `CastKind` argument 
type (by adding a corresponding `ArgTypeTraits` instantiation). Etienne, can 
you try this?


http://reviews.llvm.org/D19871



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

Reply via email to