[clang-tools-extra] [clang-tidy] Fix readability-simplify-boolean-expr for init statements (PR #172220)
via cfe-commits
cfe-commits at lists.llvm.org
Mon Dec 15 03:56:12 PST 2025
Anshul200677 wrote:
@zeyi2 , @zwuis Thanks for the feedback!!
I have addressed all the comments and the CI checks are now passing. Here is a summary of the changes:
1. LOGIC FIX: I updated the check to preserve initialization statements (e.g., `if (int x = 0; ...)` ) by wrapping the replacement in a new scope `{ ... }`.
2. TEST UPDATE: I removed the invalid test case and verified the behavior locally with "llvm-lit".
3. STYLE: I fixed the typo mentioned and resolved all `clang-format` style issues.
Ready for review!
https://github.com/llvm/llvm-project/pull/172220
More information about the cfe-commits
mailing list