-
Notifications
You must be signed in to change notification settings - Fork 3
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
Adapt to Clang 16 internal changes #71
base: mg/poplar-3.3
Are you sure you want to change the base?
Conversation
I hit more compilation errors when using the |
oh CI hasn't been upgraded to 1.11 |
Yes, I was trying that in #45. |
Maybe you want to target that branch? |
I made a debugging setup in this PR to generate wrappers locally without using a docker environment:
I got the following results on my machine:
comparing ② with ①, some nested name specifiers are dropped due to the internal changes introduced in Clang 16. (I'm currently looking for a way to fix this.) comparing ③ with ②, they are new types and methods introduced in v3.3, which may cause more compilation errors because some kinds of types are not supported and should be excluded manually like what it did here: IPUToolkit.jl/deps/generate_wrapper.jl Lines 438 to 506 in e598a1f
|
for someone who wanna debug this part, the errors below are produced after fixing (probably not all of) the NNS issues with ugly hacks 4eb3b9a
|
gen_inherit.cpp
gen_inline.cpp