-
Notifications
You must be signed in to change notification settings - Fork 7
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
Bring bindgen up-to-date and add some fixes #200
base: master
Are you sure you want to change the base?
Conversation
Segfault appeared:
With debug symbols:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thank you for making it up to date with latest Scala Native.
I believe the changes should be also applied to tests, it's not catched becouse CI is broken and would require moving it from Travis to GHA. I would not be able to allocate a time for that soon. |
bindgen/ir/Record.cpp
(for unknown reasons, I have to admit)