nemanjai marked 3 inline comments as done.
nemanjai added inline comments.
================
Comment at: clang/lib/Sema/SemaDeclAttr.cpp:3970
+ DestWidth = 128;
+ break;
case 'T':
----------------
rjmccall wrote:
> rjmccall wrote:
> > Are there interactions with the other mode specifiers? For example, should
> > this be allowed with integer modes? If so, I think this needs more tests.
> I shouldn't have said "if so" — *either way*, this needs more tests.
Very good point. Thank you. Added.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80374/new/
https://reviews.llvm.org/D80374
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
