-
-
Notifications
You must be signed in to change notification settings - Fork 67
Update to clang 10? #25
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
Comments
What are you looking for specifically in clang 10? |
But why do you think that clang 10 is fixing this? |
the issue is usually with libclang, not with clang. Libclang makes a C wrapper of their internal C++ API, but their wrapper is incomplete, so things perfectly exposed in C++ can be wrongly exposed (or not exposed at all) through libclang, that's a known issue unfortunately (for example: https://foonathan.net/2017/04/cppast/) |
Idk I thought it would help. I get correct tokens from other forms of this library |
Which one? Do you have a small repro with these other libraries that I could check? |
Not a small repo, no. I was using the main branch of https://github.com/microsoft/ClangSharp I also tried https://github.com/mono/CppSharp All three don't really work for me but in lots of different ways lol. I'm beginning to think it's impossible to generate good bindings for third party libraries. |
I will probably wait that https://github.com/microsoft/ClangSharp release NuGet packages. I had to release my own https://github.com/xoofx/libclang but I would really prefer that there is a single source to manage that. |
Their beta came out recently. I was trying to get their p/invoke gen setup but was unsuccessful. Tbh, I really like how you've setup the code gen in https://github.com/xoofx/CppAst.CodeGen better. Just makes more sense. |
So latest master is using ClangSharp 10.0 beta package, though, I'm not able to get tests running on the CI for some reasons... |
Tracking the issue with RID with ClangSharp here |
So CppAst |
Any plans to update this library to use clang 10?
The text was updated successfully, but these errors were encountered: