Anshul200677 wrote:

@zeyi2 @vbvictor I have addressed all the feedback:

Implementation: Removed the // Fix: comments and fixed the clang-format 
violations.

Formatting: The tool now generates valid C++ with proper spacing and semicolons 
({ int i = 0; foo(i); };).

Tests: I restored simplify-boolean-expr-cxx17.cpp and moved the C++17 specific 
tests there. I also updated the crash-test cases to avoid triggering unrelated 
warnings.

All tests are passing locally. Thanks for the guidance!


https://github.com/llvm/llvm-project/pull/172220
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to