[PATCH] D127976: [IR] Move vector.insert/vector.extract out of experimental namespace

[Javier Setoain via Phabricator via llvm-commits]

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.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D127976/new/

https://reviews.llvm.org/D127976