[llvm] [offload] Add properties parameter to olLaunchKernel (PR #184343)

Kseniya Tikhomirova via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 1 06:39:15 PDT 2026


================
@@ -33,6 +55,7 @@ def olLaunchKernel : Function {
         Param<"const void*", "ArgumentsData", "pointer to the kernel argument struct", PARAM_IN_OPTIONAL>,
         Param<"size_t", "ArgumentsSize", "size of the kernel argument struct", PARAM_IN>,
----------------
KseniyaTikhomirova wrote:

question came from the code review for SYCL RT implementation using this:
why do we use **uint64_t**  type for argument sizes array while ArgumentsSize uses **size_t**?

Should we align he types?

```
TaggedEtor<"size", "uint64_t *", "Array of the arguments sizes.">,
vs
 Param<"size_t", "ArgumentsSize", "size of the kernel argument struct", PARAM_IN>,
```

https://github.com/llvm/llvm-project/pull/184343


More information about the llvm-commits mailing list