[PATCH] D133102: [clang-tidy] Extend simplify-boolean-expr check
Richard via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 29 08:22:52 PDT 2023
LegalizeAdulthood added a comment.
Herald added a subscriber: PiotrZSL.
In D133102#3774151 <https://reviews.llvm.org/D133102#3774151>, @alexfh wrote:
> For example, LLVM coding standards recommends using early exits: https://llvm.org/docs/CodingStandards.html#use-early-exits-and-continue-to-simplify-code
How is the proposed replacement not an early exit? It's an unconditional exit: it either returns a bool constant or it returns a variable that is a bool.
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