Jianhui-Li wrote:

> > We really just need a "mlir-runner" with target-platform as command 
> > parameters.
> 
> What kind of "target-platform" command parameters do you have in mind? (other 
> than what we do now)

The current way of mlir-cpu-runner using the share library name to indicate 
target-platform looks good to me: Cuda, Rocm, and SYCL with this PR.  Vulkan 
could be added same way.  mlir-cpu-spirv-runner could be refactored to be 
mlir-opt passes generating spirv binary and feed to mlir-cpu-runner.  

If we reach that state, the name "mlir-cpu-runner" could be promoted to 
"mlir-runner". That would clear up a lot of misunderstanding.  The 
"mlir-runner" is really running MLIR programs on a target platform, and has 
little to do with enabling specific language or runtime features of a target 
platform.  For example, we say "SYCL runner", people think it is about running 
SYCL program on top of MLIR. 

https://github.com/llvm/llvm-project/pull/65539
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to