[NFCI][SYCL][Test E2E] Fix warnings in HierPar tests (#9517)

Author: aelovikov-intel

sycl/test-e2e/HierPar/Inputs/hier_par_wgscope_impl.hpp

@@ -46,7 +46,7 @@ static bool testWgScope(queue &Q) {
       cgh.parallel_for_work_group<class hpar_hw>(
           range<1>(N_WG), range<1>(PHYS_WG_SIZE), [=](group<1> G) {
             // offset of group'S chunk in the 'DevPtr' array:
-            int GroupOff = PHYS_WG_SIZE * G.get_id(0);
+            int GroupOff = PHYS_WG_SIZE * G.get_group_id(0);
 
             for (int CntOuter = 0; CntOuter < N_OUTER_ITER; CntOuter++) {
               // local group-shared array; declared inside a loop
@@ -64,7 +64,7 @@ static bool testWgScope(queue &Q) {
               });
 
               // only for groups with IDs 0, 1:
-              if (G.get_id(0) < GROUP_ID_SPLIT) {
+              if (G.get_group_id(0) < GROUP_ID_SPLIT) {
                 // invoke PFWI in the loop
                 // Step 2a (1) - additionally increment 1 element per group as a
                 //   "side effect" of the increment part
@@ -230,7 +230,8 @@ bool testPrivateMemory(queue &Q) {
           }
         });
   });
-  auto Ptr1 = Buf.get_access<access::mode::read>().get_pointer();
+  host_accessor HostAcc(Buf, read_only);
+  auto Ptr1 = HostAcc.get_pointer();
   bool Res = verify(0, RangeLength, Ptr1, [&](int I) -> int {
     return N_ITER * (1 + C1 + C2 + 2 * I);
   });
@@ -243,12 +244,12 @@ int run() {
     for (auto ep : L) {
       try {
         std::rethrow_exception(ep);
-      } catch (std::exception &E) {
-        std::cout << "*** std exception caught:\n";
-        std::cout << E.what();
       } catch (sycl::exception const &E1) {
         std::cout << "*** SYCL exception caught:\n";
         std::cout << E1.what();
+      } catch (std::exception &E) {
+        std::cout << "*** std exception caught:\n";
+        std::cout << E.what();
       }
     }
   });

sycl/test-e2e/HierPar/hier_par_basic.cpp

@@ -81,7 +81,7 @@ struct PFWGFunctor {
 
   void operator()(group<1> g) const {
     int v = addend; // to check constant initializer works too
-    size_t wg_offset = wg_chunk * g.get_id(0);
+    size_t wg_offset = wg_chunk * g.get_group_id(0);
     size_t wg_size = g.get_local_range(0);
 
     PFWIFunctor PFWI(wg_chunk, wg_size, wg_offset, range_length, v, dev_ptr);
@@ -139,7 +139,8 @@ int main() {
         PFWGFunctor PFWG(wg_chunk, range_length, addend, N_ITER, dev_ptr);
         cgh.parallel_for_work_group(range<1>(N_WG), PFWG);
       });
-      auto ptr1 = buf.get_access<access::mode::read>().get_pointer();
+      host_accessor hostacc(buf, read_only);
+      auto ptr1 = hostacc.get_pointer();
       passed &= verify(1, range_length, ptr1,
                        [&](int i) -> int { return N_ITER * addend; });
     }
@@ -166,7 +167,8 @@ int main() {
               }
             });
       });
-      auto ptr1 = buf.get_access<access::mode::read>().get_pointer();
+      host_accessor hostacc(buf, read_only);
+      auto ptr1 = hostacc.get_pointer();
       passed &= verify(2, range_length, ptr1, [&](int i) -> int {
         // consider increments by the first PFWI:
         int gold = (WG_SIZE_GREATER_THAN_PHYSICAL - 1) / WG_SIZE_PHYSICAL;
@@ -207,7 +209,8 @@ int main() {
               }
             });
       });
-      auto ptr1 = buf.get_access<access::mode::read>().get_pointer();
+      host_accessor hostacc(buf, read_only);
+      auto ptr1 = hostacc.get_pointer();
       passed &= verify(3, range_length, ptr1, [&](int i) -> int {
         int gold = 0;
         if (i % WG_SIZE_PHYSICAL <
@@ -256,7 +259,8 @@ int main() {
               }
             });
       });
-      auto ptr1 = buf.get_access<access::mode::read>().get_pointer();
+      host_accessor hostacc(buf, read_only);
+      auto ptr1 = hostacc.get_pointer();
       passed &= verify(3, range_length, ptr1,
                        [&](int i) -> int { return N_ITER * (1 + i + 5); });
     }
@@ -281,7 +285,7 @@ int main() {
                 g.parallel_for_work_item(
                     range<1>(WG_SIZE_GREATER_THAN_PHYSICAL), [&](h_item<1> i) {
                       size_t wg_offset = WG_SIZE_GREATER_THAN_PHYSICAL *
-                                         wi_chunk * g.get_id(0);
+                                         wi_chunk * g.get_group_id(0);
                       size_t wi_offset =
                           wg_offset +
                           i.get_logical_local_id().get(0) * wi_chunk;
@@ -291,7 +295,8 @@ int main() {
               }
             });
       });
-      auto ptr1 = buf.get_access<access::mode::read>().get_pointer();
+      host_accessor hostacc(buf, read_only);
+      auto ptr1 = hostacc.get_pointer();
       passed &= verify(5, range_length, ptr1, [&](int i) -> int {
         int gold =
             i % 2 == 0 ? WG_SIZE_GREATER_THAN_PHYSICAL : WG_SIZE_PHYSICAL;