[PATCH] D20360: [ASTMatcher] Add a node matcher for UnresolvedLookupExpr.
Aaron Ballman via cfe-commits
cfe-commits at lists.llvm.org
Wed May 18 05:05:33 PDT 2016
aaron.ballman added a comment.
In http://reviews.llvm.org/D20360#432878, @alexfh wrote:
> In http://reviews.llvm.org/D20360#432874, @aaron.ballman wrote:
>
> > The changes to docs/tools/dump_ast_matchers.py look to be spurious, can they be reverted?
>
>
> The script should be executable, so the change looks fine to me.
World executable? That's a bit presumptuous. ;-) 0744 may be fine, but 0755 does not seem correct to me.
Regardless, I think that it should be a separate commit, not part of this one as a drive-by (it has security implications, and that deserves review).
http://reviews.llvm.org/D20360
More information about the cfe-commits
mailing list