Skip to content

Commit 0c48d9c

Browse files
[NFCI][SYCL] Use convertToOpenCLType in more places (#12692)
Follow-up for #12674, updating places where `ConvertToOpenCLType_t` was used with a plain cast instead of `convertDataToType`. Not touching `multi_ptr` related uses just yet.
1 parent b06cfb5 commit 0c48d9c

File tree

4 files changed

+10
-15
lines changed

4 files changed

+10
-15
lines changed

sycl/include/sycl/detail/spirv.hpp

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -803,8 +803,7 @@ EnableIfGenericShuffle<T> SubgroupShuffleUp(T x, uint32_t delta);
803803
template <typename T>
804804
EnableIfNativeShuffle<T> SubgroupShuffle(T x, id<1> local_id) {
805805
#ifndef __NVPTX__
806-
using OCLT = detail::ConvertToOpenCLType_t<T>;
807-
return __spirv_SubgroupShuffleINTEL(OCLT(x),
806+
return __spirv_SubgroupShuffleINTEL(convertToOpenCLType(x),
808807
static_cast<uint32_t>(local_id.get(0)));
809808
#else
810809
return __nvvm_shfl_sync_idx_i32(membermask(), x, local_id.get(0), 0x1f);
@@ -814,9 +813,8 @@ EnableIfNativeShuffle<T> SubgroupShuffle(T x, id<1> local_id) {
814813
template <typename T>
815814
EnableIfNativeShuffle<T> SubgroupShuffleXor(T x, id<1> local_id) {
816815
#ifndef __NVPTX__
817-
using OCLT = detail::ConvertToOpenCLType_t<T>;
818816
return __spirv_SubgroupShuffleXorINTEL(
819-
OCLT(x), static_cast<uint32_t>(local_id.get(0)));
817+
convertToOpenCLType(x), static_cast<uint32_t>(local_id.get(0)));
820818
#else
821819
return __nvvm_shfl_sync_bfly_i32(membermask(), x, local_id.get(0), 0x1f);
822820
#endif
@@ -825,8 +823,8 @@ EnableIfNativeShuffle<T> SubgroupShuffleXor(T x, id<1> local_id) {
825823
template <typename T>
826824
EnableIfNativeShuffle<T> SubgroupShuffleDown(T x, uint32_t delta) {
827825
#ifndef __NVPTX__
828-
using OCLT = detail::ConvertToOpenCLType_t<T>;
829-
return __spirv_SubgroupShuffleDownINTEL(OCLT(x), OCLT(x), delta);
826+
return __spirv_SubgroupShuffleDownINTEL(convertToOpenCLType(x),
827+
convertToOpenCLType(x), delta);
830828
#else
831829
return __nvvm_shfl_sync_down_i32(membermask(), x, delta, 0x1f);
832830
#endif
@@ -835,8 +833,8 @@ EnableIfNativeShuffle<T> SubgroupShuffleDown(T x, uint32_t delta) {
835833
template <typename T>
836834
EnableIfNativeShuffle<T> SubgroupShuffleUp(T x, uint32_t delta) {
837835
#ifndef __NVPTX__
838-
using OCLT = detail::ConvertToOpenCLType_t<T>;
839-
return __spirv_SubgroupShuffleUpINTEL(OCLT(x), OCLT(x), delta);
836+
return __spirv_SubgroupShuffleUpINTEL(convertToOpenCLType(x),
837+
convertToOpenCLType(x), delta);
840838
#else
841839
return __nvvm_shfl_sync_up_i32(membermask(), x, delta, 0);
842840
#endif

sycl/include/sycl/ext/oneapi/experimental/non_uniform_groups.hpp

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,11 +34,9 @@ inline sycl::vec<unsigned, 4> ExtractMask(ext::oneapi::sub_group_mask Mask) {
3434
// TODO: This may need to be generalized beyond uint32_t for big masks
3535
inline uint32_t CallerPositionInMask(ext::oneapi::sub_group_mask Mask) {
3636
sycl::vec<unsigned, 4> MemberMask = ExtractMask(Mask);
37-
auto OCLMask =
38-
sycl::detail::ConvertToOpenCLType_t<sycl::vec<unsigned, 4>>(MemberMask);
3937
return __spirv_GroupNonUniformBallotBitCount(
4038
__spv::Scope::Subgroup, (int)__spv::GroupOperation::ExclusiveScan,
41-
OCLMask);
39+
sycl::detail::convertToOpenCLType(MemberMask));
4240
}
4341
#endif
4442

sycl/include/sycl/ext/oneapi/sub_group_mask.hpp

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -138,10 +138,9 @@ struct sub_group_mask {
138138
for (int i = 0; i < 4; ++i) {
139139
MemberMask[i] = TmpMArray[i];
140140
}
141-
auto OCLMask =
142-
sycl::detail::ConvertToOpenCLType_t<sycl::vec<unsigned, 4>>(MemberMask);
143141
return __spirv_GroupNonUniformBallotBitCount(
144-
__spv::Scope::Subgroup, (int)__spv::GroupOperation::Reduce, OCLMask);
142+
__spv::Scope::Subgroup, (int)__spv::GroupOperation::Reduce,
143+
sycl::detail::convertToOpenCLType(MemberMask));
145144
#else
146145
unsigned int count = 0;
147146
auto word = (Bits & valuable_bits(bits_num));

sycl/include/sycl/group.hpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
#include <sycl/detail/common.hpp> // for NDLoop, __SYCL_ASSERT
1515
#include <sycl/detail/defines.hpp> // for __SYCL_TYPE
1616
#include <sycl/detail/defines_elementary.hpp> // for __SYCL2020_DEPRECATED
17-
#include <sycl/detail/generic_type_traits.hpp> // for ConvertToOpenCLType_t
17+
#include <sycl/detail/generic_type_traits.hpp> // for convertToOpenCLType
1818
#include <sycl/detail/helpers.hpp> // for Builder, getSPIRVMemo...
1919
#include <sycl/detail/item_base.hpp> // for id, range
2020
#include <sycl/detail/type_traits.hpp> // for is_bool, change_base_...

0 commit comments

Comments
 (0)