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

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 22 21:26:11 PDT 2018


rjmccall added a comment.

There are at least three good reasons to make sure this can enabled/disabled by a flag:

- We have to anticipate that introducing new keywords will cause some compatibility problems.  New language standards that introduce new keywords can be disabled using `-std=<something earlier>`.  We shouldn't let this bypass that just because it's an out-of-band addition to the base language.
- It seems likely to me that this feature will be target-restricted.
- It seems plausible to me that development of this feature might continue past the Clang 7 branch date, and we shouldn't release a compiler with significantly-incomplete features on by default.

That said, when it's ready, I think it should probably be on by default on the targets that support it.


Repository:
  rC Clang

https://reviews.llvm.org/D46084





More information about the cfe-commits mailing list