diff --git a/sycl/source/detail/scheduler/commands.cpp b/sycl/source/detail/scheduler/commands.cpp index 10f51c6840de5..c85608faa2dd1 100644 --- a/sycl/source/detail/scheduler/commands.cpp +++ b/sycl/source/detail/scheduler/commands.cpp @@ -1321,7 +1321,7 @@ ur_result_t ReleaseCommand::enqueueImp() { UnmapEventImpl->setHandle(UREvent); std::swap(MAllocaCmd->MIsActive, MAllocaCmd->MLinkedAllocaCmd->MIsActive); EventImpls.clear(); - EventImpls.push_back(UnmapEventImpl); + EventImpls.push_back(std::move(UnmapEventImpl)); } ur_event_handle_t UREvent = nullptr; if (SkipRelease) diff --git a/sycl/source/detail/split_string.hpp b/sycl/source/detail/split_string.hpp index 557c35229c054..a24ab2a978647 100644 --- a/sycl/source/detail/split_string.hpp +++ b/sycl/source/detail/split_string.hpp @@ -31,8 +31,9 @@ inline std::vector split_string(std::string_view str, std::string LastSubStr(str.substr(Start, End - Start)); // In case str has a delimeter at the end, the substring will be empty, so // we shouldn't add it to the final vector - if (!LastSubStr.empty()) - Result.push_back(LastSubStr); + if (!LastSubStr.empty()) { + Result.push_back(std::move(LastSubStr)); + } } return Result; } diff --git a/sycl/source/handler.cpp b/sycl/source/handler.cpp index 5612d4173a7d5..2e00cea153ccc 100644 --- a/sycl/source/handler.cpp +++ b/sycl/source/handler.cpp @@ -899,7 +899,7 @@ void handler::ext_oneapi_barrier(const std::vector &WaitList) { if (EventImpl->isHost()) { depends_on(EventImpl); } - impl->MEventsWaitWithBarrier.push_back(EventImpl); + impl->MEventsWaitWithBarrier.push_back(std::move(EventImpl)); } } diff --git a/sycl/source/kernel.cpp b/sycl/source/kernel.cpp index 916a80ba4646b..8abde164c6b62 100644 --- a/sycl/source/kernel.cpp +++ b/sycl/source/kernel.cpp @@ -133,7 +133,7 @@ kernel::ext_oneapi_get_info(queue Queue, const range<1> &WorkGroupSize, template typename detail::is_kernel_queue_specific_info_desc::return_type kernel::ext_oneapi_get_info(queue Queue, const range<3> &WG) const { - return impl->ext_oneapi_get_info(Queue, WG); + return impl->ext_oneapi_get_info(std::move(Queue), WG); } template @@ -147,13 +147,13 @@ kernel::ext_oneapi_get_info(queue Queue, const range<2> &WorkGroupSize, template typename detail::is_kernel_queue_specific_info_desc::return_type kernel::ext_oneapi_get_info(queue Queue, const range<2> &WG) const { - return impl->ext_oneapi_get_info(Queue, WG); + return impl->ext_oneapi_get_info(std::move(Queue), WG); } template typename detail::is_kernel_queue_specific_info_desc::return_type kernel::ext_oneapi_get_info(queue Queue, const range<1> &WG) const { - return impl->ext_oneapi_get_info(Queue, WG); + return impl->ext_oneapi_get_info(std::move(Queue), WG); } template