[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