[clang-tools-extra] [clang-tidy] Simplify conjunction with comparison in readability-simplify-boolean-expr (PR #233080)

via cfe-commits cfe-commits at lists.llvm.org
Sat Oct 17 20:44:04 PDT 2026


github-actions[bot] wrote:

<!--LLVM CODE LINT COMMENT: clang-tidy-->


:warning: C/C++ code linter, clang-tidy found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

```bash

git diff -U0 origin/main...HEAD -- clang-tools-extra/clang-tidy/readability/SimplifyBooleanExprCheck.cpp clang-tools-extra/clang-tidy/readability/SimplifyBooleanExprCheck.h |
python3 clang-tools-extra/clang-tidy/tool/clang-tidy-diff.py   -path build -p1 -quiet
```

</details>

<details>
<summary>
View the output from clang-tidy here.
</summary>

```
clang-tools-extra/clang-tidy/readability/SimplifyBooleanExprCheck.cpp:697:3: warning: nested 'if' statements can be merged together [readability-redundant-nested-if]
  697 |   if (const auto *UnOp = dyn_cast<UnaryOperator>(E)) {
      |   ^   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~  ~
      |       const auto *UnOp = dyn_cast<UnaryOperator>(E); UnOp && (UnOp->getOpcode() == UO_LNot)
  698 |     if (UnOp->getOpcode() == UO_LNot)
      |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  699 |       return {UnOp->getSubExpr()->IgnoreParenImpCasts(), true};
  700 |   }
      |   ~
clang-tools-extra/clang-tidy/readability/SimplifyBooleanExprCheck.cpp:698:5: note: nested 'if' statement to merge declared here
  698 |     if (UnOp->getOpcode() == UO_LNot)
      |     ^
```

</details>


https://github.com/llvm/llvm-project/pull/233080


More information about the cfe-commits mailing list