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

Reply via email to