Remove calls of handler::addAccessorReq.

Author: Alexandr-Konovalov

sycl/include/sycl/handler.hpp

@@ -618,7 +618,6 @@ class __SYCL_EXPORT handler {
     detail::AccessorBaseHost *AccBase = (detail::AccessorBaseHost *)&Arg;
     detail::AccessorImplPtr AccImpl = detail::getSyclObjImpl(*AccBase);
     detail::AccessorImplHost *Req = AccImpl.get();
-    addAccessorReq(std::move(AccImpl));
     // Add accessor to the list of arguments.
     addArg(detail::kernel_param_kind_t::kind_accessor, Req,
            static_cast<int>(AccessTarget), ArgIndex);
@@ -2580,9 +2579,6 @@ class __SYCL_EXPORT handler {
 
     MSrcPtr = static_cast<void *>(AccImpl.get());
     MDstPtr = static_cast<void *>(Dst);
-    // Store copy of accessor to the local storage to make sure it is alive
-    // until we finish
-    addAccessorReq(std::move(AccImpl));
   }
 
   /// Copies the content of memory pointed by Src into the memory object
@@ -2618,9 +2614,6 @@ class __SYCL_EXPORT handler {
 
     MSrcPtr = const_cast<T_Src *>(Src);
     MDstPtr = static_cast<void *>(AccImpl.get());
-    // Store copy of accessor to the local storage to make sure it is alive
-    // until we finish
-    addAccessorReq(std::move(AccImpl));
   }
 
   /// Copies the content of memory object accessed by Src to the memory
@@ -2675,10 +2668,6 @@ class __SYCL_EXPORT handler {
 
     MSrcPtr = AccImplSrc.get();
     MDstPtr = AccImplDst.get();
-    // Store copy of accessor to the local storage to make sure it is alive
-    // until we finish
-    addAccessorReq(std::move(AccImplSrc));
-    addAccessorReq(std::move(AccImplDst));
   }
 
   /// Provides guarantees that the memory object accessed via Acc is updated
@@ -2704,7 +2693,6 @@ class __SYCL_EXPORT handler {
     detail::AccessorImplPtr AccImpl = detail::getSyclObjImpl(*AccBase);
 
     MDstPtr = static_cast<void *>(AccImpl.get());
-    addAccessorReq(std::move(AccImpl));
   }
 
 public:
@@ -3459,7 +3447,6 @@ class __SYCL_EXPORT handler {
     detail::AccessorImplPtr AccImpl = detail::getSyclObjImpl(*AccBase);
 
     MDstPtr = static_cast<void *>(AccImpl.get());
-    addAccessorReq(std::move(AccImpl));
 
     MPattern.resize(sizeof(T));
     auto PatternPtr = reinterpret_cast<T *>(MPattern.data());

sycl/source/handler.cpp

@@ -2055,21 +2055,8 @@ void handler::SetHostTask(std::function<void(interop_handle)> &&Func) {
   setType(detail::CGType::CodeplayHostTask);
 }
 
-void handler::addAccessorReq(detail::AccessorImplPtr Accessor) {
-#if 0
-  // Constructor of accessors add them to MRequirements and MAccStorage
-  // of the associated handler, so do not add duplicates if use same
-  // handler as during construction.
-  if (impl->CGData.MRequirements.end() !=
-      std::find(impl->CGData.MRequirements.begin(),
-                impl->CGData.MRequirements.end(), Accessor.get()))
-    return;
-
-  // Add accessor to the list of requirements.
-  impl->CGData.MRequirements.push_back(Accessor.get());
-  // Store copy of the accessor.
-  impl->CGData.MAccStorage.push_back(std::move(Accessor));
-#endif
+void handler::addAccessorReq(detail::AccessorImplPtr) {
+    assert(false && "The function must not be used.");
 }
 
 void handler::addLifetimeSharedPtrStorage(std::shared_ptr<const void> SPtr) {