Skip to content

Commit

Permalink
Merge pull request #2088 from kbenzie/benie/cts-fix-windows-build
Browse files Browse the repository at this point in the history
Fix Windows CTS with kernels build
  • Loading branch information
kbenzie authored Sep 17, 2024
2 parents cfecab0 + 291c0c1 commit 3cd6eae
Show file tree
Hide file tree
Showing 9 changed files with 12 additions and 4 deletions.
7 changes: 4 additions & 3 deletions source/adapters/opencl/device.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -450,15 +450,16 @@ UR_APIEXPORT ur_result_t UR_APICALL urDeviceGetInfo(ur_device_handle_t hDevice,
URValue[i].type = static_cast<ur_device_partition_t>(CLValue[0]);
switch (URValue[i].type) {
case UR_DEVICE_PARTITION_EQUALLY: {
URValue[i].value.equally = CLValue[i + 1];
URValue[i].value.equally = static_cast<uint32_t>(CLValue[i + 1]);
break;
}
case UR_DEVICE_PARTITION_BY_COUNTS: {
URValue[i].value.count = CLValue[i + 1];
URValue[i].value.count = static_cast<uint32_t>(CLValue[i + 1]);
break;
}
case UR_DEVICE_PARTITION_BY_AFFINITY_DOMAIN: {
URValue[i].value.affinity_domain = CLValue[i + 1];
URValue[i].value.affinity_domain =
static_cast<uint32_t>(CLValue[i + 1]);
break;
}
default: {
Expand Down
2 changes: 1 addition & 1 deletion source/adapters/opencl/kernel.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -302,7 +302,7 @@ UR_APIEXPORT ur_result_t UR_APICALL urKernelSetExecInfo(

switch (propName) {
case UR_KERNEL_EXEC_INFO_USM_INDIRECT_ACCESS: {
if (*(static_cast<const ur_bool_t *>(pPropValue)) == true) {
if (*(static_cast<const ur_bool_t *>(pPropValue))) {
UR_RETURN_ON_FAILURE(usmSetIndirectAccess(hKernel));
}
return UR_RESULT_SUCCESS;
Expand Down
1 change: 1 addition & 0 deletions test/conformance/enqueue/urEnqueueKernelLaunch.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
// See LICENSE.TXT
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception

#include <array>
#include <uur/fixtures.h>

struct urEnqueueKernelLaunchTest : uur::urKernelExecutionTest {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception

#include "fixtures.h"
#include <array>

// Test that updating a command-buffer with a single kernel command
// taking buffer & scalar arguments works correctly.
Expand Down
1 change: 1 addition & 0 deletions test/conformance/exp_command_buffer/commands.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception

#include "fixtures.h"
#include <array>

struct urCommandBufferCommandsTest
: uur::command_buffer::urCommandBufferExpTest {
Expand Down
1 change: 1 addition & 0 deletions test/conformance/exp_command_buffer/ndrange_update.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception

#include "fixtures.h"
#include <array>
#include <cstring>

// Test that updating a command-buffer with a single kernel command
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception

#include "fixtures.h"
#include <array>
#include <cstring>

// Test that updating a command-buffer with a single kernel command
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception

#include "fixtures.h"
#include <array>
#include <cstring>

// Test that updating a command-buffer with a single kernel command
Expand Down
1 change: 1 addition & 0 deletions test/conformance/kernel/urKernelGetGroupInfo.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
// See LICENSE.TXT
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception

#include <array>
#include <uur/fixtures.h>

using urKernelGetGroupInfoTest =
Expand Down

0 comments on commit 3cd6eae

Please sign in to comment.