These intrinsics are now fundemental for SVE code generation and have been
present for a year and a half, hence move them out of the experimental
namespace.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Hey Bradley! Thanks for pinging us about this upcoming change! Something that my colleague @jsetoain mentioned is that these vector insert/extract intrinsics can also work on fixed-length only vectors. However, the documentation and slightly tend to suggest that they should specifically be used to insert/extract fixed-length vectors into/out of scalable one. Perhaps clarifying that they can be used on fixed-length only vectors and adding an example for that case would be very helpful. Thanks!
- Clarify LangRef slightly to make it clearer that fixed types can be used
- Rebase on top of recent test changes
llvm/docs/LangRef.rst | ||
---|---|---|
17293–17295 | To answer Craig's point, I don't think it was clear before that you could use this intrinsic with purely scalable vectors before, either. I believe the confusion comes from these clarifications. If instead of explicitly stating valid options you exclude the invalid ones, there's no room for ambiguity. Something like: "Scalable vectors can only be inserted/extracted into/out of other scalable vectors". Examples of all other possible combinations might help to nail down this point. |
llvm/docs/LangRef.rst | ||
---|---|---|
17283 | For this and the other instances, please remove the % as idx must be a literal (as documented below) but using %idx makes it look like an arbitrary variable can be used. |
For this and the other instances, please remove the % as idx must be a literal (as documented below) but using %idx makes it look like an arbitrary variable can be used.