Skip to content

Commit 392f4d8

Browse files
Jaime Arteagakchusha
authored andcommitted
[SYCL][UR][L0] Fix some CI errors (#9628)
Signed-off-by: Jaime Arteaga <[email protected]>
1 parent 143e1ca commit 392f4d8

File tree

7 files changed

+21
-5
lines changed

7 files changed

+21
-5
lines changed

sycl/plugins/unified_runtime/CMakeLists.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@
33
if (NOT DEFINED UNIFIED_RUNTIME_LIBRARY OR NOT DEFINED UNIFIED_RUNTIME_INCLUDE_DIR)
44
include(FetchContent)
55

6-
set(UNIFIED_RUNTIME_REPO "https://github.com/oneapi-src/unified-runtime.git")
7-
set(UNIFIED_RUNTIME_TAG 620ddb1e8bb1f5ef6cc775edf79ba4674057fe2e)
6+
set(UNIFIED_RUNTIME_REPO "https://github.com/jandres742/unified-runtime.git")
7+
set(UNIFIED_RUNTIME_TAG c4570a25e3493454ab6c4cd0192f270962f77a3c)
88

99
message(STATUS "Will fetch Unified Runtime from ${UNIFIED_RUNTIME_REPO}")
1010
FetchContent_Declare(unified-runtime

sycl/plugins/unified_runtime/pi2ur.hpp

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1137,6 +1137,9 @@ inline pi_result piContextCreate(const pi_context_properties *Properties,
11371137
const void *PrivateInfo,
11381138
size_t CB, void *UserData),
11391139
void *UserData, pi_context *RetContext) {
1140+
std::ignore = Properties;
1141+
std::ignore = PFnNotify;
1142+
std::ignore = UserData;
11401143
auto UrDevices = reinterpret_cast<const ur_device_handle_t *>(Devices);
11411144

11421145
ur_context_handle_t *UrContext =
@@ -1213,8 +1216,8 @@ inline pi_result piContextGetInfo(pi_context Context, pi_context_info ParamName,
12131216
ContextInfoType = UR_CONTEXT_INFO_REFERENCE_COUNT;
12141217
break;
12151218
}
1216-
case PI_EXT_ONEAPI_CONTEXT_INFO_USM_FILL2D_SUPPORT: {
1217-
case PI_EXT_ONEAPI_CONTEXT_INFO_USM_MEMSET2D_SUPPORT:
1219+
case PI_EXT_ONEAPI_CONTEXT_INFO_USM_FILL2D_SUPPORT:
1220+
case PI_EXT_ONEAPI_CONTEXT_INFO_USM_MEMSET2D_SUPPORT: {
12181221
ContextInfoType = UR_CONTEXT_INFO_USM_FILL2D_SUPPORT;
12191222
break;
12201223
}
@@ -2337,6 +2340,7 @@ inline pi_result piextUSMHostAlloc(void **ResultPtr, pi_context Context,
23372340
pi_usm_mem_properties *Properties,
23382341
size_t Size, pi_uint32 Alignment) {
23392342

2343+
std::ignore = Properties;
23402344
ur_context_handle_t UrContext =
23412345
reinterpret_cast<ur_context_handle_t>(Context);
23422346
ur_usm_desc_t USMDesc{};
@@ -2744,6 +2748,7 @@ inline pi_result piextUSMDeviceAlloc(void **ResultPtr, pi_context Context,
27442748
pi_usm_mem_properties *Properties,
27452749
size_t Size, pi_uint32 Alignment) {
27462750

2751+
std::ignore = Properties;
27472752
ur_context_handle_t UrContext =
27482753
reinterpret_cast<ur_context_handle_t>(Context);
27492754
auto UrDevice = reinterpret_cast<ur_device_handle_t>(Device);
@@ -2762,6 +2767,7 @@ inline pi_result piextUSMSharedAlloc(void **ResultPtr, pi_context Context,
27622767
pi_usm_mem_properties *Properties,
27632768
size_t Size, pi_uint32 Alignment) {
27642769

2770+
std::ignore = Properties;
27652771
if (Properties && *Properties != 0) {
27662772
PI_ASSERT(*(Properties) == PI_MEM_ALLOC_FLAGS && *(Properties + 2) == 0,
27672773
PI_ERROR_INVALID_VALUE);

sycl/plugins/unified_runtime/ur/adapters/level_zero/ur_level_zero_device.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1180,6 +1180,7 @@ UR_APIEXPORT ur_result_t UR_APICALL urDeviceSelectBinary(
11801180
///< array of binaries. If a suitable binary was not
11811181
///< found the function returns ${X}_INVALID_BINARY.
11821182
) {
1183+
std::ignore = Device;
11831184
// TODO: this is a bare-bones implementation for choosing a device image
11841185
// that would be compatible with the targeted device. An AOT-compiled
11851186
// image is preferred over SPIR-V for known devices (i.e. Intel devices)

sycl/plugins/unified_runtime/ur/adapters/level_zero/ur_level_zero_mem.cpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1236,6 +1236,7 @@ UR_APIEXPORT ur_result_t UR_APICALL urEnqueueUSMPrefetch(
12361236
*OutEvent ///< [in,out][optional] return an event object that identifies
12371237
///< this particular command instance.
12381238
) {
1239+
std::ignore = Flags;
12391240
// Lock automatically releases when this goes out of scope.
12401241
std::scoped_lock<ur_shared_mutex> lock(Queue->Mutex);
12411242

@@ -1826,6 +1827,7 @@ UR_APIEXPORT ur_result_t UR_APICALL urMemBufferPartition(
18261827
ur_mem_handle_t
18271828
*RetMem ///< [out] pointer to the handle of sub buffer created
18281829
) {
1830+
std::ignore = BufferCreateType;
18291831
UR_ASSERT(Buffer && !Buffer->isImage() &&
18301832
!(static_cast<_ur_buffer *>(Buffer))->isSubBuffer(),
18311833
UR_RESULT_ERROR_INVALID_MEM_OBJECT);

sycl/plugins/unified_runtime/ur/adapters/level_zero/ur_level_zero_platform.cpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -290,6 +290,7 @@ UR_APIEXPORT ur_result_t UR_APICALL urPlatformGetApiVersion(
290290
ur_platform_handle_t Driver, ///< [in] handle of the platform
291291
ur_api_version_t *Version ///< [out] api version
292292
) {
293+
std::ignore = Driver;
293294
*Version = UR_API_VERSION_0_6;
294295
return UR_RESULT_SUCCESS;
295296
}
@@ -550,6 +551,7 @@ UR_APIEXPORT ur_result_t UR_APICALL urPlatformGetBackendOption(
550551
*PlatformOption ///< [out] returns the correct platform specific
551552
///< compiler option based on the frontend option.
552553
) {
554+
std::ignore = Platform;
553555
using namespace std::literals;
554556
if (FrontendOption == nullptr) {
555557
return UR_RESULT_SUCCESS;

sycl/plugins/unified_runtime/ur/adapters/level_zero/ur_level_zero_program.cpp

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,7 @@ UR_APIEXPORT ur_result_t UR_APICALL urProgramCreateWithIL(
5454
ur_program_handle_t
5555
*Program ///< [out] pointer to handle of program object created.
5656
) {
57+
std::ignore = Properties;
5758
try {
5859
ur_program_handle_t_ *UrProgram =
5960
new ur_program_handle_t_(ur_program_handle_t_::IL, Context, IL, Length);
@@ -78,6 +79,8 @@ UR_APIEXPORT ur_result_t UR_APICALL urProgramCreateWithBinary(
7879
ur_program_handle_t
7980
*Program ///< [out] pointer to handle of Program object created.
8081
) {
82+
std::ignore = Device;
83+
std::ignore = Properties;
8184
// In OpenCL, clCreateProgramWithBinary() can be used to load any of the
8285
// following: "program executable", "compiled program", or "library of
8386
// compiled programs". In addition, the loaded program can be either
@@ -191,7 +194,7 @@ UR_APIEXPORT ur_result_t UR_APICALL urProgramCompile(
191194
const char *Options ///< [in][optional] pointer to build options
192195
///< null-terminated string.
193196
) {
194-
197+
std::ignore = Context;
195198
std::scoped_lock<ur_shared_mutex> Guard(Program->Mutex);
196199

197200
// It's only valid to compile a program created from IL (we don't support

sycl/plugins/unified_runtime/ur/adapters/level_zero/ur_level_zero_usm.cpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -632,6 +632,7 @@ ur_result_t USMDeviceAllocImpl(void **ResultPtr, ur_context_handle_t Context,
632632
ur_device_handle_t Device,
633633
ur_usm_device_mem_flags_t *Flags, size_t Size,
634634
uint32_t Alignment) {
635+
std::ignore = Flags;
635636
// TODO: translate PI properties to Level Zero flags
636637
ZeStruct<ze_device_mem_alloc_desc_t> ZeDesc;
637638
ZeDesc.flags = 0;
@@ -693,6 +694,7 @@ ur_result_t USMSharedAllocImpl(void **ResultPtr, ur_context_handle_t Context,
693694
ur_result_t USMHostAllocImpl(void **ResultPtr, ur_context_handle_t Context,
694695
ur_usm_host_mem_flags_t *Flags, size_t Size,
695696
uint32_t Alignment) {
697+
std::ignore = Flags;
696698
// TODO: translate PI properties to Level Zero flags
697699
ZeStruct<ze_host_mem_alloc_desc_t> ZeHostDesc;
698700
ZeHostDesc.flags = 0;

0 commit comments

Comments
 (0)