aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
LGTM, with a small nit. ================ Comment at: clang-tidy/readability/SimplifyBooleanExprCheck.cpp:78 E = E->IgnoreImpCasts(); + if (auto *EC = dyn_cast<ExprWithCleanups>(E)) + E = EC->getSubExpr(); ---------------- `const auto *`, please. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D47122 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits