[NFC][SYCL] Reduce dependencies of sycl/detail/helpers.hpp

Author: aelovikov-intel

sycl/include/sycl/detail/helpers.hpp

@@ -18,11 +18,9 @@
 #endif
 
 #include <cstddef>     // for size_t
-#include <memory>      // for shared_ptr
 #include <stdint.h>    // for uint32_t
 #include <type_traits> // for enable_if_t, integral_constant
 #include <utility>     // for forward, integer_sequence, mak...
-#include <vector>      // for vector
 
 namespace sycl {
 inline namespace _V1 {
@@ -38,11 +36,6 @@ template <typename Type, std::size_t NumElements> class marray;
 enum class memory_order;
 
 namespace detail {
-class buffer_impl;
-
-__SYCL_EXPORT void
-markBufferAsInternal(const std::shared_ptr<buffer_impl> &BufImpl);
-
 template <typename T> T *declptr() { return static_cast<T *>(nullptr); }
 
 // Function to get or store id, item, nd_item, group for the host implementation

sycl/include/sycl/handler.hpp

@@ -166,6 +166,10 @@ class graph_impl;
 class dynamic_parameter_impl;
 } // namespace ext::oneapi::experimental::detail
 namespace detail {
+class buffer_impl;
+
+__SYCL_EXPORT void
+markBufferAsInternal(const std::shared_ptr<buffer_impl> &BufImpl);
 
 class type_erased_cgfo_ty {
   // From SYCL 2020,  command group function object:

sycl/source/detail/helpers.cpp

@@ -34,11 +34,6 @@ __SYCL_EXPORT void waitEvents(std::vector<sycl::event> DepEvents) {
 }
 #endif
 
-__SYCL_EXPORT void
-markBufferAsInternal(const std::shared_ptr<buffer_impl> &BufImpl) {
-  BufImpl->markAsInternal();
-}
-
 const RTDeviceBinaryImage *retrieveKernelBinary(queue_impl &Queue,
                                                 KernelNameStrRefT KernelName,
                                                 CGExecKernel *KernelCG) {

sycl/source/handler.cpp

@@ -10,6 +10,7 @@
 #include "ur_api.h"
 #include <algorithm>
 
+#include <detail/buffer_impl.hpp>
 #include <detail/config.hpp>
 #include <detail/global_handler.hpp>
 #include <detail/graph/dynamic_impl.hpp>
@@ -49,6 +50,10 @@ namespace sycl {
 inline namespace _V1 {
 
 namespace detail {
+__SYCL_EXPORT void
+markBufferAsInternal(const std::shared_ptr<buffer_impl> &BufImpl) {
+  BufImpl->markAsInternal();
+}
 
 #ifdef __INTEL_PREVIEW_BREAKING_CHANGES
 // TODO: Check if two ABI exports below are still necessary.