Anastasia added inline comments.

================
Comment at: clang/test/SemaOpenCL/unsupported.cl:11
+#ifndef BITFIELDS_EXT
+// expected-error@-2 {{bit-fields are not supported in OpenCL}}
+#endif
----------------
svenvh wrote:
> The extension has "bitfields" in the name but most diagnostics (including 
> this one) spell it as "bit-fields".  I wonder what the least surprising name 
> would be?  It seems Clang tends to use the hyphenated form in diagnostics and 
> the non-hyphenated form in e.g. option names (e.g. `fsigned-bitfields`), so 
> using the non-hyphenated form in the extension name is probably fine.
Hyphenated words are often split so we could also go for 
`__cl_clang_bit_fields`? But I think this might be more confusing...


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D101843/new/

https://reviews.llvm.org/D101843

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to