[AMDGPU] Fix a warning

kazutakahirata · kazutakahirata · commit 14006c8b5be1 · 2025-03-20T01:02:54.000-07:00
This patch fixes:

  llvm/lib/Target/AMDGPU/SILateBranchLowering.cpp:173:64: error:
  comparison of integers of different signs: 'unsigned int' and 'int'
  [-Werror,-Wsign-compare]
diff --git a/llvm/lib/Target/AMDGPU/SILateBranchLowering.cpp b/llvm/lib/Target/AMDGPU/SILateBranchLowering.cpp
@@ -170,8 +170,7 @@ void SILateBranchLowering::expandChainCall(MachineInstr &MI,
                           *TII->getNamedOperand(MI, AMDGPU::OpName::exec));
   }
 
-  for (unsigned OpIdx = MI.getNumExplicitOperands() - 1; OpIdx >= ExecIdx;
-       --OpIdx)
+  for (int OpIdx = MI.getNumExplicitOperands() - 1; OpIdx >= ExecIdx; --OpIdx)
     MI.removeOperand(OpIdx);
 
   MI.setDesc(TII->get(AMDGPU::SI_TCRETURN));

Original file line number	Diff line number	Diff line change
`@@ -170,8 +170,7 @@ void SILateBranchLowering::expandChainCall(MachineInstr &MI,`
`170`	`170`	`*TII->getNamedOperand(MI, AMDGPU::OpName::exec));`
`171`	`171`	`}`
`172`	`172`
`173`		`- for (unsigned OpIdx = MI.getNumExplicitOperands() - 1; OpIdx >= ExecIdx;`
`174`		`- --OpIdx)`
	`173`	`+ for (int OpIdx = MI.getNumExplicitOperands() - 1; OpIdx >= ExecIdx; --OpIdx)`
`175`	`174`	`MI.removeOperand(OpIdx);`
`176`	`175`
`177`	`176`	`MI.setDesc(TII->get(AMDGPU::SI_TCRETURN));`