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" 19a17f03bdSSanjay Patel #include "llvm/Analysis/TargetTransformInfo.h" 2019b62b79SSanjay Patel #include "llvm/Analysis/ValueTracking.h" 21b6050ca1SSanjay Patel #include "llvm/Analysis/VectorUtils.h" 22a17f03bdSSanjay Patel #include "llvm/IR/Dominators.h" 23a17f03bdSSanjay Patel #include "llvm/IR/Function.h" 24a17f03bdSSanjay Patel #include "llvm/IR/IRBuilder.h" 25a17f03bdSSanjay Patel #include "llvm/IR/PatternMatch.h" 26a17f03bdSSanjay Patel #include "llvm/InitializePasses.h" 27a17f03bdSSanjay Patel #include "llvm/Pass.h" 2825c6544fSSanjay Patel #include "llvm/Support/CommandLine.h" 29a17f03bdSSanjay Patel #include "llvm/Transforms/Utils/Local.h" 305006e551SSimon Pilgrim #include "llvm/Transforms/Vectorize.h" 31a17f03bdSSanjay Patel 32a17f03bdSSanjay Patel using namespace llvm; 33a17f03bdSSanjay Patel using namespace llvm::PatternMatch; 34a17f03bdSSanjay Patel 35a17f03bdSSanjay Patel #define DEBUG_TYPE "vector-combine" 36a17f03bdSSanjay Patel STATISTIC(NumVecCmp, "Number of vector compares formed"); 3719b62b79SSanjay Patel STATISTIC(NumVecBO, "Number of vector binops formed"); 380d2a0b44SSanjay Patel STATISTIC(NumScalarBO, "Number of scalar binops formed"); 39a17f03bdSSanjay Patel 4025c6544fSSanjay Patel static cl::opt<bool> DisableVectorCombine( 4125c6544fSSanjay Patel "disable-vector-combine", cl::init(false), cl::Hidden, 4225c6544fSSanjay Patel cl::desc("Disable all vector combine transforms")); 4325c6544fSSanjay Patel 44a69158c1SSanjay Patel static cl::opt<bool> DisableBinopExtractShuffle( 45a69158c1SSanjay Patel "disable-binop-extract-shuffle", cl::init(false), cl::Hidden, 46a69158c1SSanjay Patel cl::desc("Disable binop extract to shuffle transforms")); 47a69158c1SSanjay Patel 48a69158c1SSanjay Patel 49a69158c1SSanjay Patel /// Compare the relative costs of 2 extracts followed by scalar operation vs. 50a69158c1SSanjay Patel /// vector operation(s) followed by extract. Return true if the existing 51a69158c1SSanjay Patel /// instructions are cheaper than a vector alternative. Otherwise, return false 52a69158c1SSanjay Patel /// and if one of the extracts should be transformed to a shufflevector, set 53a69158c1SSanjay Patel /// \p ConvertToShuffle to that extract instruction. 5434e34855SSanjay Patel static bool isExtractExtractCheap(Instruction *Ext0, Instruction *Ext1, 5534e34855SSanjay Patel unsigned Opcode, 56a69158c1SSanjay Patel const TargetTransformInfo &TTI, 57ce97ce3aSSanjay Patel Instruction *&ConvertToShuffle, 58ce97ce3aSSanjay Patel unsigned PreferredExtractIndex) { 594fa63fd4SAustin Kerbow assert(isa<ConstantInt>(Ext0->getOperand(1)) && 60a69158c1SSanjay Patel isa<ConstantInt>(Ext1->getOperand(1)) && 61a69158c1SSanjay Patel "Expected constant extract indexes"); 6234e34855SSanjay Patel Type *ScalarTy = Ext0->getType(); 63e3056ae9SSam Parker auto *VecTy = cast<VectorType>(Ext0->getOperand(0)->getType()); 6434e34855SSanjay Patel int ScalarOpCost, VectorOpCost; 6534e34855SSanjay Patel 6634e34855SSanjay Patel // Get cost estimates for scalar and vector versions of the operation. 6734e34855SSanjay Patel bool IsBinOp = Instruction::isBinaryOp(Opcode); 6834e34855SSanjay Patel if (IsBinOp) { 6934e34855SSanjay Patel ScalarOpCost = TTI.getArithmeticInstrCost(Opcode, ScalarTy); 7034e34855SSanjay Patel VectorOpCost = TTI.getArithmeticInstrCost(Opcode, VecTy); 7134e34855SSanjay Patel } else { 7234e34855SSanjay Patel assert((Opcode == Instruction::ICmp || Opcode == Instruction::FCmp) && 7334e34855SSanjay Patel "Expected a compare"); 7434e34855SSanjay Patel ScalarOpCost = TTI.getCmpSelInstrCost(Opcode, ScalarTy, 7534e34855SSanjay Patel CmpInst::makeCmpResultType(ScalarTy)); 7634e34855SSanjay Patel VectorOpCost = TTI.getCmpSelInstrCost(Opcode, VecTy, 7734e34855SSanjay Patel CmpInst::makeCmpResultType(VecTy)); 7834e34855SSanjay Patel } 7934e34855SSanjay Patel 80a69158c1SSanjay Patel // Get cost estimates for the extract elements. These costs will factor into 8134e34855SSanjay Patel // both sequences. 82a69158c1SSanjay Patel unsigned Ext0Index = cast<ConstantInt>(Ext0->getOperand(1))->getZExtValue(); 83a69158c1SSanjay Patel unsigned Ext1Index = cast<ConstantInt>(Ext1->getOperand(1))->getZExtValue(); 84a69158c1SSanjay Patel 85a69158c1SSanjay Patel int Extract0Cost = TTI.getVectorInstrCost(Instruction::ExtractElement, 86a69158c1SSanjay Patel VecTy, Ext0Index); 87a69158c1SSanjay Patel int Extract1Cost = TTI.getVectorInstrCost(Instruction::ExtractElement, 88a69158c1SSanjay Patel VecTy, Ext1Index); 89a69158c1SSanjay Patel 90a69158c1SSanjay Patel // A more expensive extract will always be replaced by a splat shuffle. 91a69158c1SSanjay Patel // For example, if Ext0 is more expensive: 92a69158c1SSanjay Patel // opcode (extelt V0, Ext0), (ext V1, Ext1) --> 93a69158c1SSanjay Patel // extelt (opcode (splat V0, Ext0), V1), Ext1 94a69158c1SSanjay Patel // TODO: Evaluate whether that always results in lowest cost. Alternatively, 95a69158c1SSanjay Patel // check the cost of creating a broadcast shuffle and shuffling both 96a69158c1SSanjay Patel // operands to element 0. 97a69158c1SSanjay Patel int CheapExtractCost = std::min(Extract0Cost, Extract1Cost); 9834e34855SSanjay Patel 9934e34855SSanjay Patel // Extra uses of the extracts mean that we include those costs in the 10034e34855SSanjay Patel // vector total because those instructions will not be eliminated. 101e9c79a7aSSanjay Patel int OldCost, NewCost; 102a69158c1SSanjay Patel if (Ext0->getOperand(0) == Ext1->getOperand(0) && Ext0Index == Ext1Index) { 103a69158c1SSanjay Patel // Handle a special case. If the 2 extracts are identical, adjust the 10434e34855SSanjay Patel // formulas to account for that. The extra use charge allows for either the 10534e34855SSanjay Patel // CSE'd pattern or an unoptimized form with identical values: 10634e34855SSanjay Patel // opcode (extelt V, C), (extelt V, C) --> extelt (opcode V, V), C 10734e34855SSanjay Patel bool HasUseTax = Ext0 == Ext1 ? !Ext0->hasNUses(2) 10834e34855SSanjay Patel : !Ext0->hasOneUse() || !Ext1->hasOneUse(); 109a69158c1SSanjay Patel OldCost = CheapExtractCost + ScalarOpCost; 110a69158c1SSanjay Patel NewCost = VectorOpCost + CheapExtractCost + HasUseTax * CheapExtractCost; 11134e34855SSanjay Patel } else { 11234e34855SSanjay Patel // Handle the general case. Each extract is actually a different value: 113a69158c1SSanjay Patel // opcode (extelt V0, C0), (extelt V1, C1) --> extelt (opcode V0, V1), C 114a69158c1SSanjay Patel OldCost = Extract0Cost + Extract1Cost + ScalarOpCost; 115a69158c1SSanjay Patel NewCost = VectorOpCost + CheapExtractCost + 116a69158c1SSanjay Patel !Ext0->hasOneUse() * Extract0Cost + 117a69158c1SSanjay Patel !Ext1->hasOneUse() * Extract1Cost; 11834e34855SSanjay Patel } 119a69158c1SSanjay Patel 120a69158c1SSanjay Patel if (Ext0Index == Ext1Index) { 121a69158c1SSanjay Patel // If the extract indexes are identical, no shuffle is needed. 122a69158c1SSanjay Patel ConvertToShuffle = nullptr; 123a69158c1SSanjay Patel } else { 124a69158c1SSanjay Patel if (IsBinOp && DisableBinopExtractShuffle) 125a69158c1SSanjay Patel return true; 126a69158c1SSanjay Patel 127a69158c1SSanjay Patel // If we are extracting from 2 different indexes, then one operand must be 128a69158c1SSanjay Patel // shuffled before performing the vector operation. The shuffle mask is 129a69158c1SSanjay Patel // undefined except for 1 lane that is being translated to the remaining 130a69158c1SSanjay Patel // extraction lane. Therefore, it is a splat shuffle. Ex: 131a69158c1SSanjay Patel // ShufMask = { undef, undef, 0, undef } 132a69158c1SSanjay Patel // TODO: The cost model has an option for a "broadcast" shuffle 133a69158c1SSanjay Patel // (splat-from-element-0), but no option for a more general splat. 134a69158c1SSanjay Patel NewCost += 135a69158c1SSanjay Patel TTI.getShuffleCost(TargetTransformInfo::SK_PermuteSingleSrc, VecTy); 136a69158c1SSanjay Patel 137ce97ce3aSSanjay Patel // The more expensive extract will be replaced by a shuffle. If the costs 138ce97ce3aSSanjay Patel // are equal and there is a preferred extract index, shuffle the opposite 139ce97ce3aSSanjay Patel // operand. Otherwise, replace the extract with the higher index. 140a69158c1SSanjay Patel if (Extract0Cost > Extract1Cost) 141a69158c1SSanjay Patel ConvertToShuffle = Ext0; 142a69158c1SSanjay Patel else if (Extract1Cost > Extract0Cost) 143a69158c1SSanjay Patel ConvertToShuffle = Ext1; 144ce97ce3aSSanjay Patel else if (PreferredExtractIndex == Ext0Index) 145ce97ce3aSSanjay Patel ConvertToShuffle = Ext1; 146ce97ce3aSSanjay Patel else if (PreferredExtractIndex == Ext1Index) 147ce97ce3aSSanjay Patel ConvertToShuffle = Ext0; 148a69158c1SSanjay Patel else 149a69158c1SSanjay Patel ConvertToShuffle = Ext0Index > Ext1Index ? Ext0 : Ext1; 150a69158c1SSanjay Patel } 151a69158c1SSanjay Patel 15210ea01d8SSanjay Patel // Aggressively form a vector op if the cost is equal because the transform 15310ea01d8SSanjay Patel // may enable further optimization. 15410ea01d8SSanjay Patel // Codegen can reverse this transform (scalarize) if it was not profitable. 15510ea01d8SSanjay Patel return OldCost < NewCost; 15634e34855SSanjay Patel } 15734e34855SSanjay Patel 158fc445589SSanjay Patel /// Try to reduce extract element costs by converting scalar compares to vector 159fc445589SSanjay Patel /// compares followed by extract. 160e9c79a7aSSanjay Patel /// cmp (ext0 V0, C), (ext1 V1, C) 161e9c79a7aSSanjay Patel static void foldExtExtCmp(Instruction *Ext0, Instruction *Ext1, 162fc445589SSanjay Patel Instruction &I, const TargetTransformInfo &TTI) { 163fc445589SSanjay Patel assert(isa<CmpInst>(&I) && "Expected a compare"); 164a17f03bdSSanjay Patel 165a17f03bdSSanjay Patel // cmp Pred (extelt V0, C), (extelt V1, C) --> extelt (cmp Pred V0, V1), C 166a17f03bdSSanjay Patel ++NumVecCmp; 167a17f03bdSSanjay Patel IRBuilder<> Builder(&I); 168fc445589SSanjay Patel CmpInst::Predicate Pred = cast<CmpInst>(&I)->getPredicate(); 169e9c79a7aSSanjay Patel Value *V0 = Ext0->getOperand(0), *V1 = Ext1->getOperand(0); 17034e34855SSanjay Patel Value *VecCmp = 17134e34855SSanjay Patel Ext0->getType()->isFloatingPointTy() ? Builder.CreateFCmp(Pred, V0, V1) 172a17f03bdSSanjay Patel : Builder.CreateICmp(Pred, V0, V1); 173fc445589SSanjay Patel Value *Extract = Builder.CreateExtractElement(VecCmp, Ext0->getOperand(1)); 174fc445589SSanjay Patel I.replaceAllUsesWith(Extract); 175a17f03bdSSanjay Patel } 176a17f03bdSSanjay Patel 17719b62b79SSanjay Patel /// Try to reduce extract element costs by converting scalar binops to vector 17819b62b79SSanjay Patel /// binops followed by extract. 179e9c79a7aSSanjay Patel /// bo (ext0 V0, C), (ext1 V1, C) 180e9c79a7aSSanjay Patel static void foldExtExtBinop(Instruction *Ext0, Instruction *Ext1, 181fc445589SSanjay Patel Instruction &I, const TargetTransformInfo &TTI) { 182fc445589SSanjay Patel assert(isa<BinaryOperator>(&I) && "Expected a binary operator"); 18319b62b79SSanjay Patel 18434e34855SSanjay Patel // bo (extelt V0, C), (extelt V1, C) --> extelt (bo V0, V1), C 18519b62b79SSanjay Patel ++NumVecBO; 18619b62b79SSanjay Patel IRBuilder<> Builder(&I); 187e9c79a7aSSanjay Patel Value *V0 = Ext0->getOperand(0), *V1 = Ext1->getOperand(0); 188e9c79a7aSSanjay Patel Value *VecBO = 18934e34855SSanjay Patel Builder.CreateBinOp(cast<BinaryOperator>(&I)->getOpcode(), V0, V1); 190e9c79a7aSSanjay Patel 19119b62b79SSanjay Patel // All IR flags are safe to back-propagate because any potential poison 19219b62b79SSanjay Patel // created in unused vector elements is discarded by the extract. 193e9c79a7aSSanjay Patel if (auto *VecBOInst = dyn_cast<Instruction>(VecBO)) 19419b62b79SSanjay Patel VecBOInst->copyIRFlags(&I); 195e9c79a7aSSanjay Patel 196e9c79a7aSSanjay Patel Value *Extract = Builder.CreateExtractElement(VecBO, Ext0->getOperand(1)); 19719b62b79SSanjay Patel I.replaceAllUsesWith(Extract); 19819b62b79SSanjay Patel } 19919b62b79SSanjay Patel 200fc445589SSanjay Patel /// Match an instruction with extracted vector operands. 201fc445589SSanjay Patel static bool foldExtractExtract(Instruction &I, const TargetTransformInfo &TTI) { 202e9c79a7aSSanjay Patel // It is not safe to transform things like div, urem, etc. because we may 203e9c79a7aSSanjay Patel // create undefined behavior when executing those on unknown vector elements. 204e9c79a7aSSanjay Patel if (!isSafeToSpeculativelyExecute(&I)) 205e9c79a7aSSanjay Patel return false; 206e9c79a7aSSanjay Patel 207fc445589SSanjay Patel Instruction *Ext0, *Ext1; 208fc445589SSanjay Patel CmpInst::Predicate Pred = CmpInst::BAD_ICMP_PREDICATE; 209fc445589SSanjay Patel if (!match(&I, m_Cmp(Pred, m_Instruction(Ext0), m_Instruction(Ext1))) && 210fc445589SSanjay Patel !match(&I, m_BinOp(m_Instruction(Ext0), m_Instruction(Ext1)))) 211fc445589SSanjay Patel return false; 212fc445589SSanjay Patel 213fc445589SSanjay Patel Value *V0, *V1; 214fc445589SSanjay Patel uint64_t C0, C1; 2157eed772aSSanjay Patel if (!match(Ext0, m_ExtractElt(m_Value(V0), m_ConstantInt(C0))) || 2167eed772aSSanjay Patel !match(Ext1, m_ExtractElt(m_Value(V1), m_ConstantInt(C1))) || 217fc445589SSanjay Patel V0->getType() != V1->getType()) 218fc445589SSanjay Patel return false; 219fc445589SSanjay Patel 220ce97ce3aSSanjay Patel // If the scalar value 'I' is going to be re-inserted into a vector, then try 221ce97ce3aSSanjay Patel // to create an extract to that same element. The extract/insert can be 222ce97ce3aSSanjay Patel // reduced to a "select shuffle". 223ce97ce3aSSanjay Patel // TODO: If we add a larger pattern match that starts from an insert, this 224ce97ce3aSSanjay Patel // probably becomes unnecessary. 225ce97ce3aSSanjay Patel uint64_t InsertIndex = std::numeric_limits<uint64_t>::max(); 226ce97ce3aSSanjay Patel if (I.hasOneUse()) 2277eed772aSSanjay Patel match(I.user_back(), 2287eed772aSSanjay Patel m_InsertElt(m_Value(), m_Value(), m_ConstantInt(InsertIndex))); 229ce97ce3aSSanjay Patel 230a69158c1SSanjay Patel Instruction *ConvertToShuffle; 231ce97ce3aSSanjay Patel if (isExtractExtractCheap(Ext0, Ext1, I.getOpcode(), TTI, ConvertToShuffle, 232ce97ce3aSSanjay Patel InsertIndex)) 233fc445589SSanjay Patel return false; 234e9c79a7aSSanjay Patel 235a69158c1SSanjay Patel if (ConvertToShuffle) { 236a69158c1SSanjay Patel // The shuffle mask is undefined except for 1 lane that is being translated 237a69158c1SSanjay Patel // to the cheap extraction lane. Example: 238a69158c1SSanjay Patel // ShufMask = { 2, undef, undef, undef } 239a69158c1SSanjay Patel uint64_t SplatIndex = ConvertToShuffle == Ext0 ? C0 : C1; 240a69158c1SSanjay Patel uint64_t CheapExtIndex = ConvertToShuffle == Ext0 ? C1 : C0; 2413297e9b7SChristopher Tetreault auto *VecTy = cast<VectorType>(V0->getType()); 2426f64dacaSBenjamin Kramer SmallVector<int, 32> ShufMask(VecTy->getNumElements(), -1); 2436f64dacaSBenjamin Kramer ShufMask[CheapExtIndex] = SplatIndex; 244a69158c1SSanjay Patel IRBuilder<> Builder(ConvertToShuffle); 245a69158c1SSanjay Patel 246a69158c1SSanjay Patel // extelt X, C --> extelt (splat X), C' 247a69158c1SSanjay Patel Value *Shuf = Builder.CreateShuffleVector(ConvertToShuffle->getOperand(0), 2486f64dacaSBenjamin Kramer UndefValue::get(VecTy), ShufMask); 249a69158c1SSanjay Patel Value *NewExt = Builder.CreateExtractElement(Shuf, CheapExtIndex); 250a69158c1SSanjay Patel if (ConvertToShuffle == Ext0) 251a69158c1SSanjay Patel Ext0 = cast<Instruction>(NewExt); 252a69158c1SSanjay Patel else 253a69158c1SSanjay Patel Ext1 = cast<Instruction>(NewExt); 254a69158c1SSanjay Patel } 255e9c79a7aSSanjay Patel 256e9c79a7aSSanjay Patel if (Pred != CmpInst::BAD_ICMP_PREDICATE) 257e9c79a7aSSanjay Patel foldExtExtCmp(Ext0, Ext1, I, TTI); 258e9c79a7aSSanjay Patel else 259e9c79a7aSSanjay Patel foldExtExtBinop(Ext0, Ext1, I, TTI); 260e9c79a7aSSanjay Patel 261e9c79a7aSSanjay Patel return true; 262fc445589SSanjay Patel } 263fc445589SSanjay Patel 264bef6e67eSSanjay Patel /// If this is a bitcast of a shuffle, try to bitcast the source vector to the 265bef6e67eSSanjay Patel /// destination type followed by shuffle. This can enable further transforms by 266bef6e67eSSanjay Patel /// moving bitcasts or shuffles together. 267b6050ca1SSanjay Patel static bool foldBitcastShuf(Instruction &I, const TargetTransformInfo &TTI) { 268b6050ca1SSanjay Patel Value *V; 269b6050ca1SSanjay Patel ArrayRef<int> Mask; 2707eed772aSSanjay Patel if (!match(&I, m_BitCast( 2717eed772aSSanjay Patel m_OneUse(m_Shuffle(m_Value(V), m_Undef(), m_Mask(Mask)))))) 272b6050ca1SSanjay Patel return false; 273b6050ca1SSanjay Patel 274bef6e67eSSanjay Patel // Disallow non-vector casts and length-changing shuffles. 275bef6e67eSSanjay Patel // TODO: We could allow any shuffle. 2763297e9b7SChristopher Tetreault auto *DestTy = dyn_cast<VectorType>(I.getType()); 2773297e9b7SChristopher Tetreault auto *SrcTy = cast<VectorType>(V->getType()); 2783297e9b7SChristopher Tetreault if (!DestTy || I.getOperand(0)->getType() != SrcTy) 279b6050ca1SSanjay Patel return false; 280b6050ca1SSanjay Patel 281b6050ca1SSanjay Patel // The new shuffle must not cost more than the old shuffle. The bitcast is 282b6050ca1SSanjay Patel // moved ahead of the shuffle, so assume that it has the same cost as before. 283b6050ca1SSanjay Patel if (TTI.getShuffleCost(TargetTransformInfo::SK_PermuteSingleSrc, DestTy) > 284b6050ca1SSanjay Patel TTI.getShuffleCost(TargetTransformInfo::SK_PermuteSingleSrc, SrcTy)) 285b6050ca1SSanjay Patel return false; 286b6050ca1SSanjay Patel 287bef6e67eSSanjay Patel unsigned DestNumElts = DestTy->getNumElements(); 288bef6e67eSSanjay Patel unsigned SrcNumElts = SrcTy->getNumElements(); 289b6050ca1SSanjay Patel SmallVector<int, 16> NewMask; 290bef6e67eSSanjay Patel if (SrcNumElts <= DestNumElts) { 291bef6e67eSSanjay Patel // The bitcast is from wide to narrow/equal elements. The shuffle mask can 292bef6e67eSSanjay Patel // always be expanded to the equivalent form choosing narrower elements. 293b6050ca1SSanjay Patel assert(DestNumElts % SrcNumElts == 0 && "Unexpected shuffle mask"); 294b6050ca1SSanjay Patel unsigned ScaleFactor = DestNumElts / SrcNumElts; 2951318ddbcSSanjay Patel narrowShuffleMaskElts(ScaleFactor, Mask, NewMask); 296bef6e67eSSanjay Patel } else { 297bef6e67eSSanjay Patel // The bitcast is from narrow elements to wide elements. The shuffle mask 298bef6e67eSSanjay Patel // must choose consecutive elements to allow casting first. 299bef6e67eSSanjay Patel assert(SrcNumElts % DestNumElts == 0 && "Unexpected shuffle mask"); 300bef6e67eSSanjay Patel unsigned ScaleFactor = SrcNumElts / DestNumElts; 301bef6e67eSSanjay Patel if (!widenShuffleMaskElts(ScaleFactor, Mask, NewMask)) 302bef6e67eSSanjay Patel return false; 303bef6e67eSSanjay Patel } 304bef6e67eSSanjay Patel // bitcast (shuf V, MaskC) --> shuf (bitcast V), MaskC' 305bef6e67eSSanjay Patel IRBuilder<> Builder(&I); 306bef6e67eSSanjay Patel Value *CastV = Builder.CreateBitCast(V, DestTy); 3077eed772aSSanjay Patel Value *Shuf = 3087eed772aSSanjay Patel Builder.CreateShuffleVector(CastV, UndefValue::get(DestTy), NewMask); 309b6050ca1SSanjay Patel I.replaceAllUsesWith(Shuf); 310b6050ca1SSanjay Patel return true; 311b6050ca1SSanjay Patel } 312b6050ca1SSanjay Patel 3130d2a0b44SSanjay Patel /// Match a vector binop instruction with inserted scalar operands and convert 3140d2a0b44SSanjay Patel /// to scalar binop followed by insertelement. 3150d2a0b44SSanjay Patel static bool scalarizeBinop(Instruction &I, const TargetTransformInfo &TTI) { 316*5dc4e7c2SSimon Pilgrim Value *Ins0, *Ins1; 317*5dc4e7c2SSimon Pilgrim if (!match(&I, m_BinOp(m_Value(Ins0), m_Value(Ins1)))) 3180d2a0b44SSanjay Patel return false; 3190d2a0b44SSanjay Patel 320*5dc4e7c2SSimon Pilgrim // Match against one or both scalar values being inserted into constant 321*5dc4e7c2SSimon Pilgrim // vectors: 322*5dc4e7c2SSimon Pilgrim // vec_bo VecC0, (inselt VecC1, V1, Index) 323*5dc4e7c2SSimon Pilgrim // vec_bo (inselt VecC0, V0, Index), VecC1 324*5dc4e7c2SSimon Pilgrim // vec_bo (inselt VecC0, V0, Index), (inselt VecC1, V1, Index) 3250d2a0b44SSanjay Patel // TODO: Deal with mismatched index constants and variable indexes? 326*5dc4e7c2SSimon Pilgrim Constant *VecC0 = nullptr, *VecC1 = nullptr; 327*5dc4e7c2SSimon Pilgrim Value *V0 = nullptr, *V1 = nullptr; 328*5dc4e7c2SSimon Pilgrim uint64_t Index0 = 0, Index1 = 0; 3297eed772aSSanjay Patel if (!match(Ins0, m_InsertElt(m_Constant(VecC0), m_Value(V0), 330*5dc4e7c2SSimon Pilgrim m_ConstantInt(Index0))) && 331*5dc4e7c2SSimon Pilgrim !match(Ins0, m_Constant(VecC0))) 332*5dc4e7c2SSimon Pilgrim return false; 333*5dc4e7c2SSimon Pilgrim if (!match(Ins1, m_InsertElt(m_Constant(VecC1), m_Value(V1), 334*5dc4e7c2SSimon Pilgrim m_ConstantInt(Index1))) && 335*5dc4e7c2SSimon Pilgrim !match(Ins1, m_Constant(VecC1))) 3360d2a0b44SSanjay Patel return false; 3370d2a0b44SSanjay Patel 338*5dc4e7c2SSimon Pilgrim bool IsConst0 = !V0; 339*5dc4e7c2SSimon Pilgrim bool IsConst1 = !V1; 340*5dc4e7c2SSimon Pilgrim if (IsConst0 && IsConst1) 341*5dc4e7c2SSimon Pilgrim return false; 342*5dc4e7c2SSimon Pilgrim if (!IsConst0 && !IsConst1 && Index0 != Index1) 343*5dc4e7c2SSimon Pilgrim return false; 344*5dc4e7c2SSimon Pilgrim 345*5dc4e7c2SSimon Pilgrim // Bail for single insertion if it is a load. 346*5dc4e7c2SSimon Pilgrim // TODO: Handle this once getVectorInstrCost can cost for load/stores. 347*5dc4e7c2SSimon Pilgrim auto *I0 = dyn_cast_or_null<Instruction>(V0); 348*5dc4e7c2SSimon Pilgrim auto *I1 = dyn_cast_or_null<Instruction>(V1); 349*5dc4e7c2SSimon Pilgrim if ((IsConst0 && I1 && I1->mayReadFromMemory()) || 350*5dc4e7c2SSimon Pilgrim (IsConst1 && I0 && I0->mayReadFromMemory())) 351*5dc4e7c2SSimon Pilgrim return false; 352*5dc4e7c2SSimon Pilgrim 353*5dc4e7c2SSimon Pilgrim uint64_t Index = IsConst0 ? Index1 : Index0; 354*5dc4e7c2SSimon Pilgrim Type *ScalarTy = IsConst0 ? V1->getType() : V0->getType(); 3550d2a0b44SSanjay Patel Type *VecTy = I.getType(); 356*5dc4e7c2SSimon Pilgrim assert(VecTy->isVectorTy() && 357*5dc4e7c2SSimon Pilgrim (IsConst0 || IsConst1 || V0->getType() == V1->getType()) && 3580d2a0b44SSanjay Patel (ScalarTy->isIntegerTy() || ScalarTy->isFloatingPointTy()) && 3590d2a0b44SSanjay Patel "Unexpected types for insert into binop"); 3600d2a0b44SSanjay Patel 3610d2a0b44SSanjay Patel Instruction::BinaryOps Opcode = cast<BinaryOperator>(&I)->getOpcode(); 3620d2a0b44SSanjay Patel int ScalarOpCost = TTI.getArithmeticInstrCost(Opcode, ScalarTy); 3630d2a0b44SSanjay Patel int VectorOpCost = TTI.getArithmeticInstrCost(Opcode, VecTy); 3640d2a0b44SSanjay Patel 3650d2a0b44SSanjay Patel // Get cost estimate for the insert element. This cost will factor into 3660d2a0b44SSanjay Patel // both sequences. 3670d2a0b44SSanjay Patel int InsertCost = 3680d2a0b44SSanjay Patel TTI.getVectorInstrCost(Instruction::InsertElement, VecTy, Index); 369*5dc4e7c2SSimon Pilgrim int OldCost = (IsConst0 ? 0 : InsertCost) + (IsConst1 ? 0 : InsertCost) + 370*5dc4e7c2SSimon Pilgrim VectorOpCost; 3715f730b64SSanjay Patel int NewCost = ScalarOpCost + InsertCost + 372*5dc4e7c2SSimon Pilgrim (IsConst0 ? 0 : !Ins0->hasOneUse() * InsertCost) + 373*5dc4e7c2SSimon Pilgrim (IsConst1 ? 0 : !Ins1->hasOneUse() * InsertCost); 3740d2a0b44SSanjay Patel 3750d2a0b44SSanjay Patel // We want to scalarize unless the vector variant actually has lower cost. 3760d2a0b44SSanjay Patel if (OldCost < NewCost) 3770d2a0b44SSanjay Patel return false; 3780d2a0b44SSanjay Patel 3790d2a0b44SSanjay Patel // vec_bo (inselt VecC0, V0, Index), (inselt VecC1, V1, Index) --> 3800d2a0b44SSanjay Patel // inselt NewVecC, (scalar_bo V0, V1), Index 3810d2a0b44SSanjay Patel ++NumScalarBO; 3820d2a0b44SSanjay Patel IRBuilder<> Builder(&I); 383*5dc4e7c2SSimon Pilgrim 384*5dc4e7c2SSimon Pilgrim // For constant cases, extract the scalar element, this should constant fold. 385*5dc4e7c2SSimon Pilgrim if (IsConst0) 386*5dc4e7c2SSimon Pilgrim V0 = ConstantExpr::getExtractElement(VecC0, Builder.getInt64(Index)); 387*5dc4e7c2SSimon Pilgrim if (IsConst1) 388*5dc4e7c2SSimon Pilgrim V1 = ConstantExpr::getExtractElement(VecC1, Builder.getInt64(Index)); 389*5dc4e7c2SSimon Pilgrim 3900d2a0b44SSanjay Patel Value *Scalar = Builder.CreateBinOp(Opcode, V0, V1, I.getName() + ".scalar"); 3910d2a0b44SSanjay Patel 3920d2a0b44SSanjay Patel // All IR flags are safe to back-propagate. There is no potential for extra 3930d2a0b44SSanjay Patel // poison to be created by the scalar instruction. 3940d2a0b44SSanjay Patel if (auto *ScalarInst = dyn_cast<Instruction>(Scalar)) 3950d2a0b44SSanjay Patel ScalarInst->copyIRFlags(&I); 3960d2a0b44SSanjay Patel 3970d2a0b44SSanjay Patel // Fold the vector constants in the original vectors into a new base vector. 3980d2a0b44SSanjay Patel Constant *NewVecC = ConstantExpr::get(Opcode, VecC0, VecC1); 3990d2a0b44SSanjay Patel Value *Insert = Builder.CreateInsertElement(NewVecC, Scalar, Index); 4000d2a0b44SSanjay Patel I.replaceAllUsesWith(Insert); 4010d2a0b44SSanjay Patel Insert->takeName(&I); 4020d2a0b44SSanjay Patel return true; 4030d2a0b44SSanjay Patel } 4040d2a0b44SSanjay Patel 405a17f03bdSSanjay Patel /// This is the entry point for all transforms. Pass manager differences are 406a17f03bdSSanjay Patel /// handled in the callers of this function. 407a17f03bdSSanjay Patel static bool runImpl(Function &F, const TargetTransformInfo &TTI, 408a17f03bdSSanjay Patel const DominatorTree &DT) { 40925c6544fSSanjay Patel if (DisableVectorCombine) 41025c6544fSSanjay Patel return false; 41125c6544fSSanjay Patel 412a17f03bdSSanjay Patel bool MadeChange = false; 413a17f03bdSSanjay Patel for (BasicBlock &BB : F) { 414a17f03bdSSanjay Patel // Ignore unreachable basic blocks. 415a17f03bdSSanjay Patel if (!DT.isReachableFromEntry(&BB)) 416a17f03bdSSanjay Patel continue; 417a17f03bdSSanjay Patel // Do not delete instructions under here and invalidate the iterator. 41881e9ede3SSanjay Patel // Walk the block forwards to enable simple iterative chains of transforms. 419a17f03bdSSanjay Patel // TODO: It could be more efficient to remove dead instructions 420a17f03bdSSanjay Patel // iteratively in this loop rather than waiting until the end. 42181e9ede3SSanjay Patel for (Instruction &I : BB) { 422fc3cc8a4SSanjay Patel if (isa<DbgInfoIntrinsic>(I)) 423fc3cc8a4SSanjay Patel continue; 424fc445589SSanjay Patel MadeChange |= foldExtractExtract(I, TTI); 425b6050ca1SSanjay Patel MadeChange |= foldBitcastShuf(I, TTI); 4260d2a0b44SSanjay Patel MadeChange |= scalarizeBinop(I, TTI); 427a17f03bdSSanjay Patel } 428fc3cc8a4SSanjay Patel } 429a17f03bdSSanjay Patel 430a17f03bdSSanjay Patel // We're done with transforms, so remove dead instructions. 431a17f03bdSSanjay Patel if (MadeChange) 432a17f03bdSSanjay Patel for (BasicBlock &BB : F) 433a17f03bdSSanjay Patel SimplifyInstructionsInBlock(&BB); 434a17f03bdSSanjay Patel 435a17f03bdSSanjay Patel return MadeChange; 436a17f03bdSSanjay Patel } 437a17f03bdSSanjay Patel 438a17f03bdSSanjay Patel // Pass manager boilerplate below here. 439a17f03bdSSanjay Patel 440a17f03bdSSanjay Patel namespace { 441a17f03bdSSanjay Patel class VectorCombineLegacyPass : public FunctionPass { 442a17f03bdSSanjay Patel public: 443a17f03bdSSanjay Patel static char ID; 444a17f03bdSSanjay Patel VectorCombineLegacyPass() : FunctionPass(ID) { 445a17f03bdSSanjay Patel initializeVectorCombineLegacyPassPass(*PassRegistry::getPassRegistry()); 446a17f03bdSSanjay Patel } 447a17f03bdSSanjay Patel 448a17f03bdSSanjay Patel void getAnalysisUsage(AnalysisUsage &AU) const override { 449a17f03bdSSanjay Patel AU.addRequired<DominatorTreeWrapperPass>(); 450a17f03bdSSanjay Patel AU.addRequired<TargetTransformInfoWrapperPass>(); 451a17f03bdSSanjay Patel AU.setPreservesCFG(); 452a17f03bdSSanjay Patel AU.addPreserved<DominatorTreeWrapperPass>(); 453a17f03bdSSanjay Patel AU.addPreserved<GlobalsAAWrapperPass>(); 454024098aeSSanjay Patel AU.addPreserved<AAResultsWrapperPass>(); 455024098aeSSanjay Patel AU.addPreserved<BasicAAWrapperPass>(); 456a17f03bdSSanjay Patel FunctionPass::getAnalysisUsage(AU); 457a17f03bdSSanjay Patel } 458a17f03bdSSanjay Patel 459a17f03bdSSanjay Patel bool runOnFunction(Function &F) override { 460a17f03bdSSanjay Patel if (skipFunction(F)) 461a17f03bdSSanjay Patel return false; 462a17f03bdSSanjay Patel auto &TTI = getAnalysis<TargetTransformInfoWrapperPass>().getTTI(F); 463a17f03bdSSanjay Patel auto &DT = getAnalysis<DominatorTreeWrapperPass>().getDomTree(); 464a17f03bdSSanjay Patel return runImpl(F, TTI, DT); 465a17f03bdSSanjay Patel } 466a17f03bdSSanjay Patel }; 467a17f03bdSSanjay Patel } // namespace 468a17f03bdSSanjay Patel 469a17f03bdSSanjay Patel char VectorCombineLegacyPass::ID = 0; 470a17f03bdSSanjay Patel INITIALIZE_PASS_BEGIN(VectorCombineLegacyPass, "vector-combine", 471a17f03bdSSanjay Patel "Optimize scalar/vector ops", false, 472a17f03bdSSanjay Patel false) 473a17f03bdSSanjay Patel INITIALIZE_PASS_DEPENDENCY(DominatorTreeWrapperPass) 474a17f03bdSSanjay Patel INITIALIZE_PASS_END(VectorCombineLegacyPass, "vector-combine", 475a17f03bdSSanjay Patel "Optimize scalar/vector ops", false, false) 476a17f03bdSSanjay Patel Pass *llvm::createVectorCombinePass() { 477a17f03bdSSanjay Patel return new VectorCombineLegacyPass(); 478a17f03bdSSanjay Patel } 479a17f03bdSSanjay Patel 480a17f03bdSSanjay Patel PreservedAnalyses VectorCombinePass::run(Function &F, 481a17f03bdSSanjay Patel FunctionAnalysisManager &FAM) { 482a17f03bdSSanjay Patel TargetTransformInfo &TTI = FAM.getResult<TargetIRAnalysis>(F); 483a17f03bdSSanjay Patel DominatorTree &DT = FAM.getResult<DominatorTreeAnalysis>(F); 484a17f03bdSSanjay Patel if (!runImpl(F, TTI, DT)) 485a17f03bdSSanjay Patel return PreservedAnalyses::all(); 486a17f03bdSSanjay Patel PreservedAnalyses PA; 487a17f03bdSSanjay Patel PA.preserveSet<CFGAnalyses>(); 488a17f03bdSSanjay Patel PA.preserve<GlobalsAA>(); 489024098aeSSanjay Patel PA.preserve<AAManager>(); 490024098aeSSanjay Patel PA.preserve<BasicAA>(); 491a17f03bdSSanjay Patel return PA; 492a17f03bdSSanjay Patel } 493