[PATCH] D116280: [clang] adds unary type trait checks as compiler built-ins
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Apr 18 11:53:27 PDT 2022
aaron.ballman added inline comments.
Comment at: clang/test/SemaCXX/type-traits.cpp:764
+ int t30[F(__is_bounded_array(void*))];
+ int t31[F(__is_bounded_array(cvoid*))];
> aaron.ballman wrote:
> > cjdb wrote:
> > > aaron.ballman wrote:
> > > > Any reason there's not a test with something like `int` and `int`? (Same below.)
> > > Those are `IntAr` and `IntArNB` at the very top. I can rename those if you'd like.
> > Oh, yeah, I totally missed that, thank you!
> > Up to you on the rename, so long as we have the coverage, that's the crucial bit.
> Would you be against a patch where I found and replaced **all** `TypeAr` with `Type` and `TypeArNB` with `Type`? I've done several double takes myself and question the readability of `TypeAr`.
I'd be fine with that -- it would be an easy NFC change after this lands.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits