Skip to content

Add riscv64 platform support#9287

Merged
headius merged 1 commit intojruby:masterfrom
trocedge:master
Mar 4, 2026
Merged

Add riscv64 platform support#9287
headius merged 1 commit intojruby:masterfrom
trocedge:master

Conversation

@trocedge
Copy link
Copy Markdown
Contributor

@trocedge trocedge commented Mar 3, 2026

This PR adds RISC-V architecture support to JRuby (issue #9286). The changes are modeled after the LoongArch implementation in #7518 to ensure the support is as comprehensive and accurate as possible.

Copy link
Copy Markdown
Member

@headius headius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clean and mundane, just as it should be.

@headius
Copy link
Copy Markdown
Member

headius commented Mar 4, 2026

@trocedge I'm glad the changes were simple! If you could, rebase on master rather than merge, just to clean up the history a little bit.

If you're up for more help, we wouldn't turn down some help setting up a RISC-V test job here! (probably using qemu and Docker as it is in jffi).

@headius headius added this to the JRuby 10.1.0.0 milestone Mar 4, 2026
Co-authored-by: gong-flying <gongxiaofei24@iscas.ac.cn>
@trocedge
Copy link
Copy Markdown
Contributor Author

trocedge commented Mar 4, 2026

All set! I rebased the branch. To be honest, I don't use rebase very often, so it took me a minute to figure it out, but I'm glad I got it working. The history should be nice and clean now.

@trocedge
Copy link
Copy Markdown
Contributor Author

trocedge commented Mar 4, 2026

As for the RISC-V CI job, I'd be absolutely happy to help! I'll try setting it up based on what we have in JFFI.

@headius
Copy link
Copy Markdown
Member

headius commented Mar 4, 2026

@trocedge Thank you!

@headius headius merged commit a3180b4 into jruby:master Mar 4, 2026
77 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants