Skip to content

Commit 2a05505

Browse files
hdelanldrumm
andauthored
Apply suggestions from code review
Co-authored-by: ldrumm <[email protected]>
1 parent c35df73 commit 2a05505

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

sycl/source/detail/buffer_impl.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ buffer_impl::getNativeVector(backend BackendName) const {
8484
}
8585

8686
pi_native_handle Handle;
87-
Plugin->call<PiApiKind::piextMemGetNativeHandle>(NativeMem, nullptr,
87+
Plugin->call<PiApiKind::piextMemGetNativeHandle>(NativeMem, /*Dev*/nullptr,
8888
&Handle);
8989
Handles.push_back(Handle);
9090
}

sycl/source/detail/memory_manager.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,7 @@ void memBufferCreateHelper(const PluginPtr &Plugin, pi_context Ctx,
144144
// Always use call_nocheck here, because call may throw an exception,
145145
// and this lambda will be called from destructor, which in combination
146146
// rewards us with UB.
147-
Plugin->call_nocheck<PiApiKind::piextMemGetNativeHandle>(*RetMem, nullptr,
147+
Plugin->call_nocheck<PiApiKind::piextMemGetNativeHandle>(*RetMem, /*Dev*/nullptr,
148148
&Ptr);
149149
emitMemAllocEndTrace(MemObjID, (uintptr_t)(Ptr), Size, 0 /* guard zone */,
150150
CorrID);
@@ -168,7 +168,7 @@ void memReleaseHelper(const PluginPtr &Plugin, pi_mem Mem) {
168168
// Do not make unnecessary PI calls without instrumentation enabled
169169
if (xptiTraceEnabled()) {
170170
pi_native_handle PtrHandle = 0;
171-
Plugin->call<PiApiKind::piextMemGetNativeHandle>(Mem, nullptr, &PtrHandle);
171+
Plugin->call<PiApiKind::piextMemGetNativeHandle>(Mem, /*Dev*/nullptr, &PtrHandle);
172172
Ptr = (uintptr_t)(PtrHandle);
173173
}
174174
#endif

sycl/test-e2e/HostInteropTask/interop-task-hip.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ void checkBufferValues(BufferT Buffer, ValueT Value) {
2626
for (size_t Idx = 0; Idx < Acc.get_count(); ++Idx) {
2727
if (Acc[Idx] != Value) {
2828
std::cerr << "buffer[" << Idx << "] = " << Acc[Idx]
29-
<< ", expected val = " << Value << std::endl;
29+
<< ", expected val = " << Value << '\n';
3030
assert(0 && "Invalid data in the buffer");
3131
}
3232
}

0 commit comments

Comments
 (0)