[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?
More information about the cfe-commits