Skip to content

Commit

Permalink
Elaborate open issue with free function kernels
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
gmlueck committed Sep 20, 2024
1 parent ea26120 commit c7b3018
Showing 1 changed file with 7 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -963,6 +963,13 @@ int main() {
```
+
Where `kfp` would have some nicer name.
+
With either form above, it seems like we have enough type information for the
header to check that the types of the actual kernel arguments are implicitly
convertible to the types of the formal kernel parameters, and we can raise a
compile-time error if they are not.
In addition, the header can perform any necessary implicit conversions when
setting the kernel argument values.

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

0 comments on commit c7b3018

Please sign in to comment.