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
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
