capitan-davide wrote:

> Overall looks good. Did you try running it on some codebases to find any 
> FP/incorrect fixes? A good idea would be to run at least on LLVM (like 
> `clang` or `clang-tools-extra`):
> 
> ```
> python3 clang-tools-extra/clang-tidy/tool/run-clang-tidy.py 
> -checks='-*,readability-inconsistent-ifelse-braces' -clang-tidy-binary 
> build/bin/clang-tidy -quiet -hide-progress -p build/ -fix clang-tools-extra/
> ```

`clang-tools-extra` generates quite a number of matches. I tested the fixes and 
they look correct, all tests are passing.

[fixes.patch](https://github.com/user-attachments/files/24327545/fixes.patch)


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

Reply via email to