Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Test on clang 5.0+ #59

Merged
merged 1 commit into from
Nov 15, 2022
Merged

Test on clang 5.0+ #59

merged 1 commit into from
Nov 15, 2022

Conversation

ioquatix
Copy link
Owner

No description provided.

@ioquatix ioquatix force-pushed the clang-5 branch 3 times, most recently from 657dfb6 to 8fc9b76 Compare November 15, 2022 09:36
@ioquatix ioquatix merged commit d13ef85 into main Nov 15, 2022
@ioquatix ioquatix deleted the clang-5 branch November 15, 2022 09:40
@ioquatix
Copy link
Owner Author

@k0kubun fyi now testing on 4.0 and 5.0, but it fails on 6.0+ so we might need to work on the test suite.

@k0kubun
Copy link

k0kubun commented Nov 16, 2022

👀 I see. I'm using libclang 14 for MJIT. We could possibly use its make mjit-bindgen as an integration test if you wish. (We don't need to build CRuby; it just needs to install ffi-clang.gem, parse vm.c with it, and generate a Ruby script using the parsed information. If it doesn't cause any diff, we can confirm it's working in the same way as the past versions)

@ioquatix
Copy link
Owner Author

That seems reasonable to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ioquatix @k0kubun