[PATCH] D127976: [IR] Move vector.insert/vector.extract out of experimental namespace
Javier Setoain via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jun 21 10:05:11 PDT 2022
jsetoain added inline comments.
Comment at: llvm/docs/LangRef.rst:17292-17294
+``llvm.vector.insert`` can be used to insert a fixed-width vector into a
+scalable vector, but not the other way around.
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.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits