Skip to content

Commit 9cb42ef

Browse files
[SYCL][UR] do not expose UR as a backend of SYCL (#9013)
Signed-off-by: Sergey V Maslov <[email protected]>
1 parent 2880673 commit 9cb42ef

File tree

8 files changed

+7
-27
lines changed

8 files changed

+7
-27
lines changed

sycl/include/sycl/backend_types.hpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,6 @@ enum class backend : char {
3232
ext_intel_esimd_emulator,
3333
ext_oneapi_hip = 6,
3434
hip __SYCL2020_DEPRECATED("use 'ext_oneapi_hip' instead") = ext_oneapi_hip,
35-
ext_oneapi_unified_runtime = 7,
3635
};
3736

3837
template <backend Backend> class backend_traits;
@@ -64,9 +63,6 @@ inline std::ostream &operator<<(std::ostream &Out, backend be) {
6463
case backend::ext_oneapi_hip:
6564
Out << "ext_oneapi_hip";
6665
break;
67-
case backend::ext_oneapi_unified_runtime:
68-
Out << "ext_oneapi_unified_runtime";
69-
break;
7066
case backend::all:
7167
Out << "all";
7268
}

sycl/include/sycl/detail/pi.hpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -70,29 +70,25 @@ bool trace(TraceLevel level);
7070
#define __SYCL_CUDA_PLUGIN_NAME "pi_cuda.dll"
7171
#define __SYCL_ESIMD_EMULATOR_PLUGIN_NAME "pi_esimd_emulator.dll"
7272
#define __SYCL_HIP_PLUGIN_NAME "libpi_hip.dll"
73-
#define __SYCL_UNIFIED_RUNTIME_PLUGIN_NAME "pi_unified_runtime.dll"
7473
#else
7574
#define __SYCL_OPENCL_PLUGIN_NAME "libpi_opencl.dll"
7675
#define __SYCL_LEVEL_ZERO_PLUGIN_NAME "libpi_level_zero.dll"
7776
#define __SYCL_CUDA_PLUGIN_NAME "libpi_cuda.dll"
7877
#define __SYCL_ESIMD_EMULATOR_PLUGIN_NAME "libpi_esimd_emulator.dll"
7978
#define __SYCL_HIP_PLUGIN_NAME "libpi_hip.dll"
80-
#define __SYCL_UNIFIED_RUNTIME_PLUGIN_NAME "libpi_unified_runtime.dll"
8179
#endif
8280
#elif defined(__SYCL_RT_OS_LINUX)
8381
#define __SYCL_OPENCL_PLUGIN_NAME "libpi_opencl.so"
8482
#define __SYCL_LEVEL_ZERO_PLUGIN_NAME "libpi_level_zero.so"
8583
#define __SYCL_CUDA_PLUGIN_NAME "libpi_cuda.so"
8684
#define __SYCL_ESIMD_EMULATOR_PLUGIN_NAME "libpi_esimd_emulator.so"
87-
#define __SYCL_UNIFIED_RUNTIME_PLUGIN_NAME "libpi_unified_runtime.so"
8885
#define __SYCL_HIP_PLUGIN_NAME "libpi_hip.so"
8986
#elif defined(__SYCL_RT_OS_DARWIN)
9087
#define __SYCL_OPENCL_PLUGIN_NAME "libpi_opencl.dylib"
9188
#define __SYCL_LEVEL_ZERO_PLUGIN_NAME "libpi_level_zero.dylib"
9289
#define __SYCL_CUDA_PLUGIN_NAME "libpi_cuda.dylib"
9390
#define __SYCL_ESIMD_EMULATOR_PLUGIN_NAME "libpi_esimd_emulator.dylib"
9491
#define __SYCL_HIP_PLUGIN_NAME "libpi_hip.dylib"
95-
#define __SYCL_UNIFIED_RUNTIME_PLUGIN_NAME "libpi_unified_runtime.dylib"
9692
#else
9793
#error "Unsupported OS"
9894
#endif

sycl/source/detail/config.cpp

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -172,15 +172,14 @@ getSyclDeviceTypeMap() {
172172

173173
// Array is used by SYCL_DEVICE_FILTER and SYCL_DEVICE_ALLOWLIST and
174174
// ONEAPI_DEVICE_SELECTOR
175-
const std::array<std::pair<std::string, backend>, 8> &getSyclBeMap() {
176-
static const std::array<std::pair<std::string, backend>, 8> SyclBeMap = {
175+
const std::array<std::pair<std::string, backend>, 7> &getSyclBeMap() {
176+
static const std::array<std::pair<std::string, backend>, 7> SyclBeMap = {
177177
{{"host", backend::host},
178178
{"opencl", backend::opencl},
179179
{"level_zero", backend::ext_oneapi_level_zero},
180180
{"cuda", backend::ext_oneapi_cuda},
181181
{"hip", backend::ext_oneapi_hip},
182182
{"esimd_emulator", backend::ext_intel_esimd_emulator},
183-
{"ext_oneapi_unified_runtime", backend::ext_oneapi_unified_runtime},
184183
{"*", backend::all}}};
185184
return SyclBeMap;
186185
}

sycl/source/detail/config.hpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -237,7 +237,7 @@ getSyclDeviceTypeMap();
237237

238238
// Array is used by SYCL_DEVICE_FILTER and SYCL_DEVICE_ALLOWLIST and
239239
// ONEAPI_DEVICE_SELECTOR
240-
const std::array<std::pair<std::string, backend>, 8> &getSyclBeMap();
240+
const std::array<std::pair<std::string, backend>, 7> &getSyclBeMap();
241241

242242
// ---------------------------------------
243243
// ONEAPI_DEVICE_SELECTOR support

sycl/source/detail/pi.cpp

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -280,8 +280,6 @@ std::vector<std::pair<std::string, backend>> findPlugins() {
280280
"conjunction with SYCL_DEVICE_FILTER");
281281
} else if (!FilterList && !OdsTargetList) {
282282
PluginNames.emplace_back(__SYCL_OPENCL_PLUGIN_NAME, backend::opencl);
283-
PluginNames.emplace_back(__SYCL_UNIFIED_RUNTIME_PLUGIN_NAME,
284-
backend::ext_oneapi_unified_runtime);
285283
PluginNames.emplace_back(__SYCL_LEVEL_ZERO_PLUGIN_NAME,
286284
backend::ext_oneapi_level_zero);
287285
PluginNames.emplace_back(__SYCL_CUDA_PLUGIN_NAME, backend::ext_oneapi_cuda);
@@ -329,10 +327,6 @@ std::vector<std::pair<std::string, backend>> findPlugins() {
329327
if (list.backendCompatible(backend::opencl)) {
330328
PluginNames.emplace_back(__SYCL_OPENCL_PLUGIN_NAME, backend::opencl);
331329
}
332-
if (list.backendCompatible(backend::ext_oneapi_unified_runtime)) {
333-
PluginNames.emplace_back(__SYCL_UNIFIED_RUNTIME_PLUGIN_NAME,
334-
backend::ext_oneapi_unified_runtime);
335-
}
336330
if (list.backendCompatible(backend::ext_oneapi_level_zero)) {
337331
PluginNames.emplace_back(__SYCL_LEVEL_ZERO_PLUGIN_NAME,
338332
backend::ext_oneapi_level_zero);

sycl/test-e2e/Plugin/sycl-ls-unified-runtime.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
// REQUIRES: TEMPORARY_DISABLED
22
// Unified Runtime will soon be changing its reporting.
33
//
4-
// RUN: env ONEAPI_DEVICE_SELECTOR="ext_oneapi_unified_runtime:*" sycl-ls --verbose 2>&1 %GPU_CHECK_PLACEHOLDER
4+
// RUN: env ONEAPI_DEVICE_SELECTOR="ext_oneapi_level_zero:*" sycl-ls --verbose 2>&1 %GPU_CHECK_PLACEHOLDER
55

66
// CHECK: Platforms: 1
77
// CHECK-NEXT: Platform [#1]:

sycl/test-e2e/Regression/device_num.cpp

100755100644
Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@ const std::map<backend, std::string> BackendStringMap = {
2626
{backend::opencl, "opencl"},
2727
{backend::host, "host"},
2828
{backend::ext_oneapi_level_zero, "ext_oneapi_level_zero"},
29-
{backend::ext_oneapi_unified_runtime, "ext_oneapi_unified_runtime"},
3029
{backend::ext_intel_esimd_emulator, "ext_intel_esimd_emulator"},
3130
{backend::ext_oneapi_cuda, "ext_oneapi_cuda"},
3231
{backend::ext_oneapi_hip, "ext_oneapi_hip"}};

sycl/unittests/allowlist/ParseAllowList.cpp

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -167,13 +167,9 @@ TEST(ParseAllowListTests, CheckAllValidBackendNameValuesAreProcessed) {
167167
sycl::detail::AllowListParsedT ActualValue =
168168
sycl::detail::parseAllowList(AllowList);
169169
sycl::detail::AllowListParsedT ExpectedValue{
170-
{{"BackendName", "host"}},
171-
{{"BackendName", "opencl"}},
172-
{{"BackendName", "level_zero"}},
173-
{{"BackendName", "cuda"}},
174-
{{"BackendName", "hip"}},
175-
{{"BackendName", "esimd_emulator"}},
176-
{{"BackendName", "ext_oneapi_unified_runtime"}},
170+
{{"BackendName", "host"}}, {{"BackendName", "opencl"}},
171+
{{"BackendName", "level_zero"}}, {{"BackendName", "cuda"}},
172+
{{"BackendName", "hip"}}, {{"BackendName", "esimd_emulator"}},
177173
{{"BackendName", "*"}}};
178174
EXPECT_EQ(ExpectedValue, ActualValue);
179175
}

0 commit comments

Comments
 (0)