Skip to content

Commit f8480a0

Browse files
committed
Rectification of a stupid error.
Rectified merge conflict issues. Signed-off-by: Garima Gupta <[email protected]>
1 parent 7cc25c4 commit f8480a0

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

sycl/source/detail/kernel_impl.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -31,13 +31,13 @@ kernel_impl::kernel_impl(RT::PiKernel Kernel, const context &SyclContext,
3131
MCreatedFromSource(IsCreatedFromSource) {
3232

3333
RT::PiContext Context = nullptr;
34-
PI_CALL(RT::piKernelGetInfo, MKernel, CL_KERNEL_CONTEXT, sizeof(Context),
35-
&Context, nullptr);
34+
PI_CALL(piKernelGetInfo)(MKernel, CL_KERNEL_CONTEXT, sizeof(Context),
35+
&Context, nullptr);
3636
auto ContextImpl = detail::getSyclObjImpl(SyclContext);
3737
if (ContextImpl->getHandleRef() != Context)
3838
throw cl::sycl::invalid_parameter_error(
3939
"Input context must be the same as the context of cl_kernel");
40-
PI_CALL(RT::piKernelRetain, MKernel);
40+
PI_CALL(piKernelRetain)(MKernel);
4141
}
4242

4343
kernel_impl::kernel_impl(const context &SyclContext,
@@ -47,7 +47,7 @@ kernel_impl::kernel_impl(const context &SyclContext,
4747
kernel_impl::~kernel_impl() {
4848
// TODO catch an exception and put it to list of asynchronous exceptions
4949
if (!is_host()) {
50-
PI_CALL(RT::piKernelRelease, MKernel);
50+
PI_CALL(piKernelRelease)(MKernel);
5151
}
5252
}
5353

sycl/source/detail/pi.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -140,7 +140,9 @@ void assertion(bool Condition, const char *Message) {
140140
// implemented yet.
141141

142142
#define _PI_API(api) \
143-
template <> CallPi<decltype(&::api)>::CallPi() { \
143+
template <> \
144+
CallPi<decltype(&::api), \
145+
(offsetof(pi_plugin::FunctionPointers, api))>::CallPi() { \
144146
initialize(); \
145147
MFnPtr = (RT::PluginInformation.PiFunctionTable.api); \
146148
MFnName = #api; \

0 commit comments

Comments
 (0)