Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Assert tests fail on Windows Gen12 #12797

Closed
steffenlarsen opened this issue Feb 22, 2024 · 2 comments
Closed

Assert tests fail on Windows Gen12 #12797

steffenlarsen opened this issue Feb 22, 2024 · 2 comments
Labels
bug Something isn't working confirmed Windows

Comments

@steffenlarsen
Copy link
Contributor

steffenlarsen commented Feb 22, 2024 •
edited
Loading

The following tests have started failing on CI Windows E2E testing on Gen12:

  • SYCL :: Assert/assert_in_kernels_win.cpp
  • SYCL :: Assert/assert_in_multiple_tus_one_ndebug_win.cpp
  • SYCL :: Assert/assert_in_multiple_tus_win.cpp
  • SYCL :: Assert/assert_in_one_kernel_win.cpp
  • SYCL :: Assert/assert_in_simultaneous_kernels_win.cpp
  • SYCL :: Assert/assert_in_simultaneously_multiple_tus.cpp
  • SYCL :: Assert/assert_in_simultaneously_multiple_tus_one_ndebug.cpp

See for example #12778 and #12766
@steffenlarsen steffenlarsen added bug Something isn't working confirmed Windows labels Feb 22, 2024
This was referenced Feb 22, 2024
Merged
Merged
Merged
Merged
@dkhaldi dkhaldi mentioned this issue Feb 22, 2024
Merged
This was referenced Feb 23, 2024
Merged
Merged
Merged
@KornevNikita KornevNikita mentioned this issue Feb 23, 2024
Merged
@cperkinsintel cperkinsintel mentioned this issue Feb 23, 2024
Merged
@isaacault isaacault mentioned this issue Feb 23, 2024
Merged
@aelovikov-intel
Copy link
Contributor

Possibly related: #11364 (as per internal defect for win failures).

aelovikov-intel added a commit to aelovikov-intel/llvm that referenced this issue Feb 23, 2024
@aelovikov-intel
[SYCL][E2E] Disable multiple assert tests on Windows
2260495 
See intel#12797.
@aelovikov-intel aelovikov-intel mentioned this issue Feb 23, 2024
Merged
aelovikov-intel added a commit that referenced this issue Feb 23, 2024
@aelovikov-intel
[SYCL][E2E] Disable multiple assert tests on Windows (#12816)
03318d5 
See #12797.
@aelovikov-intel aelovikov-intel mentioned this issue Feb 23, 2024
Merged
@steffenlarsen steffenlarsen mentioned this issue Feb 26, 2024
Merged
@uditagarwal97 uditagarwal97 mentioned this issue Feb 26, 2024
Merged
@steffenlarsen steffenlarsen mentioned this issue Feb 27, 2024
Merged
@maarquitos14 maarquitos14 mentioned this issue Apr 1, 2024
Merged
This was referenced Aug 26, 2024
Closed
Merged
steffenlarsen pushed a commit that referenced this issue Sep 2, 2024
@lbushi25
[SYCL] Fix device assertion bug on windows (#15232)
101307f 
This PR fixes a device assertion bug on Windows where the assertion
message and information are not printed when an assertion is triggered.
Also re-enabled some tests failing because of it. See
#12797.
@lbushi25
Copy link
Contributor

Can we close this now that #15232 has been merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working confirmed Windows
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@aelovikov-intel @steffenlarsen @lbushi25