Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

[SYCL] Disable sort over sub-group for CUDA and HIP BEs #1396

Merged

Conversation

romanovvlad
Copy link

CUDA and HIP BE do not support intel::reqd_sub_group_size, so it's not possible to control sub-group size which is needed to get the size of temporary memory for sort algorithm.

CUDA and HIP BE do not support intel::reqd_sub_group_size, so it's not
possible to control sub-group size which is needed to get the size of
temporary memory for sort algorithm.
@romanovvlad romanovvlad requested review from Pennycook and a team as code owners November 17, 2022 19:26
@romanovvlad
Copy link
Author

Tagging @npmiller

@againull againull merged commit ecc67a2 into intel:intel Nov 18, 2022
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Mar 22, 2023
…ntel#1353) (intel#1396)

A number of tests currently use double and sycl::half despite them not
being supported by the used device. Although it does not necessarily
fail to run on those devices, they are not considered supported by the
SYCL specification and should not be used in testing on those devices.
This commit prevent parts of a selection of tests from running on
devices that do not support the types used.

Signed-off-by: Larsen, Steffen <steffen.larsen@intel.com>

Signed-off-by: Larsen, Steffen <steffen.larsen@intel.com>
Co-authored-by: Steffen Larsen <steffen.larsen@intel.com>
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Mar 22, 2023
CUDA and HIP BE do not support intel::reqd_sub_group_size, so it's not
possible to control sub-group size which is needed to get the size of
temporary memory for sort algorithm.
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…est-suite#1396)

CUDA and HIP BE do not support intel::reqd_sub_group_size, so it's not
possible to control sub-group size which is needed to get the size of
temporary memory for sort algorithm.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants