[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