[PATCH] D76508: [VectorUtils] move x86's scaleShuffleMask to generic VectorUtils

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 22 11:12:43 PDT 2020


RKSimon accepted this revision.
RKSimon added a comment.

LGTM



================
Comment at: llvm/include/llvm/Analysis/VectorUtils.h:346
+  assert(Scale > 0 && "Unexpected scaling factor");
+  size_t NumElts = Mask.size();
+  ScaledMask.assign(NumElts * Scale, -1);
----------------
Retaining a fast copy path for Scale == 1 would make sense


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

https://reviews.llvm.org/D76508





More information about the llvm-commits mailing list