Merge pull request #1169 from martygrant/martin/nativeCPUFixes

[NATIVECPU] Fix coverity warnings

Author: martygrant

source/adapters/native_cpu/enqueue.cpp

@@ -25,7 +25,8 @@ struct NDRDescT {
   RangeT GlobalSize;
   RangeT LocalSize;
   NDRDescT(uint32_t WorkDim, const size_t *GlobalWorkOffset,
-           const size_t *GlobalWorkSize, const size_t *LocalWorkSize) {
+           const size_t *GlobalWorkSize, const size_t *LocalWorkSize)
+      : WorkDim(WorkDim) {
     for (uint32_t I = 0; I < WorkDim; I++) {
       GlobalOffset[I] = GlobalWorkOffset[I];
       GlobalSize[I] = GlobalWorkSize[I];

source/adapters/native_cpu/kernel.cpp

@@ -97,7 +97,6 @@ UR_APIEXPORT ur_result_t UR_APICALL urKernelGetInfo(ur_kernel_handle_t hKernel,
   default:
     return UR_RESULT_ERROR_INVALID_VALUE;
   }
-  DIE_NO_IMPLEMENTATION
 }
 
 UR_APIEXPORT ur_result_t UR_APICALL

source/adapters/native_cpu/kernel.hpp

@@ -13,6 +13,7 @@
 #include "common.hpp"
 #include "nativecpu_state.hpp"
 #include <ur_api.h>
+#include <utility>
 
 namespace native_cpu {
 
@@ -39,7 +40,7 @@ struct local_arg_info_t {
 struct ur_kernel_handle_t_ : RefCounted {
 
   ur_kernel_handle_t_(const char *name, nativecpu_task_t subhandler)
-      : _name{name}, _subhandler{subhandler} {}
+      : _name{name}, _subhandler{std::move(subhandler)} {}
 
   const char *_name;
   nativecpu_task_t _subhandler;

source/adapters/native_cpu/memory.hpp

@@ -68,7 +68,7 @@ struct _ur_buffer final : ur_mem_handle_t_ {
   bool isSubBuffer() const { return SubBuffer.Parent != nullptr; }
 
   struct BB {
-    BB(_ur_buffer *b) : Parent(b) {}
+    BB(_ur_buffer *b) : Parent(b), Origin(0) {}
     BB() : BB(nullptr) {}
     _ur_buffer *const Parent;
     size_t Origin; // only valid if Parent != nullptr