Skip to content

Conversation

@uditagarwal97
Copy link
Contributor

@uditagarwal97 uditagarwal97 commented Jul 18, 2024

@uditagarwal97 uditagarwal97 marked this pull request as ready for review July 18, 2024 14:57
@uditagarwal97 uditagarwal97 requested a review from a team as a code owner July 18, 2024 14:57
@uditagarwal97 uditagarwal97 changed the title [CI] Fix interop E2E tests on Win CI [CI] Fix detection of level_zero_dev_kit LIT feature on Win CI with L0 Jul 18, 2024
@uditagarwal97
Copy link
Contributor Author

uditagarwal97 commented Jul 18, 2024

CI failures:
E2E tests on Arc, Linux:

Failed Tests (2):
  SYCL :: ESIMD/PerformanceTests/invoke_simd_smoke.cpp
  SYCL :: Regression/local-arg-align.cpp

Regression/local-arg-align.cpp test failure is related to opaque pointers support #10682
Opened a new bug report for ESIMD/PerformanceTests/invoke_simd_smoke.cpp: #14647

E2E test on Windows Gen12

Failed Tests (1):
  SYCL :: ESIMD/api/functional/ctors/ctor_load_usm_fp_extra.cpp

This test failure is related to #14542

Linux LIT tests

Failed Tests (1):
  SYCL :: invoke_simd/invoke_simd_address_space_inferral.cpp

Opened a new issue for this failure: #14648

@uditagarwal97
Copy link
Contributor Author

All the above test failures are unrelated to the changes made in this PR.
@intel/llvm-gatekeepers the PR is ready to be merged.

@steffenlarsen
Copy link
Contributor

I believe ESIMD/api/functional/ctors/ctor_load_usm_fp_extra.cpp is unrelated to #14542, but it has been seen in #14631 and #14618.

As for the other failures, I am a little weary given so many tests fail that are failing for the first time. It would be good to have @sarnex's thoughts on this, seeing as most are ESIMD.

@sarnex
Copy link
Contributor

sarnex commented Jul 19, 2024

Other two issues should be fixed in HEAD, it was introduced by a different commit. I'm comfortable merging this.

@sarnex sarnex merged commit 3bd3256 into sycl Jul 19, 2024
@uditagarwal97 uditagarwal97 deleted the sycl-devops-pr/udit/win_interop_e2e branch July 30, 2024 15:22
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.

5 participants