[PATCH] D73967: Implement _ExtInt as an extended int type specifier.

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added a comment.

In D73967#2653042 <https://reviews.llvm.org/D73967#2653042>, @keryell wrote:

> An FPGA programmer is hitting this issue from your unit test:
>
>   c++
>     signed _ExtInt(1) m; // expected-error{{signed _ExtInt must have a bit size of at least 2}}
>
> Why do you not allow a type able to represent `{-1, 0}`?

At the time this feature was first being designed for WG14, I believe ones complement and sign magnitude integer representations were still allowed, so a signed 1-bit _ExtInt made no sense. That's no longer the case and so WG14 may be more interested in hearing about this if you have a compelling use case.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73967