-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Please allow using AdaptiveCpp (acpp) as a replacement for icpx #958
Comments
@zboszor what about actual MKL? If that's not present in Yocto then there is no point in enabling adaptivecpp |
MKL is present in |
FWIW, I have already created a Yocto recipe for chipStar with only a few minimal patches needed. The samples do run with both Intel OpenCL and Level0. |
I somewhat lied. Most of the samples do run, but a few segfaults, like |
Please dump the traces here along with platform information. |
Output of
backtrace:
The crash is inside the Intel OpenCL driver and eventually in libLLVM-15.so. I will try to rebuild it with LLVM 14, as LLVM 15 support in the Intel Graphics Compiler is considered beta quality. |
Fixed the IGC / Compute Runtime build on my side with Yocto. Now it uses LLVM 14 and an updated version of the SPIRV-LLVM-Translator from the LLVM 14 compatible branch. The previously crashing fp16 related tests work and report "passed". I tried running every HIP and CUDA samples and they all work. |
@zboszor ran into issues compiling AdaptiveCpp, will try again today |
@zboszor switching to acpp for compiling the MKLShim layer is not as trivial as I expected as all of the cmake export targets from MKL (which we do not control) expect icpx
|
Take your time. Thank you very much for looking into it. |
hipBLAS and other optional libraries in chipStar use the git submodule
H4I-MKLShim
which relies on the Intelicpx
SYCL compiler.However, Yocto with its cross-compiler framework, cannot easily use
icpx
, even with meta-intel present in the build.Please, allow AdaptiveCpp to be used for compiling the SYCL sources, which may be more accessible.
The text was updated successfully, but these errors were encountered: