Skip to content

Commit

Permalink
[SYCL][ESIMD][E2E] Remove force O2 from tests (#12911)
Browse files Browse the repository at this point in the history
They all pass now, I tested on Gen12 and PVC.

Signed-off-by: Sarnie, Nick <nick.sarnie@intel.com>
  • Loading branch information
sarnex committed Mar 6, 2024
1 parent e8361f6 commit 8027266
Show file tree
Hide file tree
Showing 49 changed files with 49 additions and 97 deletions.
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/BitonicSortK.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

#include "BitonicSortK.hpp"
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/BitonicSortKv2.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

#include "BitonicSortKv2.hpp"
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/InlineAsm/asm_glb.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Force -O2 as it currently fails in O0 due to missing VC support
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

#include "../esimd_test_utils.hpp"
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/PrefixSum.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out 20

#include "esimd_test_utils.hpp"
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/Prefix_Local_sum2.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out 20

#include "esimd_test_utils.hpp"
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/Prefix_Local_sum3.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

#include "esimd_test_utils.hpp"
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/Stencil.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

#include "esimd_test_utils.hpp"
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/acc_gather_scatter_rgba.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out
//
// The test checks functionality of the gather_rgba/scatter_rgba accessor-based
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/acc_gather_scatter_rgba_stateless.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -fsycl-esimd-force-stateless-mem -o %t.out
// RUN: %{build} -fsycl-esimd-force-stateless-mem -o %t.out
// RUN: %{run} %t.out
//
// The test checks functionality of the gather_rgba/scatter_rgba accessor-based
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/accessor_gather_scatter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out
//
// The test checks functionality of the gather/scatter accessor-based ESIMD
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/accessor_gather_scatter_stateless.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -fsycl-esimd-force-stateless-mem -o %t.out
// RUN: %{build} -fsycl-esimd-force-stateless-mem -o %t.out
// RUN: %{run} %t.out
//
// The test checks functionality of the gather/scatter accessor-based ESIMD
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/accessor_stateless.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -fsycl-esimd-force-stateless-mem -D_CRT_SECURE_NO_WARNINGS=1 -o %t.out
// RUN: %{build} -fsycl-esimd-force-stateless-mem -D_CRT_SECURE_NO_WARNINGS=1 -o %t.out
// RUN: %{run} %t.out

// This test checks that accessor-based memory accesses work correctly in ESIMD
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/api/bin_and_cmp_ops_heavy.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,7 @@
//===----------------------------------------------------------------------===//
// Exclude PVC not to run same test cases twice (via the *_pvc.cpp variant).
// UNSUPPORTED: gpu-intel-pvc
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{build} -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{run} %t.out

// Tests various binary operations applied to simd objects.
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/api/esimd_pack_unpack_mask.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{build} -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{run} %t.out
//
// Smoke test for the esimd pack_mask/unpack_mask APIs.
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/api/esimd_rgba_smoke.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

// Smoke test for scatter/gather also illustrating correct use of these APIs
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/api/esimd_rgba_smoke_64.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

// Smoke test for scatter/gather also illustrating correct use of these APIs
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{build} -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{run} %t.out
//
// Test for simd lvalue select function.
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/api/simd_any_all.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{build} -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{run} %t.out
//
// Smoke test for esimd any/all operations APIs.
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/api/simd_copy_to_from.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@
//
//===----------------------------------------------------------------------===//
// UNSUPPORTED: gpu-intel-pvc
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

// This test checks simd::copy_from/to methods with alignment flags.
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/api/simd_mask.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -I%S/.. -o %t.out
// RUN: %{build} -I%S/.. -o %t.out
// RUN: %{run} %t.out

// Smoke test for simd_mask API functionality.
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/api/simd_memory_access.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{build} -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{run} %t.out
//
// The test checks functionality of the memory access APIs which are members of
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/api/simd_view_select_2d_fp.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{build} -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{run} %t.out
//
// Smoke test for 2D region select API which can be used to represent 2D tiles.
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/api/simd_view_select_2d_int.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{build} -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{run} %t.out
//
// Smoke test for 2D region select API which can be used to represent 2D tiles.
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/api/slm_gather_scatter.cpp
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{build} -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{run} %t.out
//
// The test checks functionality of the slm_gather/slm_scatter ESIMD APIs.
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/api/svm_gather_scatter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@
//
//===----------------------------------------------------------------------===//
// UNSUPPORTED: gpu-intel-pvc
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{build} -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{run} %t.out

// Regression test for SVM gather/scatter API.
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/api/svm_gather_scatter_64.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@
//
//===----------------------------------------------------------------------===//
// UNSUPOORTED: gpu-intel-pvc
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

// Regression test for gather/scatter API.
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/api/unary_ops_heavy.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,7 @@
//===----------------------------------------------------------------------===//
// Exclude PVC not to run same test cases twice (via the *_pvc.cpp variant).
// UNSUPPORTED: gpu-intel-pvc
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{build} -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{run} %t.out

// Tests various unary operations applied to simd objects.
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/histogram.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@
//
//===----------------------------------------------------------------------===//
// REQUIRES: aspect-ext_intel_legacy_image
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

#include "esimd_test_utils.hpp"
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/histogram_256_slm.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

#include "esimd_test_utils.hpp"
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/histogram_256_slm_spec_2020.cpp
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
// TODO enable on Windows
// REQUIRES: linux
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out 16

#include "esimd_test_utils.hpp"
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/histogram_2d.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@
//
//===----------------------------------------------------------------------===//
// REQUIRES: aspect-ext_intel_legacy_image
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

#include "esimd_test_utils.hpp"
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/histogram_2d_64.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@
//
//===----------------------------------------------------------------------===//
// REQUIRES: aspect-ext_intel_legacy_image
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out
//
// 64 bit offset variant of the test - uses 64 bit offsets.
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/histogram_64.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@
//
//===----------------------------------------------------------------------===//
// REQUIRES: aspect-ext_intel_legacy_image
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out
//
// 64 bit offset variant of the test - uses 64 bit offsets.
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/imulh_umulh.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

// The test verifies ESIMD API that multiplies 2 32-bit integer scalars/vectors
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/kmeans/kmeans.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -I%S/.. -o %t.out
// RUN: %{build} -I%S/.. -o %t.out
// RUN: %{run} %t.out %S/points.csv

#include "kmeans.h"
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/linear/linear.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@
//
//===----------------------------------------------------------------------===//
// REQUIRES: aspect-ext_intel_legacy_image
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -I%S/.. -o %t.out
// RUN: %{build} -I%S/.. -o %t.out
// RUN: %{run} %t.out %S/linear_in.bmp %S/linear_gold_hw.bmp

#include "bitmap_helpers.h"
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/matrix_transpose_glb.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

#include "esimd_test_utils.hpp"
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/matrix_transpose_usm.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

#include "esimd_test_utils.hpp"
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/radix_sort.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

// This test implements parallel radix sort on GPU
Expand Down
3 changes: 1 addition & 2 deletions sycl/test-e2e/ESIMD/reduction.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Use -O2 to avoid huge stack usage under -O0.
// RUN: %{build} -O2 -o %t.out
// RUN: %{build} -o %t.out
// RUN: %{run} %t.out

#include "esimd_test_utils.hpp"
Expand Down
Loading

0 comments on commit 8027266

Please sign in to comment.