[Parallel_libs-commits] [PATCH] D24213: [SE] Remove Platform*Handle classes

Jason Henline via Parallel_libs-commits parallel_libs-commits at lists.llvm.org
Fri Sep 2 18:27:54 PDT 2016


jhen created this revision.
jhen added reviewers: jprice, jlebar.
jhen added a subscriber: parallel_libs-commits.

As pointed out by jprice, these classes don't serve a purpose. Instead,
we stay consistent with the way memory is managed and let the Stream and
Kernel classes directly hold opaque handles to device Stream and Kernel
instances, respectively.

https://reviews.llvm.org/D24213

Files:
  streamexecutor/include/streamexecutor/Device.h
  streamexecutor/include/streamexecutor/Kernel.h
  streamexecutor/include/streamexecutor/PlatformInterfaces.h
  streamexecutor/include/streamexecutor/Stream.h
  streamexecutor/lib/Device.cpp
  streamexecutor/lib/Kernel.cpp
  streamexecutor/lib/PlatformInterfaces.cpp
  streamexecutor/lib/Stream.cpp
  streamexecutor/lib/unittests/SimpleHostPlatformDevice.h
  streamexecutor/lib/unittests/StreamTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D24213.70261.patch
Type: text/x-patch
Size: 18239 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/parallel_libs-commits/attachments/20160903/ba5ab41e/attachment-0001.bin>


More information about the Parallel_libs-commits mailing list