[PATCH] D24862: Workaround ASTMatcher crashes. Added some more test cases.

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 23 07:19:48 PDT 2016


aaron.ballman added a comment.

In https://reviews.llvm.org/D24862#550646, @ioeric wrote:

> Acked, and I totally agree with you :) It's just that the change in this patch would still be valid after the underlying bugs are fixed, so I thought it was fine to fix those bugs after this.


I might be confused then (this happens from time to time) -- are the changes in this patch still necessary after the underlying bugs are fixed?


https://reviews.llvm.org/D24862





More information about the cfe-commits mailing list