[llvm] 2d0ac85 - [X86] Fix gcc warning about mix of enumeral and non-enumeral types. NFC
Craig Topper via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 26 08:07:44 PDT 2023
Author: Craig Topper
Date: 2023-10-26T08:02:51-07:00
New Revision: 2d0ac85b6693eb6e25b4120c0e4e224c42a84462
URL: https://github.com/llvm/llvm-project/commit/2d0ac85b6693eb6e25b4120c0e4e224c42a84462
DIFF: https://github.com/llvm/llvm-project/commit/2d0ac85b6693eb6e25b4120c0e4e224c42a84462.diff
LOG: [X86] Fix gcc warning about mix of enumeral and non-enumeral types. NFC
Added:
Modified:
llvm/lib/Target/X86/X86ISelLowering.cpp
Removed:
################################################################################
diff --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp
index 93db31e03e116e7..6411f27da0776d4 100644
--- a/llvm/lib/Target/X86/X86ISelLowering.cpp
+++ b/llvm/lib/Target/X86/X86ISelLowering.cpp
@@ -3297,21 +3297,22 @@ unsigned X86TargetLowering::preferedOpcodeForCmpEqPiecesOfOperand(
// If the current setup has imm64 mask, then inverse will have
// at least imm32 mask (or be zext i32 -> i64).
if (VT == MVT::i64)
- return AndMask->getSignificantBits() > 32 ? ISD::SRL : ShiftOpc;
+ return AndMask->getSignificantBits() > 32 ? (unsigned)ISD::SRL
+ : ShiftOpc;
// We can only benefit if req at least 7-bit for the mask. We
// don't want to replace shl of 1,2,3 as they can be implemented
// with lea/add.
- return ShiftOrRotateAmt.uge(7) ? ISD::SRL : ShiftOpc;
+ return ShiftOrRotateAmt.uge(7) ? (unsigned)ISD::SRL : ShiftOpc;
}
if (VT == MVT::i64)
// Keep exactly 32-bit imm64, this is zext i32 -> i64 which is
// extremely efficient.
- return AndMask->getSignificantBits() > 33 ? ISD::SHL : ShiftOpc;
+ return AndMask->getSignificantBits() > 33 ? (unsigned)ISD::SHL : ShiftOpc;
// Keep small shifts as shl so we can generate add/lea.
- return ShiftOrRotateAmt.ult(7) ? ISD::SHL : ShiftOpc;
+ return ShiftOrRotateAmt.ult(7) ? (unsigned)ISD::SHL : ShiftOpc;
}
// We prefer rotate for vectors of if we won't get a zext mask with SRL
More information about the llvm-commits
mailing list