[SYCL][Graph] Remove assume_data_outlives_buffer property

Implements spec feedback to remove this property, goes with
spec PR #311

Author: Ewan Crawford

sycl/include/sycl/ext/oneapi/experimental/graph.hpp

@@ -117,19 +117,6 @@ class assume_buffer_outlives_graph
 public:
   assume_buffer_outlives_graph() = default;
 };
-
-/// Property passed to command_graph constructor to allow buffers created with
-/// host pointers. Passing this property represents a promise from the user that
-/// the host data will outlive the buffer and by extension any graph that it is
-/// used in.
-///
-class assume_data_outlives_buffer
-    : public ::sycl::detail::DataLessProperty<
-          ::sycl::detail::GraphAssumeDataOutlivesBuffer> {
-public:
-  assume_data_outlives_buffer() = default;
-};
-
 } // namespace graph
 
 namespace node {

sycl/source/detail/graph_impl.cpp

@@ -271,13 +271,6 @@ graph_impl::add(sycl::detail::CG::CGTYPE CGType,
   for (auto &Req : Requirements) {
     // Track and mark the memory objects being used by the graph.
     auto MemObj = static_cast<sycl::detail::SYCLMemObjT *>(Req->MSYCLMemObj);
-    if (MemObj->getUserPtr() && !MAllowBuffersHostPointers) {
-      throw sycl::exception(
-          make_error_code(errc::invalid),
-          "Cannot use a buffer which was created with a host pointer in a "
-          "graph without passing the assume_data_outlives_buffer property on "
-          "Graph construction.");
-    }
     bool WasInserted = MMemObjs.insert(MemObj).second;
     if (WasInserted) {
       MemObj->markBeingUsedInGraph();

sycl/source/detail/graph_impl.hpp

@@ -305,9 +305,6 @@ class graph_impl {
     if (PropList.has_property<property::graph::no_cycle_check>()) {
       MSkipCycleChecks = true;
     }
-    if (PropList.has_property<property::graph::assume_data_outlives_buffer>()) {
-      MAllowBuffersHostPointers = true;
-    }
     if (PropList
             .has_property<property::graph::assume_buffer_outlives_graph>()) {
       MAllowBuffers = true;
@@ -615,11 +612,6 @@ class graph_impl {
   /// Unique set of SYCL Memory Objects which are currently in use in the graph.
   std::set<sycl::detail::SYCLMemObjT *> MMemObjs;
 
-  /// Controls whether we allow buffers that are created with host pointers to
-  /// be used in the graph. Set by the presence of the
-  /// assume_data_outlives_buffer property.
-  bool MAllowBuffersHostPointers = false;
-
   /// Controls whether we allow buffers to be used in the graph. Set by the
   /// presence of the assume_buffer_outlives_graph property.
   bool MAllowBuffers = false;

sycl/test-e2e/Graph/Explicit/assume_data_outlives_buffer_property.cpp

@@ -28,8 +28,7 @@ int main() {
       exp_ext::command_graph Graph{
           Queue.get_context(),
           Queue.get_device(),
-          {exp_ext::property::graph::assume_buffer_outlives_graph{},
-           exp_ext::property::graph::assume_data_outlives_buffer{}}};
+          {exp_ext::property::graph::assume_buffer_outlives_graph{}}};
 
       // Add commands to graph
       add_nodes(Graph, Queue, Size, BufferA, BufferB, BufferC);

sycl/test-e2e/Graph/Inputs/assume_data_outlives_buffer_property.cpp

@@ -37,8 +37,7 @@ int main() {
     exp_ext::command_graph Graph{
         Queue.get_context(),
         Queue.get_device(),
-        {exp_ext::property::graph::assume_buffer_outlives_graph{},
-         exp_ext::property::graph::assume_data_outlives_buffer{}}};
+        {exp_ext::property::graph::assume_buffer_outlives_graph{}}};
 
     // Copy from B to A
     auto NodeA = add_node(Graph, Queue, [&](handler &CGH) {

sycl/test-e2e/Graph/Inputs/basic_buffer.cpp

@@ -38,8 +38,7 @@ int main() {
     exp_ext::command_graph Graph{
         Queue.get_context(),
         Queue.get_device(),
-        {exp_ext::property::graph::assume_buffer_outlives_graph{},
-         exp_ext::property::graph::assume_data_outlives_buffer{}}};
+        {exp_ext::property::graph::assume_buffer_outlives_graph{}}};
 
     // Copy from B to A
     auto NodeA = add_node(Graph, Queue, [&](handler &CGH) {

sycl/test-e2e/Graph/Inputs/buffer_copy.cpp

@@ -24,8 +24,7 @@ int main() {
     exp_ext::command_graph Graph{
         Queue.get_context(),
         Queue.get_device(),
-        {exp_ext::property::graph::assume_buffer_outlives_graph{},
-         exp_ext::property::graph::assume_data_outlives_buffer{}}};
+        {exp_ext::property::graph::assume_buffer_outlives_graph{}}};
 
     auto NodeA = add_node(Graph, Queue, [&](handler &CGH) {
       auto AccA = BufferA.get_access(CGH);

sycl/test-e2e/Graph/Inputs/buffer_copy_2d.cpp

@@ -25,8 +25,7 @@ int main() {
     exp_ext::command_graph Graph{
         Queue.get_context(),
         Queue.get_device(),
-        {exp_ext::property::graph::assume_buffer_outlives_graph{},
-         exp_ext::property::graph::assume_data_outlives_buffer{}}};
+        {exp_ext::property::graph::assume_buffer_outlives_graph{}}};
 
     auto NodeA = add_node(Graph, Queue, [&](handler &CGH) {
       auto AccA = BufferA.get_access<access::mode::write>(CGH);