Skip to content

Basic/std_array.cpp failing on Windows GEN12 post commit #18458

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

Open
kbenzie opened this issue May 14, 2025 · 3 comments
Open

Basic/std_array.cpp failing on Windows GEN12 post commit #18458

kbenzie opened this issue May 14, 2025 · 3 comments
Assignees
Labels
bug Something isn't working confirmed

Comments

@kbenzie
Copy link
Contributor

kbenzie commented May 14, 2025

As noted by @sarnex in this comment on #18400:

@ianayl @aelovikov-intel It looks like the std::array test is failing in Gen12 Win postcommit after this PR, can you take a look? Thx

https://github.com/intel/llvm/actions/runs/14999424550/job/42152662548

FAIL: SYCL :: Basic/std_array.cpp (407 of 2364)
******************** TEST 'SYCL :: Basic/std_array.cpp' FAILED ********************
Exit Code: 3221225477

Command Output (stdout):
--
# RUN: at line 5
D:/github/_work/llvm/llvm/install/bin/clang++.exe  -Werror --driver-mode=cl -fsycl -o D:\github\_work\llvm\llvm\build-e2e\Basic\Output\std_array.cpp.tmp.exe D:\github\_work\llvm\llvm\llvm\sycl\test-e2e\Basic\std_array.cpp /Od /MDd /Zi /EHsc
# executed command: D:/github/_work/llvm/llvm/install/bin/clang++.exe -Werror --driver-mode=cl -fsycl -o 'D:\github\_work\llvm\llvm\build-e2e\Basic\Output\std_array.cpp.tmp.exe' 'D:\github\_work\llvm\llvm\llvm\sycl\test-e2e\Basic\std_array.cpp' /Od /MDd /Zi /EHsc
# RUN: at line 6
env ONEAPI_DEVICE_SELECTOR=level_zero:gpu  D:\github\_work\llvm\llvm\build-e2e\Basic\Output\std_array.cpp.tmp.exe
# executed command: env ONEAPI_DEVICE_SELECTOR=level_zero:gpu 'D:\github\_work\llvm\llvm\build-e2e\Basic\Output\std_array.cpp.tmp.exe'
# note: command had no output on stdout or stderr
# error: command failed with exit status: 0xc0000005

--

********************
kbenzie added a commit to kbenzie/intel-llvm that referenced this issue May 14, 2025
@kbenzie kbenzie added the bug Something isn't working label May 14, 2025
kbenzie added a commit to kbenzie/intel-llvm that referenced this issue May 14, 2025
@kbenzie
Copy link
Contributor Author

kbenzie commented May 14, 2025

Since this doesn't fail in pre-commit it must be something to do with the post-commit config.

kbenzie added a commit that referenced this issue May 14, 2025
@ianayl
Copy link
Contributor

ianayl commented May 15, 2025

Reproducer:
Using oneapi compiler 2025.1 environment, gfx 101.6734, MSVC 2022 17.13.4, and spirv-tools on Windows:

#include <sycl/queue.hpp>

int main() {
  sycl::queue q;
}

Compiling with clang-cl -fsycl reproducer.cpp /MDd in 2025.1 oneapi environment and running resulting executable results in 0xc0000005 NTSTATUS error.

.... Still need to fix the issue though 😅 But it does mean the issue is unrelated to changes from #18400.

@aelovikov-intel
Copy link
Contributor

aelovikov-intel commented May 16, 2025

What happens is ur_loader.dll depends on libmmd.dll when built with OneAPI compiler. Yet

auto loadAdapter = [&](auto adapterName,
DWORD flags = LOAD_LIBRARY_SEARCH_DLL_LOAD_DIR |
LOAD_LIBRARY_SEARCH_SYSTEM32) {
auto path = LibSYCLDir / adapterName;
return LoadLibraryEx(path.wstring().c_str(), NULL, flags);
};

prohibits libmmd.dll from being loaded when trying to load ur_loader.dll.

Looking into how to fix our CI, but there is no user-visible bug here.

aelovikov-intel added a commit that referenced this issue May 19, 2025
The issue is specific to the post-commit CI using OneAPI compiler to
build SYCL RT (see #18458). Ideally,
we'd like to use `/Qno-intel-lib` during such build but OneAPI seems to
have a bug related to that (reported internally). It will take an
official release for it to reach this CI even after it's implemented, so
we need to disable the test on that specific configuration only for the
time being.
aelovikov-intel added a commit that referenced this issue May 19, 2025
The issue is specific to the post-commit CI using OneAPI compiler to
build SYCL RT (see #18458). Ideally,
we'd like to use `/Qno-intel-lib` during such build but OneAPI seems to
have a bug related to that (reported internally). It will take an
official release for it to reach this CI even after it's implemented, so
we need to disable the test on that specific configuration only for the
time being.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working confirmed
Projects
None yet
Development

No branches or pull requests

3 participants