removing unnecessary and harmful move of temp obj

danhoeflinger · danhoeflinger · commit 97454e4dee0d · 2025-06-25T09:03:54.000-04:00
Signed-off-by: Dan Hoeflinger &lt;dan.hoeflinger@intel.com&gt;
diff --git a/include/oneapi/dpl/pstl/parallel_backend_tbb.h b/include/oneapi/dpl/pstl/parallel_backend_tbb.h
@@ -195,7 +195,7 @@ __parallel_transform_reduce(oneapi::dpl::__internal::__tbb_backend_tag, _Executi
     // The grain size of 3 is used in order to provide minimum 2 elements for each body
     tbb::this_task_arena::isolate(
         [__first, __last, &__body]() { tbb::parallel_reduce(tbb::blocked_range<_Index>(__first, __last, 3), __body); });
-    return std::move(__body.sum());
+    return __body.sum();
 }
 
 //------------------------------------------------------------------------

Original file line number	Diff line number	Diff line change
`@@ -195,7 +195,7 @@ __parallel_transform_reduce(oneapi::dpl::__internal::__tbb_backend_tag, _Executi`
`195`	`195`	`// The grain size of 3 is used in order to provide minimum 2 elements for each body`
`196`	`196`	`tbb::this_task_arena::isolate(`
`197`	`197`	`[__first, __last, &__body]() { tbb::parallel_reduce(tbb::blocked_range<_Index>(__first, __last, 3), __body); });`
`198`		`- return std::move(__body.sum());`
	`198`	`+ return __body.sum();`
`199`	`199`	`}`
`200`	`200`
`201`	`201`	`//------------------------------------------------------------------------`