Skip to content

Commit

Permalink
remove unneeded build in satisfy_data_dependencies (#561)
Browse files Browse the repository at this point in the history
building the command in satisfy_data_dependencies is not needed. The
command is enqueue just after. During the enqueue the command will be
either built as a batchable command (in
cvk_command_queue::enqueue_command), or built when it will be added to
a batch (in cvk_command_batch::add_command).
  • Loading branch information
rjodinchr authored Jul 1, 2023
1 parent 98108ec commit a80c62b
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions src/queue.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -94,9 +94,6 @@ cl_int cvk_command_queue::satisfy_data_dependencies(cvk_command* cmd) {
CVK_ASSERT(mem->is_image_type());
auto initcmd =
new cvk_command_image_init(this, static_cast<cvk_image*>(mem));
if (initcmd->build() != CL_SUCCESS) {
return CL_OUT_OF_RESOURCES;
}
_cl_event* initev;
cl_int err = enqueue_command(initcmd, &initev);
if (err != CL_SUCCESS) {
Expand Down

0 comments on commit a80c62b

Please sign in to comment.