From 36dd80862175f4ad10f2eff36e68b9c57ea9b6d2 Mon Sep 17 00:00:00 2001 From: "Larsen, Steffen" Date: Mon, 12 May 2025 07:23:50 -0700 Subject: [PATCH 1/2] [SYCL][Docs] Move sycl_ext_intel_event_mode to experimental This commit moves the sycl_ext_intel_event_mode extension to the experimental namespace and defines the corresponding feature test macro. Signed-off-by: Larsen, Steffen --- .../sycl_ext_intel_event_mode.asciidoc | 14 +++++++++----- sycl/source/feature_test.hpp.in | 1 + 2 files changed, 10 insertions(+), 5 deletions(-) rename sycl/doc/extensions/{proposed => experimental}/sycl_ext_intel_event_mode.asciidoc (90%) diff --git a/sycl/doc/extensions/proposed/sycl_ext_intel_event_mode.asciidoc b/sycl/doc/extensions/experimental/sycl_ext_intel_event_mode.asciidoc similarity index 90% rename from sycl/doc/extensions/proposed/sycl_ext_intel_event_mode.asciidoc rename to sycl/doc/extensions/experimental/sycl_ext_intel_event_mode.asciidoc index 50d1e732329c8..67cb660b802ef 100644 --- a/sycl/doc/extensions/proposed/sycl_ext_intel_event_mode.asciidoc +++ b/sycl/doc/extensions/experimental/sycl_ext_intel_event_mode.asciidoc @@ -52,11 +52,12 @@ This extension also depends on the following other SYCL extensions: == Status -This is a proposed extension specification, intended to gather community -feedback. Interfaces defined in this specification may not be implemented yet -or may be in a preliminary state. The specification itself may also change in -incompatible ways before it is finalized. *Shipping software products should -not rely on APIs defined in this specification.* +This is an experimental extension specification, intended to provide early +access to features and gather community feedback. Interfaces defined in this +specification are implemented in {dpcpp}, but they are not finalized and may +change incompatibly in future versions of {dpcpp} without prior notice. +*Shipping software products should not rely on APIs defined in this +specification.* == Overview @@ -72,6 +73,9 @@ only wake up occasionally to check if the event has finished. This reduces the time the CPU spends checking finish condition of the wait, at the cost of latency. +This extension exists to solve a specific problem, and a general solution is +still being evaluated. It is not recommended for general usage. + == Specification diff --git a/sycl/source/feature_test.hpp.in b/sycl/source/feature_test.hpp.in index cccc8d462be50..64e40cba50a74 100644 --- a/sycl/source/feature_test.hpp.in +++ b/sycl/source/feature_test.hpp.in @@ -120,6 +120,7 @@ inline namespace _V1 { // In progress yet #define SYCL_EXT_ONEAPI_ATOMIC16 0 #define SYCL_KHR_DEFAULT_CONTEXT 1 +#define SYCL_EXT_INTEL_EVENT_MODE 1 #ifndef __has_include #define __has_include(x) 0 From 11d7a950647b8b9312add4571cefb9f29913fe75 Mon Sep 17 00:00:00 2001 From: "Larsen, Steffen" Date: Mon, 12 May 2025 21:31:11 -0700 Subject: [PATCH 2/2] Remove avoidance statement Signed-off-by: Larsen, Steffen --- .../extensions/experimental/sycl_ext_intel_event_mode.asciidoc | 3 --- 1 file changed, 3 deletions(-) diff --git a/sycl/doc/extensions/experimental/sycl_ext_intel_event_mode.asciidoc b/sycl/doc/extensions/experimental/sycl_ext_intel_event_mode.asciidoc index 67cb660b802ef..d3f3ca4c31aff 100644 --- a/sycl/doc/extensions/experimental/sycl_ext_intel_event_mode.asciidoc +++ b/sycl/doc/extensions/experimental/sycl_ext_intel_event_mode.asciidoc @@ -73,9 +73,6 @@ only wake up occasionally to check if the event has finished. This reduces the time the CPU spends checking finish condition of the wait, at the cost of latency. -This extension exists to solve a specific problem, and a general solution is -still being evaluated. It is not recommended for general usage. - == Specification