alexfh added inline comments.
================ Comment at: clang-tidy/readability/SimplifyBooleanExprCheck.cpp:517 + has(compoundStmt( + has(caseStmt(hasDescendant(ifStmt(hasThen(returnsBool(Value)), + unless(hasElse(stmt()))) ---------------- Please put the more expensive matcher (`hasDescendant` may be *really* expensive) last. Same below. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56303/new/ https://reviews.llvm.org/D56303 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
