[llvm] e805946 - [RISCV] Fix -Wsign-compare warning from f8ee58a

[Philip Reames via llvm-commits]

Author: Philip Reames
Date: 2025-03-29T15:46:56-07:00
New Revision: e8059467ef0041f6735b23ef680eb9070e308e33

URL: https://github.com/llvm/llvm-project/commit/e8059467ef0041f6735b23ef680eb9070e308e33
DIFF: https://github.com/llvm/llvm-project/commit/e8059467ef0041f6735b23ef680eb9070e308e33.diff

LOG: [RISCV] Fix -Wsign-compare warning from f8ee58a

lib/Target/RISCV/RISCVISelLowering.cpp:4629:26: error: comparison of integers of different signs: 'unsigned int' and 'int' [-Werror,-Wsign-compare]
 4629 |   for (unsigned i = 0; i != NumElts; ++i) {
      |                        ~ ^  ~~~~~~~
1 error generated.

Added: 
    

Modified: 
    llvm/lib/Target/RISCV/RISCVISelLowering.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/RISCV/RISCVISelLowering.cpp b/llvm/lib/Target/RISCV/RISCVISelLowering.cpp
index a8c83113854c9..70ec57798db71 100644
--- a/llvm/lib/Target/RISCV/RISCVISelLowering.cpp
+++ b/llvm/lib/Target/RISCV/RISCVISelLowering.cpp
@@ -4626,7 +4626,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,
                           ArrayRef<int> Mask, bool RequiredPolarity) {
   int NumElts = Mask.size();
-  for (unsigned i = 0; i != NumElts; ++i) {
+  for (int i = 0; i != NumElts; ++i) {
     int M = Mask[i];
     if (M < 0)
       continue;