[PATCH] D47122: [clang-tidy] SimplifyBoolenExpr doesn't add parens if unary negotiation is of ExprWithCleanups type
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon May 21 07:46:34 PDT 2018
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
More information about the cfe-commits
mailing list