Skip to content

Commit be530df

Browse files
Fix size parameters in cl_ext_buffer_device_address tests (#2585)
The `DeviceAddrFromAPI` variable is of type `cl_mem_device_address_ext`. But on 32-bit systems `sizeof(void*) = 4 < 8 = sizeof(cl_mem_device_address_ext)`. Pass `sizeof(DeviceAddrFromAPI)` instead.
1 parent 764b77a commit be530df

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

test_conformance/extensions/cl_ext_buffer_device_address/buffer_device_address.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -313,8 +313,8 @@ class BufferDeviceAddressTest {
313313
// A basic buffer used to pass the other buffer's address.
314314
error = clEnqueueWriteBuffer(queue, buffer_in_long,
315315
CL_TRUE, // block
316-
0, sizeof(cl_long), &DeviceAddrFromAPI,
317-
0, NULL, NULL);
316+
0, sizeof(DeviceAddrFromAPI),
317+
&DeviceAddrFromAPI, 0, NULL, NULL);
318318
test_error_fail(error,
319319
"clEnqueueWriteBuffer of dev_addr_buffer failed\n");
320320

@@ -325,9 +325,9 @@ class BufferDeviceAddressTest {
325325
&buffer_out_int);
326326
test_error_fail(error, "clSetKernelArg 1 failed\n");
327327

328-
error = clSetKernelExecInfo(ind_access_kernel,
329-
CL_KERNEL_EXEC_INFO_DEVICE_PTRS_EXT,
330-
sizeof(void *), &DeviceAddrFromAPI);
328+
error = clSetKernelExecInfo(
329+
ind_access_kernel, CL_KERNEL_EXEC_INFO_DEVICE_PTRS_EXT,
330+
sizeof(DeviceAddrFromAPI), &DeviceAddrFromAPI);
331331
test_error_fail(error,
332332
"Setting indirect access for "
333333
"device ptrs failed!\n");

0 commit comments

Comments
 (0)