[SYCL] Fix member initialization bugs

Signed-off-by: Hu, Peisen <[email protected]>

Author: HPS-1

sycl/test-e2e/GroupAlgorithm/root_group.cpp

@@ -75,10 +75,8 @@ void testRootGroup() {
   struct TestKernel1 {
     sycl::buffer<bool> m_dataBuf;
     sycl::handler m_h;
-    TestKernel1(sycl::buffer<bool> dataBuf, sycl::handler h) {
-      m_dataBuf = dataBuf;
-      m_h = h;
-    }
+    TestKernel1(sycl::buffer<bool> dataBuf, sycl::handler h)
+        : m_dataBuf(dataBuf), m_h(h) {}
     void operator()(sycl::nd_item<1> it) const {
       sycl::accessor data{m_dataBuf, m_h};
       volatile float X = 1.0f;
@@ -131,10 +129,8 @@ void testRootGroupFunctions() {
   struct TestKernel2 {
     sycl::buffer<bool> m_testResultsBuf;
     sycl::handler m_h;
-    TestKernel2(sycl::buffer<bool> testResultsBuf, sycl::handler h) {
-      m_testResultsBuf = testResultsBuf;
-      m_h = h;
-    }
+    TestKernel2(sycl::buffer<bool> testResultsBuf, sycl::handler h)
+        : m_testResultsBuf(testResultsBuf), m_h(h) {}
     void operator()(sycl::nd_item<1> it) const {
       sycl::accessor testResults{m_testResultsBuf, m_h};
       const auto root = it.ext_oneapi_get_root_group();

sycl/test-e2e/VirtualFunctions/2/1/1/missing-overrides.cpp

@@ -85,11 +85,8 @@ int main() try {
     sycl::buffer<int> mDataStorage;
     sycl::handler mCGH;
     KernelFunctor(sycl::buffer<storage_t> DeviceStorage,
-                  sycl::buffer<int> DataStorage, sycl::handler CGH) {
-      mDeviceStorage = DeviceStorage;
-      mDataStorage = DataStorage;
-      mCGH = CGH;
-    }
+                  sycl::buffer<int> DataStorage, sycl::handler CGH)
+        : mDeviceStorage(DeviceStorage), mDataStorage(DataStorage), mCGH(CGH) {}
 
     void operator()() const {
       sycl::accessor StorageAcc(mDeviceStorage, mCGH, sycl::write_only);

sycl/test-e2e/VirtualFunctions/2/1/1/more-complex-hierarchy.cpp

@@ -64,11 +64,8 @@ int main() try {
     sycl::buffer<int> mDataStorage;
     sycl::handler mCGH;
     KernelFunctor(sycl::buffer<storage_t> DeviceStorage,
-                  sycl::buffer<int> DataStorage, sycl::handler CGH) {
-      mDeviceStorage = DeviceStorage;
-      mDataStorage = DataStorage;
-      mCGH = CGH;
-    }
+                  sycl::buffer<int> DataStorage, sycl::handler CGH)
+        : mDeviceStorage(DeviceStorage), mDataStorage(DataStorage), mCGH(CGH) {}
 
     void operator()() const {
       sycl::accessor StorageAcc(mDeviceStorage, mCGH, sycl::write_only);

sycl/test-e2e/VirtualFunctions/2/1/1/simple-hierarchy.cpp

@@ -49,12 +49,8 @@ int main() try {
     sycl::buffer<int> mDataStorage;
     sycl::handler mCGH;
     KernelFunctor(sycl::buffer<storage_t> DeviceStorage,
-                  sycl::buffer<int> DataStorage, sycl::handler CGH) {
-      mDeviceStorage = DeviceStorage;
-      mDataStorage = DataStorage;
-      mCGH = CGH;
-    }
-
+                  sycl::buffer<int> DataStorage, sycl::handler CGH)
+        : mDeviceStorage(DeviceStorage), mDataStorage(DataStorage), mCGH(CGH) {}
     void operator()() const {
       sycl::accessor StorageAcc(mDeviceStorage, mCGH, sycl::write_only);
       sycl::accessor DataAcc(mDataStorage, mCGH, sycl::write_only);

sycl/test-e2e/VirtualFunctions/2/2/single-construct-single-use.cpp

@@ -77,12 +77,8 @@ int main() try {
     sycl::buffer<int> mDataStorage;
     sycl::handler mCGH;
     KernelFunctor(sycl::buffer<storage_t> DeviceStorage,
-                  sycl::buffer<int> DataStorage, sycl::handler CGH) {
-      mDeviceStorage = DeviceStorage;
-      mDataStorage = DataStorage;
-      mCGH = CGH;
-    }
-
+                  sycl::buffer<int> DataStorage, sycl::handler CGH)
+        : mDeviceStorage(DeviceStorage), mDataStorage(DataStorage), mCGH(CGH) {}
     void operator()() const {
       sycl::accessor StorageAcc(mDeviceStorage, mCGH, sycl::read_write);
       sycl::accessor DataAcc(mDataStorage, mCGH, sycl::write_only);

sycl/test-e2e/VirtualFunctions/misc/math.cpp

@@ -54,11 +54,8 @@ int main() try {
     sycl::buffer<int> mDataStorage;
     sycl::handler mCGH;
     KernelFunctor(T DeviceStorage, sycl::buffer<float> DataStorage,
-                  sycl::handler CGH) {
-      mDeviceStorage = DeviceStorage;
-      mDataStorage = DataStorage;
-      mCGH = CGH;
-    }
+                  sycl::handler CGH)
+        : mDeviceStorage(DeviceStorage), mDataStorage(DataStorage), mCGH(CGH) {}
 
     void operator()() const {
       sycl::accessor DataAcc(mDataStorage, mCGH, sycl::read_write);