[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