Address review comment

preames · preames · commit c359d56faa58 · 2025-02-11T13:33:35.000-08:00
diff --git a/llvm/lib/Target/RISCV/RISCVISelLowering.cpp b/llvm/lib/Target/RISCV/RISCVISelLowering.cpp
@@ -5352,8 +5352,8 @@ static bool isLocalRepeatingShuffle(ArrayRef<int> Mask, int Span) {
 
 /// Is this mask only using elements from the first span of the input?
 static bool isLowSourceShuffle(ArrayRef<int> Mask, int Span) {
-  return !any_of(Mask,
-                 [&](const auto &Idx) { return Idx != -1 && Idx >= Span; });
+  return all_of(Mask,
+                [&](const auto &Idx) { return Idx == -1 || Idx < Span; });
 }
 
 /// Try to widen element type to get a new mask value for a better permutation

Original file line number	Diff line number	Diff line change
`@@ -5352,8 +5352,8 @@ static bool isLocalRepeatingShuffle(ArrayRef<int> Mask, int Span) {`
`5352`	`5352`
`5353`	`5353`	`/// Is this mask only using elements from the first span of the input?`
`5354`	`5354`	`static bool isLowSourceShuffle(ArrayRef<int> Mask, int Span) {`
`5355`		`- return !any_of(Mask,`
`5356`		`- [&](const auto &Idx) { return Idx != -1 && Idx >= Span; });`
	`5355`	`+ return all_of(Mask,`
	`5356`	`+ [&](const auto &Idx) { return Idx == -1 \|\| Idx < Span; });`
`5357`	`5357`	`}`
`5358`	`5358`
`5359`	`5359`	`/// Try to widen element type to get a new mask value for a better permutation`