[llvm] 03d8e61 - [Transforms] Fix -Wsign-compare in DemoteRegToStack.cpp (NFC)

Jie Fu via llvm-commits llvm-commits at lists.llvm.org
Thu May 16 05:30:35 PDT 2024


Author: Jie Fu
Date: 2024-05-16T20:29:54+08:00
New Revision: 03d8e613911bdef52fec23924a4e5b4235997bb8

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

LOG: [Transforms] Fix -Wsign-compare in DemoteRegToStack.cpp (NFC)

llvm-project/llvm/lib/Transforms/Utils/DemoteRegToStack.cpp:54:23:
error: comparison of integers of different signs: 'int' and 'unsigned int' [-Werror,-Wsign-compare]
    for (int i = 0; i < CBI->getNumSuccessors(); i++) {
                    ~ ^ ~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.

Added: 
    

Modified: 
    llvm/lib/Transforms/Utils/DemoteRegToStack.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/Utils/DemoteRegToStack.cpp b/llvm/lib/Transforms/Utils/DemoteRegToStack.cpp
index f03de2f25a416..653ea8bbb72b9 100644
--- a/llvm/lib/Transforms/Utils/DemoteRegToStack.cpp
+++ b/llvm/lib/Transforms/Utils/DemoteRegToStack.cpp
@@ -51,7 +51,7 @@ AllocaInst *llvm::DemoteRegToStack(Instruction &I, bool VolatileLoads,
       (void)BB;
     }
   } else if (CallBrInst *CBI = dyn_cast<CallBrInst>(&I)) {
-    for (int i = 0; i < CBI->getNumSuccessors(); i++) {
+    for (unsigned i = 0; i < CBI->getNumSuccessors(); i++) {
       auto *Succ = CBI->getSuccessor(i);
       if (!Succ->getSinglePredecessor()) {
         assert(isCriticalEdge(II, i) && "Expected a critical edge!");


        


More information about the llvm-commits mailing list