[llvm] a5605f1 - [RISCV] Fix operand number in debug message in RISCVMergeBaseOffset.

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 2 15:27:36 PDT 2022


Author: Craig Topper
Date: 2022-08-02T15:27:23-07:00
New Revision: a5605f1f6853ee26456895f6055fa5e11a875de0

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

LOG: [RISCV] Fix operand number in debug message in RISCVMergeBaseOffset.

This used to print from the ADDI where the operand number was
correct. It recently changed to print from the LUI or AUIPC which
needs to use operand 1 instead of 2.

This shows up as a crash with -debug.

Added: 
    

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

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/RISCV/RISCVMergeBaseOffset.cpp b/llvm/lib/Target/RISCV/RISCVMergeBaseOffset.cpp
index 7b2fb9f0d6868..d18e2ef8723bb 100644
--- a/llvm/lib/Target/RISCV/RISCVMergeBaseOffset.cpp
+++ b/llvm/lib/Target/RISCV/RISCVMergeBaseOffset.cpp
@@ -426,7 +426,7 @@ bool RISCVMergeBaseOffsetOpt::runOnMachineFunction(MachineFunction &Fn) {
       if (!detectFoldable(Hi, Lo))
         continue;
       LLVM_DEBUG(dbgs() << "  Found lowered global address: "
-                        << *Hi.getOperand(2).getGlobal() << "\n");
+                        << *Hi.getOperand(1).getGlobal() << "\n");
       MadeChange |= detectAndFoldOffset(Hi, *Lo);
       MadeChange |= foldIntoMemoryOps(Hi, *Lo);
     }


        


More information about the llvm-commits mailing list