[PATCH] D79806: [CodeGen] Specify meaning of ISD opcodes for scalable vectors

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 22 04:45:18 PDT 2020


sdesmalen added a comment.

> I think I'd rather use a different opcode for the cases where you're mixing fixed/scalable vectors. There are some potentially weird edge cases to deal with there, and the existing code is much more likely to blow up. For example, can you extract an <8 x i32> from a <vscale x 4 x i32>? Does it matter if we know that vscale is 2 or more for the current subtarget?

We discussed this in the previous SVE sync-up call, but just to clarify: extracting a `<8 x i32>` from a `<vscale x 4 x i32>` would be a valid use-case for the same reason as allowing to extract an element from index `8` from e.g. a `<vscale x 4 x i32>`. I've called that out explicitly in the description of EXTRACT_VECTOR_ELT.


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

https://reviews.llvm.org/D79806





More information about the llvm-commits mailing list