Skip to content

Feature/get ref address #105

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
PokhodenkoSA merged 12 commits into from
Oct 6, 2020
Merged

Feature/get ref address #105

Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
ae6873d
A current device is always present even outside a device context.
Oct 4, 2020
96de2e1
Allow program creation only for OpenCL for the time being.
Oct 4, 2020
aec7ba3
Fix test cases.
Oct 4, 2020
f109abb
Fix errors found for Sergey.
Oct 4, 2020
9597ee1
Remove useless except+, as C++ exceptions do not cross C API.
Oct 4, 2020
1de7ff4
Add support for uint8 and ulong.
Oct 4, 2020
35c4748
Add support for uint8 and ulong.
Oct 4, 2020
1e0db44
Merge branch 'fix/issue_83' of github.com:diptorupd/dpctl into fix/is…
Oct 5, 2020
f765236
Merge branch 'diptorupd-fix/issue_83' into upstream_master
Oct 5, 2020
d10f0e3
Merge branch 'master' of github.com:IntelPython/dpctl into upstream_m…
Oct 5, 2020
d041b06
Add functions to get address of ref
reazulhoque Oct 2, 2020
b2f7831
Rename all functions to addressof_ref and add docstring.
Oct 5, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
57 changes: 57 additions & 0 deletions dpctl/sycl_core.pyx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,15 @@ cdef class SyclContext:
cdef DPPLSyclContextRef get_context_ref (self):
return self._ctxt_ref

def addressof_ref (self):
"""Returns the address of the DPPLSyclContextRef pointer as a
long.

Returns:
The address of the DPPLSyclContextRef object used to create this
SyclContext cast to a long.
"""
return int(<long>self._ctx_ref)

cdef class SyclDevice:
''' Wrapper class for a Sycl Device
Expand Down Expand Up @@ -163,6 +172,15 @@ cdef class SyclDevice:
'''
return self._device_ref

def addressof_ref (self):
"""Returns the address of the DPPLSyclDeviceRef pointer as a
long.

Returns:
The address of the DPPLSyclDeviceRef object used to create this
SyclDevice cast to a long.
"""
return int(<long>self._device_ref)

cdef class SyclEvent:
''' Wrapper class for a Sycl Event
Expand All @@ -187,6 +205,16 @@ cdef class SyclEvent:
cpdef void wait (self):
DPPLEvent_Wait(self._event_ref)

def addressof_ref (self):
"""Returns the address of the C API DPPLSyclEventRef pointer as
a long.

Returns:
The address of the DPPLSyclEventRef object used to create this
SyclEvent cast to a long.
"""
return int(<long>self._event_ref)


cdef class SyclKernel:
''' Wraps a sycl::kernel object created from an OpenCL interoperability
Expand Down Expand Up @@ -219,6 +247,15 @@ cdef class SyclKernel:
'''
return self._kernel_ref

def addressof_ref (self):
"""Returns the address of the C API DPPLSyclKernelRef pointer
as a long.

Returns:
The address of the DPPLSyclKernelRef object used to create this
SyclKernel cast to a long.
"""
return int(<long>self._kernel_ref)

cdef class SyclProgram:
''' Wraps a sycl::program object created from an OpenCL interoperability
Expand Down Expand Up @@ -250,6 +287,16 @@ cdef class SyclProgram:
name = kernel_name.encode('utf8')
return DPPLProgram_HasKernel(self._program_ref, name)

def addressof_ref (self):
"""Returns the address of the C API DPPLSyclProgramRef pointer
as a long.

Returns:
The address of the DPPLSyclProgramRef object used to create this
SyclProgram cast to a long.
"""
return int(<long>self._program_ref)
Copy link
Contributor

@oleksandr-pavlyk oleksandr-pavlyk Oct 6, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@PokhodenkoSA @diptorupd

Long is not good here. It is a 32-bit integer on Windows. We should be using <size_t> instead.


import ctypes

cdef class SyclQueue:
Expand Down Expand Up @@ -380,6 +427,16 @@ cdef class SyclQueue:
cdef DPPLSyclQueueRef get_queue_ref (self):
return self._queue_ref

def addressof_ref (self):
"""Returns the address of the C API DPPLSyclQueueRef pointer as
a long.

Returns:
The address of the DPPLSyclQueueRef object used to create this
SyclQueue cast to a long.
"""
return int(<long>self._queue_ref)

cpdef SyclEvent submit (self, SyclKernel kernel, list args, list gS, \
list lS = None, list dEvents = None):

Expand Down