[llvm] r347093 - [X86] In LowerLoad, fix assert messages and rename a variable that use Zize instead of Size. NFC
Craig Topper via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 16 13:04:56 PST 2018
Author: ctopper
Date: Fri Nov 16 13:04:56 2018
New Revision: 347093
URL: http://llvm.org/viewvc/llvm-project?rev=347093&view=rev
Log:
[X86] In LowerLoad, fix assert messages and rename a variable that use Zize instead of Size. NFC
Modified:
llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
Modified: llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86ISelLowering.cpp?rev=347093&r1=347092&r2=347093&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86ISelLowering.cpp Fri Nov 16 13:04:56 2018
@@ -20150,16 +20150,16 @@ static SDValue LowerStore(SDValue Op, co
static SDValue LowerLoad(SDValue Op, const X86Subtarget &Subtarget,
SelectionDAG &DAG) {
MVT RegVT = Op.getSimpleValueType();
- assert(RegVT.isVector() && "We only custom lower vector sext loads.");
+ assert(RegVT.isVector() && "We only custom lower vector loads.");
assert(RegVT.isInteger() &&
- "We only custom lower integer vector sext loads.");
+ "We only custom lower integer vector loads.");
LoadSDNode *Ld = cast<LoadSDNode>(Op.getNode());
SDLoc dl(Ld);
EVT MemVT = Ld->getMemoryVT();
// Without AVX512DQ, we need to use a scalar type for v2i1/v4i1/v8i1 loads.
- if (RegVT.isVector() && RegVT.getVectorElementType() == MVT::i1) {
+ if (RegVT.getVectorElementType() == MVT::i1) {
assert(EVT(RegVT) == MemVT && "Expected non-extending load");
assert(RegVT.getVectorNumElements() <= 8 && "Unexpected VT");
assert(Subtarget.hasAVX512() && !Subtarget.hasDQI() &&
@@ -20262,26 +20262,26 @@ static SDValue LowerLoad(SDValue Op, con
assert((Ext != ISD::SEXTLOAD || NumLoads == 1) &&
"Can only lower sext loads with a single scalar load!");
- unsigned loadRegZize = RegSz;
+ unsigned loadRegSize = RegSz;
if (Ext == ISD::SEXTLOAD && RegSz >= 256)
- loadRegZize = 128;
+ loadRegSize = 128;
// If we don't have BWI we won't be able to create the shuffle needed for
// v8i8->v8i64.
if (Ext == ISD::EXTLOAD && !Subtarget.hasBWI() && RegVT == MVT::v8i64 &&
MemVT == MVT::v8i8)
- loadRegZize = 128;
+ loadRegSize = 128;
// Represent our vector as a sequence of elements which are the
// largest scalar that we can load.
EVT LoadUnitVecVT = EVT::getVectorVT(
- *DAG.getContext(), SclrLoadTy, loadRegZize / SclrLoadTy.getSizeInBits());
+ *DAG.getContext(), SclrLoadTy, loadRegSize / SclrLoadTy.getSizeInBits());
// Represent the data using the same element type that is stored in
// memory. In practice, we ''widen'' MemVT.
EVT WideVecVT =
EVT::getVectorVT(*DAG.getContext(), MemVT.getScalarType(),
- loadRegZize / MemVT.getScalarSizeInBits());
+ loadRegSize / MemVT.getScalarSizeInBits());
assert(WideVecVT.getSizeInBits() == LoadUnitVecVT.getSizeInBits() &&
"Invalid vector type");
More information about the llvm-commits
mailing list