1a17f03bdSSanjay Patel //===------- VectorCombine.cpp - Optimize partial vector operations -------===// 2a17f03bdSSanjay Patel // 3a17f03bdSSanjay Patel // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. 4a17f03bdSSanjay Patel // See https://llvm.org/LICENSE.txt for license information. 5a17f03bdSSanjay Patel // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception 6a17f03bdSSanjay Patel // 7a17f03bdSSanjay Patel //===----------------------------------------------------------------------===// 8a17f03bdSSanjay Patel // 9a17f03bdSSanjay Patel // This pass optimizes scalar/vector interactions using target cost models. The 10a17f03bdSSanjay Patel // transforms implemented here may not fit in traditional loop-based or SLP 11a17f03bdSSanjay Patel // vectorization passes. 12a17f03bdSSanjay Patel // 13a17f03bdSSanjay Patel //===----------------------------------------------------------------------===// 14a17f03bdSSanjay Patel 15a17f03bdSSanjay Patel #include "llvm/Transforms/Vectorize/VectorCombine.h" 16a17f03bdSSanjay Patel #include "llvm/ADT/Statistic.h" 175006e551SSimon Pilgrim #include "llvm/Analysis/BasicAliasAnalysis.h" 18a17f03bdSSanjay Patel #include "llvm/Analysis/GlobalsModRef.h" 1943bdac29SSanjay Patel #include "llvm/Analysis/Loads.h" 20a17f03bdSSanjay Patel #include "llvm/Analysis/TargetTransformInfo.h" 2119b62b79SSanjay Patel #include "llvm/Analysis/ValueTracking.h" 22b6050ca1SSanjay Patel #include "llvm/Analysis/VectorUtils.h" 23a17f03bdSSanjay Patel #include "llvm/IR/Dominators.h" 24a17f03bdSSanjay Patel #include "llvm/IR/Function.h" 25a17f03bdSSanjay Patel #include "llvm/IR/IRBuilder.h" 26a17f03bdSSanjay Patel #include "llvm/IR/PatternMatch.h" 27a17f03bdSSanjay Patel #include "llvm/InitializePasses.h" 28a17f03bdSSanjay Patel #include "llvm/Pass.h" 2925c6544fSSanjay Patel #include "llvm/Support/CommandLine.h" 30a17f03bdSSanjay Patel #include "llvm/Transforms/Utils/Local.h" 315006e551SSimon Pilgrim #include "llvm/Transforms/Vectorize.h" 32a17f03bdSSanjay Patel 33a17f03bdSSanjay Patel using namespace llvm; 34a17f03bdSSanjay Patel using namespace llvm::PatternMatch; 35a17f03bdSSanjay Patel 36a17f03bdSSanjay Patel #define DEBUG_TYPE "vector-combine" 3743bdac29SSanjay Patel STATISTIC(NumVecLoad, "Number of vector loads formed"); 38a17f03bdSSanjay Patel STATISTIC(NumVecCmp, "Number of vector compares formed"); 3919b62b79SSanjay Patel STATISTIC(NumVecBO, "Number of vector binops formed"); 40b6315aeeSSanjay Patel STATISTIC(NumVecCmpBO, "Number of vector compare + binop formed"); 417aeb41b3SRoman Lebedev STATISTIC(NumShufOfBitcast, "Number of shuffles moved after bitcast"); 420d2a0b44SSanjay Patel STATISTIC(NumScalarBO, "Number of scalar binops formed"); 43ed67f5e7SSanjay Patel STATISTIC(NumScalarCmp, "Number of scalar compares formed"); 44a17f03bdSSanjay Patel 4525c6544fSSanjay Patel static cl::opt<bool> DisableVectorCombine( 4625c6544fSSanjay Patel "disable-vector-combine", cl::init(false), cl::Hidden, 4725c6544fSSanjay Patel cl::desc("Disable all vector combine transforms")); 4825c6544fSSanjay Patel 49a69158c1SSanjay Patel static cl::opt<bool> DisableBinopExtractShuffle( 50a69158c1SSanjay Patel "disable-binop-extract-shuffle", cl::init(false), cl::Hidden, 51a69158c1SSanjay Patel cl::desc("Disable binop extract to shuffle transforms")); 52a69158c1SSanjay Patel 53a0f96741SSanjay Patel static const unsigned InvalidIndex = std::numeric_limits<unsigned>::max(); 54a0f96741SSanjay Patel 55b4447054SBenjamin Kramer namespace { 566bdd531aSSanjay Patel class VectorCombine { 576bdd531aSSanjay Patel public: 586bdd531aSSanjay Patel VectorCombine(Function &F, const TargetTransformInfo &TTI, 596bdd531aSSanjay Patel const DominatorTree &DT) 60de65b356SSanjay Patel : F(F), Builder(F.getContext()), TTI(TTI), DT(DT) {} 616bdd531aSSanjay Patel 626bdd531aSSanjay Patel bool run(); 636bdd531aSSanjay Patel 646bdd531aSSanjay Patel private: 656bdd531aSSanjay Patel Function &F; 66de65b356SSanjay Patel IRBuilder<> Builder; 676bdd531aSSanjay Patel const TargetTransformInfo &TTI; 686bdd531aSSanjay Patel const DominatorTree &DT; 696bdd531aSSanjay Patel 7043bdac29SSanjay Patel bool vectorizeLoadInsert(Instruction &I); 713b95d834SSanjay Patel ExtractElementInst *getShuffleExtract(ExtractElementInst *Ext0, 723b95d834SSanjay Patel ExtractElementInst *Ext1, 733b95d834SSanjay Patel unsigned PreferredExtractIndex) const; 746bdd531aSSanjay Patel bool isExtractExtractCheap(ExtractElementInst *Ext0, ExtractElementInst *Ext1, 756bdd531aSSanjay Patel unsigned Opcode, 766bdd531aSSanjay Patel ExtractElementInst *&ConvertToShuffle, 776bdd531aSSanjay Patel unsigned PreferredExtractIndex); 78de65b356SSanjay Patel void foldExtExtCmp(ExtractElementInst *Ext0, ExtractElementInst *Ext1, 79de65b356SSanjay Patel Instruction &I); 80de65b356SSanjay Patel void foldExtExtBinop(ExtractElementInst *Ext0, ExtractElementInst *Ext1, 81de65b356SSanjay Patel Instruction &I); 826bdd531aSSanjay Patel bool foldExtractExtract(Instruction &I); 836bdd531aSSanjay Patel bool foldBitcastShuf(Instruction &I); 846bdd531aSSanjay Patel bool scalarizeBinopOrCmp(Instruction &I); 85b6315aeeSSanjay Patel bool foldExtractedCmps(Instruction &I); 866bdd531aSSanjay Patel }; 87b4447054SBenjamin Kramer } // namespace 88a69158c1SSanjay Patel 8998c2f4eeSSanjay Patel static void replaceValue(Value &Old, Value &New) { 9098c2f4eeSSanjay Patel Old.replaceAllUsesWith(&New); 9198c2f4eeSSanjay Patel New.takeName(&Old); 9298c2f4eeSSanjay Patel } 9398c2f4eeSSanjay Patel 9443bdac29SSanjay Patel bool VectorCombine::vectorizeLoadInsert(Instruction &I) { 95b2ef2640SSanjay Patel // Match insert into fixed vector of scalar value. 9647aaa99cSSanjay Patel // TODO: Handle non-zero insert index. 97ddd9575dSSanjay Patel auto *Ty = dyn_cast<FixedVectorType>(I.getType()); 9843bdac29SSanjay Patel Value *Scalar; 9948a23bccSSanjay Patel if (!Ty || !match(&I, m_InsertElt(m_Undef(), m_Value(Scalar), m_ZeroInt())) || 10048a23bccSSanjay Patel !Scalar->hasOneUse()) 10143bdac29SSanjay Patel return false; 102ddd9575dSSanjay Patel 103b2ef2640SSanjay Patel // Optionally match an extract from another vector. 104b2ef2640SSanjay Patel Value *X; 105b2ef2640SSanjay Patel bool HasExtract = match(Scalar, m_ExtractElt(m_Value(X), m_ZeroInt())); 106b2ef2640SSanjay Patel if (!HasExtract) 107b2ef2640SSanjay Patel X = Scalar; 108b2ef2640SSanjay Patel 109b2ef2640SSanjay Patel // Match source value as load of scalar or vector. 1104452cc40SFangrui Song // Do not vectorize scalar load (widening) if atomic/volatile or under 1114452cc40SFangrui Song // asan/hwasan/memtag/tsan. The widened load may load data from dirty regions 1124452cc40SFangrui Song // or create data races non-existent in the source. 113b2ef2640SSanjay Patel auto *Load = dyn_cast<LoadInst>(X); 114b2ef2640SSanjay Patel if (!Load || !Load->isSimple() || !Load->hasOneUse() || 1154452cc40SFangrui Song Load->getFunction()->hasFnAttribute(Attribute::SanitizeMemTag) || 1164452cc40SFangrui Song mustSuppressSpeculation(*Load)) 11743bdac29SSanjay Patel return false; 11843bdac29SSanjay Patel 11912b684aeSSanjay Patel const DataLayout &DL = I.getModule()->getDataLayout(); 12012b684aeSSanjay Patel Value *SrcPtr = Load->getPointerOperand()->stripPointerCasts(); 12112b684aeSSanjay Patel assert(isa<PointerType>(SrcPtr->getType()) && "Expected a pointer type"); 122c36c0fabSArtem Belevich 123c36c0fabSArtem Belevich // If original AS != Load's AS, we can't bitcast the original pointer and have 124c36c0fabSArtem Belevich // to use Load's operand instead. Ideally we would want to strip pointer casts 125c36c0fabSArtem Belevich // without changing AS, but there's no API to do that ATM. 12612b684aeSSanjay Patel unsigned AS = Load->getPointerAddressSpace(); 12712b684aeSSanjay Patel if (AS != SrcPtr->getType()->getPointerAddressSpace()) 12812b684aeSSanjay Patel SrcPtr = Load->getPointerOperand(); 12943bdac29SSanjay Patel 13047aaa99cSSanjay Patel // We are potentially transforming byte-sized (8-bit) memory accesses, so make 13147aaa99cSSanjay Patel // sure we have all of our type-based constraints in place for this target. 132ddd9575dSSanjay Patel Type *ScalarTy = Scalar->getType(); 13343bdac29SSanjay Patel uint64_t ScalarSize = ScalarTy->getPrimitiveSizeInBits(); 134ddd9575dSSanjay Patel unsigned MinVectorSize = TTI.getMinVectorRegisterBitWidth(); 13547aaa99cSSanjay Patel if (!ScalarSize || !MinVectorSize || MinVectorSize % ScalarSize != 0 || 13647aaa99cSSanjay Patel ScalarSize % 8 != 0) 13743bdac29SSanjay Patel return false; 13843bdac29SSanjay Patel 13943bdac29SSanjay Patel // Check safety of replacing the scalar load with a larger vector load. 140aaaf0ec7SSanjay Patel // We use minimal alignment (maximum flexibility) because we only care about 141aaaf0ec7SSanjay Patel // the dereferenceable region. When calculating cost and creating a new op, 142aaaf0ec7SSanjay Patel // we may use a larger value based on alignment attributes. 1438fb05593SSanjay Patel unsigned MinVecNumElts = MinVectorSize / ScalarSize; 1448fb05593SSanjay Patel auto *MinVecTy = VectorType::get(ScalarTy, MinVecNumElts, false); 14547aaa99cSSanjay Patel unsigned OffsetEltIndex = 0; 14647aaa99cSSanjay Patel Align Alignment = Load->getAlign(); 14747aaa99cSSanjay Patel if (!isSafeToLoadUnconditionally(SrcPtr, MinVecTy, Align(1), DL, Load, &DT)) { 14847aaa99cSSanjay Patel // It is not safe to load directly from the pointer, but we can still peek 14947aaa99cSSanjay Patel // through gep offsets and check if it safe to load from a base address with 15047aaa99cSSanjay Patel // updated alignment. If it is, we can shuffle the element(s) into place 15147aaa99cSSanjay Patel // after loading. 15247aaa99cSSanjay Patel unsigned OffsetBitWidth = DL.getIndexTypeSizeInBits(SrcPtr->getType()); 15347aaa99cSSanjay Patel APInt Offset(OffsetBitWidth, 0); 15447aaa99cSSanjay Patel SrcPtr = SrcPtr->stripAndAccumulateInBoundsConstantOffsets(DL, Offset); 15547aaa99cSSanjay Patel 15647aaa99cSSanjay Patel // We want to shuffle the result down from a high element of a vector, so 15747aaa99cSSanjay Patel // the offset must be positive. 15847aaa99cSSanjay Patel if (Offset.isNegative()) 15947aaa99cSSanjay Patel return false; 16047aaa99cSSanjay Patel 16147aaa99cSSanjay Patel // The offset must be a multiple of the scalar element to shuffle cleanly 16247aaa99cSSanjay Patel // in the element's size. 16347aaa99cSSanjay Patel uint64_t ScalarSizeInBytes = ScalarSize / 8; 16447aaa99cSSanjay Patel if (Offset.urem(ScalarSizeInBytes) != 0) 16547aaa99cSSanjay Patel return false; 16647aaa99cSSanjay Patel 16747aaa99cSSanjay Patel // If we load MinVecNumElts, will our target element still be loaded? 16847aaa99cSSanjay Patel OffsetEltIndex = Offset.udiv(ScalarSizeInBytes).getZExtValue(); 16947aaa99cSSanjay Patel if (OffsetEltIndex >= MinVecNumElts) 17047aaa99cSSanjay Patel return false; 17147aaa99cSSanjay Patel 172aaaf0ec7SSanjay Patel if (!isSafeToLoadUnconditionally(SrcPtr, MinVecTy, Align(1), DL, Load, &DT)) 17343bdac29SSanjay Patel return false; 17443bdac29SSanjay Patel 17547aaa99cSSanjay Patel // Update alignment with offset value. Note that the offset could be negated 17647aaa99cSSanjay Patel // to more accurately represent "(new) SrcPtr - Offset = (old) SrcPtr", but 17747aaa99cSSanjay Patel // negation does not change the result of the alignment calculation. 17847aaa99cSSanjay Patel Alignment = commonAlignment(Alignment, Offset.getZExtValue()); 17947aaa99cSSanjay Patel } 18047aaa99cSSanjay Patel 181b2ef2640SSanjay Patel // Original pattern: insertelt undef, load [free casts of] PtrOp, 0 18238ebc1a1SSanjay Patel // Use the greater of the alignment on the load or its source pointer. 18347aaa99cSSanjay Patel Alignment = std::max(SrcPtr->getPointerAlignment(DL), Alignment); 184b2ef2640SSanjay Patel Type *LoadTy = Load->getType(); 18536710c38SCaroline Concatto InstructionCost OldCost = 18636710c38SCaroline Concatto TTI.getMemoryOpCost(Instruction::Load, LoadTy, Alignment, AS); 1878fb05593SSanjay Patel APInt DemandedElts = APInt::getOneBitSet(MinVecNumElts, 0); 188b2ef2640SSanjay Patel OldCost += TTI.getScalarizationOverhead(MinVecTy, DemandedElts, 189b2ef2640SSanjay Patel /* Insert */ true, HasExtract); 19043bdac29SSanjay Patel 19143bdac29SSanjay Patel // New pattern: load VecPtr 19236710c38SCaroline Concatto InstructionCost NewCost = 19336710c38SCaroline Concatto TTI.getMemoryOpCost(Instruction::Load, MinVecTy, Alignment, AS); 19447aaa99cSSanjay Patel // Optionally, we are shuffling the loaded vector element(s) into place. 195*e2935dcfSDavid Green // For the mask set everything but element 0 to undef to prevent poison from 196*e2935dcfSDavid Green // propagating from the extra loaded memory. This will also optionally 197*e2935dcfSDavid Green // shrink/grow the vector from the loaded size to the output size. 198*e2935dcfSDavid Green // We assume this operation has no cost in codegen if there was no offset. 199*e2935dcfSDavid Green // Note that we could use freeze to avoid poison problems, but then we might 200*e2935dcfSDavid Green // still need a shuffle to change the vector size. 201*e2935dcfSDavid Green unsigned OutputNumElts = Ty->getNumElements(); 202*e2935dcfSDavid Green SmallVector<int, 16> Mask(OutputNumElts, UndefMaskElem); 203*e2935dcfSDavid Green assert(OffsetEltIndex < MinVecNumElts && "Address offset too big"); 204*e2935dcfSDavid Green Mask[0] = OffsetEltIndex; 20547aaa99cSSanjay Patel if (OffsetEltIndex) 206*e2935dcfSDavid Green NewCost += TTI.getShuffleCost(TTI::SK_PermuteSingleSrc, MinVecTy, Mask); 20743bdac29SSanjay Patel 20843bdac29SSanjay Patel // We can aggressively convert to the vector form because the backend can 20943bdac29SSanjay Patel // invert this transform if it does not result in a performance win. 21036710c38SCaroline Concatto if (OldCost < NewCost || !NewCost.isValid()) 21143bdac29SSanjay Patel return false; 21243bdac29SSanjay Patel 21343bdac29SSanjay Patel // It is safe and potentially profitable to load a vector directly: 21443bdac29SSanjay Patel // inselt undef, load Scalar, 0 --> load VecPtr 21543bdac29SSanjay Patel IRBuilder<> Builder(Load); 21612b684aeSSanjay Patel Value *CastedPtr = Builder.CreateBitCast(SrcPtr, MinVecTy->getPointerTo(AS)); 2178fb05593SSanjay Patel Value *VecLd = Builder.CreateAlignedLoad(MinVecTy, CastedPtr, Alignment); 2181e6b240dSSanjay Patel VecLd = Builder.CreateShuffleVector(VecLd, Mask); 219d399f870SSanjay Patel 22043bdac29SSanjay Patel replaceValue(I, *VecLd); 22143bdac29SSanjay Patel ++NumVecLoad; 22243bdac29SSanjay Patel return true; 22343bdac29SSanjay Patel } 22443bdac29SSanjay Patel 2253b95d834SSanjay Patel /// Determine which, if any, of the inputs should be replaced by a shuffle 2263b95d834SSanjay Patel /// followed by extract from a different index. 2273b95d834SSanjay Patel ExtractElementInst *VectorCombine::getShuffleExtract( 2283b95d834SSanjay Patel ExtractElementInst *Ext0, ExtractElementInst *Ext1, 2293b95d834SSanjay Patel unsigned PreferredExtractIndex = InvalidIndex) const { 2303b95d834SSanjay Patel assert(isa<ConstantInt>(Ext0->getIndexOperand()) && 2313b95d834SSanjay Patel isa<ConstantInt>(Ext1->getIndexOperand()) && 2323b95d834SSanjay Patel "Expected constant extract indexes"); 2333b95d834SSanjay Patel 2343b95d834SSanjay Patel unsigned Index0 = cast<ConstantInt>(Ext0->getIndexOperand())->getZExtValue(); 2353b95d834SSanjay Patel unsigned Index1 = cast<ConstantInt>(Ext1->getIndexOperand())->getZExtValue(); 2363b95d834SSanjay Patel 2373b95d834SSanjay Patel // If the extract indexes are identical, no shuffle is needed. 2383b95d834SSanjay Patel if (Index0 == Index1) 2393b95d834SSanjay Patel return nullptr; 2403b95d834SSanjay Patel 2413b95d834SSanjay Patel Type *VecTy = Ext0->getVectorOperand()->getType(); 2423b95d834SSanjay Patel assert(VecTy == Ext1->getVectorOperand()->getType() && "Need matching types"); 24336710c38SCaroline Concatto InstructionCost Cost0 = 24436710c38SCaroline Concatto TTI.getVectorInstrCost(Ext0->getOpcode(), VecTy, Index0); 24536710c38SCaroline Concatto InstructionCost Cost1 = 24636710c38SCaroline Concatto TTI.getVectorInstrCost(Ext1->getOpcode(), VecTy, Index1); 24736710c38SCaroline Concatto 24836710c38SCaroline Concatto // If both costs are invalid no shuffle is needed 24936710c38SCaroline Concatto if (!Cost0.isValid() && !Cost1.isValid()) 25036710c38SCaroline Concatto return nullptr; 2513b95d834SSanjay Patel 2523b95d834SSanjay Patel // We are extracting from 2 different indexes, so one operand must be shuffled 2533b95d834SSanjay Patel // before performing a vector operation and/or extract. The more expensive 2543b95d834SSanjay Patel // extract will be replaced by a shuffle. 2553b95d834SSanjay Patel if (Cost0 > Cost1) 2563b95d834SSanjay Patel return Ext0; 2573b95d834SSanjay Patel if (Cost1 > Cost0) 2583b95d834SSanjay Patel return Ext1; 2593b95d834SSanjay Patel 2603b95d834SSanjay Patel // If the costs are equal and there is a preferred extract index, shuffle the 2613b95d834SSanjay Patel // opposite operand. 2623b95d834SSanjay Patel if (PreferredExtractIndex == Index0) 2633b95d834SSanjay Patel return Ext1; 2643b95d834SSanjay Patel if (PreferredExtractIndex == Index1) 2653b95d834SSanjay Patel return Ext0; 2663b95d834SSanjay Patel 2673b95d834SSanjay Patel // Otherwise, replace the extract with the higher index. 2683b95d834SSanjay Patel return Index0 > Index1 ? Ext0 : Ext1; 2693b95d834SSanjay Patel } 2703b95d834SSanjay Patel 271a69158c1SSanjay Patel /// Compare the relative costs of 2 extracts followed by scalar operation vs. 272a69158c1SSanjay Patel /// vector operation(s) followed by extract. Return true if the existing 273a69158c1SSanjay Patel /// instructions are cheaper than a vector alternative. Otherwise, return false 274a69158c1SSanjay Patel /// and if one of the extracts should be transformed to a shufflevector, set 275a69158c1SSanjay Patel /// \p ConvertToShuffle to that extract instruction. 2766bdd531aSSanjay Patel bool VectorCombine::isExtractExtractCheap(ExtractElementInst *Ext0, 2776bdd531aSSanjay Patel ExtractElementInst *Ext1, 2786bdd531aSSanjay Patel unsigned Opcode, 279216a37bbSSanjay Patel ExtractElementInst *&ConvertToShuffle, 280ce97ce3aSSanjay Patel unsigned PreferredExtractIndex) { 2814fa63fd4SAustin Kerbow assert(isa<ConstantInt>(Ext0->getOperand(1)) && 282a69158c1SSanjay Patel isa<ConstantInt>(Ext1->getOperand(1)) && 283a69158c1SSanjay Patel "Expected constant extract indexes"); 28434e34855SSanjay Patel Type *ScalarTy = Ext0->getType(); 285e3056ae9SSam Parker auto *VecTy = cast<VectorType>(Ext0->getOperand(0)->getType()); 28636710c38SCaroline Concatto InstructionCost ScalarOpCost, VectorOpCost; 28734e34855SSanjay Patel 28834e34855SSanjay Patel // Get cost estimates for scalar and vector versions of the operation. 28934e34855SSanjay Patel bool IsBinOp = Instruction::isBinaryOp(Opcode); 29034e34855SSanjay Patel if (IsBinOp) { 29134e34855SSanjay Patel ScalarOpCost = TTI.getArithmeticInstrCost(Opcode, ScalarTy); 29234e34855SSanjay Patel VectorOpCost = TTI.getArithmeticInstrCost(Opcode, VecTy); 29334e34855SSanjay Patel } else { 29434e34855SSanjay Patel assert((Opcode == Instruction::ICmp || Opcode == Instruction::FCmp) && 29534e34855SSanjay Patel "Expected a compare"); 29634e34855SSanjay Patel ScalarOpCost = TTI.getCmpSelInstrCost(Opcode, ScalarTy, 29734e34855SSanjay Patel CmpInst::makeCmpResultType(ScalarTy)); 29834e34855SSanjay Patel VectorOpCost = TTI.getCmpSelInstrCost(Opcode, VecTy, 29934e34855SSanjay Patel CmpInst::makeCmpResultType(VecTy)); 30034e34855SSanjay Patel } 30134e34855SSanjay Patel 302a69158c1SSanjay Patel // Get cost estimates for the extract elements. These costs will factor into 30334e34855SSanjay Patel // both sequences. 304a69158c1SSanjay Patel unsigned Ext0Index = cast<ConstantInt>(Ext0->getOperand(1))->getZExtValue(); 305a69158c1SSanjay Patel unsigned Ext1Index = cast<ConstantInt>(Ext1->getOperand(1))->getZExtValue(); 306a69158c1SSanjay Patel 30736710c38SCaroline Concatto InstructionCost Extract0Cost = 3086bdd531aSSanjay Patel TTI.getVectorInstrCost(Instruction::ExtractElement, VecTy, Ext0Index); 30936710c38SCaroline Concatto InstructionCost Extract1Cost = 3106bdd531aSSanjay Patel TTI.getVectorInstrCost(Instruction::ExtractElement, VecTy, Ext1Index); 311a69158c1SSanjay Patel 312a69158c1SSanjay Patel // A more expensive extract will always be replaced by a splat shuffle. 313a69158c1SSanjay Patel // For example, if Ext0 is more expensive: 314a69158c1SSanjay Patel // opcode (extelt V0, Ext0), (ext V1, Ext1) --> 315a69158c1SSanjay Patel // extelt (opcode (splat V0, Ext0), V1), Ext1 316a69158c1SSanjay Patel // TODO: Evaluate whether that always results in lowest cost. Alternatively, 317a69158c1SSanjay Patel // check the cost of creating a broadcast shuffle and shuffling both 318a69158c1SSanjay Patel // operands to element 0. 31936710c38SCaroline Concatto InstructionCost CheapExtractCost = std::min(Extract0Cost, Extract1Cost); 32034e34855SSanjay Patel 32134e34855SSanjay Patel // Extra uses of the extracts mean that we include those costs in the 32234e34855SSanjay Patel // vector total because those instructions will not be eliminated. 32336710c38SCaroline Concatto InstructionCost OldCost, NewCost; 324a69158c1SSanjay Patel if (Ext0->getOperand(0) == Ext1->getOperand(0) && Ext0Index == Ext1Index) { 325a69158c1SSanjay Patel // Handle a special case. If the 2 extracts are identical, adjust the 32634e34855SSanjay Patel // formulas to account for that. The extra use charge allows for either the 32734e34855SSanjay Patel // CSE'd pattern or an unoptimized form with identical values: 32834e34855SSanjay Patel // opcode (extelt V, C), (extelt V, C) --> extelt (opcode V, V), C 32934e34855SSanjay Patel bool HasUseTax = Ext0 == Ext1 ? !Ext0->hasNUses(2) 33034e34855SSanjay Patel : !Ext0->hasOneUse() || !Ext1->hasOneUse(); 331a69158c1SSanjay Patel OldCost = CheapExtractCost + ScalarOpCost; 332a69158c1SSanjay Patel NewCost = VectorOpCost + CheapExtractCost + HasUseTax * CheapExtractCost; 33334e34855SSanjay Patel } else { 33434e34855SSanjay Patel // Handle the general case. Each extract is actually a different value: 335a69158c1SSanjay Patel // opcode (extelt V0, C0), (extelt V1, C1) --> extelt (opcode V0, V1), C 336a69158c1SSanjay Patel OldCost = Extract0Cost + Extract1Cost + ScalarOpCost; 337a69158c1SSanjay Patel NewCost = VectorOpCost + CheapExtractCost + 338a69158c1SSanjay Patel !Ext0->hasOneUse() * Extract0Cost + 339a69158c1SSanjay Patel !Ext1->hasOneUse() * Extract1Cost; 34034e34855SSanjay Patel } 341a69158c1SSanjay Patel 3423b95d834SSanjay Patel ConvertToShuffle = getShuffleExtract(Ext0, Ext1, PreferredExtractIndex); 3433b95d834SSanjay Patel if (ConvertToShuffle) { 344a69158c1SSanjay Patel if (IsBinOp && DisableBinopExtractShuffle) 345a69158c1SSanjay Patel return true; 346a69158c1SSanjay Patel 347a69158c1SSanjay Patel // If we are extracting from 2 different indexes, then one operand must be 348a69158c1SSanjay Patel // shuffled before performing the vector operation. The shuffle mask is 349a69158c1SSanjay Patel // undefined except for 1 lane that is being translated to the remaining 350a69158c1SSanjay Patel // extraction lane. Therefore, it is a splat shuffle. Ex: 351a69158c1SSanjay Patel // ShufMask = { undef, undef, 0, undef } 352a69158c1SSanjay Patel // TODO: The cost model has an option for a "broadcast" shuffle 353a69158c1SSanjay Patel // (splat-from-element-0), but no option for a more general splat. 354a69158c1SSanjay Patel NewCost += 355a69158c1SSanjay Patel TTI.getShuffleCost(TargetTransformInfo::SK_PermuteSingleSrc, VecTy); 356a69158c1SSanjay Patel } 357a69158c1SSanjay Patel 35810ea01d8SSanjay Patel // Aggressively form a vector op if the cost is equal because the transform 35910ea01d8SSanjay Patel // may enable further optimization. 36010ea01d8SSanjay Patel // Codegen can reverse this transform (scalarize) if it was not profitable. 36110ea01d8SSanjay Patel return OldCost < NewCost; 36234e34855SSanjay Patel } 36334e34855SSanjay Patel 3649934cc54SSanjay Patel /// Create a shuffle that translates (shifts) 1 element from the input vector 3659934cc54SSanjay Patel /// to a new element location. 3669934cc54SSanjay Patel static Value *createShiftShuffle(Value *Vec, unsigned OldIndex, 3679934cc54SSanjay Patel unsigned NewIndex, IRBuilder<> &Builder) { 3689934cc54SSanjay Patel // The shuffle mask is undefined except for 1 lane that is being translated 3699934cc54SSanjay Patel // to the new element index. Example for OldIndex == 2 and NewIndex == 0: 3709934cc54SSanjay Patel // ShufMask = { 2, undef, undef, undef } 3719934cc54SSanjay Patel auto *VecTy = cast<FixedVectorType>(Vec->getType()); 37254143e2bSSanjay Patel SmallVector<int, 32> ShufMask(VecTy->getNumElements(), UndefMaskElem); 3739934cc54SSanjay Patel ShufMask[NewIndex] = OldIndex; 3741e6b240dSSanjay Patel return Builder.CreateShuffleVector(Vec, ShufMask, "shift"); 3759934cc54SSanjay Patel } 3769934cc54SSanjay Patel 377216a37bbSSanjay Patel /// Given an extract element instruction with constant index operand, shuffle 378216a37bbSSanjay Patel /// the source vector (shift the scalar element) to a NewIndex for extraction. 379216a37bbSSanjay Patel /// Return null if the input can be constant folded, so that we are not creating 380216a37bbSSanjay Patel /// unnecessary instructions. 3819934cc54SSanjay Patel static ExtractElementInst *translateExtract(ExtractElementInst *ExtElt, 3829934cc54SSanjay Patel unsigned NewIndex, 3839934cc54SSanjay Patel IRBuilder<> &Builder) { 384216a37bbSSanjay Patel // If the extract can be constant-folded, this code is unsimplified. Defer 385216a37bbSSanjay Patel // to other passes to handle that. 386216a37bbSSanjay Patel Value *X = ExtElt->getVectorOperand(); 387216a37bbSSanjay Patel Value *C = ExtElt->getIndexOperand(); 388de65b356SSanjay Patel assert(isa<ConstantInt>(C) && "Expected a constant index operand"); 389216a37bbSSanjay Patel if (isa<Constant>(X)) 390216a37bbSSanjay Patel return nullptr; 391216a37bbSSanjay Patel 3929934cc54SSanjay Patel Value *Shuf = createShiftShuffle(X, cast<ConstantInt>(C)->getZExtValue(), 3939934cc54SSanjay Patel NewIndex, Builder); 394216a37bbSSanjay Patel return cast<ExtractElementInst>(Builder.CreateExtractElement(Shuf, NewIndex)); 395216a37bbSSanjay Patel } 396216a37bbSSanjay Patel 397fc445589SSanjay Patel /// Try to reduce extract element costs by converting scalar compares to vector 398fc445589SSanjay Patel /// compares followed by extract. 399e9c79a7aSSanjay Patel /// cmp (ext0 V0, C), (ext1 V1, C) 400de65b356SSanjay Patel void VectorCombine::foldExtExtCmp(ExtractElementInst *Ext0, 401de65b356SSanjay Patel ExtractElementInst *Ext1, Instruction &I) { 402fc445589SSanjay Patel assert(isa<CmpInst>(&I) && "Expected a compare"); 403216a37bbSSanjay Patel assert(cast<ConstantInt>(Ext0->getIndexOperand())->getZExtValue() == 404216a37bbSSanjay Patel cast<ConstantInt>(Ext1->getIndexOperand())->getZExtValue() && 405216a37bbSSanjay Patel "Expected matching constant extract indexes"); 406a17f03bdSSanjay Patel 407a17f03bdSSanjay Patel // cmp Pred (extelt V0, C), (extelt V1, C) --> extelt (cmp Pred V0, V1), C 408a17f03bdSSanjay Patel ++NumVecCmp; 409fc445589SSanjay Patel CmpInst::Predicate Pred = cast<CmpInst>(&I)->getPredicate(); 410216a37bbSSanjay Patel Value *V0 = Ext0->getVectorOperand(), *V1 = Ext1->getVectorOperand(); 41146a285adSSanjay Patel Value *VecCmp = Builder.CreateCmp(Pred, V0, V1); 412216a37bbSSanjay Patel Value *NewExt = Builder.CreateExtractElement(VecCmp, Ext0->getIndexOperand()); 41398c2f4eeSSanjay Patel replaceValue(I, *NewExt); 414a17f03bdSSanjay Patel } 415a17f03bdSSanjay Patel 41619b62b79SSanjay Patel /// Try to reduce extract element costs by converting scalar binops to vector 41719b62b79SSanjay Patel /// binops followed by extract. 418e9c79a7aSSanjay Patel /// bo (ext0 V0, C), (ext1 V1, C) 419de65b356SSanjay Patel void VectorCombine::foldExtExtBinop(ExtractElementInst *Ext0, 420de65b356SSanjay Patel ExtractElementInst *Ext1, Instruction &I) { 421fc445589SSanjay Patel assert(isa<BinaryOperator>(&I) && "Expected a binary operator"); 422216a37bbSSanjay Patel assert(cast<ConstantInt>(Ext0->getIndexOperand())->getZExtValue() == 423216a37bbSSanjay Patel cast<ConstantInt>(Ext1->getIndexOperand())->getZExtValue() && 424216a37bbSSanjay Patel "Expected matching constant extract indexes"); 42519b62b79SSanjay Patel 42634e34855SSanjay Patel // bo (extelt V0, C), (extelt V1, C) --> extelt (bo V0, V1), C 42719b62b79SSanjay Patel ++NumVecBO; 428216a37bbSSanjay Patel Value *V0 = Ext0->getVectorOperand(), *V1 = Ext1->getVectorOperand(); 429e9c79a7aSSanjay Patel Value *VecBO = 43034e34855SSanjay Patel Builder.CreateBinOp(cast<BinaryOperator>(&I)->getOpcode(), V0, V1); 431e9c79a7aSSanjay Patel 43219b62b79SSanjay Patel // All IR flags are safe to back-propagate because any potential poison 43319b62b79SSanjay Patel // created in unused vector elements is discarded by the extract. 434e9c79a7aSSanjay Patel if (auto *VecBOInst = dyn_cast<Instruction>(VecBO)) 43519b62b79SSanjay Patel VecBOInst->copyIRFlags(&I); 436e9c79a7aSSanjay Patel 437216a37bbSSanjay Patel Value *NewExt = Builder.CreateExtractElement(VecBO, Ext0->getIndexOperand()); 43898c2f4eeSSanjay Patel replaceValue(I, *NewExt); 43919b62b79SSanjay Patel } 44019b62b79SSanjay Patel 441fc445589SSanjay Patel /// Match an instruction with extracted vector operands. 4426bdd531aSSanjay Patel bool VectorCombine::foldExtractExtract(Instruction &I) { 443e9c79a7aSSanjay Patel // It is not safe to transform things like div, urem, etc. because we may 444e9c79a7aSSanjay Patel // create undefined behavior when executing those on unknown vector elements. 445e9c79a7aSSanjay Patel if (!isSafeToSpeculativelyExecute(&I)) 446e9c79a7aSSanjay Patel return false; 447e9c79a7aSSanjay Patel 448216a37bbSSanjay Patel Instruction *I0, *I1; 449fc445589SSanjay Patel CmpInst::Predicate Pred = CmpInst::BAD_ICMP_PREDICATE; 450216a37bbSSanjay Patel if (!match(&I, m_Cmp(Pred, m_Instruction(I0), m_Instruction(I1))) && 451216a37bbSSanjay Patel !match(&I, m_BinOp(m_Instruction(I0), m_Instruction(I1)))) 452fc445589SSanjay Patel return false; 453fc445589SSanjay Patel 454fc445589SSanjay Patel Value *V0, *V1; 455fc445589SSanjay Patel uint64_t C0, C1; 456216a37bbSSanjay Patel if (!match(I0, m_ExtractElt(m_Value(V0), m_ConstantInt(C0))) || 457216a37bbSSanjay Patel !match(I1, m_ExtractElt(m_Value(V1), m_ConstantInt(C1))) || 458fc445589SSanjay Patel V0->getType() != V1->getType()) 459fc445589SSanjay Patel return false; 460fc445589SSanjay Patel 461ce97ce3aSSanjay Patel // If the scalar value 'I' is going to be re-inserted into a vector, then try 462ce97ce3aSSanjay Patel // to create an extract to that same element. The extract/insert can be 463ce97ce3aSSanjay Patel // reduced to a "select shuffle". 464ce97ce3aSSanjay Patel // TODO: If we add a larger pattern match that starts from an insert, this 465ce97ce3aSSanjay Patel // probably becomes unnecessary. 466216a37bbSSanjay Patel auto *Ext0 = cast<ExtractElementInst>(I0); 467216a37bbSSanjay Patel auto *Ext1 = cast<ExtractElementInst>(I1); 468a0f96741SSanjay Patel uint64_t InsertIndex = InvalidIndex; 469ce97ce3aSSanjay Patel if (I.hasOneUse()) 4707eed772aSSanjay Patel match(I.user_back(), 4717eed772aSSanjay Patel m_InsertElt(m_Value(), m_Value(), m_ConstantInt(InsertIndex))); 472ce97ce3aSSanjay Patel 473216a37bbSSanjay Patel ExtractElementInst *ExtractToChange; 4746bdd531aSSanjay Patel if (isExtractExtractCheap(Ext0, Ext1, I.getOpcode(), ExtractToChange, 475ce97ce3aSSanjay Patel InsertIndex)) 476fc445589SSanjay Patel return false; 477e9c79a7aSSanjay Patel 478216a37bbSSanjay Patel if (ExtractToChange) { 479216a37bbSSanjay Patel unsigned CheapExtractIdx = ExtractToChange == Ext0 ? C1 : C0; 480216a37bbSSanjay Patel ExtractElementInst *NewExtract = 4819934cc54SSanjay Patel translateExtract(ExtractToChange, CheapExtractIdx, Builder); 482216a37bbSSanjay Patel if (!NewExtract) 4836d864097SSanjay Patel return false; 484216a37bbSSanjay Patel if (ExtractToChange == Ext0) 485216a37bbSSanjay Patel Ext0 = NewExtract; 486a69158c1SSanjay Patel else 487216a37bbSSanjay Patel Ext1 = NewExtract; 488a69158c1SSanjay Patel } 489e9c79a7aSSanjay Patel 490e9c79a7aSSanjay Patel if (Pred != CmpInst::BAD_ICMP_PREDICATE) 491039ff29eSSanjay Patel foldExtExtCmp(Ext0, Ext1, I); 492e9c79a7aSSanjay Patel else 493039ff29eSSanjay Patel foldExtExtBinop(Ext0, Ext1, I); 494e9c79a7aSSanjay Patel 495e9c79a7aSSanjay Patel return true; 496fc445589SSanjay Patel } 497fc445589SSanjay Patel 498bef6e67eSSanjay Patel /// If this is a bitcast of a shuffle, try to bitcast the source vector to the 499bef6e67eSSanjay Patel /// destination type followed by shuffle. This can enable further transforms by 500bef6e67eSSanjay Patel /// moving bitcasts or shuffles together. 5016bdd531aSSanjay Patel bool VectorCombine::foldBitcastShuf(Instruction &I) { 502b6050ca1SSanjay Patel Value *V; 503b6050ca1SSanjay Patel ArrayRef<int> Mask; 5047eed772aSSanjay Patel if (!match(&I, m_BitCast( 5057eed772aSSanjay Patel m_OneUse(m_Shuffle(m_Value(V), m_Undef(), m_Mask(Mask)))))) 506b6050ca1SSanjay Patel return false; 507b6050ca1SSanjay Patel 508b4f04d71SHuihui Zhang // 1) Do not fold bitcast shuffle for scalable type. First, shuffle cost for 509b4f04d71SHuihui Zhang // scalable type is unknown; Second, we cannot reason if the narrowed shuffle 510b4f04d71SHuihui Zhang // mask for scalable type is a splat or not. 511b4f04d71SHuihui Zhang // 2) Disallow non-vector casts and length-changing shuffles. 512bef6e67eSSanjay Patel // TODO: We could allow any shuffle. 513b4f04d71SHuihui Zhang auto *DestTy = dyn_cast<FixedVectorType>(I.getType()); 514b4f04d71SHuihui Zhang auto *SrcTy = dyn_cast<FixedVectorType>(V->getType()); 515b4f04d71SHuihui Zhang if (!SrcTy || !DestTy || I.getOperand(0)->getType() != SrcTy) 516b6050ca1SSanjay Patel return false; 517b6050ca1SSanjay Patel 518b4f04d71SHuihui Zhang unsigned DestNumElts = DestTy->getNumElements(); 519b4f04d71SHuihui Zhang unsigned SrcNumElts = SrcTy->getNumElements(); 520b6050ca1SSanjay Patel SmallVector<int, 16> NewMask; 521bef6e67eSSanjay Patel if (SrcNumElts <= DestNumElts) { 522bef6e67eSSanjay Patel // The bitcast is from wide to narrow/equal elements. The shuffle mask can 523bef6e67eSSanjay Patel // always be expanded to the equivalent form choosing narrower elements. 524b6050ca1SSanjay Patel assert(DestNumElts % SrcNumElts == 0 && "Unexpected shuffle mask"); 525b6050ca1SSanjay Patel unsigned ScaleFactor = DestNumElts / SrcNumElts; 5261318ddbcSSanjay Patel narrowShuffleMaskElts(ScaleFactor, Mask, NewMask); 527bef6e67eSSanjay Patel } else { 528bef6e67eSSanjay Patel // The bitcast is from narrow elements to wide elements. The shuffle mask 529bef6e67eSSanjay Patel // must choose consecutive elements to allow casting first. 530bef6e67eSSanjay Patel assert(SrcNumElts % DestNumElts == 0 && "Unexpected shuffle mask"); 531bef6e67eSSanjay Patel unsigned ScaleFactor = SrcNumElts / DestNumElts; 532bef6e67eSSanjay Patel if (!widenShuffleMaskElts(ScaleFactor, Mask, NewMask)) 533bef6e67eSSanjay Patel return false; 534bef6e67eSSanjay Patel } 535*e2935dcfSDavid Green 536*e2935dcfSDavid Green // The new shuffle must not cost more than the old shuffle. The bitcast is 537*e2935dcfSDavid Green // moved ahead of the shuffle, so assume that it has the same cost as before. 538*e2935dcfSDavid Green InstructionCost DestCost = TTI.getShuffleCost( 539*e2935dcfSDavid Green TargetTransformInfo::SK_PermuteSingleSrc, DestTy, NewMask); 540*e2935dcfSDavid Green InstructionCost SrcCost = 541*e2935dcfSDavid Green TTI.getShuffleCost(TargetTransformInfo::SK_PermuteSingleSrc, SrcTy, Mask); 542*e2935dcfSDavid Green if (DestCost > SrcCost || !DestCost.isValid()) 543*e2935dcfSDavid Green return false; 544*e2935dcfSDavid Green 545bef6e67eSSanjay Patel // bitcast (shuf V, MaskC) --> shuf (bitcast V), MaskC' 5467aeb41b3SRoman Lebedev ++NumShufOfBitcast; 547bef6e67eSSanjay Patel Value *CastV = Builder.CreateBitCast(V, DestTy); 5481e6b240dSSanjay Patel Value *Shuf = Builder.CreateShuffleVector(CastV, NewMask); 54998c2f4eeSSanjay Patel replaceValue(I, *Shuf); 550b6050ca1SSanjay Patel return true; 551b6050ca1SSanjay Patel } 552b6050ca1SSanjay Patel 553ed67f5e7SSanjay Patel /// Match a vector binop or compare instruction with at least one inserted 554ed67f5e7SSanjay Patel /// scalar operand and convert to scalar binop/cmp followed by insertelement. 5556bdd531aSSanjay Patel bool VectorCombine::scalarizeBinopOrCmp(Instruction &I) { 556ed67f5e7SSanjay Patel CmpInst::Predicate Pred = CmpInst::BAD_ICMP_PREDICATE; 5575dc4e7c2SSimon Pilgrim Value *Ins0, *Ins1; 558ed67f5e7SSanjay Patel if (!match(&I, m_BinOp(m_Value(Ins0), m_Value(Ins1))) && 559ed67f5e7SSanjay Patel !match(&I, m_Cmp(Pred, m_Value(Ins0), m_Value(Ins1)))) 560ed67f5e7SSanjay Patel return false; 561ed67f5e7SSanjay Patel 562ed67f5e7SSanjay Patel // Do not convert the vector condition of a vector select into a scalar 563ed67f5e7SSanjay Patel // condition. That may cause problems for codegen because of differences in 564ed67f5e7SSanjay Patel // boolean formats and register-file transfers. 565ed67f5e7SSanjay Patel // TODO: Can we account for that in the cost model? 566ed67f5e7SSanjay Patel bool IsCmp = Pred != CmpInst::Predicate::BAD_ICMP_PREDICATE; 567ed67f5e7SSanjay Patel if (IsCmp) 568ed67f5e7SSanjay Patel for (User *U : I.users()) 569ed67f5e7SSanjay Patel if (match(U, m_Select(m_Specific(&I), m_Value(), m_Value()))) 5700d2a0b44SSanjay Patel return false; 5710d2a0b44SSanjay Patel 5725dc4e7c2SSimon Pilgrim // Match against one or both scalar values being inserted into constant 5735dc4e7c2SSimon Pilgrim // vectors: 574ed67f5e7SSanjay Patel // vec_op VecC0, (inselt VecC1, V1, Index) 575ed67f5e7SSanjay Patel // vec_op (inselt VecC0, V0, Index), VecC1 576ed67f5e7SSanjay Patel // vec_op (inselt VecC0, V0, Index), (inselt VecC1, V1, Index) 5770d2a0b44SSanjay Patel // TODO: Deal with mismatched index constants and variable indexes? 5785dc4e7c2SSimon Pilgrim Constant *VecC0 = nullptr, *VecC1 = nullptr; 5795dc4e7c2SSimon Pilgrim Value *V0 = nullptr, *V1 = nullptr; 5805dc4e7c2SSimon Pilgrim uint64_t Index0 = 0, Index1 = 0; 5817eed772aSSanjay Patel if (!match(Ins0, m_InsertElt(m_Constant(VecC0), m_Value(V0), 5825dc4e7c2SSimon Pilgrim m_ConstantInt(Index0))) && 5835dc4e7c2SSimon Pilgrim !match(Ins0, m_Constant(VecC0))) 5845dc4e7c2SSimon Pilgrim return false; 5855dc4e7c2SSimon Pilgrim if (!match(Ins1, m_InsertElt(m_Constant(VecC1), m_Value(V1), 5865dc4e7c2SSimon Pilgrim m_ConstantInt(Index1))) && 5875dc4e7c2SSimon Pilgrim !match(Ins1, m_Constant(VecC1))) 5880d2a0b44SSanjay Patel return false; 5890d2a0b44SSanjay Patel 5905dc4e7c2SSimon Pilgrim bool IsConst0 = !V0; 5915dc4e7c2SSimon Pilgrim bool IsConst1 = !V1; 5925dc4e7c2SSimon Pilgrim if (IsConst0 && IsConst1) 5935dc4e7c2SSimon Pilgrim return false; 5945dc4e7c2SSimon Pilgrim if (!IsConst0 && !IsConst1 && Index0 != Index1) 5955dc4e7c2SSimon Pilgrim return false; 5965dc4e7c2SSimon Pilgrim 5975dc4e7c2SSimon Pilgrim // Bail for single insertion if it is a load. 5985dc4e7c2SSimon Pilgrim // TODO: Handle this once getVectorInstrCost can cost for load/stores. 5995dc4e7c2SSimon Pilgrim auto *I0 = dyn_cast_or_null<Instruction>(V0); 6005dc4e7c2SSimon Pilgrim auto *I1 = dyn_cast_or_null<Instruction>(V1); 6015dc4e7c2SSimon Pilgrim if ((IsConst0 && I1 && I1->mayReadFromMemory()) || 6025dc4e7c2SSimon Pilgrim (IsConst1 && I0 && I0->mayReadFromMemory())) 6035dc4e7c2SSimon Pilgrim return false; 6045dc4e7c2SSimon Pilgrim 6055dc4e7c2SSimon Pilgrim uint64_t Index = IsConst0 ? Index1 : Index0; 6065dc4e7c2SSimon Pilgrim Type *ScalarTy = IsConst0 ? V1->getType() : V0->getType(); 6070d2a0b44SSanjay Patel Type *VecTy = I.getType(); 6085dc4e7c2SSimon Pilgrim assert(VecTy->isVectorTy() && 6095dc4e7c2SSimon Pilgrim (IsConst0 || IsConst1 || V0->getType() == V1->getType()) && 610741e20f3SSanjay Patel (ScalarTy->isIntegerTy() || ScalarTy->isFloatingPointTy() || 611741e20f3SSanjay Patel ScalarTy->isPointerTy()) && 612741e20f3SSanjay Patel "Unexpected types for insert element into binop or cmp"); 6130d2a0b44SSanjay Patel 614ed67f5e7SSanjay Patel unsigned Opcode = I.getOpcode(); 61536710c38SCaroline Concatto InstructionCost ScalarOpCost, VectorOpCost; 616ed67f5e7SSanjay Patel if (IsCmp) { 617ed67f5e7SSanjay Patel ScalarOpCost = TTI.getCmpSelInstrCost(Opcode, ScalarTy); 618ed67f5e7SSanjay Patel VectorOpCost = TTI.getCmpSelInstrCost(Opcode, VecTy); 619ed67f5e7SSanjay Patel } else { 620ed67f5e7SSanjay Patel ScalarOpCost = TTI.getArithmeticInstrCost(Opcode, ScalarTy); 621ed67f5e7SSanjay Patel VectorOpCost = TTI.getArithmeticInstrCost(Opcode, VecTy); 622ed67f5e7SSanjay Patel } 6230d2a0b44SSanjay Patel 6240d2a0b44SSanjay Patel // Get cost estimate for the insert element. This cost will factor into 6250d2a0b44SSanjay Patel // both sequences. 62636710c38SCaroline Concatto InstructionCost InsertCost = 6270d2a0b44SSanjay Patel TTI.getVectorInstrCost(Instruction::InsertElement, VecTy, Index); 62836710c38SCaroline Concatto InstructionCost OldCost = 62936710c38SCaroline Concatto (IsConst0 ? 0 : InsertCost) + (IsConst1 ? 0 : InsertCost) + VectorOpCost; 63036710c38SCaroline Concatto InstructionCost NewCost = ScalarOpCost + InsertCost + 6315dc4e7c2SSimon Pilgrim (IsConst0 ? 0 : !Ins0->hasOneUse() * InsertCost) + 6325dc4e7c2SSimon Pilgrim (IsConst1 ? 0 : !Ins1->hasOneUse() * InsertCost); 6330d2a0b44SSanjay Patel 6340d2a0b44SSanjay Patel // We want to scalarize unless the vector variant actually has lower cost. 63536710c38SCaroline Concatto if (OldCost < NewCost || !NewCost.isValid()) 6360d2a0b44SSanjay Patel return false; 6370d2a0b44SSanjay Patel 638ed67f5e7SSanjay Patel // vec_op (inselt VecC0, V0, Index), (inselt VecC1, V1, Index) --> 639ed67f5e7SSanjay Patel // inselt NewVecC, (scalar_op V0, V1), Index 640ed67f5e7SSanjay Patel if (IsCmp) 641ed67f5e7SSanjay Patel ++NumScalarCmp; 642ed67f5e7SSanjay Patel else 6430d2a0b44SSanjay Patel ++NumScalarBO; 6445dc4e7c2SSimon Pilgrim 6455dc4e7c2SSimon Pilgrim // For constant cases, extract the scalar element, this should constant fold. 6465dc4e7c2SSimon Pilgrim if (IsConst0) 6475dc4e7c2SSimon Pilgrim V0 = ConstantExpr::getExtractElement(VecC0, Builder.getInt64(Index)); 6485dc4e7c2SSimon Pilgrim if (IsConst1) 6495dc4e7c2SSimon Pilgrim V1 = ConstantExpr::getExtractElement(VecC1, Builder.getInt64(Index)); 6505dc4e7c2SSimon Pilgrim 651ed67f5e7SSanjay Patel Value *Scalar = 65246a285adSSanjay Patel IsCmp ? Builder.CreateCmp(Pred, V0, V1) 653ed67f5e7SSanjay Patel : Builder.CreateBinOp((Instruction::BinaryOps)Opcode, V0, V1); 654ed67f5e7SSanjay Patel 655ed67f5e7SSanjay Patel Scalar->setName(I.getName() + ".scalar"); 6560d2a0b44SSanjay Patel 6570d2a0b44SSanjay Patel // All IR flags are safe to back-propagate. There is no potential for extra 6580d2a0b44SSanjay Patel // poison to be created by the scalar instruction. 6590d2a0b44SSanjay Patel if (auto *ScalarInst = dyn_cast<Instruction>(Scalar)) 6600d2a0b44SSanjay Patel ScalarInst->copyIRFlags(&I); 6610d2a0b44SSanjay Patel 6620d2a0b44SSanjay Patel // Fold the vector constants in the original vectors into a new base vector. 663ed67f5e7SSanjay Patel Constant *NewVecC = IsCmp ? ConstantExpr::getCompare(Pred, VecC0, VecC1) 664ed67f5e7SSanjay Patel : ConstantExpr::get(Opcode, VecC0, VecC1); 6650d2a0b44SSanjay Patel Value *Insert = Builder.CreateInsertElement(NewVecC, Scalar, Index); 66698c2f4eeSSanjay Patel replaceValue(I, *Insert); 6670d2a0b44SSanjay Patel return true; 6680d2a0b44SSanjay Patel } 6690d2a0b44SSanjay Patel 670b6315aeeSSanjay Patel /// Try to combine a scalar binop + 2 scalar compares of extracted elements of 671b6315aeeSSanjay Patel /// a vector into vector operations followed by extract. Note: The SLP pass 672b6315aeeSSanjay Patel /// may miss this pattern because of implementation problems. 673b6315aeeSSanjay Patel bool VectorCombine::foldExtractedCmps(Instruction &I) { 674b6315aeeSSanjay Patel // We are looking for a scalar binop of booleans. 675b6315aeeSSanjay Patel // binop i1 (cmp Pred I0, C0), (cmp Pred I1, C1) 676b6315aeeSSanjay Patel if (!I.isBinaryOp() || !I.getType()->isIntegerTy(1)) 677b6315aeeSSanjay Patel return false; 678b6315aeeSSanjay Patel 679b6315aeeSSanjay Patel // The compare predicates should match, and each compare should have a 680b6315aeeSSanjay Patel // constant operand. 681b6315aeeSSanjay Patel // TODO: Relax the one-use constraints. 682b6315aeeSSanjay Patel Value *B0 = I.getOperand(0), *B1 = I.getOperand(1); 683b6315aeeSSanjay Patel Instruction *I0, *I1; 684b6315aeeSSanjay Patel Constant *C0, *C1; 685b6315aeeSSanjay Patel CmpInst::Predicate P0, P1; 686b6315aeeSSanjay Patel if (!match(B0, m_OneUse(m_Cmp(P0, m_Instruction(I0), m_Constant(C0)))) || 687b6315aeeSSanjay Patel !match(B1, m_OneUse(m_Cmp(P1, m_Instruction(I1), m_Constant(C1)))) || 688b6315aeeSSanjay Patel P0 != P1) 689b6315aeeSSanjay Patel return false; 690b6315aeeSSanjay Patel 691b6315aeeSSanjay Patel // The compare operands must be extracts of the same vector with constant 692b6315aeeSSanjay Patel // extract indexes. 693b6315aeeSSanjay Patel // TODO: Relax the one-use constraints. 694b6315aeeSSanjay Patel Value *X; 695b6315aeeSSanjay Patel uint64_t Index0, Index1; 696b6315aeeSSanjay Patel if (!match(I0, m_OneUse(m_ExtractElt(m_Value(X), m_ConstantInt(Index0)))) || 697b6315aeeSSanjay Patel !match(I1, m_OneUse(m_ExtractElt(m_Specific(X), m_ConstantInt(Index1))))) 698b6315aeeSSanjay Patel return false; 699b6315aeeSSanjay Patel 700b6315aeeSSanjay Patel auto *Ext0 = cast<ExtractElementInst>(I0); 701b6315aeeSSanjay Patel auto *Ext1 = cast<ExtractElementInst>(I1); 702b6315aeeSSanjay Patel ExtractElementInst *ConvertToShuf = getShuffleExtract(Ext0, Ext1); 703b6315aeeSSanjay Patel if (!ConvertToShuf) 704b6315aeeSSanjay Patel return false; 705b6315aeeSSanjay Patel 706b6315aeeSSanjay Patel // The original scalar pattern is: 707b6315aeeSSanjay Patel // binop i1 (cmp Pred (ext X, Index0), C0), (cmp Pred (ext X, Index1), C1) 708b6315aeeSSanjay Patel CmpInst::Predicate Pred = P0; 709b6315aeeSSanjay Patel unsigned CmpOpcode = CmpInst::isFPPredicate(Pred) ? Instruction::FCmp 710b6315aeeSSanjay Patel : Instruction::ICmp; 711b6315aeeSSanjay Patel auto *VecTy = dyn_cast<FixedVectorType>(X->getType()); 712b6315aeeSSanjay Patel if (!VecTy) 713b6315aeeSSanjay Patel return false; 714b6315aeeSSanjay Patel 71536710c38SCaroline Concatto InstructionCost OldCost = 71636710c38SCaroline Concatto TTI.getVectorInstrCost(Ext0->getOpcode(), VecTy, Index0); 717b6315aeeSSanjay Patel OldCost += TTI.getVectorInstrCost(Ext1->getOpcode(), VecTy, Index1); 718b6315aeeSSanjay Patel OldCost += TTI.getCmpSelInstrCost(CmpOpcode, I0->getType()) * 2; 719b6315aeeSSanjay Patel OldCost += TTI.getArithmeticInstrCost(I.getOpcode(), I.getType()); 720b6315aeeSSanjay Patel 721b6315aeeSSanjay Patel // The proposed vector pattern is: 722b6315aeeSSanjay Patel // vcmp = cmp Pred X, VecC 723b6315aeeSSanjay Patel // ext (binop vNi1 vcmp, (shuffle vcmp, Index1)), Index0 724b6315aeeSSanjay Patel int CheapIndex = ConvertToShuf == Ext0 ? Index1 : Index0; 725b6315aeeSSanjay Patel int ExpensiveIndex = ConvertToShuf == Ext0 ? Index0 : Index1; 726b6315aeeSSanjay Patel auto *CmpTy = cast<FixedVectorType>(CmpInst::makeCmpResultType(X->getType())); 72736710c38SCaroline Concatto InstructionCost NewCost = TTI.getCmpSelInstrCost(CmpOpcode, X->getType()); 728*e2935dcfSDavid Green SmallVector<int, 32> ShufMask(VecTy->getNumElements(), UndefMaskElem); 729*e2935dcfSDavid Green ShufMask[CheapIndex] = ExpensiveIndex; 730*e2935dcfSDavid Green NewCost += TTI.getShuffleCost(TargetTransformInfo::SK_PermuteSingleSrc, CmpTy, 731*e2935dcfSDavid Green ShufMask); 732b6315aeeSSanjay Patel NewCost += TTI.getArithmeticInstrCost(I.getOpcode(), CmpTy); 733b6315aeeSSanjay Patel NewCost += TTI.getVectorInstrCost(Ext0->getOpcode(), CmpTy, CheapIndex); 734b6315aeeSSanjay Patel 735b6315aeeSSanjay Patel // Aggressively form vector ops if the cost is equal because the transform 736b6315aeeSSanjay Patel // may enable further optimization. 737b6315aeeSSanjay Patel // Codegen can reverse this transform (scalarize) if it was not profitable. 73836710c38SCaroline Concatto if (OldCost < NewCost || !NewCost.isValid()) 739b6315aeeSSanjay Patel return false; 740b6315aeeSSanjay Patel 741b6315aeeSSanjay Patel // Create a vector constant from the 2 scalar constants. 742b6315aeeSSanjay Patel SmallVector<Constant *, 32> CmpC(VecTy->getNumElements(), 743b6315aeeSSanjay Patel UndefValue::get(VecTy->getElementType())); 744b6315aeeSSanjay Patel CmpC[Index0] = C0; 745b6315aeeSSanjay Patel CmpC[Index1] = C1; 746b6315aeeSSanjay Patel Value *VCmp = Builder.CreateCmp(Pred, X, ConstantVector::get(CmpC)); 747b6315aeeSSanjay Patel 748b6315aeeSSanjay Patel Value *Shuf = createShiftShuffle(VCmp, ExpensiveIndex, CheapIndex, Builder); 749b6315aeeSSanjay Patel Value *VecLogic = Builder.CreateBinOp(cast<BinaryOperator>(I).getOpcode(), 750b6315aeeSSanjay Patel VCmp, Shuf); 751b6315aeeSSanjay Patel Value *NewExt = Builder.CreateExtractElement(VecLogic, CheapIndex); 752b6315aeeSSanjay Patel replaceValue(I, *NewExt); 753b6315aeeSSanjay Patel ++NumVecCmpBO; 754b6315aeeSSanjay Patel return true; 755b6315aeeSSanjay Patel } 756b6315aeeSSanjay Patel 757a17f03bdSSanjay Patel /// This is the entry point for all transforms. Pass manager differences are 758a17f03bdSSanjay Patel /// handled in the callers of this function. 7596bdd531aSSanjay Patel bool VectorCombine::run() { 76025c6544fSSanjay Patel if (DisableVectorCombine) 76125c6544fSSanjay Patel return false; 76225c6544fSSanjay Patel 763cc892fd9SSanjay Patel // Don't attempt vectorization if the target does not support vectors. 764cc892fd9SSanjay Patel if (!TTI.getNumberOfRegisters(TTI.getRegisterClassForType(/*Vector*/ true))) 765cc892fd9SSanjay Patel return false; 766cc892fd9SSanjay Patel 767a17f03bdSSanjay Patel bool MadeChange = false; 768a17f03bdSSanjay Patel for (BasicBlock &BB : F) { 769a17f03bdSSanjay Patel // Ignore unreachable basic blocks. 770a17f03bdSSanjay Patel if (!DT.isReachableFromEntry(&BB)) 771a17f03bdSSanjay Patel continue; 772a17f03bdSSanjay Patel // Do not delete instructions under here and invalidate the iterator. 77381e9ede3SSanjay Patel // Walk the block forwards to enable simple iterative chains of transforms. 774a17f03bdSSanjay Patel // TODO: It could be more efficient to remove dead instructions 775a17f03bdSSanjay Patel // iteratively in this loop rather than waiting until the end. 77681e9ede3SSanjay Patel for (Instruction &I : BB) { 777fc3cc8a4SSanjay Patel if (isa<DbgInfoIntrinsic>(I)) 778fc3cc8a4SSanjay Patel continue; 779de65b356SSanjay Patel Builder.SetInsertPoint(&I); 78043bdac29SSanjay Patel MadeChange |= vectorizeLoadInsert(I); 7816bdd531aSSanjay Patel MadeChange |= foldExtractExtract(I); 7826bdd531aSSanjay Patel MadeChange |= foldBitcastShuf(I); 7836bdd531aSSanjay Patel MadeChange |= scalarizeBinopOrCmp(I); 784b6315aeeSSanjay Patel MadeChange |= foldExtractedCmps(I); 785a17f03bdSSanjay Patel } 786fc3cc8a4SSanjay Patel } 787a17f03bdSSanjay Patel 788a17f03bdSSanjay Patel // We're done with transforms, so remove dead instructions. 789a17f03bdSSanjay Patel if (MadeChange) 790a17f03bdSSanjay Patel for (BasicBlock &BB : F) 791a17f03bdSSanjay Patel SimplifyInstructionsInBlock(&BB); 792a17f03bdSSanjay Patel 793a17f03bdSSanjay Patel return MadeChange; 794a17f03bdSSanjay Patel } 795a17f03bdSSanjay Patel 796a17f03bdSSanjay Patel // Pass manager boilerplate below here. 797a17f03bdSSanjay Patel 798a17f03bdSSanjay Patel namespace { 799a17f03bdSSanjay Patel class VectorCombineLegacyPass : public FunctionPass { 800a17f03bdSSanjay Patel public: 801a17f03bdSSanjay Patel static char ID; 802a17f03bdSSanjay Patel VectorCombineLegacyPass() : FunctionPass(ID) { 803a17f03bdSSanjay Patel initializeVectorCombineLegacyPassPass(*PassRegistry::getPassRegistry()); 804a17f03bdSSanjay Patel } 805a17f03bdSSanjay Patel 806a17f03bdSSanjay Patel void getAnalysisUsage(AnalysisUsage &AU) const override { 807a17f03bdSSanjay Patel AU.addRequired<DominatorTreeWrapperPass>(); 808a17f03bdSSanjay Patel AU.addRequired<TargetTransformInfoWrapperPass>(); 809a17f03bdSSanjay Patel AU.setPreservesCFG(); 810a17f03bdSSanjay Patel AU.addPreserved<DominatorTreeWrapperPass>(); 811a17f03bdSSanjay Patel AU.addPreserved<GlobalsAAWrapperPass>(); 812024098aeSSanjay Patel AU.addPreserved<AAResultsWrapperPass>(); 813024098aeSSanjay Patel AU.addPreserved<BasicAAWrapperPass>(); 814a17f03bdSSanjay Patel FunctionPass::getAnalysisUsage(AU); 815a17f03bdSSanjay Patel } 816a17f03bdSSanjay Patel 817a17f03bdSSanjay Patel bool runOnFunction(Function &F) override { 818a17f03bdSSanjay Patel if (skipFunction(F)) 819a17f03bdSSanjay Patel return false; 820a17f03bdSSanjay Patel auto &TTI = getAnalysis<TargetTransformInfoWrapperPass>().getTTI(F); 821a17f03bdSSanjay Patel auto &DT = getAnalysis<DominatorTreeWrapperPass>().getDomTree(); 8226bdd531aSSanjay Patel VectorCombine Combiner(F, TTI, DT); 8236bdd531aSSanjay Patel return Combiner.run(); 824a17f03bdSSanjay Patel } 825a17f03bdSSanjay Patel }; 826a17f03bdSSanjay Patel } // namespace 827a17f03bdSSanjay Patel 828a17f03bdSSanjay Patel char VectorCombineLegacyPass::ID = 0; 829a17f03bdSSanjay Patel INITIALIZE_PASS_BEGIN(VectorCombineLegacyPass, "vector-combine", 830a17f03bdSSanjay Patel "Optimize scalar/vector ops", false, 831a17f03bdSSanjay Patel false) 832a17f03bdSSanjay Patel INITIALIZE_PASS_DEPENDENCY(DominatorTreeWrapperPass) 833a17f03bdSSanjay Patel INITIALIZE_PASS_END(VectorCombineLegacyPass, "vector-combine", 834a17f03bdSSanjay Patel "Optimize scalar/vector ops", false, false) 835a17f03bdSSanjay Patel Pass *llvm::createVectorCombinePass() { 836a17f03bdSSanjay Patel return new VectorCombineLegacyPass(); 837a17f03bdSSanjay Patel } 838a17f03bdSSanjay Patel 839a17f03bdSSanjay Patel PreservedAnalyses VectorCombinePass::run(Function &F, 840a17f03bdSSanjay Patel FunctionAnalysisManager &FAM) { 841a17f03bdSSanjay Patel TargetTransformInfo &TTI = FAM.getResult<TargetIRAnalysis>(F); 842a17f03bdSSanjay Patel DominatorTree &DT = FAM.getResult<DominatorTreeAnalysis>(F); 8436bdd531aSSanjay Patel VectorCombine Combiner(F, TTI, DT); 8446bdd531aSSanjay Patel if (!Combiner.run()) 845a17f03bdSSanjay Patel return PreservedAnalyses::all(); 846a17f03bdSSanjay Patel PreservedAnalyses PA; 847a17f03bdSSanjay Patel PA.preserveSet<CFGAnalyses>(); 848a17f03bdSSanjay Patel PA.preserve<GlobalsAA>(); 849024098aeSSanjay Patel PA.preserve<AAManager>(); 850024098aeSSanjay Patel PA.preserve<BasicAA>(); 851a17f03bdSSanjay Patel return PA; 852a17f03bdSSanjay Patel } 853