Carla icon indicating copy to clipboard operation
Carla copied to clipboard

add riscv64 support

Open moodyhunter opened this issue 2 years ago • 4 comments

Currently, it's difficult for WDL eel to support riscv64 'natively', so we are enabling EEL_TARGET_PORTABLE as a workaround.

moodyhunter avatar Jul 20 '22 23:07 moodyhunter

Maybe a better approach here is to enable the generic mode if cpu is not within an allow list? Otherwise we will need yet another check if building for a different cpu.

@jpcima is there a list of cpus that ysfx/jsfx has optimizations for? perhaps it is just i686, x64 and arm64?

falkTX avatar Jul 20 '22 23:07 falkTX

That makes sense, well, it's up to you guys to decide whether we can set "generic mode" for all other architectures. I'll update this PR once that's confirmed :)

moodyhunter avatar Jul 20 '22 23:07 moodyhunter

perhaps it is just i686, x64 and arm64?

yes that's it exactly and the arm32 support which is broken at the moment (according to eel makefile "armv6t2 + VFP")

jpcima avatar Jul 21 '22 09:07 jpcima

Hi, the PR has been updated, now it has a better way to check for those architectures which have "optimised" code.

moodyhunter avatar Jul 27 '22 17:07 moodyhunter

I kinda forgot about this, and ended up doing something similar but in a slightly different way (somewhat more ugly..)

Please check if it builds for you, thanks!

falkTX avatar Sep 15 '22 00:09 falkTX

Hi @falkTX, I've tested and the commit https://github.com/falkTX/Carla/commit/c9fc68aaa370088b588ad6aea7af60efbc16530c works fine :)

moodyhunter avatar Sep 15 '22 13:09 moodyhunter

Cool thanks for confirming. Closing the PR then.

btw, which board are you testing with?

falkTX avatar Sep 15 '22 13:09 falkTX

We currently have 1) StarFive VisionFive, 2) AllWinner D1 but I was testing on a qemu-system virtual machine.

moodyhunter avatar Sep 15 '22 13:09 moodyhunter