Skip to content

Commit 6825615

Browse files
authored
Elaborate open issue with free function kernels (#15463)
I forget if we identified this already as an advantage to the enqueue function approach, but it came up recently in a discussion with @tahonermann. I wanted to record it here, so we don't forget when we get around to implementing this.
1 parent 2b4cb11 commit 6825615

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

sycl/doc/extensions/proposed/sycl_ext_oneapi_free_function_kernels.asciidoc

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -963,6 +963,13 @@ int main() {
963963
```
964964
+
965965
Where `kfp` would have some nicer name.
966+
+
967+
With either form above, it seems like we have enough type information for the
968+
header to check that the types of the actual kernel arguments are implicitly
969+
convertible to the types of the formal kernel parameters, and we can raise a
970+
compile-time error if they are not.
971+
In addition, the header can perform any necessary implicit conversions when
972+
setting the kernel argument values.
966973

967974
* We are debating whether we should allow a free function kernel to be defined
968975
with an initial "iteration index" parameter such as:

0 commit comments

Comments
 (0)