Change notion of macro so that it allows for both enabling and disabling of fallback assert

Signed-off-by: Sergey Kanaev <[email protected]>

Author: Sergey Kanaev

sycl/doc/PreprocessorMacros.md

@@ -38,9 +38,9 @@ This file describes macros that have effect on SYCL compiler and run-time.
 
   Disables a message which warns about unsupported C++ version.
 
-- **SYCL_ENABLE_FALLBACK_ASSERT**
+- **SYCL_FALLBACK_ASSERT**
 
-  Defining this macro enables the fallback assert feature even on devices
+  Defining as non-zero enables the fallback assert feature even on devices
   without native support. Be aware that this will add some overhead that is
   associated with submitting kernels that call `assert()`. When this macro is
   not defined, the logic for detecting assertion failures in kernels is
@@ -50,6 +50,7 @@ This file describes macros that have effect on SYCL compiler and run-time.
   these devices regardless of whether this macro is defined because that logic
   does not add any extra overhead. One can check to see if a device has native
   support for `assert()` via `aspect::ext_oneapi_native_assert`.
+  Default value is 0.
 
 ## Version macros
 

sycl/include/CL/sycl/queue.hpp

@@ -67,8 +67,8 @@
 
 // Helper macro to identify if fallback assert is needed
 // FIXME remove __NVPTX__ condition once devicelib supports CUDA
-#if defined(SYCL_ENABLE_FALLBACK_ASSERT)
-#define __SYCL_USE_FALLBACK_ASSERT 1
+#if defined(SYCL_FALLBACK_ASSERT)
+#define __SYCL_USE_FALLBACK_ASSERT SYCL_FALLBACK_ASSERT
 #else
 #define __SYCL_USE_FALLBACK_ASSERT 0
 #endif

sycl/unittests/assert/assert.cpp

@@ -18,7 +18,7 @@
  * pipe.
  */
 
-#define SYCL_ENABLE_FALLBACK_ASSERT 1
+#define SYCL_FALLBACK_ASSERT 1
 // Enable use of interop kernel c-tor
 #define __SYCL_INTERNAL_API
 #include <CL/sycl.hpp>