malcolm.parsons added a comment. In https://reviews.llvm.org/D32942#780090, @lebedev.ri wrote:
> In https://reviews.llvm.org/D32942#780053, @malcolm.parsons wrote: > > > My prototype of this feature used `ifStmt(stmt().bind("if"), > > unless(hasParent(ifStmt(hasElse(equalsBoundNode("if"))))))`. > > > The fix should ideally be fully contained in the `TraverseStmt()`, and should > not be noticeably slower. Yes, my prototype wasn't part of readability-function-size. Repository: rL LLVM https://reviews.llvm.org/D32942 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits