rsmith added a comment.

In D73967#1861757 <https://reviews.llvm.org/D73967#1861757>, @erichkeane wrote:

> Extended-vector types don't really make sense for non-powers-of-two (plus 
> have some odd gotchas when it comes to vectors of i1 for example), so I've 
> added a test that shows that this is rejected.


OK, that seems fine to me.

> _Complex _ExtInt(N) is rejected when parsing _Complex.  It doesn't seem to 
> make sense to me to support them, so I've added a test that shows it is 
> invalid.  Do you disagree?

We allow _Complex T for integral types T in general, so this seems 
inconsistent. Are there problems supporting this? Is it just a parser 
limitation or something deeper? If there's some reason it doesn't naturally 
work (as there is for at least non-power-of-two-sized integers in vectors) then 
rejecting it seems fine.


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

https://reviews.llvm.org/D73967



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to