[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