[PATCH] D133102: [clang-tidy] Extend simplify-boolean-expr check
Alexander Kornienko via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Sep 7 04:12:08 PDT 2022
alexfh requested changes to this revision.
alexfh added a comment.
This revision now requires changes to proceed.
FWIW, I'm not sure this will usually be a simplification of the code. Splitting the condition and using early return is quite a frequent recommendation in code reviews, and I would expect a strong pushback to automated suggestions going in the opposite direction.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D133102/new/
https://reviews.llvm.org/D133102
More information about the cfe-commits
mailing list