Skip to content

Commit 463d6e4

Browse files
committed
only setup the OpenCL device, not the OpenCL platform
Very few samples need the OpenCL platform, and when they do, they can query it from the device.
1 parent e6e40cf commit 463d6e4

39 files changed

Lines changed: 54 additions & 85 deletions

File tree

include/util.hpp

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -124,8 +124,7 @@ static bool checkDeviceIndex(
124124
return true;
125125
}
126126

127-
static bool setupPlatformAndDevice(
128-
cl::Platform& platform,
127+
static bool setupDevice(
129128
cl::Device& device,
130129
int platformIndex,
131130
int deviceIndex,
@@ -138,7 +137,7 @@ static bool setupPlatformAndDevice(
138137
return false;
139138
}
140139

141-
platform = std::move(platforms[platformIndex]);
140+
const cl::Platform& platform = platforms[platformIndex];
142141
printf("Running on platform: %s\n",
143142
platform.getInfo<CL_PLATFORM_NAME>().c_str() );
144143

samples/01_copybuffer/main.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,9 +40,8 @@ int main(int argc, char** argv)
4040
}
4141
}
4242

43-
cl::Platform platform;
4443
cl::Device device;
45-
if (!setupPlatformAndDevice(platform, device, platformIndex, deviceIndex, verbose)) {
44+
if (!setupDevice(device, platformIndex, deviceIndex, verbose)) {
4645
return -1;
4746
}
4847

samples/02_copybufferkernel/main.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,8 @@ int main(int argc, char** argv)
4848
}
4949
}
5050

51-
cl::Platform platform;
5251
cl::Device device;
53-
if (!setupPlatformAndDevice(platform, device, platformIndex, deviceIndex, verbose)) {
52+
if (!setupDevice(device, platformIndex, deviceIndex, verbose)) {
5453
return -1;
5554
}
5655

samples/03_mandelbrot/main.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -83,9 +83,8 @@ int main(int argc, char** argv)
8383
}
8484
}
8585

86-
cl::Platform platform;
8786
cl::Device device;
88-
if (!setupPlatformAndDevice(platform, device, platformIndex, deviceIndex, verbose)) {
87+
if (!setupDevice(device, platformIndex, deviceIndex, verbose)) {
8988
return -1;
9089
}
9190

samples/04_julia/main.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -103,9 +103,8 @@ int main(int argc, char** argv)
103103
}
104104
}
105105

106-
cl::Platform platform;
107106
cl::Device device;
108-
if (!setupPlatformAndDevice(platform, device, platformIndex, deviceIndex, verbose)) {
107+
if (!setupDevice(device, platformIndex, deviceIndex, verbose)) {
109108
return -1;
110109
}
111110

samples/04_sobel/main.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -150,9 +150,8 @@ int main(int argc, char** argv)
150150
}
151151
}
152152

153-
cl::Platform platform;
154153
cl::Device device;
155-
if (!setupPlatformAndDevice(platform, device, platformIndex, deviceIndex, verbose)) {
154+
if (!setupDevice(device, platformIndex, deviceIndex, verbose)) {
156155
return -1;
157156
}
158157

samples/05_kernelfromfile/main.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,9 +50,8 @@ int main(int argc, char** argv)
5050
}
5151
}
5252

53-
cl::Platform platform;
5453
cl::Device device;
55-
if (!setupPlatformAndDevice(platform, device, platformIndex, deviceIndex, verbose)) {
54+
if (!setupDevice(device, platformIndex, deviceIndex, verbose)) {
5655
return -1;
5756
}
5857

samples/05_spirvkernelfromfile/main.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -132,9 +132,8 @@ int main(int argc, char** argv)
132132
}
133133
}
134134

135-
cl::Platform platform;
136135
cl::Device device;
137-
if (!setupPlatformAndDevice(platform, device, platformIndex, deviceIndex, verbose)) {
136+
if (!setupDevice(device, platformIndex, deviceIndex, verbose)) {
138137
return -1;
139138
}
140139

samples/06_ndrangekernelfromfile/main.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,9 +65,8 @@ int main(int argc, char** argv)
6565
}
6666
}
6767

68-
cl::Platform platform;
6968
cl::Device device;
70-
if (!setupPlatformAndDevice(platform, device, platformIndex, deviceIndex, verbose)) {
69+
if (!setupDevice(device, platformIndex, deviceIndex, verbose)) {
7170
return -1;
7271
}
7372

samples/10_queueexperiments/main.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -412,9 +412,8 @@ int main(int argc, char** argv)
412412
numKernels = maxKernels;
413413
}
414414

415-
cl::Platform platform;
416415
cl::Device device;
417-
if (!setupPlatformAndDevice(platform, device, platformIndex, deviceIndex, verbose)) {
416+
if (!setupDevice(device, platformIndex, deviceIndex, verbose)) {
418417
return -1;
419418
}
420419

0 commit comments

Comments
 (0)