[clang] [git-clang-format] Fix: make the tool backward compatible (PR #108721)

2024-09-15 Thread Eden Reich via cfe-commits
https://github.com/edenreich closed https://github.com/llvm/llvm-project/pull/108721 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [git-clang-format] Fix: make the tool backward compatible (PR #108721)

2024-09-15 Thread Eden Reich via cfe-commits
edenreich wrote: Because I was trying to use this tool with the current stable release of clang-format, and it's currently broken? IDK, or I just wait for the next release of clang-format before I can consider using git-clang-format 🤷‍♂️ https://github.com/llvm/llvm-project/pull/108721 __

[clang] [git-clang-format] Fix: make the tool backward compatible (PR #108721)

2024-09-15 Thread Owen Pan via cfe-commits
owenca wrote: Why do we need to make newer versions of git-clang-format "compatible" with older versions of clang-format? Shouldn't both be from the same LLVM release? https://github.com/llvm/llvm-project/pull/108721 ___ cfe-commits mailing list cfe-c

[clang] [git-clang-format] Fix: make the tool backward compatible (PR #108721)

2024-09-14 Thread Eden Reich via cfe-commits
https://github.com/edenreich edited https://github.com/llvm/llvm-project/pull/108721 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [git-clang-format] Fix: make the tool backward compatible (PR #108721)

2024-09-14 Thread Eden Reich via cfe-commits
https://github.com/edenreich edited https://github.com/llvm/llvm-project/pull/108721 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [git-clang-format] Fix: make the tool backward compatible (PR #108721)

2024-09-14 Thread Eden Reich via cfe-commits
https://github.com/edenreich edited https://github.com/llvm/llvm-project/pull/108721 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [git-clang-format] Fix: make the tool backward compatible (PR #108721)

2024-09-14 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-format Author: Eden Reich (edenreich) Changes ## Summary Currently it's not backward compatible, because the flag -list-ignored doesn't exists on older version of clang-format. To make this tool still backward compatible, we can first check if t

[clang] [git-clang-format] Fix: make the tool backward compatible (PR #108721)

2024-09-14 Thread via cfe-commits
github-actions[bot] wrote: Thank you for submitting a Pull Request (PR) to the LLVM Project! This PR will be automatically labeled and the relevant teams will be notified. If you wish to, you can add reviewers by using the "Reviewers" section on this page. If this is not working for you, it

[clang] [git-clang-format] Fix: make the tool backward compatible (PR #108721)

2024-09-14 Thread Eden Reich via cfe-commits
https://github.com/edenreich created https://github.com/llvm/llvm-project/pull/108721 ## Summary Currently it's not backward compatible, because the flag -list-ignored doesn't exists on older version of clang-format. To make this tool still backward compatible, we can first check if the .cla