[Lldb-commits] [mlir] [flang] [clang-tools-extra] [lldb] [llvm] [libc] [lld] [compiler-rt] [libcxx] [clang] [MLIR] Enable GPU Dialect to SYCL runtime integration (PR #71430)

2023-11-26 Thread Lei Zhang via lldb-commits

https://github.com/antiagainst approved this pull request.

LGTM regarding SPIR-V. But please address comments from others. :)

https://github.com/llvm/llvm-project/pull/71430
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits


[Lldb-commits] [libc] [compiler-rt] [llvm] [lldb] [lld] [clang-tools-extra] [mlir] [clang] [openmp] [libcxx] [flang] [MLIR] Enabling Intel GPU Integration. (PR #65539)

2023-11-26 Thread Lei Zhang via lldb-commits

antiagainst wrote:

Thanks for breaking down this pull request into various smaller pieces to make 
it easier for review. I looked at various pieces; LGTM. Looking forward to see 
this being supported! :)

from @joker-eph:
> Vulkan and Spirv still have dedicated runners on the model of the original 
> cuda-runner, but I suspect this is just legacy?

from @Jianhui-Li:
> 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.

Yup; it's legacy. +1 to the idea of unifying! I've created 
https://github.com/llvm/llvm-project/issues/73457 to track this issue to make 
it more visible. I might not have the bandwidth to work on this; if somebody 
else is interested that'd be nice! So maked it as "good first issue" (not sure 
whether I'm pushing the limit of "good first issue" but hoping to get traction 
there).


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


[Lldb-commits] [compiler-rt] [llvm] [clang-tools-extra] [clang] [flang] [lldb] [mlir] [libcxx] [mlir] Prepare convert-gpu-to-spirv for OpenCL support (PR #69941)

2023-11-05 Thread Lei Zhang via lldb-commits

https://github.com/antiagainst closed 
https://github.com/llvm/llvm-project/pull/69941
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits


[Lldb-commits] [compiler-rt] [llvm] [clang-tools-extra] [clang] [flang] [lldb] [mlir] [libcxx] [mlir] Prepare convert-gpu-to-spirv for OpenCL support (PR #69941)

2023-11-05 Thread Lei Zhang via lldb-commits

antiagainst wrote:

> @antiagainst @joker-eph Can someone merge this PR? Thanks in advance.

Done. Thanks for the contribution again!

https://github.com/llvm/llvm-project/pull/69941
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits