Skip to content

Feature/program interface #47

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.

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

Closed

Conversation

diptorupd
Copy link
Contributor

Adds wrappers for program and kernel. Needs more unit tests to check building program from SPIR-V and the Cython interface.

PokhodenkoSA and others added 30 commits September 15, 2020 10:33
…erfaces.

     - Separates out the queue manager from the sycl queue interface wrapper.
     - Added API to get context, device froma  Sycl queue.
     - Added C-API wrappers for sycl device, queue, context interfaces exposing
       some of the API functionalities. More can be added as needed.
     - The dppl extension now defines extension types for sycl quque, device, context to
       mimic the C API.
     - Sycl queues are now returned to callers as a SyclQueue extension type rather than
       a Py_capsule.
export OpenCL_LIBDIR=${DPCPP_ROOT}/lib
export DPPL_OPENCL_INTERFACE_LIBDIR=dpctl
export DPPL_OPENCL_INTERFACE_INCLDIR=dpctl/include
export OpenCL_LIBDIR=${DPCPP_ROOT}/libw
Copy link
Contributor

Choose a reason for hiding this comment

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

libw -> lib? typo?

Copy link
Contributor

@PokhodenkoSA PokhodenkoSA Sep 29, 2020

Choose a reason for hiding this comment

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

I think lib is correct.

@diptorupd
Copy link
Contributor Author

Superseded by #80. Closing this PR.

@diptorupd diptorupd closed this Sep 30, 2020
@diptorupd diptorupd deleted the feature/program_interface branch October 3, 2020 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants