[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