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.

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

[NFC][SYCL][Bindless] Define VERBOSE_PRINT before include user_types_common.hpp #12796

Merged

Conversation

wenju-he
Copy link
Contributor

Because VERBOSE_PRINT is also used inside file user_types_common.hpp

…common.hpp

Because VERBOSE_PRINT is also used inside file user_types_common.hpp
@wenju-he
Copy link
Contributor Author

Failures in SYCL Pre Commit on Windows / e2e / Intel GEN12 Graphics with Level Zero (pull_request) are not related to this PR

********************
Failed Tests (8):
  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
  SYCL :: Plugin/sycl-ls-unified-runtime.cpp


Testing Time: 2640.04s

Total Discovered Tests: 1902
  Unsupported      :  856 (45.01%)
  Passed           : 1028 (54.05%)
  Expectedly Failed:   10 (0.53%)
  Failed           :    8 (0.42%)

@wenju-he
Copy link
Contributor Author

@intel/llvm-gatekeepers could you please merge?

@steffenlarsen
Copy link
Contributor

Windows Gen12:

Failed Tests (8):
  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
  SYCL :: Plugin/sycl-ls-unified-runtime.cpp

Reported in #12797 and #12798

@steffenlarsen steffenlarsen merged commit 7b41068 into intel:sycl Feb 27, 2024
10 of 11 checks passed
@wenju-he wenju-he deleted the bindless_images-user_types-VERBOSE_PRINT branch February 27, 2024 08:09
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