[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