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

Commit fb0da8f

Browse files
[SYCL] Fix MemObjRemapping for devices without host unified memory (#167)
The check for host unified memory within the test was insufficient, since the use of FileCheck ignores it. Instead, force the runtime to behave as if all devices support host unified memory for this test.
1 parent 5e9e234 commit fb0da8f

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

SYCL/Scheduler/MemObjRemapping.cpp

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
// RUN: %clangxx -fsycl %s -o %t.out
2-
// RUN: env SYCL_PI_TRACE=2 %CPU_RUN_PLACEHOLDER %t.out 2>&1 %CPU_CHECK_PLACEHOLDER
3-
// RUN: env SYCL_PI_TRACE=2 %GPU_RUN_PLACEHOLDER %t.out 2>&1 %GPU_CHECK_PLACEHOLDER
2+
// RUN: env SYCL_HOST_UNIFIED_MEMORY=1 SYCL_PI_TRACE=2 %CPU_RUN_PLACEHOLDER %t.out 2>&1 %CPU_CHECK_PLACEHOLDER
3+
// RUN: env SYCL_HOST_UNIFIED_MEMORY=1 SYCL_PI_TRACE=2 %GPU_RUN_PLACEHOLDER %t.out 2>&1 %GPU_CHECK_PLACEHOLDER
44
// XFAIL: cuda
55
#include <CL/sycl.hpp>
66
#include <cassert>
@@ -16,9 +16,6 @@ class Bar;
1616
// read-write.
1717
int main() {
1818
queue Q;
19-
// Mapping is not used for devices without host unified memory support.
20-
if (!Q.get_device().get_info<info::device::host_unified_memory>())
21-
return 0;
2219

2320
std::size_t Size = 64;
2421
range<1> Range{Size};

0 commit comments

Comments
 (0)