Contributor
This pull request partially address issue: #6448 by adding support for retrieving the constant memory size utilized by a kernel from the Dispatcher.
Contributor Author
@gmarkall I updated the docstring with you recommendation.
Member
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Many thanks for the PR - there are some comments on the diff to address.
Contributor Author
@gmarkall I updated the PR with all your comments and your patch.
Member
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Many thanks for resolving everything - this looks good now.
@esc / @stuartarchibald Could this have a CUDA buildfarm run please?
Contributor
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the patch. Approved for the purposes of testing on the build farm.