search

ROCm / clr

MIT License
89 stars 47 forks source link

Support non-x86_64 platform #32

Closed Xeonacid closed 1 week ago

Xeonacid commented 10 months ago

Hi, Some code uses <immintrin.h> which cannot be built on non_x86_64 platform. Could you add a fallback generic implementation? Thanks!

cjatin commented 10 months ago

can you share a bit more information.

Which platform are you using? The file from which the error is originating? you gpu config?

Xeonacid commented 10 months ago

Which platform are you using?

RISC-V. In my case.

The file from which the error is originating?

https://github.com/ROCm-Developer-Tools/clr/blob/develop/rocclr/device/rocm/rocvirtual.cpp as you can see in #35

cjatin commented 10 months ago

raised a PR internally for this.

cjatin commented 9 months ago

PR: https://github.com/ROCm/clr/commit/12461dbd6a26e02d03b572399b6d629f44b0a318

darkbasic commented 7 months ago

At least on ppc64le I do have such header:

./usr/lib/gcc/powerpc64le-unknown-linux-gnu/13/include/immintrin.h
./usr/lib/clang/15.0.7/include/ppc_wrappers/immintrin.h
./usr/lib/clang/15.0.7/include/immintrin.h
./usr/lib/clang/17/include/ppc_wrappers/immintrin.h
./usr/lib/clang/17/include/immintrin.h

Unfortunately even if it compiles fine I get runtime errors: https://github.com/ROCm/clr/issues/61

@Xeonacid does it work for you on RISC-V?

ppanchad-amd commented 1 week ago

@Xeonacid Closing ticket. Please re-open if you still need further assistance with this ticket. Thanks!