[llvm] 83e01ea - Fix MSVC signed/unsigned mismatch warning. NFC.
Simon Pilgrim via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 4 03:11:42 PST 2023
Author: Simon Pilgrim
Date: 2023-12-04T11:11:33Z
New Revision: 83e01ea1a559a7da0e8f41620a05f91c9ae542a2
URL: https://github.com/llvm/llvm-project/commit/83e01ea1a559a7da0e8f41620a05f91c9ae542a2
DIFF: https://github.com/llvm/llvm-project/commit/83e01ea1a559a7da0e8f41620a05f91c9ae542a2.diff
LOG: Fix MSVC signed/unsigned mismatch warning. NFC.
Added:
Modified:
llvm/lib/Target/ARM/ARMBaseInstrInfo.cpp
Removed:
################################################################################
diff --git a/llvm/lib/Target/ARM/ARMBaseInstrInfo.cpp b/llvm/lib/Target/ARM/ARMBaseInstrInfo.cpp
index ce65c8e5245b1..476a9bb15edbf 100644
--- a/llvm/lib/Target/ARM/ARMBaseInstrInfo.cpp
+++ b/llvm/lib/Target/ARM/ARMBaseInstrInfo.cpp
@@ -4473,7 +4473,7 @@ ARMBaseInstrInfo::getOperandLatency(const InstrItineraryData *ItinData,
ItinData->getOperandCycle(DefMCID.getSchedClass(), DefIdx);
int Adj = Subtarget.getPreISelOperandLatencyAdjustment();
int Threshold = 1 + Adj;
- return !Latency || Latency <= Threshold ? 1 : *Latency - Adj;
+ return !Latency || Latency <= (unsigned)Threshold ? 1 : *Latency - Adj;
}
const MCInstrDesc &UseMCID = get(UseNode->getMachineOpcode());
@@ -4490,7 +4490,7 @@ ARMBaseInstrInfo::getOperandLatency(const InstrItineraryData *ItinData,
if (!Latency)
return std::nullopt;
- if (Latency > 1 &&
+ if (Latency > 1U &&
(Subtarget.isCortexA8() || Subtarget.isLikeA9() ||
Subtarget.isCortexA7())) {
// FIXME: Shifter op hack: no shift (i.e. [r +/- r]) or [r + r << 2]
@@ -4519,7 +4519,7 @@ ARMBaseInstrInfo::getOperandLatency(const InstrItineraryData *ItinData,
break;
}
}
- } else if (DefIdx == 0 && Latency > 2 && Subtarget.isSwift()) {
+ } else if (DefIdx == 0 && Latency > 2U && Subtarget.isSwift()) {
// FIXME: Properly handle all of the latency adjustments for address
// writeback.
switch (DefMCID.getOpcode()) {
@@ -4836,7 +4836,7 @@ bool ARMBaseInstrInfo::hasLowDefLatency(const TargetSchedModel &SchedModel,
unsigned DefClass = DefMI.getDesc().getSchedClass();
std::optional<unsigned> DefCycle =
ItinData->getOperandCycle(DefClass, DefIdx);
- return DefCycle <= 2;
+ return DefCycle <= 2U;
}
return false;
}
More information about the llvm-commits
mailing list