Skip to content

The git-clang-format.py is not in sync with the llvm-18 version #41

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
ClausKlein opened this issue Sep 12, 2024 · 1 comment · Fixed by #42
Closed

The git-clang-format.py is not in sync with the llvm-18 version #41

ClausKlein opened this issue Sep 12, 2024 · 1 comment · Fixed by #42

Comments

@ClausKlein
Copy link
Contributor

diff --git a/git-clang-format.py b/git-clang-format.py
index 30a2add..2e7b6d8 100644
--- a/git-clang-format.py
+++ b/git-clang-format.py
@@ -25,6 +25,7 @@ Requires Python 2.7 or Python 3
 """
 
 from __future__ import absolute_import, division, print_function
+
 import argparse
 import collections
 import contextlib
@@ -79,14 +80,17 @@ def main():
       'c', 'h',  # C
       'm',  # ObjC
       'mm',  # ObjC++
-      'cc', 'cp', 'cpp', 'c++', 'cxx', 'hh', 'hpp', 'hxx', 'inc', 'inl',  # C++
-      'cu',  # CUDA
+      'cc', 'cp', 'cpp', 'c++', 'cxx', 'hh', 'hpp', 'hxx', 'inc',  # C++
+      'ccm', 'cppm', 'cxxm', 'c++m',  # C++ Modules
+      'cu', 'cuh',  # CUDA
       # Other languages that clang-format supports
       'proto', 'protodevel',  # Protocol Buffers
       'java',  # Java
       'js',  # JavaScript
       'ts',  # TypeScript
       'cs',  # C Sharp
+      'json',  # Json
+      'sv', 'svh', 'v', 'vh', # Verilog
       ])
 
   p = argparse.ArgumentParser(
@ClausKlein
Copy link
Contributor Author

@TheLartians Note: Homebrew clang version is v19.1.3 toddays!

breyerml added a commit to breyerml/Format.cmake that referenced this issue Nov 13, 2024
@TheLartians TheLartians linked a pull request Dec 22, 2024 that will close this issue
TheLartians added a commit that referenced this issue Dec 29, 2024
* Add .hip extension.

* Add new extensions according to issue #41

* Remove wrong brace.

* Remove Json.

```
Configuration file(s) do(es) not support Json
```
with clang-formt 18.0.0

* Readd JSON.

* Add inl and ipp file endings.

---------

Co-authored-by: Lars Melchior <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant