[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