Corrections due to code review

shajder · Mar 25, 2024 · fbf6ac0 · fbf6ac0
1 parent f91aafa
commit fbf6ac0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 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
@@ -11,7 +11,7 @@ set(${MODULE_NAME}_SOURCES
     mutable_command_global_offset.cpp
     mutable_command_full_dispatch.cpp
     mutable_command_multiple_dispatches.cpp
-	mutable_command_iterative_arg_update.cpp
+    mutable_command_iterative_arg_update.cpp
     ../basic_command_buffer.cpp
 )
 

diff --git a/...nd_buffer/cl_khr_command_buffer_mutable_dispatch/mutable_command_iterative_arg_update.cpp b/...nd_buffer/cl_khr_command_buffer_mutable_dispatch/mutable_command_iterative_arg_update.cpp
@@ -25,7 +25,9 @@
 namespace {
 
 ////////////////////////////////////////////////////////////////////////////////
-// command buffer with iterative update of mutable parameters test
+// Test clUpdateMutableCommandsKHR() being called twice on the same command
+// before an enqueue, but with different arguments. Verifies that the combined
+// updates are made correctly.
 
 struct IterativeArgUpdateDispatch : BasicMutableCommandBufferTest
 {
@@ -70,7 +72,7 @@ struct IterativeArgUpdateDispatch : BasicMutableCommandBufferTest
         test_error(error, "Failed to build program");
 
         kernel = clCreateKernel(program, "fill", &error);
-        test_error(error, "Failed to create copy kernel");
+        test_error(error, "Failed to create fill kernel");
 
         return CL_SUCCESS;
     }