Make events related logic inline

Author: slawekptak

sycl/source/detail/queue_impl.cpp

@@ -461,12 +461,44 @@ detail::EventImplPtr queue_impl::submit_kernel_direct_impl(
     std::vector<detail::ArgDesc> Args;
     bool DiscardEvent = !CallerNeedsEvent && supportsDiscardingPiEvents();
 
-    bool SchedulerBypass = detail::Scheduler::areEventsSafeForSchedulerBypass(
-        CGData.MEvents, getContextImpl());
+    bool SchedulerBypass = std::all_of(CGData.MEvents.begin(), CGData.MEvents.end(),
+      [&](EventImplPtr &Event) {
+      // Events that don't have an initialized context are throwaway events that
+      // don't represent actual dependencies. Calling getContextImpl() would set
+      // their context, which we wish to avoid as it is expensive.
+      // NOP events also don't represent actual dependencies.
+      if (Event->isDefaultConstructed() || Event->isNOP())
+        return true;
+
+      if (Event->isHost())
+        return Event->isCompleted();
+
+      // Cross-context dependencies can't be passed to the backend directly.
+      if (&Event->getContextImpl() != &getContextImpl())
+        return false;
+
+      // A nullptr here means that the commmand does not produce a UR event or it
+      // hasn't been enqueued yet.
+      return Event->getHandle() != nullptr;
+    });
 
     if (SchedulerBypass) {
-      std::vector<ur_event_handle_t> RawEvents =
-          detail::Command::getUrEvents(CGData.MEvents, this, false);
+      std::vector<ur_event_handle_t> RawEvents;
+
+      for (EventImplPtr &Event : CGData.MEvents) {
+        auto Handle = Event->getHandle();
+        if (Handle == nullptr)
+          continue;
+
+        // Do not add redundant event dependencies for in-order queues.
+        // At this stage dependency is definitely ur task and need to check if
+        // current one is a host task. In this case we should not skip ur event due
+        // to different sync mechanisms for different task types on in-order queue.
+        if (Event->getWorkerQueue().get() == this && isInOrder())
+          continue;
+
+        RawEvents.push_back(Handle);
+      }
 
       std::shared_ptr<detail::event_impl> ResultEvent =
           DiscardEvent ? nullptr