[cfe-dev] AST matchers and const ASTContext

[Julian Bangert via cfe-dev]

Hello,

is there a particular reason why clang::ast_matchers::match (
http://clang.llvm.org/doxygen/classclang_1_1ast__matchers_1_1MatchFinder.html#a8b5a0b1b4d21c32009ad9e75ef8d5502)
requires a non-const reference to a clang::ASTContext, but the matchresuly
only has a const clang::ASTContext *field? This makes it hard to further
apply matchers on the match result.

At least conceptually, shouldn't the AST matcher not modify the AST it
operates on (and therefore be able to accept a const context)?
If  the AST matcher has state that some matchers might modify, should we
instead return a mutable ASTContext in matchresult?

Regards,
Julian
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20160902/cb7b8c1f/attachment.html>