Changed to uint8_t and improved fix as suggested

Author: konradkusiak97

source/adapters/opencl/usm.cpp

@@ -276,14 +276,13 @@ UR_APIEXPORT ur_result_t UR_APICALL urEnqueueUSMFill(
       cl_ext::MemBlockingFreeName, &USMFree));
 
   cl_int ClErr = CL_SUCCESS;
-  auto HostBuffer = static_cast<unsigned char *>(
-      HostMemAlloc(CLContext, nullptr, size, 0, &ClErr));
+  auto HostBuffer =
+      static_cast<uint8_t *>(HostMemAlloc(CLContext, nullptr, size, 0, &ClErr));
   CL_RETURN_ON_FAILURE(ClErr);
 
-  auto NumChunks = size / patternSize;
-  for (size_t i = 0; i < NumChunks; i++) {
-    auto Dest = HostBuffer + i * patternSize;
-    memcpy(Dest, pPattern, patternSize);
+  auto *End = HostBuffer + size;
+  for (auto *Iter = HostBuffer; Iter < End; Iter += patternSize) {
+    std::memcpy(Iter, pPattern, patternSize);
   }
 
   cl_event CopyEvent = nullptr;