Skip to content

[SYCL][E2E] Switch KernelFusion/* tests to use <sycl/detail/core.hpp> #13129

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 25, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion sycl/test-e2e/KernelFusion/GroupAlgorithm/all_of.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,10 @@
#include <cassert>
#include <iostream>
#include <numeric>
#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/group_algorithm.hpp>
#include <sycl/properties/all_properties.hpp>

// COM: Check all_of works with kernel fusion.

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,9 @@
#include <iostream>
#include <limits>
#include <numeric>
#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>
#include <vector>
using namespace sycl;

Expand Down
4 changes: 3 additions & 1 deletion sycl/test-e2e/KernelFusion/GroupAlgorithm/permute.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,9 @@

// Test fusion works with permute and remapping.

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

#include "../helpers.hpp"
#include "sycl/group_algorithm.hpp"
Expand Down
4 changes: 3 additions & 1 deletion sycl/test-e2e/KernelFusion/GroupAlgorithm/support.h
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
#include <iostream>
#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;
using namespace sycl::ext::oneapi;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,10 @@
// Test complete_fusion preserves barriers by launching a kernel that requires a
// barrier for correctness.

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/group_barrier.hpp>
#include <sycl/properties/all_properties.hpp>

#include "../helpers.hpp"

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,10 @@

// Test fusion works with group_broadcast.

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/group_algorithm.hpp>
#include <sycl/properties/all_properties.hpp>

#include "../helpers.hpp"

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,10 @@

// Test fusion works with group_broadcast and remapping.

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/group_algorithm.hpp>
#include <sycl/properties/all_properties.hpp>

#include "../helpers.hpp"

Expand Down
6 changes: 4 additions & 2 deletions sycl/test-e2e/KernelFusion/Reduction/reduction.hpp
Original file line number Diff line number Diff line change
@@ -1,10 +1,12 @@
// Test fusion works with reductions. Only algorithms automatically selected by
// `sycl::reduction` are supported.

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

#include "../helpers.hpp"
#include "sycl/detail/reduction_forward.hpp"
#include "sycl/reduction.hpp"

using namespace sycl;

Expand Down
5 changes: 4 additions & 1 deletion sycl/test-e2e/KernelFusion/abort_fusion.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,10 @@
// to abort fusion due to constraint violations for fusion. Also check that
// warnings are printed when SYCL_RT_WARNING_LEVEL=1.

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>

#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
4 changes: 3 additions & 1 deletion sycl/test-e2e/KernelFusion/abort_internalization.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,9 @@
// to abort internalization due to target or parameter mismatch. Also check that
// warnings are printed when SYCL_RT_WARNING_LEVEL=1.

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,9 @@
// it just fails for a slightly different reason.
// CHECK-HIP-NEXT: Failed to promote argument 0 of function {{.*}}: Do not know how to handle value to promote

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

#include <array>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,9 @@
// Test complete fusion with local internalization and a combination of kernels
// that require a work-group barrier to be inserted by fusion.

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
4 changes: 3 additions & 1 deletion sycl/test-e2e/KernelFusion/buffer_internalization.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,9 @@
// Test complete fusion with private internalization specified on the
// buffer.

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
4 changes: 3 additions & 1 deletion sycl/test-e2e/KernelFusion/cached_ndrange.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,9 @@
// printed when SYCL_RT_WARNING_LEVEL=1.

#include <iostream>
#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
4 changes: 3 additions & 1 deletion sycl/test-e2e/KernelFusion/cancel_fusion.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,9 @@

// Test cancel fusion

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
4 changes: 3 additions & 1 deletion sycl/test-e2e/KernelFusion/complete_fusion.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,9 @@

// Test complete fusion without any internalization

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
4 changes: 3 additions & 1 deletion sycl/test-e2e/KernelFusion/cooperative_kernel.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,9 @@
// fusion enabled.
// XFAIL: hip_amd

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
4 changes: 3 additions & 1 deletion sycl/test-e2e/KernelFusion/device_info_descriptor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,9 @@

// Test correct return from device information descriptor.

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
4 changes: 3 additions & 1 deletion sycl/test-e2e/KernelFusion/diamond_shape.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,9 @@
// accessors for a combination of four kernels, forming a diamond-like shape and
// repeating one of the kernels.

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
4 changes: 3 additions & 1 deletion sycl/test-e2e/KernelFusion/diamond_shape_local.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,9 @@
// accessors for a combination of four kernels, forming a diamond-like shape and
// repeating one of the kernels.

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
6 changes: 5 additions & 1 deletion sycl/test-e2e/KernelFusion/different_nd_ranges.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,11 @@
// Kernels with different ND-ranges should be fused.
// CHECK-NOT: Cannot fuse kernels with different offsets or local sizes

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>

#include <sycl/builtins.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

#include <algorithm>

Expand Down
5 changes: 4 additions & 1 deletion sycl/test-e2e/KernelFusion/event_wait_cancel.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,10 @@

#include "fusion_event_test_common.h"

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>
#include <sycl/usm.hpp>

using namespace sycl;

Expand Down
5 changes: 4 additions & 1 deletion sycl/test-e2e/KernelFusion/event_wait_complete.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,10 @@

#include "fusion_event_test_common.h"

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>
#include <sycl/usm.hpp>

using namespace sycl;

Expand Down
4 changes: 3 additions & 1 deletion sycl/test-e2e/KernelFusion/existing_local_accessor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,9 @@
// Test complete fusion with local internalization and an local accessor that
// already exists in one of the input kernels.

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
4 changes: 3 additions & 1 deletion sycl/test-e2e/KernelFusion/fusion_event_test_common.h
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
4 changes: 3 additions & 1 deletion sycl/test-e2e/KernelFusion/helpers.hpp
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;
using namespace sycl::ext::codeplay;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,10 @@

#include "fusion_event_test_common.h"

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>
#include <sycl/usm.hpp>

using namespace sycl;

Expand Down
6 changes: 5 additions & 1 deletion sycl/test-e2e/KernelFusion/internalize_array_wrapper.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,11 @@

#include <array>

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>

#include <sycl/builtins.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,11 @@

#include <array>

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>

#include <sycl/builtins.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
6 changes: 5 additions & 1 deletion sycl/test-e2e/KernelFusion/internalize_deep.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,11 @@

#include <type_traits>

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>

#include <sycl/builtins.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
4 changes: 3 additions & 1 deletion sycl/test-e2e/KernelFusion/internalize_multi_ptr.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,9 @@
// Test complete fusion with private internalization specified on the
// accessors for a device kernel using multi_ptr to global address space.

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,9 @@
// - `tmp2` is an `i8` buffer. The corresponding `i8`-typed GEPs must be
// remapped during internalization.

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
6 changes: 5 additions & 1 deletion sycl/test-e2e/KernelFusion/internalize_vec.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,11 @@

// Test complete fusion with internalization of a struct type.

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>

#include <sycl/builtins.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
4 changes: 3 additions & 1 deletion sycl/test-e2e/KernelFusion/internalize_vfunc.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,9 @@

#define VEC 4

#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
4 changes: 3 additions & 1 deletion sycl/test-e2e/KernelFusion/jit_caching.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,9 @@
// printed when SYCL_RT_WARNING_LEVEL=1.

#include <iostream>
#include <sycl/sycl.hpp>
#include <sycl/detail/core.hpp>
#include <sycl/ext/codeplay/experimental/fusion_wrapper.hpp>
#include <sycl/properties/all_properties.hpp>

using namespace sycl;

Expand Down
Loading