[PATCH] D120254: [OpenCL] Align subgroup builtin guards
Sven van Haastregt via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Feb 23 04:21:54 PST 2022
svenvh marked an inline comment as done.
svenvh added inline comments.
================
Comment at: clang/lib/Headers/opencl-c-base.h:85
+// Internal feature macro to provide subgroup builtins.
+#define __opencl_subgroup_builtins 1
+#endif
----------------
azabaznov wrote:
> svenvh wrote:
> > I'm in doubt whether we could just reuse `__opencl_c_subgroups` for this?
> I think we couldn't. Those subgroup features/extensions are different, as implementation may support the extension but not the feature. The difference is in subgroup independent forward progress: for example, it's required by the extension, but optional in OpenCL C 3.0 feature.
Thanks for confirming, I will keep the separate internal feature macro then.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D120254/new/
https://reviews.llvm.org/D120254
More information about the cfe-commits
mailing list