Skip to content

Commit

Permalink
[SYCL][E2E] Use size_t rather than unsigned long in WorkGroupScratchM…
Browse files Browse the repository at this point in the history
…emory tests (#17100)

The max work group size is a `size_t`, rather than an `unsigned long`.
This trips up older versions of MSVC where `size_t` happens to be
`unsigned long long`.
  • Loading branch information
RossBrunton authored Feb 24, 2025
1 parent ed8b38e commit 59a1bab
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion sycl/test-e2e/WorkGroupScratchMemory/copy_dynamic_size.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ int main() {
sycl::queue queue;
auto size = std::min(
queue.get_device().get_info<sycl::info::device::max_work_group_size>(),
1024ul);
size_t{1024});

DataType *a = sycl::malloc_device<DataType>(size, queue);
DataType *b = sycl::malloc_device<DataType>(size, queue);
Expand Down
2 changes: 1 addition & 1 deletion sycl/test-e2e/WorkGroupScratchMemory/dynamic_unused.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ int main() {
sycl::queue queue;
auto size = std::min(
queue.get_device().get_info<sycl::info::device::max_work_group_size>(),
1024ul);
size_t{1024});

DataType *a = sycl::malloc_device<DataType>(size, queue);
DataType *b = sycl::malloc_device<DataType>(size, queue);
Expand Down

0 comments on commit 59a1bab

Please sign in to comment.