-
Notifications
You must be signed in to change notification settings - Fork 5
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
Suggestion: Replace libclang
to clang ast-dump=json
#2
Comments
that looks interesting, but is there a reason to not require LLVM? Do other apps support generating a Clang AST JSON file? I'll be happy to integrate this if there's a significant benefit to it, but if we have to install Clang anyway I'd prefer to keep linking against Clang for code simplicity. |
In addition to Zig which has portable LLVM. Until now, I don't know! Having libclang means the user is forced to have the LLVM (+ clang) toolchain. There are some Linux distributions that subdivide the LLVM and clang package. |
Even, if the project does not add C macros support. Then ast-dump will be enough. Preprocessors is not translated in clang AST. |
I'll think about it but I'm not sure that I'm ready to put in the effort yet. |
Hi @LorenDB,
Based on:
Also used on c2z(C++ to Zig bindings [generator]), replacing
clang
tozig cc
and add-lc++
(llvm-libc++) flag.The purpose is to make your application more portable with no LLVM dependencies.
another reference
The text was updated successfully, but these errors were encountered: