[SYCL][XPTI] Fix compilation issue when building SYCL without XPTI (#5289)

Also the XPTIRegistry::createTraceEvent static method has been moved into
the private part of the class since it is used in the class only.

Closes #5274

Author: Pavel Samolysov

sycl/source/detail/xpti_registry.cpp

@@ -16,6 +16,7 @@
 __SYCL_INLINE_NAMESPACE(cl) {
 namespace sycl {
 namespace detail {
+#ifdef XPTI_ENABLE_INSTRUMENTATION
 xpti::trace_event_data_t *
 XPTIRegistry::createTraceEvent(void *Obj, const char *ObjName, uint64_t &IId,
                                const detail::code_location &CodeLoc,
@@ -39,6 +40,8 @@ XPTIRegistry::createTraceEvent(void *Obj, const char *ObjName, uint64_t &IId,
   return xptiMakeEvent(Name.c_str(), &Payload, TraceEventType, xpti_at::active,
                        &IId);
 }
+#endif // XPTI_ENABLE_INSTRUMENTATION
+
 void XPTIRegistry::bufferConstructorNotification(
     void *UserObj, const detail::code_location &CodeLoc) {
   (void)UserObj;

sycl/source/detail/xpti_registry.hpp

@@ -98,14 +98,17 @@ class XPTIRegistry {
   static void bufferAccessorNotification(void *UserObj, void *AccessorObj,
                                          uint32_t Target, uint32_t Mode,
                                          const detail::code_location &CodeLoc);
-  static xpti::trace_event_data_t *
-  createTraceEvent(void *Obj, const char *ObjName, uint64_t &IId,
-                   const detail::code_location &CodeLoc,
-                   uint16_t TraceEventType);
 
 private:
   std::unordered_set<std::string> MActiveStreams;
   std::once_flag MInitialized;
+
+#ifdef XPTI_ENABLE_INSTRUMENTATION
+  static xpti::trace_event_data_t *
+  createTraceEvent(void *Obj, const char *ObjName, uint64_t &IId,
+                   const detail::code_location &CodeLoc,
+                   uint16_t TraceEventType);
+#endif // XPTI_ENABLE_INSTRUMENTATION
 };
 } // namespace detail
 } // namespace sycl