Skip to content

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
@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
@nyalloc nyalloc force-pushed the stuart/piextContextSetExtendedDeleter branch from 8f67cb2 to 88a7ece Compare April 10, 2020 16:38
Signed-off-by: Stuart Adams <[email protected]>
@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