[Parallel_libs-commits] [PATCH] D26050: [Acxxel] Remove setActiveDeviceForThread

Jason Henline via Parallel_libs-commits parallel_libs-commits at lists.llvm.org
Thu Oct 27 17:25:47 PDT 2016


jhen marked an inline comment as done.
jhen added inline comments.


================
Comment at: acxxel/acxxel.h:502
+  Expected<DeviceMemorySpan<ElementType>> getSymbolMemory(ElementType *Symbol,
+                                                          int DeviceIndex) {
+    Expected<void *> MaybeAddress =
----------------
jlebar wrote:
> Is there a reason we don't default DeviceIndex here as we do elsewhere, or is that just an oversight?
Oops, I accidentally missed that one. Thanks for catching it!


https://reviews.llvm.org/D26050





More information about the Parallel_libs-commits mailing list