[SYCL] Follow-up for intel#19990

Downstream CI uncovered another scenario where
`detail::CompileTimeKernelInfo<KernelName>.Name` might be empty (other
than non-SYCL compilation in some of the [unit]tests). That happens when
compiling the same TU using multiple offload models (e.g., both SYCL and
OMP) and the device compilation for non-SYCL model doesn't use SYCL
integration header to provide kernel information.

Ideally, we should be just guarding `static_assert`s with some `#if
__SYCL_WHATEVER` but we don't set any SYCL-related macro when using 3rd
party host compilers. Deal with that by turning compile-time
`static_assert` into a run-time `assert` unless we can guarantee that
SYCL kernel information is available.

Author: aelovikov-intel

sycl/include/sycl/handler.hpp

@@ -836,26 +836,22 @@ class __SYCL_EXPORT handler {
                                               Dims>());
 #endif
 
-    // SYCL unittests are built without sycl compiler, so "host" information
-    // about kernels isn't provided (e.g., via integration headers or compiler
-    // builtins).
-    //
-    // However, some copy/fill USM operation are implemented via SYCL kernels
-    // and are instantiated resulting in all the `static_assert` checks being
-    // exercised. Without kernel information that would fail, so we explicitly
-    // disable such checks when this macro is defined. Note that the unittests
-    // don't actually execute those operation, that's why disabling
-    // unconditional `static_asserts`s is enough for now.
-#ifndef __SYCL_UNITTESTS_BYPASS_KERNEL_NAME_CHECK
     constexpr auto Info = detail::CompileTimeKernelInfo<KernelName>;
 
+#ifdef SYCL_LANGUAGE_VERSION
     static_assert(Info.Name != std::string_view{}, "Kernel must have a name!");
+#else
+    // Either non-SYCL compilation (e.g., OpenMP device code generation) or
+    // using custom host compiler.
+    assert(Info.Name != std::string_view{}, "Kernel must have a name!");
+#endif
 
     // Some host compilers may have different captures from Clang. Currently
     // there is no stable way of handling this when extracting the captures,
     // so a static assert is made to fail for incompatible kernel lambdas.
     static_assert(
-        sizeof(KernelType) == Info.KernelSize,
+        Info.Name == std::string_view{} ||
+            sizeof(KernelType) == Info.KernelSize,
         "Unexpected kernel lambda size. This can be caused by an "
         "external host compiler producing a lambda with an "
         "unexpected layout. This is a limitation of the compiler."
@@ -866,7 +862,6 @@ class __SYCL_EXPORT handler {
         "In case of MSVC, passing "
         "-fsycl-host-compiler-options='/std:c++latest' "
         "might also help.");
-#endif
 
     setDeviceKernelInfo<KernelName>((void *)MHostKernel->getPtr());
 

sycl/test/lit.cfg.py

@@ -127,7 +127,7 @@
 llvm_config.with_environment("LLVM_SYMBOLIZER_PATH", llvm_symbolizer)
 
 sycl_host_only_options = (
-    "-std=c++17 -Xclang -fsycl-is-host -D__SYCL_UNITTESTS_BYPASS_KERNEL_NAME_CHECK=1"
+    "-std=c++17 -Xclang -fsycl-is-host"
 )
 for include_dir in [
     config.sycl_include,

sycl/unittests/CMakeLists.txt

@@ -9,8 +9,6 @@ endforeach()
 
 add_compile_definitions(SYCL2020_DISABLE_DEPRECATION_WARNINGS SYCL_DISABLE_FSYCL_SYCLHPP_WARNING)
 
-add_compile_definitions(__SYCL_UNITTESTS_BYPASS_KERNEL_NAME_CHECK)
-
 # suppress warnings which came from Google Test sources
 if (CXX_SUPPORTS_SUGGEST_OVERRIDE_FLAG)
   add_compile_options("-Wno-suggest-override")