[Parallel_libs-commits] [PATCH] D24043: [StreamExecutor] Simplify Kernel classes

[Justin Lebar via Parallel_libs-commits]

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

Well, that does look simpler.

Thanks a lot for all these reviews, jprice.


================
Comment at: streamexecutor/include/streamexecutor/Device.h:33
@@ -31,6 +32,3 @@
 
-  /// Gets the kernel implementation for the underlying platform.
-  virtual Expected<std::unique_ptr<KernelInterface>>
-  getKernelImplementation(const MultiKernelLoaderSpec &Spec) {
-    // TODO(jhen): Implement this.
-    return nullptr;
+  /// Creates a kernel object for this device.
+  ///
----------------
Can we be explicit that if this doesn't return an error, it is guaranteed not to return a null value?


https://reviews.llvm.org/D24043