[PATCH] D130973: [AArch64] Always allow the __bf16 type
Simon Tatham via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 3 03:02:21 PDT 2022
simon_tatham accepted this revision.
simon_tatham added a comment.
This revision is now accepted and ready to land.
Ah, yes, I misread what the test changes were about – I didn't notice that you'd only just introduced the `scalar` error context, and thought you were //enabling// all those operations in at least one situation, rather than changing the form of the check that they were unconditionally disabled.
Right, so the effect is that you can //talk about// the type, put it in struct fields, query its size and alignment, and if you really must, declare variables of the type that you never do anything with. That all sounds safe enough :-)
In that case, LGTM.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130973/new/
https://reviews.llvm.org/D130973
More information about the llvm-commits
mailing list