Skip to content

[SYCL][Joint Matrix] Add test for multiple elem-wise ops #10258

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

Merged
merged 3 commits into from
Jul 26, 2023

Conversation

YuriPlyakhin
Copy link
Contributor

Add new test for multiple elem-wise ops for different matrix types in the same kernel

@YuriPlyakhin YuriPlyakhin temporarily deployed to aws July 6, 2023 23:41 — with GitHub Actions Inactive
@YuriPlyakhin YuriPlyakhin temporarily deployed to aws July 7, 2023 00:24 — with GitHub Actions Inactive
@YuriPlyakhin YuriPlyakhin temporarily deployed to aws July 7, 2023 15:43 — with GitHub Actions Inactive
@YuriPlyakhin YuriPlyakhin temporarily deployed to aws July 7, 2023 16:17 — with GitHub Actions Inactive
@YuriPlyakhin YuriPlyakhin marked this pull request as ready for review July 7, 2023 17:13
@YuriPlyakhin
Copy link
Contributor Author

Failures:
AMD

Failed Tests (3):
  SYCL :: Basic/built-ins/host_math.cpp
  SYCL :: Basic/get_backend.cpp
  SYCL :: Regression/device_num.cpp

CUDA (tagging @intel/llvm-reviewers-cuda):

Failed Tests (7):
  SYCL :: Basic/built-ins/host_math.cpp
  SYCL :: Basic/get_backend.cpp
  SYCL :: Regression/device_num.cpp
  SYCL :: Regression/invalid_reqd_wg_size_correct_exception.cpp
  SYCL :: Tracing/usm/queue_copy_released_pointer.cpp
  SYCL :: Tracing/usm/queue_single_task_nullptr.cpp
  SYCL :: Tracing/usm/queue_single_task_released_pointer.cpp

are unrelated, as this PR just adds one more e2e test. Also noticed these same tests failing in other PR (#10259)

@YuriPlyakhin
Copy link
Contributor Author

@yubingex007-a11y , please, review

@YuriPlyakhin
Copy link
Contributor Author

@intel/llvm-gatekeepers, please merge

@aelovikov-intel
Copy link
Contributor

@intel/llvm-gatekeepers, please merge

We can't merge PRs with two weeks old test results.

Add new test for multiple elem-wise ops for different
matrix types in the same kernel
@YuriPlyakhin YuriPlyakhin force-pushed the jm_test_multi_elem_print branch from f7f8b0f to 9a6d701 Compare July 25, 2023 21:45
@YuriPlyakhin YuriPlyakhin temporarily deployed to aws July 25, 2023 21:51 — with GitHub Actions Inactive
@YuriPlyakhin YuriPlyakhin temporarily deployed to aws July 25, 2023 22:30 — with GitHub Actions Inactive
@YuriPlyakhin
Copy link
Contributor Author

@intel/llvm-gatekeepers , please, merge. All green.

@steffenlarsen steffenlarsen merged commit 57098b0 into intel:sycl Jul 26, 2023
mdtoguchi pushed a commit to mdtoguchi/llvm that referenced this pull request Oct 18, 2023
Add new test for multiple elem-wise ops for different matrix types in
the same kernel
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.

4 participants