[PATCH] D46084: [Fixed Point Arithmetic] Addition of the Fixed Point _Accum type

JF Bastien via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 30 10:41:05 PDT 2018


jfb added inline comments.


================
Comment at: include/clang/Basic/TokenKinds.def:393
+// ISO/IEC JTC1 SC22 WG14 N1169 Extension
+KEYWORD(_Accum                      , KEYALL)
+
----------------
ebevhan wrote:
> I believe that having KEYALL will enable the keyword even if -fno-fixed-point is given. Is this desired? It will mean that `_Accum` will not be a valid identifier in standard C regardless of the flag.
That seems fine: identifiers starting with underscore and a capital letter already aren't valid identifiers in C and C++ because they're reserved for the implementation.


Repository:
  rC Clang

https://reviews.llvm.org/D46084





More information about the cfe-commits mailing list