Superfluous/Misleading generic parameter in `ExecuteKernel::set_arg_local_buffer`
Closed this issue · 2 comments
KarelPeeters commented
Currently ExecuteKernel::set_arg_local_buffer
has a generic type parameter T
. Even though it's never actually used. Initially I thought this meant the size was the length of the buffer (with the given data type), but it's indeed the size in bytes.
I think this generic parameter can just be removed.
Lines 368 to 389 in f8f972e
kenba commented
Agreed, generic type parameter removed on develop branch.
KarelPeeters commented
Great, thanks!