Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

Commit 8cb1db6

Browse files
[SYCL] Prepare FilterSelector tests for migrating to sycl::oneapi (#335)
This patch prepares FilterSelector tests for transitioning from sycl::ONEAPI to sycl::oneapi namespace.
1 parent 0487154 commit 8cb1db6

File tree

2 files changed

+29
-21
lines changed

2 files changed

+29
-21
lines changed

SYCL/FilterSelector/reuse.cpp

+7-3
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,8 @@
1515
#include <CL/sycl.hpp>
1616

1717
using namespace cl::sycl;
18+
// TODO: change to 'using namespace cl::sycl::oneapi' after PR intel/llvm:4014
19+
// is merged
1820
using namespace cl::sycl::ONEAPI;
1921

2022
int main() {
@@ -25,15 +27,17 @@ int main() {
2527
std::cout << "# Devices found: " << Devs.size() << std::endl;
2628

2729
if (Devs.size() > 1) {
28-
filter_selector filter("1");
30+
// TODO: change all occurrences of filter_selector to 'filter_selector' or
31+
// 'oneapi::filter_selector' after PR intel/llvm:4014 is merged
32+
ONEAPI::filter_selector filter("1");
2933

3034
device d1(filter);
3135
device d2(filter);
3236

3337
assert(d1 == d2);
3438

35-
filter_selector f1("0");
36-
filter_selector f2("1");
39+
ONEAPI::filter_selector f1("0");
40+
ONEAPI::filter_selector f2("1");
3741
device d3(f1);
3842
device d4(f2);
3943

SYCL/FilterSelector/select.cpp

+22-18
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,8 @@
1515
#include <CL/sycl.hpp>
1616

1717
using namespace cl::sycl;
18+
// TODO: change to 'using namespace cl::sycl::oneapi' after PR intel/llvm:4014
19+
// is merged
1820
using namespace cl::sycl::ONEAPI;
1921

2022
int main() {
@@ -67,60 +69,62 @@ int main() {
6769

6870
if (!CPUs.empty()) {
6971
std::cout << "Test 'cpu'";
70-
device d1(filter_selector("cpu"));
72+
// TODO: change all occurrences of filter_selector to 'filter_selector' or
73+
// 'oneapi::filter_selector' after PR intel/llvm:4014 is merged
74+
device d1(ONEAPI::filter_selector("cpu"));
7175
assert(d1.is_cpu());
7276
std::cout << "...PASS" << std::endl;
7377
}
7478

7579
if (!GPUs.empty()) {
7680
std::cout << "Test 'gpu'";
77-
device d2(filter_selector("gpu"));
81+
device d2(ONEAPI::filter_selector("gpu"));
7882
assert(d2.is_gpu());
7983
std::cout << "...PASS" << std::endl;
8084
}
8185

8286
if (!CPUs.empty() || !GPUs.empty()) {
8387
std::cout << "Test 'cpu,gpu'";
84-
device d3(filter_selector("cpu,gpu"));
88+
device d3(ONEAPI::filter_selector("cpu,gpu"));
8589
assert((d3.is_gpu() || d3.is_cpu()));
8690
std::cout << "...PASS" << std::endl;
8791
}
8892

8993
if (HasOpenCLDevices) {
9094
std::cout << "Test 'opencl'";
91-
device d4(filter_selector("opencl"));
95+
device d4(ONEAPI::filter_selector("opencl"));
9296
assert(d4.get_platform().get_backend() == backend::opencl);
9397
std::cout << "...PASS" << std::endl;
9498

9599
if (!CPUs.empty()) {
96100
std::cout << "Test 'opencl:cpu'";
97-
device d5(filter_selector("opencl:cpu"));
101+
device d5(ONEAPI::filter_selector("opencl:cpu"));
98102
assert(d5.is_cpu() && d5.get_platform().get_backend() == backend::opencl);
99103
std::cout << "...PASS" << std::endl;
100104

101105
std::cout << "Test 'opencl:cpu:0'";
102-
device d6(filter_selector("opencl:cpu:0"));
106+
device d6(ONEAPI::filter_selector("opencl:cpu:0"));
103107
assert(d6.is_cpu() && d6.get_platform().get_backend() == backend::opencl);
104108
std::cout << "...PASS" << std::endl;
105109
}
106110

107111
if (HasOpenCLGPU) {
108112
std::cout << "Test 'opencl:gpu'" << std::endl;
109-
device d7(filter_selector("opencl:gpu"));
113+
device d7(ONEAPI::filter_selector("opencl:gpu"));
110114
assert(d7.is_gpu() && d7.get_platform().get_backend() == backend::opencl);
111115
}
112116
}
113117

114118
std::cout << "Test '0'";
115-
device d8(filter_selector("0"));
119+
device d8(ONEAPI::filter_selector("0"));
116120
std::cout << "...PASS" << std::endl;
117121

118122
std::string ErrorMesg(
119123
"Could not find a device that matches the specified filter(s)!");
120124

121125
try {
122126
// pick something crazy
123-
device d9(filter_selector("gpu:999"));
127+
device d9(ONEAPI::filter_selector("gpu:999"));
124128
std::cout << "d9 = " << d9.get_info<info::device::name>() << std::endl;
125129
} catch (const sycl::runtime_error &e) {
126130
assert(ErrorMesg.find_first_of(e.what()) == 0);
@@ -129,7 +133,7 @@ int main() {
129133

130134
try {
131135
// pick something crazy
132-
device d10(filter_selector("bob:gpu"));
136+
device d10(ONEAPI::filter_selector("bob:gpu"));
133137
std::cout << "d10 = " << d10.get_info<info::device::name>() << std::endl;
134138
} catch (const sycl::runtime_error &e) {
135139
assert(ErrorMesg.find_first_of(e.what()) == 0);
@@ -138,7 +142,7 @@ int main() {
138142

139143
try {
140144
// pick something crazy
141-
device d11(filter_selector("opencl:bob"));
145+
device d11(ONEAPI::filter_selector("opencl:bob"));
142146
std::cout << "d11 = " << d11.get_info<info::device::name>() << std::endl;
143147
} catch (const sycl::runtime_error &e) {
144148
assert(ErrorMesg.find_first_of(e.what()) == 0);
@@ -147,19 +151,19 @@ int main() {
147151

148152
if (HasLevelZeroDevices && HasLevelZeroGPU) {
149153
std::cout << "Test 'level_zero'";
150-
device d12(filter_selector("level_zero"));
154+
device d12(ONEAPI::filter_selector("level_zero"));
151155
assert(d12.get_platform().get_backend() == backend::level_zero);
152156
std::cout << "...PASS" << std::endl;
153157

154158
std::cout << "Test 'level_zero:gpu'";
155-
device d13(filter_selector("level_zero:gpu"));
159+
device d13(ONEAPI::filter_selector("level_zero:gpu"));
156160
assert(d13.is_gpu() &&
157161
d13.get_platform().get_backend() == backend::level_zero);
158162
std::cout << "...PASS" << std::endl;
159163

160164
if (HasOpenCLDevices && !CPUs.empty()) {
161165
std::cout << "Test 'level_zero:gpu,cpu'";
162-
device d14(filter_selector("level_zero:gpu,cpu"));
166+
device d14(ONEAPI::filter_selector("level_zero:gpu,cpu"));
163167
assert((d14.is_gpu() || d14.is_cpu()));
164168
std::cout << "...PASS 1/2" << std::endl;
165169
if (d14.is_gpu()) {
@@ -171,25 +175,25 @@ int main() {
171175

172176
if (Devs.size() > 1) {
173177
std::cout << "Test '1'";
174-
device d15(filter_selector("1"));
178+
device d15(ONEAPI::filter_selector("1"));
175179
std::cout << "...PASS" << std::endl;
176180
}
177181

178182
if (HasCUDADevices) {
179183
std::cout << "Test 'cuda'";
180-
device d16(filter_selector("cuda"));
184+
device d16(ONEAPI::filter_selector("cuda"));
181185
assert(d16.get_platform().get_backend() == backend::cuda);
182186
std::cout << "...PASS" << std::endl;
183187

184188
std::cout << "Test 'cuda:gpu'";
185-
device d17(filter_selector("cuda:gpu"));
189+
device d17(ONEAPI::filter_selector("cuda:gpu"));
186190
assert(d17.is_gpu() && d17.get_platform().get_backend() == backend::cuda);
187191
std::cout << "...PASS" << std::endl;
188192
}
189193

190194
if (!Accels.empty()) {
191195
std::cout << "Test 'accelerator'";
192-
device d18(filter_selector("accelerator"));
196+
device d18(ONEAPI::filter_selector("accelerator"));
193197
assert(d18.is_accelerator());
194198
std::cout << "...PASS" << std::endl;
195199
}

0 commit comments

Comments
 (0)