Added command buffer test with all mutable parameters dispatch (Khron…

…osGroup#1905) * Added command buffer with full mutable dispatch test According to KhronosGroup#1481 issue description, point 2.1 * Corrected the test to handle all available mutable properties According to KhronosGroup#1481 issue description, point 2.1
ouakheli · Mar 12, 2024 · ffb0265 · ffb0265
1 parent 6530ae4
commit ffb0265
Show file tree

Hide file tree

Showing 5 changed files with 521 additions and 10 deletions.
diff --git a/...ce/extensions/cl_khr_command_buffer/cl_khr_command_buffer_mutable_dispatch/CMakeLists.txt b/...ce/extensions/cl_khr_command_buffer/cl_khr_command_buffer_mutable_dispatch/CMakeLists.txt
@@ -9,6 +9,7 @@ set(${MODULE_NAME}_SOURCES
     mutable_command_global_size.cpp
     mutable_command_local_size.cpp
     mutable_command_global_offset.cpp
+    mutable_command_full_dispatch.cpp
     ../basic_command_buffer.cpp
 )
 

diff --git a/...formance/extensions/cl_khr_command_buffer/cl_khr_command_buffer_mutable_dispatch/main.cpp b/...formance/extensions/cl_khr_command_buffer/cl_khr_command_buffer_mutable_dispatch/main.cpp
@@ -26,6 +26,7 @@ test_definition test_list[] = {
     ADD_TEST(mutable_command_info_global_work_offset),
     ADD_TEST(mutable_command_info_local_work_size),
     ADD_TEST(mutable_command_info_global_work_size),
+    ADD_TEST(mutable_command_full_dispatch),
     ADD_TEST(mutable_dispatch_image_1d_arguments),
     ADD_TEST(mutable_dispatch_image_2d_arguments),
     ADD_TEST(mutable_dispatch_out_of_order),

diff --git a/...ions/cl_khr_command_buffer/cl_khr_command_buffer_mutable_dispatch/mutable_command_basic.h b/...ions/cl_khr_command_buffer/cl_khr_command_buffer_mutable_dispatch/mutable_command_basic.h
@@ -26,6 +26,28 @@ struct BasicMutableCommandBufferTest : BasicCommandBufferTest
         : BasicCommandBufferTest(device, context, queue)
     {}
 
+    virtual cl_int SetUpKernel() override
+    {
+        cl_int error = CL_SUCCESS;
+        clProgramWrapper program = clCreateProgramWithSource(
+            context, 1, &kernelString, nullptr, &error);
+        test_error(error, "Unable to create program");
+
+        error = clBuildProgram(program, 1, &device, nullptr, nullptr, nullptr);
+        test_error(error, "Unable to build program");
+
+        kernel = clCreateKernel(program, "empty", &error);
+        test_error(error, "Unable to create kernel");
+
+        return CL_SUCCESS;
+    }
+
+    virtual cl_int SetUpKernelArgs() override
+    {
+        /* Left blank intentionally */
+        return CL_SUCCESS;
+    }
+
     virtual cl_int SetUp(int elements) override
     {
         BasicCommandBufferTest::SetUp(elements);
@@ -42,16 +64,6 @@ struct BasicMutableCommandBufferTest : BasicCommandBufferTest
         command_buffer = clCreateCommandBufferKHR(1, &queue, props, &error);
         test_error(error, "Unable to create command buffer");
 
-        clProgramWrapper program = clCreateProgramWithSource(
-            context, 1, &kernelString, nullptr, &error);
-        test_error(error, "Unable to create program");
-
-        error = clBuildProgram(program, 1, &device, nullptr, nullptr, nullptr);
-        test_error(error, "Unable to build program");
-
-        kernel = clCreateKernel(program, "empty", &error);
-        test_error(error, "Unable to create kernel");
-
         return error;
     }