[PATCH] D73967: Implement _ExtInt as an extended int type specifier.
Erich Keane via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 26 08:48:00 PDT 2021
erichkeane 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}`?
Our FPGA team didn't see any value in it, so we didn't propose it to WG14, so we didn't implement it here. I'm sure a follow-up paper to WG14 would be considered, and if it shows promise/positive reception, we'd welcome a patch here.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D73967/new/
https://reviews.llvm.org/D73967
More information about the cfe-commits
mailing list