Skip to content
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

[SYCL][CUDA] Expose context extended deleters on PI API #1483

Merged
merged 2 commits into from
Apr 13, 2020

Conversation

nyalloc
Copy link
Contributor

@nyalloc nyalloc commented Apr 6, 2020

This PR replaces the PI CUDA backend's context callback mechanism with a PI API extension. Users of PI API can now register function pointers to be called before the context is destroyed. Prior to this PR, registering a function required including pi_cuda.hpp. Now that pi_cuda.hpp is private and not accessible to external projects, exposing this functionality on public PI API has become necessary.

@nyalloc nyalloc force-pushed the stuart/piextContextSetExtendedDeleter branch 3 times, most recently from 76f292f to 31ce837 Compare April 7, 2020 11:37
@nyalloc nyalloc changed the title Expose context extended deleters on PI API [SYCL][CUDA] Expose context extended deleters on PI API Apr 7, 2020
@nyalloc nyalloc force-pushed the stuart/piextContextSetExtendedDeleter branch 2 times, most recently from 28256d2 to 1a65d46 Compare April 7, 2020 14:22
sycl/source/detail/pi.cpp Outdated Show resolved Hide resolved
sycl/include/CL/sycl/detail/pi.hpp Outdated Show resolved Hide resolved
@bader bader added the cuda CUDA back-end label Apr 8, 2020
@bader
Copy link
Contributor

bader commented Apr 9, 2020

@StuartDAdams, please, fix failed checks.

@bader bader requested a review from romanovvlad April 9, 2020 09:21
sycl/source/detail/pi.cpp Outdated Show resolved Hide resolved
Signed-off-by: Stuart Adams stuart.adams@codeplay.com
@nyalloc nyalloc force-pushed the stuart/piextContextSetExtendedDeleter branch from 8f67cb2 to 88a7ece Compare April 10, 2020 16:38
Signed-off-by: Stuart Adams <stuart.adams@codeplay.com>
@bader bader merged commit 86acff3 into intel:sycl Apr 13, 2020
alexbatashev pushed a commit to alexbatashev/llvm that referenced this pull request Apr 15, 2020
…duler_docs

* origin/sycl:
  [SYCL][PI][CUDA] Implements get_native interoperability (intel#1332)
  [SYCL] Fix check-sycl test suite on systems w/o OpenCL (intel#1503)
  [SYCL][Doc] Update ExtendedAtomics documentation (intel#1487)
  [SYCL][CUDA] Expose context extended deleters on PI API (intel#1483)
  [SYCL][NFC] Remove a dropped environment variable from a test (intel#1506)
  [SYCL] Add opencl-aot to sycl-toolchain target (intel#1504)
  [SYCL] Allow to run deploy LIT tests from particular directory
  [SYCL][CUDA] Fix LIT testing with CUDA devices (intel#1300)
  [SYCL] Remove operator name keywords (intel#1501)
  [Driver][SYCL] Consider .lo files as static archives (intel#1500)
  [SYCL-PTX] Update the compiler design to describe the CUDA target (intel#1408)
  [SYCL] Fix library build on Windows (intel#1499)
  [SYCL][NFC] Refactor lit.cfg.py (intel#1452)
  [SYCL] Fixed sub-buffer memory allocation update (intel#1486)
  [SYCL] Ensure proper definition of spirv builtins for SYCL (intel#1393)
  [SYCL][CUDA] LIT XFAIL/UNSUPPORTED (intel#1303)
  [SYCL][Doc] Function-type kernel attribute extension (intel#1494)
alexbatashev pushed a commit to alexbatashev/llvm that referenced this pull request Apr 15, 2020
…c_abi_checks

* origin/sycl: (32 commits)
  [SYCL] Do not force LLVM_INCLUDE_TESTS variable (intel#1505)
  [SYCL][NFC] Align nd_item members with constructor initialization list (intel#1521)
  [SYCL] Move get_info_host implementation to header (intel#1514)
  [SYCL] Always use dynamic CRT for Unit tests (intel#1515)
  [SYCL][NFC] Temporarily disable sporadically failing test (intel#1526)
  [SYCL] Fix inline namespaces (intel#1525)
  [SYCL] Release notes for March'20 DPCPP implementation update (intel#1511)
  [SYCL][PI][CUDA] Implements get_native interoperability (intel#1332)
  [SYCL] Fix check-sycl test suite on systems w/o OpenCL (intel#1503)
  [SYCL][Doc] Update ExtendedAtomics documentation (intel#1487)
  [SYCL][CUDA] Expose context extended deleters on PI API (intel#1483)
  [SYCL][NFC] Remove a dropped environment variable from a test (intel#1506)
  [SYCL] Add opencl-aot to sycl-toolchain target (intel#1504)
  [SYCL] Allow to run deploy LIT tests from particular directory
  [SYCL][CUDA] Fix LIT testing with CUDA devices (intel#1300)
  [SYCL] Remove operator name keywords (intel#1501)
  [Driver][SYCL] Consider .lo files as static archives (intel#1500)
  [SYCL-PTX] Update the compiler design to describe the CUDA target (intel#1408)
  [SYCL] Fix library build on Windows (intel#1499)
  [SYCL][NFC] Refactor lit.cfg.py (intel#1452)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cuda CUDA back-end
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants