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.

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

upgpatch: python-llvmlite #3137

Merged
merged 1 commit into from
Oct 19, 2023
Merged

Conversation

Xunop
Copy link
Contributor

@Xunop Xunop commented Oct 19, 2023

OrcJIT is still in the experimental stage, and there are some issues where the OrcJIT test fails when tested on non-x86 platforms (e.g. Issue numba#1000), so skip writing this test on non-x86 platforms.

Skip OrcJIT tests on riscv64.

OrcJIT is still in the experimental stage, and there are some issues
where the OrcJIT test fails when tested on non-x86 platforms (e.g. Issue [numba#1000](numba/llvmlite#1000)),
so skip writing this test on non-x86 platforms.
@felixonmars felixonmars merged commit 3920159 into felixonmars:master Oct 19, 2023
1 check passed
@gmarkall
Copy link

Just saw this linked from numba/llvmlite#1000 - if you do want to use OrcJIT on RISC-V in llvmlite, you can backport a few JITLink patches - I outlined the ones that are needed in a post here: https://big-grey.co.uk/2023/01/26/using-risc-v-jit-on-llvm-14/

@Xunop
Copy link
Contributor Author

Xunop commented Oct 27, 2023

Just saw this linked from numba/llvmlite#1000 - if you do want to use OrcJIT on RISC-V in llvmlite, you can backport a few JITLink patches - I outlined the ones that are needed in a post here: https://big-grey.co.uk/2023/01/26/using-risc-v-jit-on-llvm-14/

Thanks for sharing this information. I'll look into the outlined JITLink patches for enabling OrcJIT on RISC-V in llvmlite. I appreciate your guidance on this.

@gmarkall
Copy link

You're welcome - do reach out to me if you have any questions!

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.

3 participants