[SYCL][E2E] Remove warnings in Basic e2e tests (#13994)

Author: ayylol

sycl/test-e2e/Basic/built-ins/host_math.cpp

@@ -44,7 +44,8 @@ void testRemquo() {
     int quo = 0;
     float rem = sycl::remquo(
         86.0f, 10.0f,
-        sycl::multi_ptr<int, sycl::access::address_space::global_space>{&quo});
+        sycl::address_space_cast<sycl::access::address_space::global_space,
+                                 sycl::access::decorated::no, int>(&quo));
     assert(quo == 9);
     assert(rem == -4);
   }
@@ -53,7 +54,8 @@ void testRemquo() {
     int quo = 0;
     float rem = sycl::remquo(
         -10.0, 3.0,
-        sycl::multi_ptr<int, sycl::access::address_space::global_space>{&quo});
+        sycl::address_space_cast<sycl::access::address_space::global_space,
+                                 sycl::access::decorated::no, int>(&quo));
     assert(quo == -3);
     assert(rem == -1);
   }
@@ -62,7 +64,8 @@ void testRemquo() {
     int quo = 0;
     float rem = sycl::remquo(
         0.552879f, 0.219282f,
-        sycl::multi_ptr<int, sycl::access::address_space::global_space>{&quo});
+        sycl::address_space_cast<sycl::access::address_space::global_space,
+                                 sycl::access::decorated::no, int>(&quo));
     assert(quo == 3);
     assert(rem == -0.10496702790260315f);
   }

sycl/test-e2e/Basic/device_event.cpp

@@ -57,8 +57,7 @@ int test_strideN(size_t stride) {
     nElemsToCopy++;
 
   try {
-    default_selector selector;
-    queue myQueue(selector, [](exception_list l) {
+    queue myQueue(default_selector_v, [](exception_list l) {
       for (auto ep : l) {
         try {
           std::rethrow_exception(ep);
@@ -88,7 +87,7 @@ int test_strideN(size_t stride) {
             local_acc.get_multi_ptr<access::decorated::yes>();
         decorated_global_ptr<int> gptr =
             out_ptr.get_multi_ptr<access::decorated::yes>() +
-            grp.get_id()[0] * 16;
+            grp.get_group_id()[0] * 16;
 
         // Write the values 700, 701, ..., 763 to global memory.
         // Why? Well, a) to ensure that something is written into that memory

sycl/test-e2e/Basic/event.cpp

@@ -52,15 +52,13 @@ int main() {
   }
 
   {
-    struct exception : public sycl::exception {};
-
     std::cout << "wait_and_throw() check" << std::endl;
     bool failed = true;
     auto handler = [&](sycl::exception_list l) { failed = false; };
 
     sycl::queue queue(handler);
     sycl::event e = queue.submit([&](sycl::handler &cgh) {
-      cgh.host_task([=]() { throw exception{}; });
+      cgh.host_task([=]() { throw sycl::exception{sycl::errc::runtime}; });
     });
     e.wait_and_throw();
     assert(failed == false);

sycl/test-e2e/Basic/host-task-dependency.cpp

@@ -32,10 +32,10 @@ struct Context {
 
 S::event HostTask_CopyBuf1ToBuf2(Context *Ctx) {
   S::event Event = Ctx->Queue.submit([&](S::handler &CGH) {
-    S::accessor<int, 1, S::access::mode::read, S::access::target::host_buffer>
-        CopierSrcAcc(Ctx->Buf1, CGH);
-    S::accessor<int, 1, S::access::mode::write, S::access::target::host_buffer>
-        CopierDstAcc(Ctx->Buf2, CGH);
+    S::host_accessor<int, 1, S::access::mode::read> CopierSrcAcc(Ctx->Buf1,
+                                                                 CGH);
+    S::host_accessor<int, 1, S::access::mode::write> CopierDstAcc(Ctx->Buf2,
+                                                                  CGH);
 
     auto CopierHostTask = [=] {
       for (size_t Idx = 0; Idx < CopierDstAcc.size(); ++Idx)
@@ -59,24 +59,21 @@ S::event HostTask_CopyBuf1ToBuf2(Context *Ctx) {
 void Thread1Fn(Context *Ctx) {
   // 0. initialize resulting buffer with apriori wrong result
   {
-    S::accessor<int, 1, S::access::mode::write, S::access::target::host_buffer>
-        Acc(Ctx->Buf1);
+    S::host_accessor<int, 1, S::access::mode::write> Acc(Ctx->Buf1);
 
     for (size_t Idx = 0; Idx < Acc.size(); ++Idx)
       Acc[Idx] = -1;
   }
 
   {
-    S::accessor<int, 1, S::access::mode::write, S::access::target::host_buffer>
-        Acc(Ctx->Buf2);
+    S::host_accessor<int, 1, S::access::mode::write> Acc(Ctx->Buf2);
 
     for (size_t Idx = 0; Idx < Acc.size(); ++Idx)
       Acc[Idx] = -2;
   }
 
   {
-    S::accessor<int, 1, S::access::mode::write, S::access::target::host_buffer>
-        Acc(Ctx->Buf3);
+    S::host_accessor<int, 1, S::access::mode::write> Acc(Ctx->Buf3);
 
     for (size_t Idx = 0; Idx < Acc.size(); ++Idx)
       Acc[Idx] = -3;
@@ -117,8 +114,7 @@ void Thread1Fn(Context *Ctx) {
 
   // 4. check data in buffer #3
   {
-    S::accessor<int, 1, S::access::mode::read, S::access::target::host_buffer>
-        Acc(Ctx->Buf3);
+    S::host_accessor<int, 1, S::access::mode::read> Acc(Ctx->Buf3);
 
     bool Failure = false;
 
@@ -163,8 +159,7 @@ void test() {
 
   // 3. check via host accessor that buf 2 contains valid data
   {
-    S::accessor<int, 1, S::access::mode::read, S::access::target::host_buffer>
-        ResultAcc(Ctx.Buf2);
+    S::host_accessor<int, 1, S::access::mode::read> ResultAcc(Ctx.Buf2);
 
     bool Failure = false;
     for (size_t Idx = 0; Idx < ResultAcc.size(); ++Idx) {

sycl/test-e2e/Basic/sycl_2020_images/common.hpp

@@ -269,7 +269,7 @@ template <typename T, int Dims> bool AllTrue(const vec<T, Dims> &Vec) {
 
 template <typename T, int Dims>
 bool ApproxEq(const vec<T, Dims> &LHS, const vec<T, Dims> &RHS,
-              T Precision = 0.1) {
+              T Precision = (T)0.1) {
   if constexpr (std::is_integral_v<T>)
     return AllTrue(sycl::abs(LHS - RHS) <= Precision);
   else