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

[SYCL] Add test cases for muptiplies,bit_or,bit_xor,bit_and subgroup … #158

Merged
merged 4 commits into from
Mar 4, 2021

Conversation

v-klochkov
Copy link

…algorithms

Signed-off-by: Vyacheslav N Klochkov vyacheslav.n.klochkov@intel.com

v-klochkov added a commit to v-klochkov/llvm that referenced this pull request Feb 26, 2021
This enables SPIR-V operations defined in version 1.3.
For example, it becomes possible to use sub-group algorithms
reduce(), exclusive_scan(), inclusive_scan() with multiply,
bitwise OR/XOR/AND operations.

The corresponding LIT test update is here:
intel/llvm-test-suite#158

Signed-off-by: Vyacheslav N Klochkov <vyacheslav.n.klochkov@intel.com>
@v-klochkov v-klochkov force-pushed the public_vklochkov_spiprv_1_3 branch 2 times, most recently from a8171a9 to 93992f8 Compare February 26, 2021 04:43
pvchupin pushed a commit to intel/llvm that referenced this pull request Feb 26, 2021
This patch enables SPIR-V operations defined in versions 1.2 and 1.3.

The major change in SPIR-V 1.3 is adding more sub-group operations
and this patch is motivated by the need to enable more sub-group algorithms
such as reduce(), exclusive_scan(), inclusive_scan() with multiply,bitwise OR/XOR/AND operations.

Lifting the max version to 1.3 should not cause backward compatibility issues
as up to SPIR-V 1.3 there are no deleted SPIR-V operations.
This though lifts the requirements to OpenCL and Level0 RT.

The corresponding LIT test update is here:
intel/llvm-test-suite#158

Signed-off-by: Vyacheslav N Klochkov <vyacheslav.n.klochkov@intel.com>
…algorithms

These new test cases verify intel/llvm#3267

Signed-off-by: Vyacheslav N Klochkov <vyacheslav.n.klochkov@intel.com>
Signed-off-by: Vyacheslav N Klochkov <vyacheslav.n.klochkov@intel.com>
…n device had support for sub-groups

Signed-off-by: Vyacheslav N Klochkov <vyacheslav.n.klochkov@intel.com>
@v-klochkov v-klochkov force-pushed the public_vklochkov_spiprv_1_3 branch from 93992f8 to ac90159 Compare March 1, 2021 21:32
@v-klochkov v-klochkov requested a review from vladimirlaz March 1, 2021 23:21
@v-klochkov
Copy link
Author

The buildbot on Linux reports an issue (SYCL::Plugin/interop-level-zero-keep-ownership.cpp) unrelated to this patch.

Signed-off-by: Vyacheslav N Klochkov <vyacheslav.n.klochkov@intel.com>
@v-klochkov
Copy link
Author

@vladimirlaz - thank you for the approval. The only failed test is on linux buildbot: it is for unrelated test: SYCL::Plugin/interop-level-zero-keep-ownership.cpp

@vladimirlaz vladimirlaz merged commit d565b8a into intel:intel Mar 4, 2021
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
intel/llvm-test-suite#158)

* Move spir-v 1.3 test cases to separate files to avoid failure on CUDA
* Fix the check for sub-groups availability. It returned false even when device had support for sub-groups
* Add checks for cl_khr_fp16/fp64
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