Skip to content

Conversation

@KornevNikita
Copy link
Contributor

@KornevNikita KornevNikita commented Apr 2, 2025

Update REQUIRES

@sarnex
Copy link
Contributor

sarnex commented Apr 2, 2025

env: '/__w/llvm/llvm/build-e2e/OptionalKernelFeatures/is_compatible/Output/is_compatible_amdgcn.cpp.tmp.out': No such file or directory

test bug?

@KornevNikita
Copy link
Contributor Author

@sarnex the test still fails with:

# .---command stderr------------
# | clang++: error: cannot find ROCm device library; provide its path via '--rocm-path' or '--rocm-device-lib-path', or pass '-nogpulib' to build without ROCm device library
# | clang++: error: cannot find ROCm device library; provide its path via '--rocm-path' or '--rocm-device-lib-path', or pass '-nogpulib' to build without ROCm device library
# `-----------------------------
# error: command failed with exit status: 1

and

# .---command stderr------------
# | clang++: error: cannot find libdevice for sm_50; provide path to different CUDA installation via '--cuda-path', or pass '-nocudalib' to build without linking with libdevice
# | clang++: error: cannot find libdevice for sm_50; provide path to different CUDA installation via '--cuda-path', or pass '-nocudalib' to build without linking with libdevice
# `-----------------------------
# error: command failed with exit status: 1

Though there are cuda_dev_kit and hip_dev_kit in the list of available features.
Could you please check?
See: https://github.com/intel/llvm/actions/runs/14225481446/job/39865116049?pr=17809

@sarnex
Copy link
Contributor

sarnex commented Apr 2, 2025

thanks probably i missed setting some path

@sarnex
Copy link
Contributor

sarnex commented Apr 4, 2025

Ok the current problem is a compiler bug here, gonna try to fix it.

@KornevNikita
Copy link
Contributor Author

@intel/llvm-gatekeepers could you please merge?
@sarnex need to merge this to calm down our internal testing. I'll remove UNSUPPORTED: windows later when it's possible.

@kbenzie kbenzie merged commit 8552343 into intel:sycl Apr 8, 2025
36 of 37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants