[PATCH] D24862: Workaround ASTMatcher crashes. Added some more test cases.
Eric Liu via cfe-commits
cfe-commits at lists.llvm.org
Fri Sep 23 06:58:54 PDT 2016
ioeric added a comment.
In https://reviews.llvm.org/D24862#550615, @aaron.ballman wrote:
> Should this perhaps be fixed in the AST matchers rather than in the check itself?
I'll file bugs for the crashes, but the fix in this patch is not that "dirty" - it simply changes the order of matchers to workaround the crashes. So I guess this doesn't need to wait for the fix in matcher library.
https://reviews.llvm.org/D24862
More information about the cfe-commits
mailing list