search

brummer10 / Xmonk.lv2

An LV2 toy
Other
6 stars 3 forks source link

add riscv64 support #8

Closed IEAST closed 3 years ago

IEAST commented 3 years ago

When I tried to build Xmonk, I found that it was optimized for intel cup.

g++: error: unrecognized command-line option -msse3
g++: error: unrecognized command-line option -mfpmath=sse

But there is no corresponding parameter defined on RISC-V, so I decided not to add CFLAGS if the architecture is RISC-V in the Mikefile.

brummer10 commented 3 years ago

That is really strange. As, the makefile checks if the CPU support any of the sse optimisations in /proc/cpuinfo If not, it defaults to empty the SSE_CFLAGS anyway. So I wonder why that seems to not work with your riscv64, if not supported, there shouldn't been any sse flags set. What is the output of your /proc/cpuinfo

IEAST commented 3 years ago

Thank you for your reply. I probably know why, currently I am using Qemu again to simulate the RISC-V environment for compilation, so the CPU information read is local (Ps: my computer's cpu information). I'll try to compile it in unmatched later.

brummer10 commented 3 years ago

You could overwrite the SSE_CFLAGS simply on command-line in that case: make SSE_CFLAGS="" to avoid reading the CPU information from your local host.

IEAST commented 3 years ago

thank you. I will close my pr.