On Thu, 10 Oct 2024 13:56:23 GMT, Hamlin Li <m...@openjdk.org> wrote:

> > FYI: I witnessed some different kinds of compile errors about libsleef 
> > while building with GCC version 13.2.0 (Ubuntu 13.2.0-23ubuntu4) shipped 
> > with Ubuntu 24.04.1 LTS riscv64. Seems that GCC-13 doesn't have a good 
> > support for these RVV intrinsics as it does not recognize quite a few of 
> > them. If libsleef claims GCC-14+, maybe it's more reasonable to simply 
> > check the GCC version instead requiring 14+ when configuring openjdk?
> 
> From https://github.com/shibatch/sleef, it state clearly that gcc 14+ is 
> required, I think we should be fine to check gcc version.
> 
> For llvm, there is no version requirement stated in the page, it could still 
> require a little `test` program to tell whether rvv intrinsicv are really 
> supported or not (If `__riscv_v_intrinsic` can not be used as a test macro 
> for that purpose like gcc(e.g. gcc-13) ).

FYI: I see they claim LLVM 17+ from the page:

Similarly, the RISC-V interface in SLEEF is based on version 1.0 of the 
intrinsics, which is only supported from llvm version 17 and gcc version 14 
onwards.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/21442#issuecomment-2405183494

Reply via email to