[Parallel_libs-commits] [PATCH] D23857: [StreamExecutor] Add Platform and PlatformManager

[Justin Lebar via Parallel_libs-commits]

jlebar accepted this revision.
This revision is now accepted and ready to land.

================
Comment at: streamexecutor/include/streamexecutor/Platform.h:34
@@ +33,3 @@
+
+  /// Gets a pointer to a Device with the given index for this platform.
+  virtual Expected<Device *> getDevice(size_t DeviceIndex) = 0;
----------------
Do we want to say here that ownership isn't transferred to the caller?  I'm on the fence, maybe it's obvious.


https://reviews.llvm.org/D23857