Review comments

Author: preames

llvm/lib/Target/RISCV/RISCVISelLowering.cpp

@@ -4624,7 +4624,7 @@ static bool isElementRotate(const std::array<std::pair<int, int>, 2> &SrcInfo,
 }
 
 static bool isAlternating(const std::array<std::pair<int, int>, 2> &SrcInfo,
-                          const ArrayRef<int> Mask, bool RequiredPolarity) {
+                          ArrayRef<int> Mask, bool RequiredPolarity) {
   int NumElts = Mask.size();
   for (unsigned i = 0; i != NumElts; ++i) {
     int M = Mask[i];
@@ -4647,7 +4647,7 @@ static bool isAlternating(const std::array<std::pair<int, int>, 2> &SrcInfo,
 /// vs1: b0 b1 b2 b3
 /// vd:  a0 b0 a2 b2
 static bool isZipEven(const std::array<std::pair<int, int>, 2> &SrcInfo,
-                      const ArrayRef<int> Mask) {
+                      ArrayRef<int> Mask) {
   return SrcInfo[0].second == 0 && SrcInfo[1].second == 1 &&
          isAlternating(SrcInfo, Mask, true);
 }
@@ -4660,7 +4660,7 @@ static bool isZipEven(const std::array<std::pair<int, int>, 2> &SrcInfo,
 /// Note that the operand order is swapped due to the way we canonicalize
 /// the slides, so SrCInfo[0] is vs1, and SrcInfo[1] is vs2.
 static bool isZipOdd(const std::array<std::pair<int, int>, 2> &SrcInfo,
-                     const ArrayRef<int> Mask) {
+                     ArrayRef<int> Mask) {
   return SrcInfo[0].second == 0 && SrcInfo[1].second == -1 &&
          isAlternating(SrcInfo, Mask, false);
 }