[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