[PATCH] D122732: [Clang][AArch64][SVE] Allow subscript operator for SVE types

2022-04-14 Thread David Truby via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG53fd8db79192: [Clang][AArch64][SVE] Allow subscript operator for SVE types (authored by DavidTruby). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D122732: [Clang][AArch64][SVE] Allow subscript operator for SVE types

2022-04-08 Thread Bradley Smith via Phabricator via cfe-commits
bsmith accepted this revision. bsmith added a comment. This revision is now accepted and ready to land. LGTM! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122732/new/ https://reviews.llvm.org/D122732

[PATCH] D122732: [Clang][AArch64][SVE] Allow subscript operator for SVE types

2022-03-31 Thread David Truby via Phabricator via cfe-commits
DavidTruby added a comment. > Not sure what you mean by this; LLVM supports extractelement on ` x i1>` vectors. I guess the fact that it's a "vscale x 16" element vector > might not be intuitive? It's a native operation at the LLVM level but not at the ISA level, unlike the data registers.

[PATCH] D122732: [Clang][AArch64][SVE] Allow subscript operator for SVE types

2022-03-30 Thread Eli Friedman via Phabricator via cfe-commits
efriedma added a comment. > Subscript on svbool_t is not allowed as this doesn't really have meaningful > semantics. Not sure what you mean by this; LLVM supports extractelement on `` vectors. I guess the fact that it's a "vscale x 16" element vector might not be intuitive? Repository: rG

[PATCH] D122732: [Clang][AArch64][SVE] Allow subscript operator for SVE types

2022-03-30 Thread David Truby via Phabricator via cfe-commits
DavidTruby created this revision. Herald added subscribers: psnobl, kristof.beyls, tschuett. Herald added a reviewer: efriedma. Herald added a project: All. DavidTruby requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Undefined behaviour is