llvm icon indicating copy to clipboard operation
llvm copied to clipboard

[SYCL] Test SYCL :: ESIMD/hardware_dispatch.cpp is failing on windows CI

Open lbushi25 opened this issue 1 year ago • 4 comments

Describe the bug

The test SYCL :: ESIMD/hardware_dispatch.cpp is failing on Windows pre-commit CI. Upon resolution, please remove the XFAIL from the test source.

To reproduce

clang++ -fsycl -fsycl-targets=intel_gpu_bdw hardware_dispatch.cpp -Xs "-options -vc-codegen" -o hardware_dispatch_test hardware_dispatch_test.exe

Environment

  • OS: Windows

Additional context

No response

lbushi25 avatar Jun 27 '24 21:06 lbushi25

@lbushi25 Can you send me a precommit CI run that shows the failure? Thanks

sarnex avatar Jun 28 '24 14:06 sarnex

@lbushi25 Can you send me a precommit CI run that shows the failure? Thanks

Hi @sarnex , I'm on it! Expect an update soon!

lbushi25 avatar Jun 28 '24 16:06 lbushi25

I've submitted a draft PR here purely for precommit CI purposes where I have removed all XFAIL comments from E2E tests to get an accurate summary of the failures. This is part of a larger ongoing effort to make sure that every XFAIL is documented with a GH issue linked in the test source.

lbushi25 avatar Jun 28 '24 17:06 lbushi25

@lbushi25 Can you send me a precommit CI run that shows the failure? Thanks

The CI run: https://github.com/intel/llvm/pull/14354/checks The E2E runs specifically have not started yet so I would check later/on Monday given how slow the CI runners have been to pick up jobs these last couple of days.

lbushi25 avatar Jun 28 '24 20:06 lbushi25

Thanks, I'll take a look

sarnex avatar Jul 01 '24 20:07 sarnex

We need to use tgllp instead of bdw on Windows because the Windows driver doesn't support bdw anymore, but then we need the changes in https://github.com/intel/llvm/pull/14188, so waiting for that.

sarnex avatar Jul 08 '24 19:07 sarnex