[PATCH] D96332: [RISCV] Add support for selecting vid.v from build_vector

Fraser Cormack via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 9 04:32:25 PST 2021


frasercrmck added a comment.

This was partly motivated to test the waters on how best to bring the fixed-length support up to speed with the scalable support.  The main concern is obviously the number of patterns we'd introduce, and having more-or-less copy/pasted code. Adding the `_VL` seems to me like a reasonable approach for our existing custom nodes like `VID`, but what about for SETCC, INSERT_ELEMENT, etc? Should scalable- and fixed- nodes both be custom-lowered to a VL version?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96332



More information about the llvm-commits mailing list