[SYCL][L0] address Coverity issue with active_barrier::clear (#8084)

Signed-off-by: Sergey V Maslov <[email protected]>

Author: smaslov-intel

sycl/plugins/level_zero/pi_level_zero.cpp

@@ -2017,7 +2017,8 @@ pi_result _pi_queue::insertActiveBarriers(pi_command_list_ptr_t &CmdList,
     return Res;
 
   // We can now replace active barriers with the ones in the wait list.
-  ActiveBarriers.clear();
+  if (auto Res = ActiveBarriers.clear())
+    return Res;
 
   if (ActiveBarriersWaitList.Length == 0) {
     return PI_SUCCESS;
@@ -6215,10 +6216,11 @@ void _pi_queue::active_barriers::add(pi_event &Event) {
   Events.push_back(Event);
 }
 
-void _pi_queue::active_barriers::clear() {
+pi_result _pi_queue::active_barriers::clear() {
   for (const auto &Event : Events)
-    piEventReleaseInternal(Event);
+    PI_CALL(piEventReleaseInternal(Event));
   Events.clear();
+  return PI_SUCCESS;
 }
 
 static pi_result piEventReleaseInternal(pi_event Event) {
@@ -6751,7 +6753,8 @@ pi_result piEnqueueEventsWaitWithBarrier(pi_queue Queue,
     if (auto Res = Queue->executeCommandList(CmdList, false, OkToBatch))
       return Res;
 
-  Queue->ActiveBarriers.clear();
+  if (auto Res = Queue->ActiveBarriers.clear())
+    return Res;
   Queue->ActiveBarriers.add(*Event);
   return PI_SUCCESS;
 }
@@ -6859,7 +6862,9 @@ pi_result _pi_queue::synchronize() {
 
   // With the entire queue synchronized, the active barriers must be done so we
   // can remove them.
-  ActiveBarriers.clear();
+  if (auto Res = ActiveBarriers.clear())
+    return Res;
+
   return PI_SUCCESS;
 }
 

sycl/plugins/level_zero/pi_level_zero.hpp

@@ -805,7 +805,7 @@ struct _pi_queue : _pi_object {
   struct active_barriers {
     std::vector<pi_event> Events;
     void add(pi_event &Event);
-    void clear();
+    pi_result clear();
     bool empty() { return Events.empty(); }
     std::vector<pi_event> &vector() { return Events; }
   };