Add RISC-V 64 Support for LuaJIT v2.1 - #236
Conversation
|
Oops, seems we have some issues with amd64 target…… |
Should have been fixed. |
b2c6b14 to
74fe9b0
Compare
|
Can we rerun Travis CI? |
I'll do a patch refresh soon. Btw, I'm considering dropping DynASM |
74fe9b0 to
72932c4
Compare
Done, all passing. |
|
@zhuizhuhaomeng PTAL, thanks |
cac9628 to
59e9671
Compare
59e9671 to
f25e678
Compare
4e7448d to
887b468
Compare
3a2022a to
ea214b2
Compare
|
@zhuizhuhaomeng Could you please take a look? 😇 |
|
Is there a chance this can be reviewed? |
Ultimately, it depends on upstream opinion (i.e., Mike). I am currently employed full-time at PLCT Lab. As Mike focuses on version 3.0, we will ensure that a functional fork for version 2.1 is available. |
Also refresh 020-riscv64-support.patch from the current state of openresty/luajit2#236. The previous snapshot defined asm_tail_prep(ASMState *as), but since openresty/luajit2 commit 68354f44 the callers pass a second TraceNo argument, so the riscv64 build failed with: lj_asm.c:2559:5: error: too many arguments to function 'asm_tail_prep' The refreshed patch carries the upstream adaptation and restores the build. The other four patches apply unchanged. Signed-off-by: Josef Schlehofer <pepe.schlehofer@gmail.com>
Also refresh 020-riscv64-support.patch from the current state of openresty/luajit2#236. The previous snapshot defined asm_tail_prep(ASMState *as), but since openresty/luajit2 commit 68354f44 the callers pass a second TraceNo argument, so the riscv64 build failed with: lj_asm.c:2559:5: error: too many arguments to function 'asm_tail_prep' The refreshed patch carries the upstream adaptation and restores the build. The other four patches apply unchanged. Signed-off-by: Josef Schlehofer <pepe.schlehofer@gmail.com> (cherry picked from commit 578758e)
Also refresh 020-riscv64-support.patch from the current state of openresty/luajit2#236. The previous snapshot defined asm_tail_prep(ASMState *as), but since openresty/luajit2 commit 68354f44 the callers pass a second TraceNo argument, so the riscv64 build failed with: lj_asm.c:2559:5: error: too many arguments to function 'asm_tail_prep' The refreshed patch carries the upstream adaptation and restores the build. The other four patches apply unchanged. Signed-off-by: Josef Schlehofer <pepe.schlehofer@gmail.com> (cherry picked from commit 578758e)
Also refresh 020-riscv64-support.patch from the current state of openresty/luajit2#236. The previous snapshot defined asm_tail_prep(ASMState *as), but since openresty/luajit2 commit 68354f44 the callers pass a second TraceNo argument, so the riscv64 build failed with: lj_asm.c:2559:5: error: too many arguments to function 'asm_tail_prep' The refreshed patch carries the upstream adaptation and restores the build. The other four patches apply unchanged. Signed-off-by: Josef Schlehofer <pepe.schlehofer@gmail.com> (cherry picked from commit 578758e) (cherry picked from commit f8b35d4)
Co-authored-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
e00e8aa to
ce14aff
Compare
This set of patches adds support for RV64G LP64D on Linux, with full interpreter and JIT support.
Patch ported from PLCT LuaJIT, tested with LuaJIT/LuaJIT-test-cleanup.
See also upstream pr LuaJIT/LuaJIT#1267