Skip to content

Commit c5f88aa

Browse files
[SYCL] Use sycl/detail/core.hpp in test-e2e/SubGroupMask/* (#13842)
Continuation of changes started in #12890.
1 parent 00ab727 commit c5f88aa

File tree

5 files changed

+32
-10
lines changed

5 files changed

+32
-10
lines changed

sycl/include/sycl/ext/oneapi/sub_group_mask.hpp

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -7,13 +7,14 @@
77
//===----------------------------------------------------------------------===//
88
#pragma once
99

10-
#include <sycl/detail/helpers.hpp> // for Builder
11-
#include <sycl/detail/memcpy.hpp> // detail::memcpy
12-
#include <sycl/exception.hpp> // for errc, exception
13-
#include <sycl/feature_test.hpp> // for SYCL_EXT_ONEAPI_SUB_GROUP_MASK
14-
#include <sycl/id.hpp> // for id
15-
#include <sycl/marray.hpp> // for marray
16-
#include <sycl/types.hpp> // for vec
10+
#include <sycl/builtins.hpp> // for assert
11+
#include <sycl/detail/helpers.hpp> // for Builder
12+
#include <sycl/detail/memcpy.hpp> // detail::memcpy
13+
#include <sycl/exception.hpp> // for errc, exception
14+
#include <sycl/feature_test.hpp> // for SYCL_EXT_ONEAPI_SUB_GROUP_MASK
15+
#include <sycl/id.hpp> // for id
16+
#include <sycl/marray.hpp> // for marray
17+
#include <sycl/types.hpp> // for vec
1718

1819
#include <assert.h> // for assert
1920
#include <climits> // for CHAR_BIT

sycl/test-e2e/SubGroupMask/Basic.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,10 @@
1616
//
1717
//===----------------------------------------------------------------------===//
1818

19+
#include <sycl/detail/core.hpp>
20+
#include <sycl/ext/oneapi/sub_group_mask.hpp>
21+
1922
#include <iostream>
20-
#include <sycl/sycl.hpp>
2123
using namespace sycl;
2224
constexpr int global_size = 128;
2325
constexpr int local_size = 32;

sycl/test-e2e/SubGroupMask/GroupSize.cpp

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,13 @@
1515
//
1616
//===----------------------------------------------------------------------===//
1717

18+
// TODO: switch to <sycl/detail/core.hpp> once we decide how feature macros
19+
// should work in that scenario.
20+
#include <sycl/detail/core.hpp>
21+
#include <sycl/ext/oneapi/sub_group_mask.hpp>
22+
1823
#include <iostream>
19-
#include <sycl/sycl.hpp>
24+
2025
using namespace sycl;
2126
#ifdef SYCL_EXT_ONEAPI_SUB_GROUP_MASK
2227
constexpr int global_size = 128;

sycl/test-e2e/SubGroupMask/sub_group_mask_ver2.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,8 @@
22
// RUN: %{run} %t.out
33

44
#include <iostream>
5-
#include <sycl/sycl.hpp>
5+
#include <sycl/detail/core.hpp>
6+
#include <sycl/ext/oneapi/sub_group_mask.hpp>
67

78
#define TEST_ON_DEVICE(TEST_BODY) \
89
{ \

sycl/test/include_deps/sycl_detail_core.hpp.cpp

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -143,6 +143,19 @@
143143
// CHECK-NEXT: detail/spirv.hpp
144144
// CHECK-NEXT: ext/oneapi/experimental/non_uniform_groups.hpp
145145
// CHECK-NEXT: ext/oneapi/sub_group_mask.hpp
146+
// CHECK-NEXT: builtins.hpp
147+
// CHECK-NEXT: detail/builtins/builtins.hpp
148+
// CHECK-NEXT: builtins_utils_vec.hpp
149+
// CHECK-NEXT: builtins_utils_scalar.hpp
150+
// CHECK-NEXT: detail/boolean.hpp
151+
// CHECK-NEXT: detail/builtins/common_functions.inc
152+
// CHECK-NEXT: detail/builtins/helper_macros.hpp
153+
// CHECK-NEXT: detail/builtins/geometric_functions.inc
154+
// CHECK-NEXT: detail/builtins/half_precision_math_functions.inc
155+
// CHECK-NEXT: detail/builtins/integer_functions.inc
156+
// CHECK-NEXT: detail/builtins/math_functions.inc
157+
// CHECK-NEXT: detail/builtins/native_math_functions.inc
158+
// CHECK-NEXT: detail/builtins/relational_functions.inc
146159
// CHECK-NEXT: feature_test.hpp
147160
// CHECK-NEXT: functional.hpp
148161
// CHECK-NEXT: device.hpp

0 commit comments

Comments
 (0)