Apply suggestions from code review

Author: sndmitriev

SYCL/ESIMD/Prefix_Local_sum1.cpp

@@ -131,7 +131,6 @@ int main(int argc, char *argv[]) {
 
   auto dev = q.get_device();
   std::cout << "Running on " << dev.get_info<info::device::name>() << "\n";
-  auto ctxt = q.get_context();
 
   // allocate and initialized input
   unsigned int *pInputs = static_cast<unsigned int *>(
@@ -141,8 +140,8 @@ int main(int argc, char *argv[]) {
   }
 
   // allocate device buffer
-  unsigned int *pDeviceOutputs = static_cast<unsigned int *>(
-      malloc_shared(size * TUPLE_SZ * sizeof(unsigned int), dev, ctxt));
+  unsigned int *pDeviceOutputs =
+      malloc_shared<unsigned int>(size * TUPLE_SZ, q);
 
   // allocate & compute expected result
   unsigned int *pExpectOutputs = static_cast<unsigned int *>(
@@ -177,7 +176,7 @@ int main(int argc, char *argv[]) {
     }
   } catch (cl::sycl::exception const &e) {
     std::cout << "SYCL exception caught: " << e.what() << '\n';
-    free(pInputs, ctxt);
+    free(pInputs, q);
     free(pExpectOutputs);
     return 1;
   }
@@ -193,7 +192,7 @@ int main(int argc, char *argv[]) {
   std::cout << "Prefix " << (pass ? "=> PASSED" : "=> FAILED") << std::endl
             << std::endl;
 
-  free(pDeviceOutputs, ctxt);
+  free(pDeviceOutputs, q);
   free(pExpectOutputs);
   free(pInputs);
   return 0;

SYCL/ESIMD/Prefix_Local_sum2.cpp

@@ -126,7 +126,6 @@ int main(int argc, char *argv[]) {
 
   auto dev = q.get_device();
   std::cout << "Running on " << dev.get_info<info::device::name>() << "\n";
-  auto ctxt = q.get_context();
 
   // allocate and initialized input
   unsigned int *pInputs = static_cast<unsigned int *>(
@@ -136,8 +135,8 @@ int main(int argc, char *argv[]) {
   }
 
   // allocate device buffer
-  unsigned int *pDeviceOutputs = static_cast<unsigned int *>(
-      malloc_shared(size * TUPLE_SZ * sizeof(unsigned int), dev, ctxt));
+  unsigned int *pDeviceOutputs =
+      malloc_shared<unsigned int>(size * TUPLE_SZ, q);
 
   // allocate & compute expected result
   unsigned int *pExpectOutputs = static_cast<unsigned int *>(
@@ -171,7 +170,7 @@ int main(int argc, char *argv[]) {
     }
   } catch (cl::sycl::exception const &e) {
     std::cout << "SYCL exception caught: " << e.what() << '\n';
-    free(pInputs, ctxt);
+    free(pInputs, q);
     free(pExpectOutputs);
     return 1;
   }
@@ -187,7 +186,7 @@ int main(int argc, char *argv[]) {
   std::cout << "Prefix " << (pass ? "=> PASSED" : "=> FAILED") << std::endl
             << std::endl;
 
-  free(pDeviceOutputs, ctxt);
+  free(pDeviceOutputs, q);
   free(pExpectOutputs);
   free(pInputs);
   return 0;

SYCL/ESIMD/Prefix_Local_sum3.cpp

@@ -332,7 +332,6 @@ int main(int argc, char *argv[]) {
 
   auto dev = q.get_device();
   std::cout << "Running on " << dev.get_info<info::device::name>() << "\n";
-  auto ctxt = q.get_context();
 
   // allocate and initialized input data
   unsigned int *pInputs = static_cast<unsigned int *>(
@@ -342,8 +341,8 @@ int main(int argc, char *argv[]) {
   }
 
   // allocate kernel buffer
-  unsigned int *pDeviceOutputs = static_cast<unsigned int *>(
-      malloc_shared(size * TUPLE_SZ * sizeof(unsigned int), dev, ctxt));
+  unsigned int *pDeviceOutputs =
+      malloc_shared<unsigned int>(size * TUPLE_SZ, q);
 
   // allocate & intialize expected result
   unsigned int *pExpectOutputs = static_cast<unsigned int *>(
@@ -389,7 +388,7 @@ int main(int argc, char *argv[]) {
   std::cout << "Prefix " << (pass ? "=> PASSED" : "=> FAILED") << std::endl
             << std::endl;
 
-  free(pDeviceOutputs, ctxt);
+  free(pDeviceOutputs, q);
   free(pExpectOutputs);
   free(pInputs);
   return 0;