Merge pull request #2609 from kbenzie/benie/fix-large-obj-noref-arg

Fix passing struct object by value
oneapi-src · Jan 24, 2025 · e370a2b · e370a2b
2 parents 2f5f4bb + f71ef62
commit e370a2b
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/test/adapters/level_zero/event_cache_tests.cpp b/test/adapters/level_zero/event_cache_tests.cpp
@@ -30,7 +30,7 @@ static std::shared_ptr<_zel_tracer_handle_t> tracer = [] {
   zel_core_callbacks_t prologue_callbacks{};
   prologue_callbacks.Event.pfnCreateCb = OnEnterEventCreate;
   prologue_callbacks.Event.pfnDestroyCb = OnEnterEventDestroy;
-  return enableTracing(prologue_callbacks, {});
+  return enableTracing(std::move(prologue_callbacks), {});
 }();
 
 template <typename... Args> auto combineFlags(std::tuple<Args...> tuple) {

diff --git a/test/adapters/level_zero/multi_device_event_cache_tests.cpp b/test/adapters/level_zero/multi_device_event_cache_tests.cpp
@@ -20,7 +20,7 @@ static std::shared_ptr<_zel_tracer_handle_t> tracer = [] {
   zel_core_callbacks_t prologue_callbacks{};
   prologue_callbacks.CommandList.pfnAppendWaitOnEventsCb =
       OnAppendWaitOnEventsCb;
-  return enableTracing(prologue_callbacks, {});
+  return enableTracing(std::move(prologue_callbacks), {});
 }();
 
 using urMultiQueueMultiDeviceEventCacheTest = uur::urAllDevicesTest;

diff --git a/test/adapters/level_zero/ze_tracer_common.hpp b/test/adapters/level_zero/ze_tracer_common.hpp
@@ -11,9 +11,9 @@
 
 #include <memory>
 
-std::shared_ptr<_zel_tracer_handle_t>
-enableTracing(zel_core_callbacks_t prologueCallbacks,
-              zel_core_callbacks_t epilogueCallbacks) {
+inline std::shared_ptr<_zel_tracer_handle_t>
+enableTracing(zel_core_callbacks_t &&prologueCallbacks,
+              zel_core_callbacks_t &&epilogueCallbacks) {
   EXPECT_EQ(zeInit(ZE_INIT_FLAG_GPU_ONLY), ZE_RESULT_SUCCESS);
 
   zel_tracer_desc_t tracer_desc = {ZEL_STRUCTURE_TYPE_TRACER_EXP_DESC, nullptr,