Do not add computation event to host_task_lists

oleksandr-pavlyk · oleksandr-pavlyk · commit d34de92bf6e4 · 2024-08-24T10:33:21.000-05:00
Fix two additional instances pointed out by @ndgrigorian during PR review.
diff --git a/dpctl/tensor/libtensor/source/boolean_advanced_indexing.cpp b/dpctl/tensor/libtensor/source/boolean_advanced_indexing.cpp
@@ -370,8 +370,6 @@ py_extract(const dpctl::tensor::usm_ndarray &src,
         host_task_events.push_back(cleanup_tmp_allocations_ev);
     }
 
-    host_task_events.push_back(extract_ev);
-
     sycl::event py_obj_management_host_task_ev = dpctl::utils::keep_args_alive(
         exec_q, {src, cumsum, dst}, host_task_events);
 
diff --git a/dpctl/tensor/libtensor/source/repeat.cpp b/dpctl/tensor/libtensor/source/repeat.cpp
@@ -356,8 +356,6 @@ py_repeat_by_sequence(const dpctl::tensor::usm_ndarray &src,
         host_task_events.push_back(cleanup_tmp_allocations_ev);
     }
 
-    host_task_events.push_back(repeat_ev);
-
     sycl::event py_obj_management_host_task_ev = dpctl::utils::keep_args_alive(
         exec_q, {src, reps, cumsum, dst}, host_task_events);
 

Original file line number	Diff line number	Diff line change
`@@ -370,8 +370,6 @@ py_extract(const dpctl::tensor::usm_ndarray &src,`
`370`	`370`	`host_task_events.push_back(cleanup_tmp_allocations_ev);`
`371`	`371`	`}`
`372`	`372`
`373`		`- host_task_events.push_back(extract_ev);`
`374`		`-`
`375`	`373`	`sycl::event py_obj_management_host_task_ev = dpctl::utils::keep_args_alive(`
`376`	`374`	`exec_q, {src, cumsum, dst}, host_task_events);`
`377`	`375`
Original file line number	Diff line number	Diff line change
`@@ -356,8 +356,6 @@ py_repeat_by_sequence(const dpctl::tensor::usm_ndarray &src,`
`356`	`356`	`host_task_events.push_back(cleanup_tmp_allocations_ev);`
`357`	`357`	`}`
`358`	`358`
`359`		`- host_task_events.push_back(repeat_ev);`
`360`		`-`
`361`	`359`	`sycl::event py_obj_management_host_task_ev = dpctl::utils::keep_args_alive(`
`362`	`360`	`exec_q, {src, reps, cumsum, dst}, host_task_events);`
`363`	`361`