[PATCH] D46084: [Fixed Point Arithmetic] Addition of the Fixed Point _Accum type
Leonard Chan via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed May 23 11:22:53 PDT 2018
leonardchan added inline comments.
================
Comment at: include/clang/Basic/DiagnosticCommonKinds.td:172
+def err_fixed_point_only_allowed_in_c : Error<
+ "Fixed point types are only allowed in C">;
----------------
leonardchan wrote:
> rsmith wrote:
> > Diagnostics should not be capitalized. Also, we generally allow conforming C extensions to be used in other languages unless there is a really good reason not to.
> We decided not to allow fixed point types in other languages because there is no specification provided in N1169 for addressing some features in other languages. Using C++ as an example, N1169 does not provide recommended characters when name mangling so we do not allow this in C++.
Actually, scratch that. We will be enabling it since GCC does. Will update this and other relevant C++ related code appropriately.
Repository:
rC Clang
https://reviews.llvm.org/D46084
More information about the cfe-commits
mailing list