From 3ac96b5fc8f3b2e70dc6720eb9fa944007e535ef Mon Sep 17 00:00:00 2001 From: "Ahn, Dongkyun" Date: Tue, 13 Sep 2022 22:30:58 -0700 Subject: [PATCH] Revert "[SYCL][ESIMD]LSC Atomic update for emulator backend (#1259)" PR#1259 was based on changes applied in intel/llvm-PR#6661. As the PR#6661 is still under review (as of Sept 14th), changes from PR#1259 causes unexpected failures for the ESIMD_EMULATOR backend for other PRs in intel/llvm, such as PR#6780 in above comment. --- SYCL/ESIMD/dword_atomic_cmpxchg.cpp | 4 ++-- SYCL/ESIMD/dword_atomic_smoke.cpp | 2 ++ SYCL/ESIMD/lsc/Inputs/lsc_surf.hpp | 4 ++-- SYCL/ESIMD/lsc/lsc_fence.cpp | 2 +- SYCL/ESIMD/lsc/lsc_slm.cpp | 4 ++-- SYCL/ESIMD/lsc/lsc_usm.cpp | 4 ++-- 6 files changed, 11 insertions(+), 9 deletions(-) diff --git a/SYCL/ESIMD/dword_atomic_cmpxchg.cpp b/SYCL/ESIMD/dword_atomic_cmpxchg.cpp index 7674af83a7..42b98bfe2d 100644 --- a/SYCL/ESIMD/dword_atomic_cmpxchg.cpp +++ b/SYCL/ESIMD/dword_atomic_cmpxchg.cpp @@ -9,8 +9,8 @@ //===----------------------------------------------------------------------===// // REQUIRES: gpu // UNSUPPORTED: cuda || hip -// TODO: esimd_emulator fails due to random timeouts (_XFAIL_: esimd_emulator) -// UNSUPPORTED: esimd_emulator +// TODO: esimd_emulator fails due to unsupported __esimd_svm_atomic0/1/2 +// XFAIL: esimd_emulator // RUN: %clangxx -fsycl %s -o %t.out // RUN: %GPU_RUN_PLACEHOLDER %t.out diff --git a/SYCL/ESIMD/dword_atomic_smoke.cpp b/SYCL/ESIMD/dword_atomic_smoke.cpp index d0fd376413..261710277c 100644 --- a/SYCL/ESIMD/dword_atomic_smoke.cpp +++ b/SYCL/ESIMD/dword_atomic_smoke.cpp @@ -9,6 +9,8 @@ //===----------------------------------------------------------------------===// // REQUIRES: gpu // UNSUPPORTED: cuda || hip +// TODO: esimd_emulator fails due to unsupported __esimd_svm_atomic0/1/2 +// XFAIL: esimd_emulator // RUN: %clangxx -fsycl %s -o %t.out // RUN: %GPU_RUN_PLACEHOLDER %t.out diff --git a/SYCL/ESIMD/lsc/Inputs/lsc_surf.hpp b/SYCL/ESIMD/lsc/Inputs/lsc_surf.hpp index 14d62c8ed4..2e1f7302e7 100644 --- a/SYCL/ESIMD/lsc/Inputs/lsc_surf.hpp +++ b/SYCL/ESIMD/lsc/Inputs/lsc_surf.hpp @@ -75,8 +75,8 @@ int main() { lsc_atomic_update(access_2, offsets, pred); lsc_atomic_update(access_3, offsets, add, pred); - lsc_atomic_update(access_4, offsets, swap, - compare, pred); + lsc_atomic_update(access_4, offsets, + compare, swap, pred); }); }); q.wait(); diff --git a/SYCL/ESIMD/lsc/lsc_fence.cpp b/SYCL/ESIMD/lsc/lsc_fence.cpp index 400f1295a6..99cc6bc912 100644 --- a/SYCL/ESIMD/lsc/lsc_fence.cpp +++ b/SYCL/ESIMD/lsc/lsc_fence.cpp @@ -5,7 +5,7 @@ // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception // //===----------------------------------------------------------------------===// -// REQUIRES: gpu-intel-pvc || esimd_emulator +// REQUIRES: gpu-intel-pvc // UNSUPPORTED: cuda || hip // RUN: %clangxx -fsycl %s -o %t.out // RUN: %GPU_RUN_PLACEHOLDER %t.out diff --git a/SYCL/ESIMD/lsc/lsc_slm.cpp b/SYCL/ESIMD/lsc/lsc_slm.cpp index 1924112888..cf050bd8b4 100644 --- a/SYCL/ESIMD/lsc/lsc_slm.cpp +++ b/SYCL/ESIMD/lsc/lsc_slm.cpp @@ -81,8 +81,8 @@ int main() { lsc_block_store(access_3, offset, data_3); lsc_slm_block_store(offset, data); - lsc_slm_atomic_update(offsets, swap, - compare, pred); + lsc_slm_atomic_update(offsets, compare, + swap, pred); auto data_4 = lsc_slm_block_load(offset); lsc_block_store(access_4, offset, data_4); }); diff --git a/SYCL/ESIMD/lsc/lsc_usm.cpp b/SYCL/ESIMD/lsc/lsc_usm.cpp index f46987f93d..4d6e5c501a 100644 --- a/SYCL/ESIMD/lsc/lsc_usm.cpp +++ b/SYCL/ESIMD/lsc/lsc_usm.cpp @@ -69,8 +69,8 @@ int main() { lsc_atomic_update(vec_2, offsets, pred); lsc_atomic_update(vec_3, offsets, add, pred); - lsc_atomic_update(vec_4, offsets, swap, - compare, pred); + lsc_atomic_update(vec_4, offsets, compare, + swap, pred); }); }); q.wait();