We are now enabling some Intel OpenCL extension and find in extension cl_intel_required_subgroup_size (https://www.khronos.org/registry/OpenCL/extensions/intel/cl_intel_required_subgroup_size.txt) has an attribute intel_reqd_sub_group_size that is very useful for intel subgroup usage and does not supported by clang now. And I want to send a patch to enable it, what do you think about this?
This looks like a simple attribute to parse, attribute((intel_reqd_sub_group_size())), and I don’t see why this would be controversial. I recommend just sending a patch for review.
I’m assuming that you’re just going to emit corresponding kernel metadata. If the handling will be more involved than that, then it might be worth discussing beforehand. -Hal
Sure, feel free to create the review.
Sorry for delay, I missed that somehow! L
It just a simple metadata, and I will create review soon.