Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 1 | //===- InstCombineVectorOps.cpp -------------------------------------------===// |
| 2 | // |
Chandler Carruth | 2946cd7 | 2019-01-19 08:50:56 +0000 | [diff] [blame] | 3 | // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. |
| 4 | // See https://llvm.org/LICENSE.txt for license information. |
| 5 | // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 6 | // |
| 7 | //===----------------------------------------------------------------------===// |
| 8 | // |
| 9 | // This file implements instcombine for ExtractElement, InsertElement and |
| 10 | // ShuffleVector. |
| 11 | // |
| 12 | //===----------------------------------------------------------------------===// |
| 13 | |
Chandler Carruth | a917458 | 2015-01-22 05:25:13 +0000 | [diff] [blame] | 14 | #include "InstCombineInternal.h" |
Eugene Zelenko | 7f0f9bc | 2017-10-24 21:24:53 +0000 | [diff] [blame] | 15 | #include "llvm/ADT/APInt.h" |
| 16 | #include "llvm/ADT/ArrayRef.h" |
JF Bastien | d52c990 | 2015-02-25 22:30:51 +0000 | [diff] [blame] | 17 | #include "llvm/ADT/DenseMap.h" |
Eugene Zelenko | 7f0f9bc | 2017-10-24 21:24:53 +0000 | [diff] [blame] | 18 | #include "llvm/ADT/STLExtras.h" |
| 19 | #include "llvm/ADT/SmallVector.h" |
David Majnemer | 599ca44 | 2015-07-13 01:15:53 +0000 | [diff] [blame] | 20 | #include "llvm/Analysis/InstructionSimplify.h" |
| 21 | #include "llvm/Analysis/VectorUtils.h" |
Eugene Zelenko | 7f0f9bc | 2017-10-24 21:24:53 +0000 | [diff] [blame] | 22 | #include "llvm/IR/BasicBlock.h" |
| 23 | #include "llvm/IR/Constant.h" |
| 24 | #include "llvm/IR/Constants.h" |
| 25 | #include "llvm/IR/DerivedTypes.h" |
| 26 | #include "llvm/IR/InstrTypes.h" |
| 27 | #include "llvm/IR/Instruction.h" |
| 28 | #include "llvm/IR/Instructions.h" |
| 29 | #include "llvm/IR/Operator.h" |
Chandler Carruth | 820a908 | 2014-03-04 11:08:18 +0000 | [diff] [blame] | 30 | #include "llvm/IR/PatternMatch.h" |
Eugene Zelenko | 7f0f9bc | 2017-10-24 21:24:53 +0000 | [diff] [blame] | 31 | #include "llvm/IR/Type.h" |
| 32 | #include "llvm/IR/User.h" |
| 33 | #include "llvm/IR/Value.h" |
| 34 | #include "llvm/Support/Casting.h" |
| 35 | #include "llvm/Support/ErrorHandling.h" |
| 36 | #include "llvm/Transforms/InstCombine/InstCombineWorklist.h" |
| 37 | #include <cassert> |
| 38 | #include <cstdint> |
| 39 | #include <iterator> |
| 40 | #include <utility> |
| 41 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 42 | using namespace llvm; |
Nadav Rotem | 7df8509 | 2013-01-15 23:43:14 +0000 | [diff] [blame] | 43 | using namespace PatternMatch; |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 44 | |
Chandler Carruth | 964daaa | 2014-04-22 02:55:47 +0000 | [diff] [blame] | 45 | #define DEBUG_TYPE "instcombine" |
| 46 | |
Sanjay Patel | 6eccf48 | 2015-09-09 15:24:36 +0000 | [diff] [blame] | 47 | /// Return true if the value is cheaper to scalarize than it is to leave as a |
Sanjay Patel | e51d5bd | 2018-12-18 19:07:38 +0000 | [diff] [blame] | 48 | /// vector operation. IsConstantExtractIndex indicates whether we are extracting |
| 49 | /// one known element from a vector constant. |
| 50 | /// |
| 51 | /// FIXME: It's possible to create more instructions than previously existed. |
| 52 | static bool cheapToScalarize(Value *V, bool IsConstantExtractIndex) { |
| 53 | // If we can pick a scalar constant value out of a vector, that is free. |
| 54 | if (auto *C = dyn_cast<Constant>(V)) |
| 55 | return IsConstantExtractIndex || C->getSplatValue(); |
Chris Lattner | 8326bd8 | 2012-01-26 00:42:34 +0000 | [diff] [blame] | 56 | |
Sanjay Patel | e51d5bd | 2018-12-18 19:07:38 +0000 | [diff] [blame] | 57 | // An insertelement to the same constant index as our extract will simplify |
| 58 | // to the scalar inserted element. An insertelement to a different constant |
| 59 | // index is irrelevant to our extract. |
| 60 | if (match(V, m_InsertElement(m_Value(), m_Value(), m_ConstantInt()))) |
| 61 | return IsConstantExtractIndex; |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 62 | |
Sanjay Patel | e51d5bd | 2018-12-18 19:07:38 +0000 | [diff] [blame] | 63 | if (match(V, m_OneUse(m_Load(m_Value())))) |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 64 | return true; |
Sanjay Patel | e51d5bd | 2018-12-18 19:07:38 +0000 | [diff] [blame] | 65 | |
| 66 | Value *V0, *V1; |
| 67 | if (match(V, m_OneUse(m_BinOp(m_Value(V0), m_Value(V1))))) |
| 68 | if (cheapToScalarize(V0, IsConstantExtractIndex) || |
| 69 | cheapToScalarize(V1, IsConstantExtractIndex)) |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 70 | return true; |
Sanjay Patel | e51d5bd | 2018-12-18 19:07:38 +0000 | [diff] [blame] | 71 | |
| 72 | CmpInst::Predicate UnusedPred; |
| 73 | if (match(V, m_OneUse(m_Cmp(UnusedPred, m_Value(V0), m_Value(V1))))) |
| 74 | if (cheapToScalarize(V0, IsConstantExtractIndex) || |
| 75 | cheapToScalarize(V1, IsConstantExtractIndex)) |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 76 | return true; |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 77 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 78 | return false; |
| 79 | } |
| 80 | |
Michael Kuperstein | a0c6ae0 | 2016-06-06 23:38:33 +0000 | [diff] [blame] | 81 | // If we have a PHI node with a vector type that is only used to feed |
Matt Arsenault | 3887473 | 2013-08-28 22:17:26 +0000 | [diff] [blame] | 82 | // itself and be an operand of extractelement at a constant location, |
| 83 | // try to replace the PHI of the vector type with a PHI of a scalar type. |
Anat Shemer | 0c95efa | 2013-04-18 19:35:39 +0000 | [diff] [blame] | 84 | Instruction *InstCombiner::scalarizePHI(ExtractElementInst &EI, PHINode *PN) { |
Michael Kuperstein | a0c6ae0 | 2016-06-06 23:38:33 +0000 | [diff] [blame] | 85 | SmallVector<Instruction *, 2> Extracts; |
| 86 | // The users we want the PHI to have are: |
| 87 | // 1) The EI ExtractElement (we already know this) |
| 88 | // 2) Possibly more ExtractElements with the same index. |
| 89 | // 3) Another operand, which will feed back into the PHI. |
| 90 | Instruction *PHIUser = nullptr; |
| 91 | for (auto U : PN->users()) { |
| 92 | if (ExtractElementInst *EU = dyn_cast<ExtractElementInst>(U)) { |
| 93 | if (EI.getIndexOperand() == EU->getIndexOperand()) |
| 94 | Extracts.push_back(EU); |
| 95 | else |
| 96 | return nullptr; |
| 97 | } else if (!PHIUser) { |
| 98 | PHIUser = cast<Instruction>(U); |
| 99 | } else { |
| 100 | return nullptr; |
| 101 | } |
| 102 | } |
Anat Shemer | 0c95efa | 2013-04-18 19:35:39 +0000 | [diff] [blame] | 103 | |
Michael Kuperstein | a0c6ae0 | 2016-06-06 23:38:33 +0000 | [diff] [blame] | 104 | if (!PHIUser) |
| 105 | return nullptr; |
Anat Shemer | 0c95efa | 2013-04-18 19:35:39 +0000 | [diff] [blame] | 106 | |
| 107 | // Verify that this PHI user has one use, which is the PHI itself, |
| 108 | // and that it is a binary operation which is cheap to scalarize. |
Eugene Zelenko | 7f0f9bc | 2017-10-24 21:24:53 +0000 | [diff] [blame] | 109 | // otherwise return nullptr. |
Chandler Carruth | cdf4788 | 2014-03-09 03:16:01 +0000 | [diff] [blame] | 110 | if (!PHIUser->hasOneUse() || !(PHIUser->user_back() == PN) || |
Sanjay Patel | 431e114 | 2015-11-17 17:24:08 +0000 | [diff] [blame] | 111 | !(isa<BinaryOperator>(PHIUser)) || !cheapToScalarize(PHIUser, true)) |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 112 | return nullptr; |
Anat Shemer | 0c95efa | 2013-04-18 19:35:39 +0000 | [diff] [blame] | 113 | |
| 114 | // Create a scalar PHI node that will replace the vector PHI node |
| 115 | // just before the current PHI node. |
Joey Gouly | b34294d | 2013-05-24 12:33:28 +0000 | [diff] [blame] | 116 | PHINode *scalarPHI = cast<PHINode>(InsertNewInstWith( |
| 117 | PHINode::Create(EI.getType(), PN->getNumIncomingValues(), ""), *PN)); |
Anat Shemer | 0c95efa | 2013-04-18 19:35:39 +0000 | [diff] [blame] | 118 | // Scalarize each PHI operand. |
Joey Gouly | b34294d | 2013-05-24 12:33:28 +0000 | [diff] [blame] | 119 | for (unsigned i = 0; i < PN->getNumIncomingValues(); i++) { |
Anat Shemer | 0c95efa | 2013-04-18 19:35:39 +0000 | [diff] [blame] | 120 | Value *PHIInVal = PN->getIncomingValue(i); |
| 121 | BasicBlock *inBB = PN->getIncomingBlock(i); |
| 122 | Value *Elt = EI.getIndexOperand(); |
| 123 | // If the operand is the PHI induction variable: |
| 124 | if (PHIInVal == PHIUser) { |
| 125 | // Scalarize the binary operation. Its first operand is the |
Sanjay Patel | 70af1fd | 2014-07-07 22:13:58 +0000 | [diff] [blame] | 126 | // scalar PHI, and the second operand is extracted from the other |
Anat Shemer | 0c95efa | 2013-04-18 19:35:39 +0000 | [diff] [blame] | 127 | // vector operand. |
| 128 | BinaryOperator *B0 = cast<BinaryOperator>(PHIUser); |
Joey Gouly | b34294d | 2013-05-24 12:33:28 +0000 | [diff] [blame] | 129 | unsigned opId = (B0->getOperand(0) == PN) ? 1 : 0; |
Joey Gouly | 8369928 | 2013-05-24 12:29:54 +0000 | [diff] [blame] | 130 | Value *Op = InsertNewInstWith( |
| 131 | ExtractElementInst::Create(B0->getOperand(opId), Elt, |
| 132 | B0->getOperand(opId)->getName() + ".Elt"), |
| 133 | *B0); |
Anat Shemer | 0c95efa | 2013-04-18 19:35:39 +0000 | [diff] [blame] | 134 | Value *newPHIUser = InsertNewInstWith( |
Owen Anderson | 7ea02fc | 2016-03-01 19:35:52 +0000 | [diff] [blame] | 135 | BinaryOperator::CreateWithCopiedFlags(B0->getOpcode(), |
| 136 | scalarPHI, Op, B0), *B0); |
Anat Shemer | 0c95efa | 2013-04-18 19:35:39 +0000 | [diff] [blame] | 137 | scalarPHI->addIncoming(newPHIUser, inBB); |
| 138 | } else { |
| 139 | // Scalarize PHI input: |
Joey Gouly | b34294d | 2013-05-24 12:33:28 +0000 | [diff] [blame] | 140 | Instruction *newEI = ExtractElementInst::Create(PHIInVal, Elt, ""); |
Anat Shemer | 0c95efa | 2013-04-18 19:35:39 +0000 | [diff] [blame] | 141 | // Insert the new instruction into the predecessor basic block. |
| 142 | Instruction *pos = dyn_cast<Instruction>(PHIInVal); |
| 143 | BasicBlock::iterator InsertPos; |
| 144 | if (pos && !isa<PHINode>(pos)) { |
Duncan P. N. Exon Smith | 9f8aaf2 | 2015-10-13 16:59:33 +0000 | [diff] [blame] | 145 | InsertPos = ++pos->getIterator(); |
Anat Shemer | 0c95efa | 2013-04-18 19:35:39 +0000 | [diff] [blame] | 146 | } else { |
| 147 | InsertPos = inBB->getFirstInsertionPt(); |
| 148 | } |
| 149 | |
| 150 | InsertNewInstWith(newEI, *InsertPos); |
| 151 | |
| 152 | scalarPHI->addIncoming(newEI, inBB); |
| 153 | } |
| 154 | } |
Michael Kuperstein | a0c6ae0 | 2016-06-06 23:38:33 +0000 | [diff] [blame] | 155 | |
| 156 | for (auto E : Extracts) |
| 157 | replaceInstUsesWith(*E, scalarPHI); |
| 158 | |
| 159 | return &EI; |
Anat Shemer | 0c95efa | 2013-04-18 19:35:39 +0000 | [diff] [blame] | 160 | } |
| 161 | |
Sanjay Patel | 4674c77 | 2018-09-24 20:41:22 +0000 | [diff] [blame] | 162 | static Instruction *foldBitcastExtElt(ExtractElementInst &Ext, |
Sanjay Patel | 31b0719 | 2018-10-01 14:40:00 +0000 | [diff] [blame] | 163 | InstCombiner::BuilderTy &Builder, |
| 164 | bool IsBigEndian) { |
Sanjay Patel | 4674c77 | 2018-09-24 20:41:22 +0000 | [diff] [blame] | 165 | Value *X; |
| 166 | uint64_t ExtIndexC; |
| 167 | if (!match(Ext.getVectorOperand(), m_BitCast(m_Value(X))) || |
| 168 | !X->getType()->isVectorTy() || |
| 169 | !match(Ext.getIndexOperand(), m_ConstantInt(ExtIndexC))) |
| 170 | return nullptr; |
| 171 | |
| 172 | // If this extractelement is using a bitcast from a vector of the same number |
| 173 | // of elements, see if we can find the source element from the source vector: |
| 174 | // extelt (bitcast VecX), IndexC --> bitcast X[IndexC] |
| 175 | Type *SrcTy = X->getType(); |
| 176 | Type *DestTy = Ext.getType(); |
| 177 | unsigned NumSrcElts = SrcTy->getVectorNumElements(); |
| 178 | unsigned NumElts = Ext.getVectorOperandType()->getNumElements(); |
| 179 | if (NumSrcElts == NumElts) |
| 180 | if (Value *Elt = findScalarElement(X, ExtIndexC)) |
| 181 | return new BitCastInst(Elt, DestTy); |
| 182 | |
Sanjay Patel | 31b0719 | 2018-10-01 14:40:00 +0000 | [diff] [blame] | 183 | // If the source elements are wider than the destination, try to shift and |
| 184 | // truncate a subset of scalar bits of an insert op. |
Sanjay Patel | 3746e11 | 2018-10-04 16:25:05 +0000 | [diff] [blame] | 185 | if (NumSrcElts < NumElts) { |
Sanjay Patel | 31b0719 | 2018-10-01 14:40:00 +0000 | [diff] [blame] | 186 | Value *Scalar; |
| 187 | uint64_t InsIndexC; |
| 188 | if (!match(X, m_InsertElement(m_Value(), m_Value(Scalar), |
| 189 | m_ConstantInt(InsIndexC)))) |
| 190 | return nullptr; |
| 191 | |
| 192 | // The extract must be from the subset of vector elements that we inserted |
| 193 | // into. Example: if we inserted element 1 of a <2 x i64> and we are |
| 194 | // extracting an i16 (narrowing ratio = 4), then this extract must be from 1 |
| 195 | // of elements 4-7 of the bitcasted vector. |
| 196 | unsigned NarrowingRatio = NumElts / NumSrcElts; |
| 197 | if (ExtIndexC / NarrowingRatio != InsIndexC) |
| 198 | return nullptr; |
| 199 | |
| 200 | // We are extracting part of the original scalar. How that scalar is |
| 201 | // inserted into the vector depends on the endian-ness. Example: |
| 202 | // Vector Byte Elt Index: 0 1 2 3 4 5 6 7 |
| 203 | // +--+--+--+--+--+--+--+--+ |
| 204 | // inselt <2 x i32> V, <i32> S, 1: |V0|V1|V2|V3|S0|S1|S2|S3| |
| 205 | // extelt <4 x i16> V', 3: | |S2|S3| |
| 206 | // +--+--+--+--+--+--+--+--+ |
| 207 | // If this is little-endian, S2|S3 are the MSB of the 32-bit 'S' value. |
| 208 | // If this is big-endian, S2|S3 are the LSB of the 32-bit 'S' value. |
| 209 | // In this example, we must right-shift little-endian. Big-endian is just a |
| 210 | // truncate. |
| 211 | unsigned Chunk = ExtIndexC % NarrowingRatio; |
| 212 | if (IsBigEndian) |
| 213 | Chunk = NarrowingRatio - 1 - Chunk; |
Sanjay Patel | 3746e11 | 2018-10-04 16:25:05 +0000 | [diff] [blame] | 214 | |
| 215 | // Bail out if this is an FP vector to FP vector sequence. That would take |
| 216 | // more instructions than we started with unless there is no shift, and it |
| 217 | // may not be handled as well in the backend. |
| 218 | bool NeedSrcBitcast = SrcTy->getScalarType()->isFloatingPointTy(); |
| 219 | bool NeedDestBitcast = DestTy->isFloatingPointTy(); |
| 220 | if (NeedSrcBitcast && NeedDestBitcast) |
| 221 | return nullptr; |
| 222 | |
| 223 | unsigned SrcWidth = SrcTy->getScalarSizeInBits(); |
| 224 | unsigned DestWidth = DestTy->getPrimitiveSizeInBits(); |
| 225 | unsigned ShAmt = Chunk * DestWidth; |
| 226 | |
| 227 | // TODO: This limitation is more strict than necessary. We could sum the |
| 228 | // number of new instructions and subtract the number eliminated to know if |
| 229 | // we can proceed. |
| 230 | if (!X->hasOneUse() || !Ext.getVectorOperand()->hasOneUse()) |
| 231 | if (NeedSrcBitcast || NeedDestBitcast) |
| 232 | return nullptr; |
| 233 | |
| 234 | if (NeedSrcBitcast) { |
| 235 | Type *SrcIntTy = IntegerType::getIntNTy(Scalar->getContext(), SrcWidth); |
| 236 | Scalar = Builder.CreateBitCast(Scalar, SrcIntTy); |
| 237 | } |
| 238 | |
Sanjay Patel | 31b0719 | 2018-10-01 14:40:00 +0000 | [diff] [blame] | 239 | if (ShAmt) { |
| 240 | // Bail out if we could end with more instructions than we started with. |
| 241 | if (!Ext.getVectorOperand()->hasOneUse()) |
| 242 | return nullptr; |
| 243 | Scalar = Builder.CreateLShr(Scalar, ShAmt); |
| 244 | } |
Sanjay Patel | 3746e11 | 2018-10-04 16:25:05 +0000 | [diff] [blame] | 245 | |
| 246 | if (NeedDestBitcast) { |
| 247 | Type *DestIntTy = IntegerType::getIntNTy(Scalar->getContext(), DestWidth); |
| 248 | return new BitCastInst(Builder.CreateTrunc(Scalar, DestIntTy), DestTy); |
| 249 | } |
Sanjay Patel | 31b0719 | 2018-10-01 14:40:00 +0000 | [diff] [blame] | 250 | return new TruncInst(Scalar, DestTy); |
| 251 | } |
| 252 | |
Sanjay Patel | 4674c77 | 2018-09-24 20:41:22 +0000 | [diff] [blame] | 253 | return nullptr; |
| 254 | } |
| 255 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 256 | Instruction *InstCombiner::visitExtractElementInst(ExtractElementInst &EI) { |
Sanjay Patel | 47b3b4b | 2018-12-05 21:57:51 +0000 | [diff] [blame] | 257 | Value *SrcVec = EI.getVectorOperand(); |
| 258 | Value *Index = EI.getIndexOperand(); |
| 259 | if (Value *V = SimplifyExtractElementInst(SrcVec, Index, |
Craig Topper | a420562 | 2017-06-09 03:21:29 +0000 | [diff] [blame] | 260 | SQ.getWithInstruction(&EI))) |
Sanjay Patel | 4b19880 | 2016-02-01 22:23:39 +0000 | [diff] [blame] | 261 | return replaceInstUsesWith(EI, V); |
David Majnemer | 599ca44 | 2015-07-13 01:15:53 +0000 | [diff] [blame] | 262 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 263 | // If extracting a specified index from the vector, see if we can recursively |
| 264 | // find a previously computed scalar that was inserted into the vector. |
Sanjay Patel | 47b3b4b | 2018-12-05 21:57:51 +0000 | [diff] [blame] | 265 | auto *IndexC = dyn_cast<ConstantInt>(Index); |
| 266 | if (IndexC) { |
Sanjay Patel | 7a52626 | 2018-09-24 16:39:03 +0000 | [diff] [blame] | 267 | unsigned NumElts = EI.getVectorOperandType()->getNumElements(); |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 268 | |
Simon Pilgrim | e7d032f | 2017-12-27 12:00:18 +0000 | [diff] [blame] | 269 | // InstSimplify should handle cases where the index is invalid. |
Sanjay Patel | 47b3b4b | 2018-12-05 21:57:51 +0000 | [diff] [blame] | 270 | if (!IndexC->getValue().ule(NumElts)) |
Simon Pilgrim | e7d032f | 2017-12-27 12:00:18 +0000 | [diff] [blame] | 271 | return nullptr; |
| 272 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 273 | // This instruction only demands the single element from the input vector. |
| 274 | // If the input vector has a single use, simplify it based on this use |
| 275 | // property. |
Sanjay Patel | 47b3b4b | 2018-12-05 21:57:51 +0000 | [diff] [blame] | 276 | if (SrcVec->hasOneUse() && NumElts != 1) { |
Sanjay Patel | 7a52626 | 2018-09-24 16:39:03 +0000 | [diff] [blame] | 277 | APInt UndefElts(NumElts, 0); |
Sanjay Patel | 47b3b4b | 2018-12-05 21:57:51 +0000 | [diff] [blame] | 278 | APInt DemandedElts(NumElts, 0); |
| 279 | DemandedElts.setBit(IndexC->getZExtValue()); |
| 280 | if (Value *V = SimplifyDemandedVectorElts(SrcVec, DemandedElts, |
Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 281 | UndefElts)) { |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 282 | EI.setOperand(0, V); |
| 283 | return &EI; |
| 284 | } |
| 285 | } |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 286 | |
Sanjay Patel | 31b0719 | 2018-10-01 14:40:00 +0000 | [diff] [blame] | 287 | if (Instruction *I = foldBitcastExtElt(EI, Builder, DL.isBigEndian())) |
Sanjay Patel | 4674c77 | 2018-09-24 20:41:22 +0000 | [diff] [blame] | 288 | return I; |
Anat Shemer | 0c95efa | 2013-04-18 19:35:39 +0000 | [diff] [blame] | 289 | |
| 290 | // If there's a vector PHI feeding a scalar use through this extractelement |
| 291 | // instruction, try to scalarize the PHI. |
Sanjay Patel | 47b3b4b | 2018-12-05 21:57:51 +0000 | [diff] [blame] | 292 | if (auto *Phi = dyn_cast<PHINode>(SrcVec)) |
| 293 | if (Instruction *ScalarPHI = scalarizePHI(EI, Phi)) |
| 294 | return ScalarPHI; |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 295 | } |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 296 | |
Sanjay Patel | 47b3b4b | 2018-12-05 21:57:51 +0000 | [diff] [blame] | 297 | BinaryOperator *BO; |
| 298 | if (match(SrcVec, m_BinOp(BO)) && cheapToScalarize(SrcVec, IndexC)) { |
| 299 | // extelt (binop X, Y), Index --> binop (extelt X, Index), (extelt Y, Index) |
| 300 | Value *X = BO->getOperand(0), *Y = BO->getOperand(1); |
| 301 | Value *E0 = Builder.CreateExtractElement(X, Index); |
| 302 | Value *E1 = Builder.CreateExtractElement(Y, Index); |
| 303 | return BinaryOperator::CreateWithCopiedFlags(BO->getOpcode(), E0, E1, BO); |
| 304 | } |
| 305 | |
Matt Arsenault | 9ccde61 | 2018-12-10 21:50:54 +0000 | [diff] [blame] | 306 | Value *X, *Y; |
| 307 | CmpInst::Predicate Pred; |
| 308 | if (match(SrcVec, m_Cmp(Pred, m_Value(X), m_Value(Y))) && |
| 309 | cheapToScalarize(SrcVec, IndexC)) { |
| 310 | // extelt (cmp X, Y), Index --> cmp (extelt X, Index), (extelt Y, Index) |
| 311 | Value *E0 = Builder.CreateExtractElement(X, Index); |
| 312 | Value *E1 = Builder.CreateExtractElement(Y, Index); |
| 313 | return CmpInst::Create(cast<CmpInst>(SrcVec)->getOpcode(), Pred, E0, E1); |
| 314 | } |
| 315 | |
Sanjay Patel | 47b3b4b | 2018-12-05 21:57:51 +0000 | [diff] [blame] | 316 | if (auto *I = dyn_cast<Instruction>(SrcVec)) { |
| 317 | if (auto *IE = dyn_cast<InsertElementInst>(I)) { |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 318 | // Extracting the inserted element? |
Sanjay Patel | 47b3b4b | 2018-12-05 21:57:51 +0000 | [diff] [blame] | 319 | if (IE->getOperand(2) == Index) |
Sanjay Patel | 4b19880 | 2016-02-01 22:23:39 +0000 | [diff] [blame] | 320 | return replaceInstUsesWith(EI, IE->getOperand(1)); |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 321 | // If the inserted and extracted elements are constants, they must not |
| 322 | // be the same value, extract from the pre-inserted value instead. |
Sanjay Patel | 47b3b4b | 2018-12-05 21:57:51 +0000 | [diff] [blame] | 323 | if (isa<Constant>(IE->getOperand(2)) && IndexC) { |
| 324 | Worklist.AddValue(SrcVec); |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 325 | EI.setOperand(0, IE->getOperand(0)); |
| 326 | return &EI; |
| 327 | } |
Sanjay Patel | 47b3b4b | 2018-12-05 21:57:51 +0000 | [diff] [blame] | 328 | } else if (auto *SVI = dyn_cast<ShuffleVectorInst>(I)) { |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 329 | // If this is extracting an element from a shufflevector, figure out where |
| 330 | // it came from and extract from the appropriate input element instead. |
Sanjay Patel | 47b3b4b | 2018-12-05 21:57:51 +0000 | [diff] [blame] | 331 | if (auto *Elt = dyn_cast<ConstantInt>(Index)) { |
Eli Friedman | 303c81c | 2011-10-21 19:11:34 +0000 | [diff] [blame] | 332 | int SrcIdx = SVI->getMaskValue(Elt->getZExtValue()); |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 333 | Value *Src; |
| 334 | unsigned LHSWidth = |
Chris Lattner | 8326bd8 | 2012-01-26 00:42:34 +0000 | [diff] [blame] | 335 | SVI->getOperand(0)->getType()->getVectorNumElements(); |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 336 | |
Bob Wilson | 11ee456 | 2010-10-29 22:03:05 +0000 | [diff] [blame] | 337 | if (SrcIdx < 0) |
Sanjay Patel | 4b19880 | 2016-02-01 22:23:39 +0000 | [diff] [blame] | 338 | return replaceInstUsesWith(EI, UndefValue::get(EI.getType())); |
Bob Wilson | 11ee456 | 2010-10-29 22:03:05 +0000 | [diff] [blame] | 339 | if (SrcIdx < (int)LHSWidth) |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 340 | Src = SVI->getOperand(0); |
Bob Wilson | 11ee456 | 2010-10-29 22:03:05 +0000 | [diff] [blame] | 341 | else { |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 342 | SrcIdx -= LHSWidth; |
| 343 | Src = SVI->getOperand(1); |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 344 | } |
Chris Lattner | 229907c | 2011-07-18 04:54:35 +0000 | [diff] [blame] | 345 | Type *Int32Ty = Type::getInt32Ty(EI.getContext()); |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 346 | return ExtractElementInst::Create(Src, |
Bob Wilson | 9d07f39 | 2010-10-29 22:03:07 +0000 | [diff] [blame] | 347 | ConstantInt::get(Int32Ty, |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 348 | SrcIdx, false)); |
| 349 | } |
Sanjay Patel | 47b3b4b | 2018-12-05 21:57:51 +0000 | [diff] [blame] | 350 | } else if (auto *CI = dyn_cast<CastInst>(I)) { |
Sanjay Patel | b67076c | 2015-11-29 22:09:34 +0000 | [diff] [blame] | 351 | // Canonicalize extractelement(cast) -> cast(extractelement). |
| 352 | // Bitcasts can change the number of vector elements, and they cost |
| 353 | // nothing. |
Anat Shemer | 5570318 | 2013-04-18 19:56:44 +0000 | [diff] [blame] | 354 | if (CI->hasOneUse() && (CI->getOpcode() != Instruction::BitCast)) { |
Sanjay Patel | 47b3b4b | 2018-12-05 21:57:51 +0000 | [diff] [blame] | 355 | Value *EE = Builder.CreateExtractElement(CI->getOperand(0), Index); |
Anat Shemer | 10260a7 | 2013-04-22 20:51:10 +0000 | [diff] [blame] | 356 | Worklist.AddValue(EE); |
Nadav Rotem | d74b72b | 2011-03-31 22:57:29 +0000 | [diff] [blame] | 357 | return CastInst::Create(CI->getOpcode(), EE, EI.getType()); |
| 358 | } |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 359 | } |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 360 | } |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 361 | return nullptr; |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 362 | } |
| 363 | |
Sanjay Patel | 6eccf48 | 2015-09-09 15:24:36 +0000 | [diff] [blame] | 364 | /// If V is a shuffle of values that ONLY returns elements from either LHS or |
| 365 | /// RHS, return the shuffle mask and true. Otherwise, return false. |
Sanjay Patel | 431e114 | 2015-11-17 17:24:08 +0000 | [diff] [blame] | 366 | static bool collectSingleShuffleElements(Value *V, Value *LHS, Value *RHS, |
Chris Lattner | 0256be9 | 2012-01-27 03:08:05 +0000 | [diff] [blame] | 367 | SmallVectorImpl<Constant*> &Mask) { |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 368 | assert(LHS->getType() == RHS->getType() && |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 369 | "Invalid CollectSingleShuffleElements"); |
Matt Arsenault | 8227b9f | 2013-09-06 00:37:24 +0000 | [diff] [blame] | 370 | unsigned NumElts = V->getType()->getVectorNumElements(); |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 371 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 372 | if (isa<UndefValue>(V)) { |
| 373 | Mask.assign(NumElts, UndefValue::get(Type::getInt32Ty(V->getContext()))); |
| 374 | return true; |
| 375 | } |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 376 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 377 | if (V == LHS) { |
| 378 | for (unsigned i = 0; i != NumElts; ++i) |
| 379 | Mask.push_back(ConstantInt::get(Type::getInt32Ty(V->getContext()), i)); |
| 380 | return true; |
| 381 | } |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 382 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 383 | if (V == RHS) { |
| 384 | for (unsigned i = 0; i != NumElts; ++i) |
| 385 | Mask.push_back(ConstantInt::get(Type::getInt32Ty(V->getContext()), |
| 386 | i+NumElts)); |
| 387 | return true; |
| 388 | } |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 389 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 390 | if (InsertElementInst *IEI = dyn_cast<InsertElementInst>(V)) { |
| 391 | // If this is an insert of an extract from some other vector, include it. |
| 392 | Value *VecOp = IEI->getOperand(0); |
| 393 | Value *ScalarOp = IEI->getOperand(1); |
| 394 | Value *IdxOp = IEI->getOperand(2); |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 395 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 396 | if (!isa<ConstantInt>(IdxOp)) |
| 397 | return false; |
| 398 | unsigned InsertedIdx = cast<ConstantInt>(IdxOp)->getZExtValue(); |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 399 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 400 | if (isa<UndefValue>(ScalarOp)) { // inserting undef into vector. |
Sanjay Patel | 70af1fd | 2014-07-07 22:13:58 +0000 | [diff] [blame] | 401 | // We can handle this if the vector we are inserting into is |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 402 | // transitively ok. |
Sanjay Patel | 431e114 | 2015-11-17 17:24:08 +0000 | [diff] [blame] | 403 | if (collectSingleShuffleElements(VecOp, LHS, RHS, Mask)) { |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 404 | // If so, update the mask to reflect the inserted undef. |
| 405 | Mask[InsertedIdx] = UndefValue::get(Type::getInt32Ty(V->getContext())); |
| 406 | return true; |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 407 | } |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 408 | } else if (ExtractElementInst *EI = dyn_cast<ExtractElementInst>(ScalarOp)){ |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 409 | if (isa<ConstantInt>(EI->getOperand(1))) { |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 410 | unsigned ExtractedIdx = |
| 411 | cast<ConstantInt>(EI->getOperand(1))->getZExtValue(); |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 412 | unsigned NumLHSElts = LHS->getType()->getVectorNumElements(); |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 413 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 414 | // This must be extracting from either LHS or RHS. |
| 415 | if (EI->getOperand(0) == LHS || EI->getOperand(0) == RHS) { |
Sanjay Patel | 70af1fd | 2014-07-07 22:13:58 +0000 | [diff] [blame] | 416 | // We can handle this if the vector we are inserting into is |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 417 | // transitively ok. |
Sanjay Patel | 431e114 | 2015-11-17 17:24:08 +0000 | [diff] [blame] | 418 | if (collectSingleShuffleElements(VecOp, LHS, RHS, Mask)) { |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 419 | // If so, update the mask to reflect the inserted value. |
| 420 | if (EI->getOperand(0) == LHS) { |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 421 | Mask[InsertedIdx % NumElts] = |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 422 | ConstantInt::get(Type::getInt32Ty(V->getContext()), |
| 423 | ExtractedIdx); |
| 424 | } else { |
| 425 | assert(EI->getOperand(0) == RHS); |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 426 | Mask[InsertedIdx % NumElts] = |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 427 | ConstantInt::get(Type::getInt32Ty(V->getContext()), |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 428 | ExtractedIdx + NumLHSElts); |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 429 | } |
| 430 | return true; |
| 431 | } |
| 432 | } |
| 433 | } |
| 434 | } |
| 435 | } |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 436 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 437 | return false; |
| 438 | } |
| 439 | |
Sanjay Patel | ae945e7 | 2015-12-24 21:17:56 +0000 | [diff] [blame] | 440 | /// If we have insertion into a vector that is wider than the vector that we |
| 441 | /// are extracting from, try to widen the source vector to allow a single |
| 442 | /// shufflevector to replace one or more insert/extract pairs. |
| 443 | static void replaceExtractElements(InsertElementInst *InsElt, |
| 444 | ExtractElementInst *ExtElt, |
| 445 | InstCombiner &IC) { |
| 446 | VectorType *InsVecType = InsElt->getType(); |
| 447 | VectorType *ExtVecType = ExtElt->getVectorOperandType(); |
| 448 | unsigned NumInsElts = InsVecType->getVectorNumElements(); |
| 449 | unsigned NumExtElts = ExtVecType->getVectorNumElements(); |
| 450 | |
| 451 | // The inserted-to vector must be wider than the extracted-from vector. |
| 452 | if (InsVecType->getElementType() != ExtVecType->getElementType() || |
| 453 | NumExtElts >= NumInsElts) |
| 454 | return; |
| 455 | |
| 456 | // Create a shuffle mask to widen the extended-from vector using undefined |
| 457 | // values. The mask selects all of the values of the original vector followed |
| 458 | // by as many undefined values as needed to create a vector of the same length |
| 459 | // as the inserted-to vector. |
| 460 | SmallVector<Constant *, 16> ExtendMask; |
| 461 | IntegerType *IntType = Type::getInt32Ty(InsElt->getContext()); |
| 462 | for (unsigned i = 0; i < NumExtElts; ++i) |
| 463 | ExtendMask.push_back(ConstantInt::get(IntType, i)); |
| 464 | for (unsigned i = NumExtElts; i < NumInsElts; ++i) |
| 465 | ExtendMask.push_back(UndefValue::get(IntType)); |
| 466 | |
| 467 | Value *ExtVecOp = ExtElt->getVectorOperand(); |
Sanjay Patel | 66fff73 | 2016-01-29 20:21:02 +0000 | [diff] [blame] | 468 | auto *ExtVecOpInst = dyn_cast<Instruction>(ExtVecOp); |
| 469 | BasicBlock *InsertionBlock = (ExtVecOpInst && !isa<PHINode>(ExtVecOpInst)) |
| 470 | ? ExtVecOpInst->getParent() |
| 471 | : ExtElt->getParent(); |
| 472 | |
| 473 | // TODO: This restriction matches the basic block check below when creating |
| 474 | // new extractelement instructions. If that limitation is removed, this one |
| 475 | // could also be removed. But for now, we just bail out to ensure that we |
| 476 | // will replace the extractelement instruction that is feeding our |
| 477 | // insertelement instruction. This allows the insertelement to then be |
| 478 | // replaced by a shufflevector. If the insertelement is not replaced, we can |
| 479 | // induce infinite looping because there's an optimization for extractelement |
| 480 | // that will delete our widening shuffle. This would trigger another attempt |
| 481 | // here to create that shuffle, and we spin forever. |
| 482 | if (InsertionBlock != InsElt->getParent()) |
| 483 | return; |
| 484 | |
Sanjay Patel | 4e1b5a5 | 2016-11-10 00:15:14 +0000 | [diff] [blame] | 485 | // TODO: This restriction matches the check in visitInsertElementInst() and |
| 486 | // prevents an infinite loop caused by not turning the extract/insert pair |
| 487 | // into a shuffle. We really should not need either check, but we're lacking |
| 488 | // folds for shufflevectors because we're afraid to generate shuffle masks |
| 489 | // that the backend can't handle. |
| 490 | if (InsElt->hasOneUse() && isa<InsertElementInst>(InsElt->user_back())) |
| 491 | return; |
| 492 | |
Sanjay Patel | ae945e7 | 2015-12-24 21:17:56 +0000 | [diff] [blame] | 493 | auto *WideVec = new ShuffleVectorInst(ExtVecOp, UndefValue::get(ExtVecType), |
| 494 | ConstantVector::get(ExtendMask)); |
| 495 | |
Sanjay Patel | a1c5347 | 2016-01-05 19:09:47 +0000 | [diff] [blame] | 496 | // Insert the new shuffle after the vector operand of the extract is defined |
Sanjay Patel | d72a458 | 2016-01-08 01:39:16 +0000 | [diff] [blame] | 497 | // (as long as it's not a PHI) or at the start of the basic block of the |
| 498 | // extract, so any subsequent extracts in the same basic block can use it. |
| 499 | // TODO: Insert before the earliest ExtractElementInst that is replaced. |
Sanjay Patel | d72a458 | 2016-01-08 01:39:16 +0000 | [diff] [blame] | 500 | if (ExtVecOpInst && !isa<PHINode>(ExtVecOpInst)) |
Sanjay Patel | a1c5347 | 2016-01-05 19:09:47 +0000 | [diff] [blame] | 501 | WideVec->insertAfter(ExtVecOpInst); |
Sanjay Patel | d72a458 | 2016-01-08 01:39:16 +0000 | [diff] [blame] | 502 | else |
Sanjay Patel | a1c5347 | 2016-01-05 19:09:47 +0000 | [diff] [blame] | 503 | IC.InsertNewInstWith(WideVec, *ExtElt->getParent()->getFirstInsertionPt()); |
Sanjay Patel | a1c5347 | 2016-01-05 19:09:47 +0000 | [diff] [blame] | 504 | |
| 505 | // Replace extracts from the original narrow vector with extracts from the new |
| 506 | // wide vector. |
Sanjay Patel | ae945e7 | 2015-12-24 21:17:56 +0000 | [diff] [blame] | 507 | for (User *U : ExtVecOp->users()) { |
Sanjay Patel | a1c5347 | 2016-01-05 19:09:47 +0000 | [diff] [blame] | 508 | ExtractElementInst *OldExt = dyn_cast<ExtractElementInst>(U); |
Sanjay Patel | d72a458 | 2016-01-08 01:39:16 +0000 | [diff] [blame] | 509 | if (!OldExt || OldExt->getParent() != WideVec->getParent()) |
Sanjay Patel | a1c5347 | 2016-01-05 19:09:47 +0000 | [diff] [blame] | 510 | continue; |
| 511 | auto *NewExt = ExtractElementInst::Create(WideVec, OldExt->getOperand(1)); |
Sven van Haastregt | 78819e0 | 2017-06-05 09:18:10 +0000 | [diff] [blame] | 512 | NewExt->insertAfter(OldExt); |
Sanjay Patel | 4b19880 | 2016-02-01 22:23:39 +0000 | [diff] [blame] | 513 | IC.replaceInstUsesWith(*OldExt, NewExt); |
Sanjay Patel | ae945e7 | 2015-12-24 21:17:56 +0000 | [diff] [blame] | 514 | } |
| 515 | } |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 516 | |
| 517 | /// We are building a shuffle to create V, which is a sequence of insertelement, |
| 518 | /// extractelement pairs. If PermittedRHS is set, then we must either use it or |
Sanjay Patel | 70af1fd | 2014-07-07 22:13:58 +0000 | [diff] [blame] | 519 | /// not rely on the second vector source. Return a std::pair containing the |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 520 | /// left and right vectors of the proposed shuffle (or 0), and set the Mask |
| 521 | /// parameter as required. |
| 522 | /// |
| 523 | /// Note: we intentionally don't try to fold earlier shuffles since they have |
| 524 | /// often been chosen carefully to be efficiently implementable on the target. |
Eugene Zelenko | 7f0f9bc | 2017-10-24 21:24:53 +0000 | [diff] [blame] | 525 | using ShuffleOps = std::pair<Value *, Value *>; |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 526 | |
Sanjay Patel | 431e114 | 2015-11-17 17:24:08 +0000 | [diff] [blame] | 527 | static ShuffleOps collectShuffleElements(Value *V, |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 528 | SmallVectorImpl<Constant *> &Mask, |
Sanjay Patel | ae945e7 | 2015-12-24 21:17:56 +0000 | [diff] [blame] | 529 | Value *PermittedRHS, |
| 530 | InstCombiner &IC) { |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 531 | assert(V->getType()->isVectorTy() && "Invalid shuffle!"); |
Craig Topper | 17b5568 | 2016-12-29 07:03:18 +0000 | [diff] [blame] | 532 | unsigned NumElts = V->getType()->getVectorNumElements(); |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 533 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 534 | if (isa<UndefValue>(V)) { |
| 535 | Mask.assign(NumElts, UndefValue::get(Type::getInt32Ty(V->getContext()))); |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 536 | return std::make_pair( |
| 537 | PermittedRHS ? UndefValue::get(PermittedRHS->getType()) : V, nullptr); |
Chris Lattner | a0d01ff | 2012-01-24 14:31:22 +0000 | [diff] [blame] | 538 | } |
Craig Topper | 2ea22b0 | 2013-01-18 05:09:16 +0000 | [diff] [blame] | 539 | |
Chris Lattner | a0d01ff | 2012-01-24 14:31:22 +0000 | [diff] [blame] | 540 | if (isa<ConstantAggregateZero>(V)) { |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 541 | Mask.assign(NumElts, ConstantInt::get(Type::getInt32Ty(V->getContext()),0)); |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 542 | return std::make_pair(V, nullptr); |
Chris Lattner | a0d01ff | 2012-01-24 14:31:22 +0000 | [diff] [blame] | 543 | } |
Craig Topper | 2ea22b0 | 2013-01-18 05:09:16 +0000 | [diff] [blame] | 544 | |
Chris Lattner | a0d01ff | 2012-01-24 14:31:22 +0000 | [diff] [blame] | 545 | if (InsertElementInst *IEI = dyn_cast<InsertElementInst>(V)) { |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 546 | // If this is an insert of an extract from some other vector, include it. |
| 547 | Value *VecOp = IEI->getOperand(0); |
| 548 | Value *ScalarOp = IEI->getOperand(1); |
| 549 | Value *IdxOp = IEI->getOperand(2); |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 550 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 551 | if (ExtractElementInst *EI = dyn_cast<ExtractElementInst>(ScalarOp)) { |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 552 | if (isa<ConstantInt>(EI->getOperand(1)) && isa<ConstantInt>(IdxOp)) { |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 553 | unsigned ExtractedIdx = |
Bob Wilson | 67a6f32 | 2010-10-29 22:20:45 +0000 | [diff] [blame] | 554 | cast<ConstantInt>(EI->getOperand(1))->getZExtValue(); |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 555 | unsigned InsertedIdx = cast<ConstantInt>(IdxOp)->getZExtValue(); |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 556 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 557 | // Either the extracted from or inserted into vector must be RHSVec, |
| 558 | // otherwise we'd end up with a shuffle of three inputs. |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 559 | if (EI->getOperand(0) == PermittedRHS || PermittedRHS == nullptr) { |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 560 | Value *RHS = EI->getOperand(0); |
Sanjay Patel | ae945e7 | 2015-12-24 21:17:56 +0000 | [diff] [blame] | 561 | ShuffleOps LR = collectShuffleElements(VecOp, Mask, RHS, IC); |
Craig Topper | e73658d | 2014-04-28 04:05:08 +0000 | [diff] [blame] | 562 | assert(LR.second == nullptr || LR.second == RHS); |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 563 | |
| 564 | if (LR.first->getType() != RHS->getType()) { |
Sanjay Patel | ae945e7 | 2015-12-24 21:17:56 +0000 | [diff] [blame] | 565 | // Although we are giving up for now, see if we can create extracts |
| 566 | // that match the inserts for another round of combining. |
| 567 | replaceExtractElements(IEI, EI, IC); |
| 568 | |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 569 | // We tried our best, but we can't find anything compatible with RHS |
| 570 | // further up the chain. Return a trivial shuffle. |
| 571 | for (unsigned i = 0; i < NumElts; ++i) |
| 572 | Mask[i] = ConstantInt::get(Type::getInt32Ty(V->getContext()), i); |
| 573 | return std::make_pair(V, nullptr); |
| 574 | } |
| 575 | |
| 576 | unsigned NumLHSElts = RHS->getType()->getVectorNumElements(); |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 577 | Mask[InsertedIdx % NumElts] = |
Bob Wilson | 67a6f32 | 2010-10-29 22:20:45 +0000 | [diff] [blame] | 578 | ConstantInt::get(Type::getInt32Ty(V->getContext()), |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 579 | NumLHSElts+ExtractedIdx); |
| 580 | return std::make_pair(LR.first, RHS); |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 581 | } |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 582 | |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 583 | if (VecOp == PermittedRHS) { |
| 584 | // We've gone as far as we can: anything on the other side of the |
| 585 | // extractelement will already have been converted into a shuffle. |
| 586 | unsigned NumLHSElts = |
| 587 | EI->getOperand(0)->getType()->getVectorNumElements(); |
| 588 | for (unsigned i = 0; i != NumElts; ++i) |
| 589 | Mask.push_back(ConstantInt::get( |
| 590 | Type::getInt32Ty(V->getContext()), |
| 591 | i == InsertedIdx ? ExtractedIdx : NumLHSElts + i)); |
| 592 | return std::make_pair(EI->getOperand(0), PermittedRHS); |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 593 | } |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 594 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 595 | // If this insertelement is a chain that comes from exactly these two |
| 596 | // vectors, return the vector and the effective shuffle. |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 597 | if (EI->getOperand(0)->getType() == PermittedRHS->getType() && |
Sanjay Patel | 431e114 | 2015-11-17 17:24:08 +0000 | [diff] [blame] | 598 | collectSingleShuffleElements(IEI, EI->getOperand(0), PermittedRHS, |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 599 | Mask)) |
| 600 | return std::make_pair(EI->getOperand(0), PermittedRHS); |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 601 | } |
| 602 | } |
| 603 | } |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 604 | |
Sanjay Patel | b67076c | 2015-11-29 22:09:34 +0000 | [diff] [blame] | 605 | // Otherwise, we can't do anything fancy. Return an identity vector. |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 606 | for (unsigned i = 0; i != NumElts; ++i) |
| 607 | Mask.push_back(ConstantInt::get(Type::getInt32Ty(V->getContext()), i)); |
Tim Northover | fad2761 | 2014-03-07 10:24:44 +0000 | [diff] [blame] | 608 | return std::make_pair(V, nullptr); |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 609 | } |
| 610 | |
Michael Zolotukhin | 7d6293a | 2014-05-07 14:30:18 +0000 | [diff] [blame] | 611 | /// Try to find redundant insertvalue instructions, like the following ones: |
| 612 | /// %0 = insertvalue { i8, i32 } undef, i8 %x, 0 |
| 613 | /// %1 = insertvalue { i8, i32 } %0, i8 %y, 0 |
| 614 | /// Here the second instruction inserts values at the same indices, as the |
| 615 | /// first one, making the first one redundant. |
| 616 | /// It should be transformed to: |
| 617 | /// %0 = insertvalue { i8, i32 } undef, i8 %y, 0 |
| 618 | Instruction *InstCombiner::visitInsertValueInst(InsertValueInst &I) { |
| 619 | bool IsRedundant = false; |
| 620 | ArrayRef<unsigned int> FirstIndices = I.getIndices(); |
| 621 | |
| 622 | // If there is a chain of insertvalue instructions (each of them except the |
| 623 | // last one has only one use and it's another insertvalue insn from this |
| 624 | // chain), check if any of the 'children' uses the same indices as the first |
| 625 | // instruction. In this case, the first one is redundant. |
| 626 | Value *V = &I; |
Michael Zolotukhin | 292d3ca | 2014-05-08 19:50:24 +0000 | [diff] [blame] | 627 | unsigned Depth = 0; |
Michael Zolotukhin | 7d6293a | 2014-05-07 14:30:18 +0000 | [diff] [blame] | 628 | while (V->hasOneUse() && Depth < 10) { |
| 629 | User *U = V->user_back(); |
Michael Zolotukhin | 292d3ca | 2014-05-08 19:50:24 +0000 | [diff] [blame] | 630 | auto UserInsInst = dyn_cast<InsertValueInst>(U); |
| 631 | if (!UserInsInst || U->getOperand(0) != V) |
Michael Zolotukhin | 7d6293a | 2014-05-07 14:30:18 +0000 | [diff] [blame] | 632 | break; |
Michael Zolotukhin | 7d6293a | 2014-05-07 14:30:18 +0000 | [diff] [blame] | 633 | if (UserInsInst->getIndices() == FirstIndices) { |
| 634 | IsRedundant = true; |
| 635 | break; |
| 636 | } |
| 637 | V = UserInsInst; |
| 638 | Depth++; |
| 639 | } |
| 640 | |
| 641 | if (IsRedundant) |
Sanjay Patel | 4b19880 | 2016-02-01 22:23:39 +0000 | [diff] [blame] | 642 | return replaceInstUsesWith(I, I.getOperand(0)); |
Michael Zolotukhin | 7d6293a | 2014-05-07 14:30:18 +0000 | [diff] [blame] | 643 | return nullptr; |
| 644 | } |
| 645 | |
Sanjay Patel | 521f19f | 2016-09-02 17:05:43 +0000 | [diff] [blame] | 646 | static bool isShuffleEquivalentToSelect(ShuffleVectorInst &Shuf) { |
| 647 | int MaskSize = Shuf.getMask()->getType()->getVectorNumElements(); |
| 648 | int VecSize = Shuf.getOperand(0)->getType()->getVectorNumElements(); |
| 649 | |
| 650 | // A vector select does not change the size of the operands. |
| 651 | if (MaskSize != VecSize) |
| 652 | return false; |
| 653 | |
| 654 | // Each mask element must be undefined or choose a vector element from one of |
| 655 | // the source operands without crossing vector lanes. |
| 656 | for (int i = 0; i != MaskSize; ++i) { |
| 657 | int Elt = Shuf.getMaskValue(i); |
| 658 | if (Elt != -1 && Elt != i && Elt != i + VecSize) |
| 659 | return false; |
| 660 | } |
| 661 | |
| 662 | return true; |
| 663 | } |
| 664 | |
Michael Kuperstein | cd7ad71 | 2016-12-28 00:18:08 +0000 | [diff] [blame] | 665 | // Turn a chain of inserts that splats a value into a canonical insert + shuffle |
| 666 | // splat. That is: |
| 667 | // insertelt(insertelt(insertelt(insertelt X, %k, 0), %k, 1), %k, 2) ... -> |
| 668 | // shufflevector(insertelt(X, %k, 0), undef, zero) |
| 669 | static Instruction *foldInsSequenceIntoBroadcast(InsertElementInst &InsElt) { |
| 670 | // We are interested in the last insert in a chain. So, if this insert |
| 671 | // has a single user, and that user is an insert, bail. |
| 672 | if (InsElt.hasOneUse() && isa<InsertElementInst>(InsElt.user_back())) |
| 673 | return nullptr; |
| 674 | |
| 675 | VectorType *VT = cast<VectorType>(InsElt.getType()); |
| 676 | int NumElements = VT->getNumElements(); |
| 677 | |
| 678 | // Do not try to do this for a one-element vector, since that's a nop, |
| 679 | // and will cause an inf-loop. |
| 680 | if (NumElements == 1) |
| 681 | return nullptr; |
| 682 | |
| 683 | Value *SplatVal = InsElt.getOperand(1); |
Fangrui Song | f78650a | 2018-07-30 19:41:25 +0000 | [diff] [blame] | 684 | InsertElementInst *CurrIE = &InsElt; |
Michael Kuperstein | cd7ad71 | 2016-12-28 00:18:08 +0000 | [diff] [blame] | 685 | SmallVector<bool, 16> ElementPresent(NumElements, false); |
Florian Hahn | b992fee | 2017-08-30 10:54:21 +0000 | [diff] [blame] | 686 | InsertElementInst *FirstIE = nullptr; |
Michael Kuperstein | cd7ad71 | 2016-12-28 00:18:08 +0000 | [diff] [blame] | 687 | |
| 688 | // Walk the chain backwards, keeping track of which indices we inserted into, |
| 689 | // until we hit something that isn't an insert of the splatted value. |
| 690 | while (CurrIE) { |
Sanjay Patel | 863d494 | 2017-11-27 18:19:32 +0000 | [diff] [blame] | 691 | auto *Idx = dyn_cast<ConstantInt>(CurrIE->getOperand(2)); |
Michael Kuperstein | cd7ad71 | 2016-12-28 00:18:08 +0000 | [diff] [blame] | 692 | if (!Idx || CurrIE->getOperand(1) != SplatVal) |
| 693 | return nullptr; |
| 694 | |
Sanjay Patel | 863d494 | 2017-11-27 18:19:32 +0000 | [diff] [blame] | 695 | auto *NextIE = dyn_cast<InsertElementInst>(CurrIE->getOperand(0)); |
Florian Hahn | b992fee | 2017-08-30 10:54:21 +0000 | [diff] [blame] | 696 | // Check none of the intermediate steps have any additional uses, except |
| 697 | // for the root insertelement instruction, which can be re-used, if it |
| 698 | // inserts at position 0. |
| 699 | if (CurrIE != &InsElt && |
| 700 | (!CurrIE->hasOneUse() && (NextIE != nullptr || !Idx->isZero()))) |
Michael Kuperstein | cd7ad71 | 2016-12-28 00:18:08 +0000 | [diff] [blame] | 701 | return nullptr; |
| 702 | |
| 703 | ElementPresent[Idx->getZExtValue()] = true; |
Florian Hahn | b992fee | 2017-08-30 10:54:21 +0000 | [diff] [blame] | 704 | FirstIE = CurrIE; |
| 705 | CurrIE = NextIE; |
Michael Kuperstein | cd7ad71 | 2016-12-28 00:18:08 +0000 | [diff] [blame] | 706 | } |
| 707 | |
| 708 | // Make sure we've seen an insert into every element. |
| 709 | if (llvm::any_of(ElementPresent, [](bool Present) { return !Present; })) |
| 710 | return nullptr; |
| 711 | |
| 712 | // All right, create the insert + shuffle. |
Florian Hahn | b992fee | 2017-08-30 10:54:21 +0000 | [diff] [blame] | 713 | Instruction *InsertFirst; |
| 714 | if (cast<ConstantInt>(FirstIE->getOperand(2))->isZero()) |
| 715 | InsertFirst = FirstIE; |
| 716 | else |
| 717 | InsertFirst = InsertElementInst::Create( |
| 718 | UndefValue::get(VT), SplatVal, |
| 719 | ConstantInt::get(Type::getInt32Ty(InsElt.getContext()), 0), |
| 720 | "", &InsElt); |
Michael Kuperstein | cd7ad71 | 2016-12-28 00:18:08 +0000 | [diff] [blame] | 721 | |
| 722 | Constant *ZeroMask = ConstantAggregateZero::get( |
| 723 | VectorType::get(Type::getInt32Ty(InsElt.getContext()), NumElements)); |
| 724 | |
| 725 | return new ShuffleVectorInst(InsertFirst, UndefValue::get(VT), ZeroMask); |
| 726 | } |
| 727 | |
Sanjay Patel | 2f602ce | 2017-03-22 17:10:44 +0000 | [diff] [blame] | 728 | /// If we have an insertelement instruction feeding into another insertelement |
| 729 | /// and the 2nd is inserting a constant into the vector, canonicalize that |
| 730 | /// constant insertion before the insertion of a variable: |
| 731 | /// |
| 732 | /// insertelement (insertelement X, Y, IdxC1), ScalarC, IdxC2 --> |
| 733 | /// insertelement (insertelement X, ScalarC, IdxC2), Y, IdxC1 |
| 734 | /// |
| 735 | /// This has the potential of eliminating the 2nd insertelement instruction |
| 736 | /// via constant folding of the scalar constant into a vector constant. |
| 737 | static Instruction *hoistInsEltConst(InsertElementInst &InsElt2, |
| 738 | InstCombiner::BuilderTy &Builder) { |
| 739 | auto *InsElt1 = dyn_cast<InsertElementInst>(InsElt2.getOperand(0)); |
| 740 | if (!InsElt1 || !InsElt1->hasOneUse()) |
| 741 | return nullptr; |
| 742 | |
| 743 | Value *X, *Y; |
| 744 | Constant *ScalarC; |
| 745 | ConstantInt *IdxC1, *IdxC2; |
| 746 | if (match(InsElt1->getOperand(0), m_Value(X)) && |
| 747 | match(InsElt1->getOperand(1), m_Value(Y)) && !isa<Constant>(Y) && |
| 748 | match(InsElt1->getOperand(2), m_ConstantInt(IdxC1)) && |
| 749 | match(InsElt2.getOperand(1), m_Constant(ScalarC)) && |
| 750 | match(InsElt2.getOperand(2), m_ConstantInt(IdxC2)) && IdxC1 != IdxC2) { |
| 751 | Value *NewInsElt1 = Builder.CreateInsertElement(X, ScalarC, IdxC2); |
| 752 | return InsertElementInst::Create(NewInsElt1, Y, IdxC1); |
| 753 | } |
| 754 | |
| 755 | return nullptr; |
| 756 | } |
| 757 | |
Alexey Bataev | fee9078 | 2016-09-23 09:14:08 +0000 | [diff] [blame] | 758 | /// insertelt (shufflevector X, CVec, Mask|insertelt X, C1, CIndex1), C, CIndex |
| 759 | /// --> shufflevector X, CVec', Mask' |
Sanjay Patel | 521f19f | 2016-09-02 17:05:43 +0000 | [diff] [blame] | 760 | static Instruction *foldConstantInsEltIntoShuffle(InsertElementInst &InsElt) { |
Alexey Bataev | fee9078 | 2016-09-23 09:14:08 +0000 | [diff] [blame] | 761 | auto *Inst = dyn_cast<Instruction>(InsElt.getOperand(0)); |
| 762 | // Bail out if the parent has more than one use. In that case, we'd be |
Sanjay Patel | 521f19f | 2016-09-02 17:05:43 +0000 | [diff] [blame] | 763 | // replacing the insertelt with a shuffle, and that's not a clear win. |
Alexey Bataev | fee9078 | 2016-09-23 09:14:08 +0000 | [diff] [blame] | 764 | if (!Inst || !Inst->hasOneUse()) |
Sanjay Patel | 521f19f | 2016-09-02 17:05:43 +0000 | [diff] [blame] | 765 | return nullptr; |
Alexey Bataev | fee9078 | 2016-09-23 09:14:08 +0000 | [diff] [blame] | 766 | if (auto *Shuf = dyn_cast<ShuffleVectorInst>(InsElt.getOperand(0))) { |
| 767 | // The shuffle must have a constant vector operand. The insertelt must have |
| 768 | // a constant scalar being inserted at a constant position in the vector. |
| 769 | Constant *ShufConstVec, *InsEltScalar; |
| 770 | uint64_t InsEltIndex; |
| 771 | if (!match(Shuf->getOperand(1), m_Constant(ShufConstVec)) || |
| 772 | !match(InsElt.getOperand(1), m_Constant(InsEltScalar)) || |
| 773 | !match(InsElt.getOperand(2), m_ConstantInt(InsEltIndex))) |
| 774 | return nullptr; |
Sanjay Patel | 521f19f | 2016-09-02 17:05:43 +0000 | [diff] [blame] | 775 | |
Alexey Bataev | fee9078 | 2016-09-23 09:14:08 +0000 | [diff] [blame] | 776 | // Adding an element to an arbitrary shuffle could be expensive, but a |
| 777 | // shuffle that selects elements from vectors without crossing lanes is |
| 778 | // assumed cheap. |
| 779 | // If we're just adding a constant into that shuffle, it will still be |
| 780 | // cheap. |
| 781 | if (!isShuffleEquivalentToSelect(*Shuf)) |
| 782 | return nullptr; |
Sanjay Patel | 521f19f | 2016-09-02 17:05:43 +0000 | [diff] [blame] | 783 | |
Alexey Bataev | fee9078 | 2016-09-23 09:14:08 +0000 | [diff] [blame] | 784 | // From the above 'select' check, we know that the mask has the same number |
| 785 | // of elements as the vector input operands. We also know that each constant |
| 786 | // input element is used in its lane and can not be used more than once by |
| 787 | // the shuffle. Therefore, replace the constant in the shuffle's constant |
| 788 | // vector with the insertelt constant. Replace the constant in the shuffle's |
| 789 | // mask vector with the insertelt index plus the length of the vector |
| 790 | // (because the constant vector operand of a shuffle is always the 2nd |
| 791 | // operand). |
| 792 | Constant *Mask = Shuf->getMask(); |
| 793 | unsigned NumElts = Mask->getType()->getVectorNumElements(); |
| 794 | SmallVector<Constant *, 16> NewShufElts(NumElts); |
| 795 | SmallVector<Constant *, 16> NewMaskElts(NumElts); |
| 796 | for (unsigned I = 0; I != NumElts; ++I) { |
| 797 | if (I == InsEltIndex) { |
| 798 | NewShufElts[I] = InsEltScalar; |
| 799 | Type *Int32Ty = Type::getInt32Ty(Shuf->getContext()); |
| 800 | NewMaskElts[I] = ConstantInt::get(Int32Ty, InsEltIndex + NumElts); |
| 801 | } else { |
| 802 | // Copy over the existing values. |
| 803 | NewShufElts[I] = ShufConstVec->getAggregateElement(I); |
| 804 | NewMaskElts[I] = Mask->getAggregateElement(I); |
| 805 | } |
Sanjay Patel | 521f19f | 2016-09-02 17:05:43 +0000 | [diff] [blame] | 806 | } |
Sanjay Patel | 521f19f | 2016-09-02 17:05:43 +0000 | [diff] [blame] | 807 | |
Alexey Bataev | fee9078 | 2016-09-23 09:14:08 +0000 | [diff] [blame] | 808 | // Create new operands for a shuffle that includes the constant of the |
| 809 | // original insertelt. The old shuffle will be dead now. |
| 810 | return new ShuffleVectorInst(Shuf->getOperand(0), |
| 811 | ConstantVector::get(NewShufElts), |
| 812 | ConstantVector::get(NewMaskElts)); |
| 813 | } else if (auto *IEI = dyn_cast<InsertElementInst>(Inst)) { |
| 814 | // Transform sequences of insertelements ops with constant data/indexes into |
| 815 | // a single shuffle op. |
| 816 | unsigned NumElts = InsElt.getType()->getNumElements(); |
| 817 | |
| 818 | uint64_t InsertIdx[2]; |
| 819 | Constant *Val[2]; |
| 820 | if (!match(InsElt.getOperand(2), m_ConstantInt(InsertIdx[0])) || |
| 821 | !match(InsElt.getOperand(1), m_Constant(Val[0])) || |
| 822 | !match(IEI->getOperand(2), m_ConstantInt(InsertIdx[1])) || |
| 823 | !match(IEI->getOperand(1), m_Constant(Val[1]))) |
| 824 | return nullptr; |
| 825 | SmallVector<Constant *, 16> Values(NumElts); |
| 826 | SmallVector<Constant *, 16> Mask(NumElts); |
| 827 | auto ValI = std::begin(Val); |
| 828 | // Generate new constant vector and mask. |
| 829 | // We have 2 values/masks from the insertelements instructions. Insert them |
| 830 | // into new value/mask vectors. |
| 831 | for (uint64_t I : InsertIdx) { |
| 832 | if (!Values[I]) { |
| 833 | assert(!Mask[I]); |
| 834 | Values[I] = *ValI; |
| 835 | Mask[I] = ConstantInt::get(Type::getInt32Ty(InsElt.getContext()), |
| 836 | NumElts + I); |
| 837 | } |
| 838 | ++ValI; |
| 839 | } |
| 840 | // Remaining values are filled with 'undef' values. |
| 841 | for (unsigned I = 0; I < NumElts; ++I) { |
| 842 | if (!Values[I]) { |
| 843 | assert(!Mask[I]); |
| 844 | Values[I] = UndefValue::get(InsElt.getType()->getElementType()); |
| 845 | Mask[I] = ConstantInt::get(Type::getInt32Ty(InsElt.getContext()), I); |
| 846 | } |
| 847 | } |
| 848 | // Create new operands for a shuffle that includes the constant of the |
| 849 | // original insertelt. |
| 850 | return new ShuffleVectorInst(IEI->getOperand(0), |
| 851 | ConstantVector::get(Values), |
| 852 | ConstantVector::get(Mask)); |
| 853 | } |
| 854 | return nullptr; |
Sanjay Patel | 521f19f | 2016-09-02 17:05:43 +0000 | [diff] [blame] | 855 | } |
| 856 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 857 | Instruction *InstCombiner::visitInsertElementInst(InsertElementInst &IE) { |
| 858 | Value *VecOp = IE.getOperand(0); |
| 859 | Value *ScalarOp = IE.getOperand(1); |
| 860 | Value *IdxOp = IE.getOperand(2); |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 861 | |
Igor Laevsky | e0edb66 | 2017-12-13 11:21:18 +0000 | [diff] [blame] | 862 | if (auto *V = SimplifyInsertElementInst( |
| 863 | VecOp, ScalarOp, IdxOp, SQ.getWithInstruction(&IE))) |
| 864 | return replaceInstUsesWith(IE, V); |
| 865 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 866 | // Inserting an undef or into an undefined place, remove this. |
| 867 | if (isa<UndefValue>(ScalarOp) || isa<UndefValue>(IdxOp)) |
Sanjay Patel | 4b19880 | 2016-02-01 22:23:39 +0000 | [diff] [blame] | 868 | replaceInstUsesWith(IE, VecOp); |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 869 | |
Sanjay Patel | 0522b0d | 2018-10-20 17:15:57 +0000 | [diff] [blame] | 870 | // If the inserted element was extracted from some other vector and both |
| 871 | // indexes are constant, try to turn this into a shuffle. |
| 872 | uint64_t InsertedIdx, ExtractedIdx; |
| 873 | Value *ExtVecOp; |
| 874 | if (match(IdxOp, m_ConstantInt(InsertedIdx)) && |
| 875 | match(ScalarOp, m_ExtractElement(m_Value(ExtVecOp), |
| 876 | m_ConstantInt(ExtractedIdx)))) { |
| 877 | unsigned NumInsertVectorElts = IE.getType()->getNumElements(); |
| 878 | unsigned NumExtractVectorElts = ExtVecOp->getType()->getVectorNumElements(); |
| 879 | if (ExtractedIdx >= NumExtractVectorElts) // Out of range extract. |
| 880 | return replaceInstUsesWith(IE, VecOp); |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 881 | |
Sanjay Patel | 0522b0d | 2018-10-20 17:15:57 +0000 | [diff] [blame] | 882 | if (InsertedIdx >= NumInsertVectorElts) // Out of range insert. |
| 883 | return replaceInstUsesWith(IE, UndefValue::get(IE.getType())); |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 884 | |
Sanjay Patel | 0522b0d | 2018-10-20 17:15:57 +0000 | [diff] [blame] | 885 | // If we are extracting a value from a vector, then inserting it right |
| 886 | // back into the same place, just use the input vector. |
| 887 | if (ExtVecOp == VecOp && ExtractedIdx == InsertedIdx) |
| 888 | return replaceInstUsesWith(IE, VecOp); |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 889 | |
Sanjay Patel | 0522b0d | 2018-10-20 17:15:57 +0000 | [diff] [blame] | 890 | // TODO: Looking at the user(s) to determine if this insert is a |
| 891 | // fold-to-shuffle opportunity does not match the usual instcombine |
| 892 | // constraints. We should decide if the transform is worthy based only |
| 893 | // on this instruction and its operands, but that may not work currently. |
| 894 | // |
| 895 | // Here, we are trying to avoid creating shuffles before reaching |
| 896 | // the end of a chain of extract-insert pairs. This is complicated because |
| 897 | // we do not generally form arbitrary shuffle masks in instcombine |
| 898 | // (because those may codegen poorly), but collectShuffleElements() does |
| 899 | // exactly that. |
| 900 | // |
| 901 | // The rules for determining what is an acceptable target-independent |
| 902 | // shuffle mask are fuzzy because they evolve based on the backend's |
| 903 | // capabilities and real-world impact. |
| 904 | auto isShuffleRootCandidate = [](InsertElementInst &Insert) { |
| 905 | if (!Insert.hasOneUse()) |
| 906 | return true; |
| 907 | auto *InsertUser = dyn_cast<InsertElementInst>(Insert.user_back()); |
| 908 | if (!InsertUser) |
| 909 | return true; |
| 910 | return false; |
| 911 | }; |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 912 | |
Sanjay Patel | 0522b0d | 2018-10-20 17:15:57 +0000 | [diff] [blame] | 913 | // Try to form a shuffle from a chain of extract-insert ops. |
| 914 | if (isShuffleRootCandidate(IE)) { |
| 915 | SmallVector<Constant*, 16> Mask; |
| 916 | ShuffleOps LR = collectShuffleElements(&IE, Mask, nullptr, *this); |
Sanjay Patel | 729c436 | 2018-10-20 16:25:55 +0000 | [diff] [blame] | 917 | |
Sanjay Patel | 0522b0d | 2018-10-20 17:15:57 +0000 | [diff] [blame] | 918 | // The proposed shuffle may be trivial, in which case we shouldn't |
| 919 | // perform the combine. |
| 920 | if (LR.first != &IE && LR.second != &IE) { |
| 921 | // We now have a shuffle of LHS, RHS, Mask. |
| 922 | if (LR.second == nullptr) |
| 923 | LR.second = UndefValue::get(LR.first->getType()); |
| 924 | return new ShuffleVectorInst(LR.first, LR.second, |
| 925 | ConstantVector::get(Mask)); |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 926 | } |
| 927 | } |
| 928 | } |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 929 | |
Craig Topper | 17b5568 | 2016-12-29 07:03:18 +0000 | [diff] [blame] | 930 | unsigned VWidth = VecOp->getType()->getVectorNumElements(); |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 931 | APInt UndefElts(VWidth, 0); |
| 932 | APInt AllOnesEltMask(APInt::getAllOnesValue(VWidth)); |
Eli Friedman | ef200db | 2011-02-19 22:42:40 +0000 | [diff] [blame] | 933 | if (Value *V = SimplifyDemandedVectorElts(&IE, AllOnesEltMask, UndefElts)) { |
| 934 | if (V != &IE) |
Sanjay Patel | 4b19880 | 2016-02-01 22:23:39 +0000 | [diff] [blame] | 935 | return replaceInstUsesWith(IE, V); |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 936 | return &IE; |
Eli Friedman | ef200db | 2011-02-19 22:42:40 +0000 | [diff] [blame] | 937 | } |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 938 | |
Sanjay Patel | 521f19f | 2016-09-02 17:05:43 +0000 | [diff] [blame] | 939 | if (Instruction *Shuf = foldConstantInsEltIntoShuffle(IE)) |
| 940 | return Shuf; |
| 941 | |
Craig Topper | bb4069e | 2017-07-07 23:16:26 +0000 | [diff] [blame] | 942 | if (Instruction *NewInsElt = hoistInsEltConst(IE, Builder)) |
Sanjay Patel | 2f602ce | 2017-03-22 17:10:44 +0000 | [diff] [blame] | 943 | return NewInsElt; |
| 944 | |
Michael Kuperstein | cd7ad71 | 2016-12-28 00:18:08 +0000 | [diff] [blame] | 945 | // Turn a sequence of inserts that broadcasts a scalar into a single |
| 946 | // insert + shufflevector. |
| 947 | if (Instruction *Broadcast = foldInsSequenceIntoBroadcast(IE)) |
| 948 | return Broadcast; |
| 949 | |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 950 | return nullptr; |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 951 | } |
| 952 | |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 953 | /// Return true if we can evaluate the specified expression tree if the vector |
| 954 | /// elements were shuffled in a different order. |
Sanjay Patel | 54d31ef | 2018-09-29 15:05:24 +0000 | [diff] [blame] | 955 | static bool canEvaluateShuffled(Value *V, ArrayRef<int> Mask, |
Nick Lewycky | 3f715e2 | 2013-06-01 20:51:31 +0000 | [diff] [blame] | 956 | unsigned Depth = 5) { |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 957 | // We can always reorder the elements of a constant. |
| 958 | if (isa<Constant>(V)) |
| 959 | return true; |
| 960 | |
| 961 | // We won't reorder vector arguments. No IPO here. |
| 962 | Instruction *I = dyn_cast<Instruction>(V); |
| 963 | if (!I) return false; |
| 964 | |
| 965 | // Two users may expect different orders of the elements. Don't try it. |
| 966 | if (!I->hasOneUse()) |
| 967 | return false; |
| 968 | |
| 969 | if (Depth == 0) return false; |
| 970 | |
| 971 | switch (I->getOpcode()) { |
| 972 | case Instruction::Add: |
| 973 | case Instruction::FAdd: |
| 974 | case Instruction::Sub: |
| 975 | case Instruction::FSub: |
| 976 | case Instruction::Mul: |
| 977 | case Instruction::FMul: |
| 978 | case Instruction::UDiv: |
| 979 | case Instruction::SDiv: |
| 980 | case Instruction::FDiv: |
| 981 | case Instruction::URem: |
| 982 | case Instruction::SRem: |
| 983 | case Instruction::FRem: |
| 984 | case Instruction::Shl: |
| 985 | case Instruction::LShr: |
| 986 | case Instruction::AShr: |
| 987 | case Instruction::And: |
| 988 | case Instruction::Or: |
| 989 | case Instruction::Xor: |
| 990 | case Instruction::ICmp: |
| 991 | case Instruction::FCmp: |
| 992 | case Instruction::Trunc: |
| 993 | case Instruction::ZExt: |
| 994 | case Instruction::SExt: |
| 995 | case Instruction::FPToUI: |
| 996 | case Instruction::FPToSI: |
| 997 | case Instruction::UIToFP: |
| 998 | case Instruction::SIToFP: |
| 999 | case Instruction::FPTrunc: |
| 1000 | case Instruction::FPExt: |
| 1001 | case Instruction::GetElementPtr: { |
Sanjay Patel | 26c119a | 2018-09-30 13:50:42 +0000 | [diff] [blame] | 1002 | // Bail out if we would create longer vector ops. We could allow creating |
| 1003 | // longer vector ops, but that may result in more expensive codegen. We |
| 1004 | // would also need to limit the transform to avoid undefined behavior for |
| 1005 | // integer div/rem. |
| 1006 | Type *ITy = I->getType(); |
| 1007 | if (ITy->isVectorTy() && Mask.size() > ITy->getVectorNumElements()) |
| 1008 | return false; |
Sanjay Patel | 4e2875314 | 2015-11-16 22:16:52 +0000 | [diff] [blame] | 1009 | for (Value *Operand : I->operands()) { |
Sanjay Patel | 54d31ef | 2018-09-29 15:05:24 +0000 | [diff] [blame] | 1010 | if (!canEvaluateShuffled(Operand, Mask, Depth - 1)) |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 1011 | return false; |
| 1012 | } |
| 1013 | return true; |
| 1014 | } |
| 1015 | case Instruction::InsertElement: { |
| 1016 | ConstantInt *CI = dyn_cast<ConstantInt>(I->getOperand(2)); |
| 1017 | if (!CI) return false; |
| 1018 | int ElementNumber = CI->getLimitedValue(); |
| 1019 | |
| 1020 | // Verify that 'CI' does not occur twice in Mask. A single 'insertelement' |
| 1021 | // can't put an element into multiple indices. |
| 1022 | bool SeenOnce = false; |
| 1023 | for (int i = 0, e = Mask.size(); i != e; ++i) { |
| 1024 | if (Mask[i] == ElementNumber) { |
| 1025 | if (SeenOnce) |
| 1026 | return false; |
| 1027 | SeenOnce = true; |
| 1028 | } |
| 1029 | } |
Sanjay Patel | 54d31ef | 2018-09-29 15:05:24 +0000 | [diff] [blame] | 1030 | return canEvaluateShuffled(I->getOperand(0), Mask, Depth - 1); |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 1031 | } |
| 1032 | } |
| 1033 | return false; |
| 1034 | } |
| 1035 | |
| 1036 | /// Rebuild a new instruction just like 'I' but with the new operands given. |
| 1037 | /// In the event of type mismatch, the type of the operands is correct. |
Sanjay Patel | 431e114 | 2015-11-17 17:24:08 +0000 | [diff] [blame] | 1038 | static Value *buildNew(Instruction *I, ArrayRef<Value*> NewOps) { |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 1039 | // We don't want to use the IRBuilder here because we want the replacement |
| 1040 | // instructions to appear next to 'I', not the builder's insertion point. |
| 1041 | switch (I->getOpcode()) { |
| 1042 | case Instruction::Add: |
| 1043 | case Instruction::FAdd: |
| 1044 | case Instruction::Sub: |
| 1045 | case Instruction::FSub: |
| 1046 | case Instruction::Mul: |
| 1047 | case Instruction::FMul: |
| 1048 | case Instruction::UDiv: |
| 1049 | case Instruction::SDiv: |
| 1050 | case Instruction::FDiv: |
| 1051 | case Instruction::URem: |
| 1052 | case Instruction::SRem: |
| 1053 | case Instruction::FRem: |
| 1054 | case Instruction::Shl: |
| 1055 | case Instruction::LShr: |
| 1056 | case Instruction::AShr: |
| 1057 | case Instruction::And: |
| 1058 | case Instruction::Or: |
| 1059 | case Instruction::Xor: { |
| 1060 | BinaryOperator *BO = cast<BinaryOperator>(I); |
| 1061 | assert(NewOps.size() == 2 && "binary operator with #ops != 2"); |
| 1062 | BinaryOperator *New = |
| 1063 | BinaryOperator::Create(cast<BinaryOperator>(I)->getOpcode(), |
| 1064 | NewOps[0], NewOps[1], "", BO); |
| 1065 | if (isa<OverflowingBinaryOperator>(BO)) { |
| 1066 | New->setHasNoUnsignedWrap(BO->hasNoUnsignedWrap()); |
| 1067 | New->setHasNoSignedWrap(BO->hasNoSignedWrap()); |
| 1068 | } |
| 1069 | if (isa<PossiblyExactOperator>(BO)) { |
| 1070 | New->setIsExact(BO->isExact()); |
| 1071 | } |
Owen Anderson | 48b842e | 2014-01-18 00:48:14 +0000 | [diff] [blame] | 1072 | if (isa<FPMathOperator>(BO)) |
| 1073 | New->copyFastMathFlags(I); |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 1074 | return New; |
| 1075 | } |
| 1076 | case Instruction::ICmp: |
| 1077 | assert(NewOps.size() == 2 && "icmp with #ops != 2"); |
| 1078 | return new ICmpInst(I, cast<ICmpInst>(I)->getPredicate(), |
| 1079 | NewOps[0], NewOps[1]); |
| 1080 | case Instruction::FCmp: |
| 1081 | assert(NewOps.size() == 2 && "fcmp with #ops != 2"); |
| 1082 | return new FCmpInst(I, cast<FCmpInst>(I)->getPredicate(), |
| 1083 | NewOps[0], NewOps[1]); |
| 1084 | case Instruction::Trunc: |
| 1085 | case Instruction::ZExt: |
| 1086 | case Instruction::SExt: |
| 1087 | case Instruction::FPToUI: |
| 1088 | case Instruction::FPToSI: |
| 1089 | case Instruction::UIToFP: |
| 1090 | case Instruction::SIToFP: |
| 1091 | case Instruction::FPTrunc: |
| 1092 | case Instruction::FPExt: { |
| 1093 | // It's possible that the mask has a different number of elements from |
| 1094 | // the original cast. We recompute the destination type to match the mask. |
| 1095 | Type *DestTy = |
| 1096 | VectorType::get(I->getType()->getScalarType(), |
| 1097 | NewOps[0]->getType()->getVectorNumElements()); |
| 1098 | assert(NewOps.size() == 1 && "cast with #ops != 1"); |
| 1099 | return CastInst::Create(cast<CastInst>(I)->getOpcode(), NewOps[0], DestTy, |
| 1100 | "", I); |
| 1101 | } |
| 1102 | case Instruction::GetElementPtr: { |
| 1103 | Value *Ptr = NewOps[0]; |
| 1104 | ArrayRef<Value*> Idx = NewOps.slice(1); |
David Blaikie | 22319eb | 2015-03-14 19:24:04 +0000 | [diff] [blame] | 1105 | GetElementPtrInst *GEP = GetElementPtrInst::Create( |
| 1106 | cast<GetElementPtrInst>(I)->getSourceElementType(), Ptr, Idx, "", I); |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 1107 | GEP->setIsInBounds(cast<GetElementPtrInst>(I)->isInBounds()); |
| 1108 | return GEP; |
| 1109 | } |
| 1110 | } |
| 1111 | llvm_unreachable("failed to rebuild vector instructions"); |
| 1112 | } |
| 1113 | |
Sanjay Patel | 54d31ef | 2018-09-29 15:05:24 +0000 | [diff] [blame] | 1114 | static Value *evaluateInDifferentElementOrder(Value *V, ArrayRef<int> Mask) { |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 1115 | // Mask.size() does not need to be equal to the number of vector elements. |
| 1116 | |
| 1117 | assert(V->getType()->isVectorTy() && "can't reorder non-vector elements"); |
Sanjay Patel | ce36b03 | 2017-10-09 17:54:46 +0000 | [diff] [blame] | 1118 | Type *EltTy = V->getType()->getScalarType(); |
Sanjay Patel | 54d31ef | 2018-09-29 15:05:24 +0000 | [diff] [blame] | 1119 | Type *I32Ty = IntegerType::getInt32Ty(V->getContext()); |
Sanjay Patel | ce36b03 | 2017-10-09 17:54:46 +0000 | [diff] [blame] | 1120 | if (isa<UndefValue>(V)) |
| 1121 | return UndefValue::get(VectorType::get(EltTy, Mask.size())); |
| 1122 | |
| 1123 | if (isa<ConstantAggregateZero>(V)) |
| 1124 | return ConstantAggregateZero::get(VectorType::get(EltTy, Mask.size())); |
| 1125 | |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 1126 | if (Constant *C = dyn_cast<Constant>(V)) { |
| 1127 | SmallVector<Constant *, 16> MaskValues; |
| 1128 | for (int i = 0, e = Mask.size(); i != e; ++i) { |
| 1129 | if (Mask[i] == -1) |
Sanjay Patel | 54d31ef | 2018-09-29 15:05:24 +0000 | [diff] [blame] | 1130 | MaskValues.push_back(UndefValue::get(I32Ty)); |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 1131 | else |
Sanjay Patel | 54d31ef | 2018-09-29 15:05:24 +0000 | [diff] [blame] | 1132 | MaskValues.push_back(ConstantInt::get(I32Ty, Mask[i])); |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 1133 | } |
| 1134 | return ConstantExpr::getShuffleVector(C, UndefValue::get(C->getType()), |
| 1135 | ConstantVector::get(MaskValues)); |
| 1136 | } |
| 1137 | |
| 1138 | Instruction *I = cast<Instruction>(V); |
| 1139 | switch (I->getOpcode()) { |
| 1140 | case Instruction::Add: |
| 1141 | case Instruction::FAdd: |
| 1142 | case Instruction::Sub: |
| 1143 | case Instruction::FSub: |
| 1144 | case Instruction::Mul: |
| 1145 | case Instruction::FMul: |
| 1146 | case Instruction::UDiv: |
| 1147 | case Instruction::SDiv: |
| 1148 | case Instruction::FDiv: |
| 1149 | case Instruction::URem: |
| 1150 | case Instruction::SRem: |
| 1151 | case Instruction::FRem: |
| 1152 | case Instruction::Shl: |
| 1153 | case Instruction::LShr: |
| 1154 | case Instruction::AShr: |
| 1155 | case Instruction::And: |
| 1156 | case Instruction::Or: |
| 1157 | case Instruction::Xor: |
| 1158 | case Instruction::ICmp: |
| 1159 | case Instruction::FCmp: |
| 1160 | case Instruction::Trunc: |
| 1161 | case Instruction::ZExt: |
| 1162 | case Instruction::SExt: |
| 1163 | case Instruction::FPToUI: |
| 1164 | case Instruction::FPToSI: |
| 1165 | case Instruction::UIToFP: |
| 1166 | case Instruction::SIToFP: |
| 1167 | case Instruction::FPTrunc: |
| 1168 | case Instruction::FPExt: |
| 1169 | case Instruction::Select: |
| 1170 | case Instruction::GetElementPtr: { |
| 1171 | SmallVector<Value*, 8> NewOps; |
| 1172 | bool NeedsRebuild = (Mask.size() != I->getType()->getVectorNumElements()); |
| 1173 | for (int i = 0, e = I->getNumOperands(); i != e; ++i) { |
Mikael Holmen | 150a7ec | 2019-04-01 14:10:10 +0000 | [diff] [blame^] | 1174 | Value *V; |
| 1175 | // Recursively call evaluateInDifferentElementOrder on vector arguments |
| 1176 | // as well. E.g. GetElementPtr may have scalar operands even if the |
| 1177 | // return value is a vector, so we need to examine the operand type. |
| 1178 | if (I->getOperand(i)->getType()->isVectorTy()) |
| 1179 | V = evaluateInDifferentElementOrder(I->getOperand(i), Mask); |
| 1180 | else |
| 1181 | V = I->getOperand(i); |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 1182 | NewOps.push_back(V); |
| 1183 | NeedsRebuild |= (V != I->getOperand(i)); |
| 1184 | } |
| 1185 | if (NeedsRebuild) { |
Sanjay Patel | 431e114 | 2015-11-17 17:24:08 +0000 | [diff] [blame] | 1186 | return buildNew(I, NewOps); |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 1187 | } |
| 1188 | return I; |
| 1189 | } |
| 1190 | case Instruction::InsertElement: { |
| 1191 | int Element = cast<ConstantInt>(I->getOperand(2))->getLimitedValue(); |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 1192 | |
| 1193 | // The insertelement was inserting at Element. Figure out which element |
| 1194 | // that becomes after shuffling. The answer is guaranteed to be unique |
| 1195 | // by CanEvaluateShuffled. |
Nick Lewycky | 3f715e2 | 2013-06-01 20:51:31 +0000 | [diff] [blame] | 1196 | bool Found = false; |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 1197 | int Index = 0; |
Nick Lewycky | 3f715e2 | 2013-06-01 20:51:31 +0000 | [diff] [blame] | 1198 | for (int e = Mask.size(); Index != e; ++Index) { |
| 1199 | if (Mask[Index] == Element) { |
| 1200 | Found = true; |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 1201 | break; |
Nick Lewycky | 3f715e2 | 2013-06-01 20:51:31 +0000 | [diff] [blame] | 1202 | } |
| 1203 | } |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 1204 | |
Hao Liu | 26abebb | 2014-01-08 03:06:15 +0000 | [diff] [blame] | 1205 | // If element is not in Mask, no need to handle the operand 1 (element to |
| 1206 | // be inserted). Just evaluate values in operand 0 according to Mask. |
Nick Lewycky | 3f715e2 | 2013-06-01 20:51:31 +0000 | [diff] [blame] | 1207 | if (!Found) |
Sanjay Patel | 54d31ef | 2018-09-29 15:05:24 +0000 | [diff] [blame] | 1208 | return evaluateInDifferentElementOrder(I->getOperand(0), Mask); |
Joey Gouly | a3250f2 | 2013-07-12 23:08:06 +0000 | [diff] [blame] | 1209 | |
Sanjay Patel | 54d31ef | 2018-09-29 15:05:24 +0000 | [diff] [blame] | 1210 | Value *V = evaluateInDifferentElementOrder(I->getOperand(0), Mask); |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 1211 | return InsertElementInst::Create(V, I->getOperand(1), |
Sanjay Patel | 54d31ef | 2018-09-29 15:05:24 +0000 | [diff] [blame] | 1212 | ConstantInt::get(I32Ty, Index), "", I); |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 1213 | } |
| 1214 | } |
| 1215 | llvm_unreachable("failed to reorder elements of vector instruction!"); |
| 1216 | } |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 1217 | |
Sanjay Patel | 431e114 | 2015-11-17 17:24:08 +0000 | [diff] [blame] | 1218 | static void recognizeIdentityMask(const SmallVectorImpl<int> &Mask, |
Serge Pavlov | 9ef66a8 | 2014-05-11 08:46:12 +0000 | [diff] [blame] | 1219 | bool &isLHSID, bool &isRHSID) { |
| 1220 | isLHSID = isRHSID = true; |
| 1221 | |
| 1222 | for (unsigned i = 0, e = Mask.size(); i != e; ++i) { |
| 1223 | if (Mask[i] < 0) continue; // Ignore undef values. |
| 1224 | // Is this an identity shuffle of the LHS value? |
| 1225 | isLHSID &= (Mask[i] == (int)i); |
| 1226 | |
| 1227 | // Is this an identity shuffle of the RHS value? |
| 1228 | isRHSID &= (Mask[i]-e == i); |
| 1229 | } |
| 1230 | } |
| 1231 | |
JF Bastien | d52c990 | 2015-02-25 22:30:51 +0000 | [diff] [blame] | 1232 | // Returns true if the shuffle is extracting a contiguous range of values from |
| 1233 | // LHS, for example: |
| 1234 | // +--+--+--+--+--+--+--+--+--+--+--+--+--+--+--+--+ |
| 1235 | // Input: |AA|BB|CC|DD|EE|FF|GG|HH|II|JJ|KK|LL|MM|NN|OO|PP| |
| 1236 | // Shuffles to: |EE|FF|GG|HH| |
| 1237 | // +--+--+--+--+ |
| 1238 | static bool isShuffleExtractingFromLHS(ShuffleVectorInst &SVI, |
| 1239 | SmallVector<int, 16> &Mask) { |
Craig Topper | 17b5568 | 2016-12-29 07:03:18 +0000 | [diff] [blame] | 1240 | unsigned LHSElems = SVI.getOperand(0)->getType()->getVectorNumElements(); |
JF Bastien | d52c990 | 2015-02-25 22:30:51 +0000 | [diff] [blame] | 1241 | unsigned MaskElems = Mask.size(); |
| 1242 | unsigned BegIdx = Mask.front(); |
| 1243 | unsigned EndIdx = Mask.back(); |
| 1244 | if (BegIdx > EndIdx || EndIdx >= LHSElems || EndIdx - BegIdx != MaskElems - 1) |
| 1245 | return false; |
| 1246 | for (unsigned I = 0; I != MaskElems; ++I) |
| 1247 | if (static_cast<unsigned>(Mask[I]) != BegIdx + I) |
| 1248 | return false; |
| 1249 | return true; |
| 1250 | } |
| 1251 | |
Sanjay Patel | b999d74 | 2018-07-02 17:42:29 +0000 | [diff] [blame] | 1252 | /// These are the ingredients in an alternate form binary operator as described |
| 1253 | /// below. |
| 1254 | struct BinopElts { |
| 1255 | BinaryOperator::BinaryOps Opcode; |
| 1256 | Value *Op0; |
| 1257 | Value *Op1; |
| 1258 | BinopElts(BinaryOperator::BinaryOps Opc = (BinaryOperator::BinaryOps)0, |
| 1259 | Value *V0 = nullptr, Value *V1 = nullptr) : |
| 1260 | Opcode(Opc), Op0(V0), Op1(V1) {} |
| 1261 | operator bool() const { return Opcode != 0; } |
| 1262 | }; |
| 1263 | |
| 1264 | /// Binops may be transformed into binops with different opcodes and operands. |
| 1265 | /// Reverse the usual canonicalization to enable folds with the non-canonical |
| 1266 | /// form of the binop. If a transform is possible, return the elements of the |
| 1267 | /// new binop. If not, return invalid elements. |
| 1268 | static BinopElts getAlternateBinop(BinaryOperator *BO, const DataLayout &DL) { |
| 1269 | Value *BO0 = BO->getOperand(0), *BO1 = BO->getOperand(1); |
| 1270 | Type *Ty = BO->getType(); |
| 1271 | switch (BO->getOpcode()) { |
| 1272 | case Instruction::Shl: { |
| 1273 | // shl X, C --> mul X, (1 << C) |
| 1274 | Constant *C; |
| 1275 | if (match(BO1, m_Constant(C))) { |
| 1276 | Constant *ShlOne = ConstantExpr::getShl(ConstantInt::get(Ty, 1), C); |
| 1277 | return { Instruction::Mul, BO0, ShlOne }; |
| 1278 | } |
| 1279 | break; |
| 1280 | } |
| 1281 | case Instruction::Or: { |
| 1282 | // or X, C --> add X, C (when X and C have no common bits set) |
| 1283 | const APInt *C; |
| 1284 | if (match(BO1, m_APInt(C)) && MaskedValueIsZero(BO0, *C, DL)) |
| 1285 | return { Instruction::Add, BO0, BO1 }; |
| 1286 | break; |
| 1287 | } |
| 1288 | default: |
| 1289 | break; |
| 1290 | } |
| 1291 | return {}; |
| 1292 | } |
| 1293 | |
Sanjay Patel | 3074b9e | 2018-07-03 13:44:22 +0000 | [diff] [blame] | 1294 | static Instruction *foldSelectShuffleWith1Binop(ShuffleVectorInst &Shuf) { |
| 1295 | assert(Shuf.isSelect() && "Must have select-equivalent shuffle"); |
| 1296 | |
| 1297 | // Are we shuffling together some value and that same value after it has been |
| 1298 | // modified by a binop with a constant? |
| 1299 | Value *Op0 = Shuf.getOperand(0), *Op1 = Shuf.getOperand(1); |
| 1300 | Constant *C; |
| 1301 | bool Op0IsBinop; |
| 1302 | if (match(Op0, m_BinOp(m_Specific(Op1), m_Constant(C)))) |
| 1303 | Op0IsBinop = true; |
| 1304 | else if (match(Op1, m_BinOp(m_Specific(Op0), m_Constant(C)))) |
| 1305 | Op0IsBinop = false; |
| 1306 | else |
| 1307 | return nullptr; |
| 1308 | |
Sanjay Patel | 3074b9e | 2018-07-03 13:44:22 +0000 | [diff] [blame] | 1309 | // The identity constant for a binop leaves a variable operand unchanged. For |
| 1310 | // a vector, this is a splat of something like 0, -1, or 1. |
| 1311 | // If there's no identity constant for this binop, we're done. |
Sanjay Patel | 509a1e7 | 2018-07-10 15:12:31 +0000 | [diff] [blame] | 1312 | auto *BO = cast<BinaryOperator>(Op0IsBinop ? Op0 : Op1); |
Sanjay Patel | 3074b9e | 2018-07-03 13:44:22 +0000 | [diff] [blame] | 1313 | BinaryOperator::BinaryOps BOpcode = BO->getOpcode(); |
Sanjay Patel | 509a1e7 | 2018-07-10 15:12:31 +0000 | [diff] [blame] | 1314 | Constant *IdC = ConstantExpr::getBinOpIdentity(BOpcode, Shuf.getType(), true); |
Sanjay Patel | 3074b9e | 2018-07-03 13:44:22 +0000 | [diff] [blame] | 1315 | if (!IdC) |
| 1316 | return nullptr; |
| 1317 | |
| 1318 | // Shuffle identity constants into the lanes that return the original value. |
| 1319 | // Example: shuf (mul X, {-1,-2,-3,-4}), X, {0,5,6,3} --> mul X, {-1,1,1,-4} |
| 1320 | // Example: shuf X, (add X, {-1,-2,-3,-4}), {0,1,6,7} --> add X, {0,0,-3,-4} |
| 1321 | // The existing binop constant vector remains in the same operand position. |
| 1322 | Constant *Mask = Shuf.getMask(); |
| 1323 | Constant *NewC = Op0IsBinop ? ConstantExpr::getShuffleVector(C, IdC, Mask) : |
| 1324 | ConstantExpr::getShuffleVector(IdC, C, Mask); |
| 1325 | |
Sanjay Patel | 509a1e7 | 2018-07-10 15:12:31 +0000 | [diff] [blame] | 1326 | bool MightCreatePoisonOrUB = |
| 1327 | Mask->containsUndefElement() && |
| 1328 | (Instruction::isIntDivRem(BOpcode) || Instruction::isShift(BOpcode)); |
| 1329 | if (MightCreatePoisonOrUB) |
| 1330 | NewC = getSafeVectorConstantForBinop(BOpcode, NewC, true); |
| 1331 | |
Sanjay Patel | 3074b9e | 2018-07-03 13:44:22 +0000 | [diff] [blame] | 1332 | // shuf (bop X, C), X, M --> bop X, C' |
| 1333 | // shuf X, (bop X, C), M --> bop X, C' |
Sanjay Patel | 509a1e7 | 2018-07-10 15:12:31 +0000 | [diff] [blame] | 1334 | Value *X = Op0IsBinop ? Op1 : Op0; |
Sanjay Patel | 3074b9e | 2018-07-03 13:44:22 +0000 | [diff] [blame] | 1335 | Instruction *NewBO = BinaryOperator::Create(BOpcode, X, NewC); |
| 1336 | NewBO->copyIRFlags(BO); |
Sanjay Patel | 3333106 | 2018-07-10 14:27:55 +0000 | [diff] [blame] | 1337 | |
| 1338 | // An undef shuffle mask element may propagate as an undef constant element in |
| 1339 | // the new binop. That would produce poison where the original code might not. |
Sanjay Patel | 509a1e7 | 2018-07-10 15:12:31 +0000 | [diff] [blame] | 1340 | // If we already made a safe constant, then there's no danger. |
| 1341 | if (Mask->containsUndefElement() && !MightCreatePoisonOrUB) |
Sanjay Patel | 3333106 | 2018-07-10 14:27:55 +0000 | [diff] [blame] | 1342 | NewBO->dropPoisonGeneratingFlags(); |
Sanjay Patel | 3074b9e | 2018-07-03 13:44:22 +0000 | [diff] [blame] | 1343 | return NewBO; |
| 1344 | } |
| 1345 | |
Sanjay Patel | b999d74 | 2018-07-02 17:42:29 +0000 | [diff] [blame] | 1346 | /// Try to fold shuffles that are the equivalent of a vector select. |
Sanjay Patel | da66753 | 2018-06-29 13:44:06 +0000 | [diff] [blame] | 1347 | static Instruction *foldSelectShuffle(ShuffleVectorInst &Shuf, |
Sanjay Patel | b999d74 | 2018-07-02 17:42:29 +0000 | [diff] [blame] | 1348 | InstCombiner::BuilderTy &Builder, |
| 1349 | const DataLayout &DL) { |
Sanjay Patel | a76b700 | 2018-06-21 20:15:09 +0000 | [diff] [blame] | 1350 | if (!Shuf.isSelect()) |
| 1351 | return nullptr; |
| 1352 | |
Sanjay Patel | b276dd1 | 2019-03-31 15:01:30 +0000 | [diff] [blame] | 1353 | // Canonicalize to choose from operand 0 first. |
| 1354 | unsigned NumElts = Shuf.getType()->getVectorNumElements(); |
| 1355 | if (Shuf.getMaskValue(0) >= (int)NumElts) { |
| 1356 | assert(!isa<UndefValue>(Shuf.getOperand(1)) && |
| 1357 | "Not expecting undef shuffle operand with select mask"); |
| 1358 | Shuf.commute(); |
| 1359 | return &Shuf; |
| 1360 | } |
| 1361 | |
Sanjay Patel | 3074b9e | 2018-07-03 13:44:22 +0000 | [diff] [blame] | 1362 | if (Instruction *I = foldSelectShuffleWith1Binop(Shuf)) |
| 1363 | return I; |
| 1364 | |
Sanjay Patel | a76b700 | 2018-06-21 20:15:09 +0000 | [diff] [blame] | 1365 | BinaryOperator *B0, *B1; |
| 1366 | if (!match(Shuf.getOperand(0), m_BinOp(B0)) || |
| 1367 | !match(Shuf.getOperand(1), m_BinOp(B1))) |
| 1368 | return nullptr; |
| 1369 | |
Sanjay Patel | da66753 | 2018-06-29 13:44:06 +0000 | [diff] [blame] | 1370 | Value *X, *Y; |
Sanjay Patel | a76b700 | 2018-06-21 20:15:09 +0000 | [diff] [blame] | 1371 | Constant *C0, *C1; |
Sanjay Patel | a52963b | 2018-06-22 12:46:16 +0000 | [diff] [blame] | 1372 | bool ConstantsAreOp1; |
| 1373 | if (match(B0, m_BinOp(m_Value(X), m_Constant(C0))) && |
Sanjay Patel | da66753 | 2018-06-29 13:44:06 +0000 | [diff] [blame] | 1374 | match(B1, m_BinOp(m_Value(Y), m_Constant(C1)))) |
Sanjay Patel | a52963b | 2018-06-22 12:46:16 +0000 | [diff] [blame] | 1375 | ConstantsAreOp1 = true; |
| 1376 | else if (match(B0, m_BinOp(m_Constant(C0), m_Value(X))) && |
Sanjay Patel | da66753 | 2018-06-29 13:44:06 +0000 | [diff] [blame] | 1377 | match(B1, m_BinOp(m_Constant(C1), m_Value(Y)))) |
Sanjay Patel | a52963b | 2018-06-22 12:46:16 +0000 | [diff] [blame] | 1378 | ConstantsAreOp1 = false; |
| 1379 | else |
Sanjay Patel | a76b700 | 2018-06-21 20:15:09 +0000 | [diff] [blame] | 1380 | return nullptr; |
| 1381 | |
Sanjay Patel | 57bda36 | 2018-06-28 17:48:04 +0000 | [diff] [blame] | 1382 | // We need matching binops to fold the lanes together. |
| 1383 | BinaryOperator::BinaryOps Opc0 = B0->getOpcode(); |
| 1384 | BinaryOperator::BinaryOps Opc1 = B1->getOpcode(); |
| 1385 | bool DropNSW = false; |
| 1386 | if (ConstantsAreOp1 && Opc0 != Opc1) { |
Sanjay Patel | 57bda36 | 2018-06-28 17:48:04 +0000 | [diff] [blame] | 1387 | // TODO: We drop "nsw" if shift is converted into multiply because it may |
| 1388 | // not be correct when the shift amount is BitWidth - 1. We could examine |
| 1389 | // each vector element to determine if it is safe to keep that flag. |
Sanjay Patel | b999d74 | 2018-07-02 17:42:29 +0000 | [diff] [blame] | 1390 | if (Opc0 == Instruction::Shl || Opc1 == Instruction::Shl) |
Sanjay Patel | 57bda36 | 2018-06-28 17:48:04 +0000 | [diff] [blame] | 1391 | DropNSW = true; |
Sanjay Patel | b999d74 | 2018-07-02 17:42:29 +0000 | [diff] [blame] | 1392 | if (BinopElts AltB0 = getAlternateBinop(B0, DL)) { |
| 1393 | assert(isa<Constant>(AltB0.Op1) && "Expecting constant with alt binop"); |
| 1394 | Opc0 = AltB0.Opcode; |
| 1395 | C0 = cast<Constant>(AltB0.Op1); |
| 1396 | } else if (BinopElts AltB1 = getAlternateBinop(B1, DL)) { |
| 1397 | assert(isa<Constant>(AltB1.Op1) && "Expecting constant with alt binop"); |
| 1398 | Opc1 = AltB1.Opcode; |
| 1399 | C1 = cast<Constant>(AltB1.Op1); |
Sanjay Patel | 57bda36 | 2018-06-28 17:48:04 +0000 | [diff] [blame] | 1400 | } |
| 1401 | } |
| 1402 | |
| 1403 | if (Opc0 != Opc1) |
Sanjay Patel | 4784e15 | 2018-06-21 23:56:59 +0000 | [diff] [blame] | 1404 | return nullptr; |
| 1405 | |
Sanjay Patel | 57bda36 | 2018-06-28 17:48:04 +0000 | [diff] [blame] | 1406 | // The opcodes must be the same. Use a new name to make that clear. |
| 1407 | BinaryOperator::BinaryOps BOpc = Opc0; |
| 1408 | |
Sanjay Patel | 06ea420 | 2018-07-10 13:33:26 +0000 | [diff] [blame] | 1409 | // Select the constant elements needed for the single binop. |
| 1410 | Constant *Mask = Shuf.getMask(); |
| 1411 | Constant *NewC = ConstantExpr::getShuffleVector(C0, C1, Mask); |
| 1412 | |
Sanjay Patel | 5bd3664 | 2018-07-09 13:21:46 +0000 | [diff] [blame] | 1413 | // We are moving a binop after a shuffle. When a shuffle has an undefined |
| 1414 | // mask element, the result is undefined, but it is not poison or undefined |
| 1415 | // behavior. That is not necessarily true for div/rem/shift. |
Sanjay Patel | 5bd3664 | 2018-07-09 13:21:46 +0000 | [diff] [blame] | 1416 | bool MightCreatePoisonOrUB = |
| 1417 | Mask->containsUndefElement() && |
| 1418 | (Instruction::isIntDivRem(BOpc) || Instruction::isShift(BOpc)); |
Sanjay Patel | 06ea420 | 2018-07-10 13:33:26 +0000 | [diff] [blame] | 1419 | if (MightCreatePoisonOrUB) |
| 1420 | NewC = getSafeVectorConstantForBinop(BOpc, NewC, ConstantsAreOp1); |
Sanjay Patel | 5bd3664 | 2018-07-09 13:21:46 +0000 | [diff] [blame] | 1421 | |
Sanjay Patel | da66753 | 2018-06-29 13:44:06 +0000 | [diff] [blame] | 1422 | Value *V; |
| 1423 | if (X == Y) { |
| 1424 | // Remove a binop and the shuffle by rearranging the constant: |
| 1425 | // shuffle (op V, C0), (op V, C1), M --> op V, C' |
| 1426 | // shuffle (op C0, V), (op C1, V), M --> op C', V |
| 1427 | V = X; |
Sanjay Patel | 5bd3664 | 2018-07-09 13:21:46 +0000 | [diff] [blame] | 1428 | } else { |
Sanjay Patel | da66753 | 2018-06-29 13:44:06 +0000 | [diff] [blame] | 1429 | // If there are 2 different variable operands, we must create a new shuffle |
| 1430 | // (select) first, so check uses to ensure that we don't end up with more |
| 1431 | // instructions than we started with. |
Sanjay Patel | 5bd3664 | 2018-07-09 13:21:46 +0000 | [diff] [blame] | 1432 | if (!B0->hasOneUse() && !B1->hasOneUse()) |
| 1433 | return nullptr; |
| 1434 | |
Sanjay Patel | 06ea420 | 2018-07-10 13:33:26 +0000 | [diff] [blame] | 1435 | // If we use the original shuffle mask and op1 is *variable*, we would be |
| 1436 | // putting an undef into operand 1 of div/rem/shift. This is either UB or |
| 1437 | // poison. We do not have to guard against UB when *constants* are op1 |
| 1438 | // because safe constants guarantee that we do not overflow sdiv/srem (and |
| 1439 | // there's no danger for other opcodes). |
| 1440 | // TODO: To allow this case, create a new shuffle mask with no undefs. |
| 1441 | if (MightCreatePoisonOrUB && !ConstantsAreOp1) |
Sanjay Patel | 5bd3664 | 2018-07-09 13:21:46 +0000 | [diff] [blame] | 1442 | return nullptr; |
| 1443 | |
Sanjay Patel | da66753 | 2018-06-29 13:44:06 +0000 | [diff] [blame] | 1444 | // Note: In general, we do not create new shuffles in InstCombine because we |
| 1445 | // do not know if a target can lower an arbitrary shuffle optimally. In this |
| 1446 | // case, the shuffle uses the existing mask, so there is no additional risk. |
Sanjay Patel | da66753 | 2018-06-29 13:44:06 +0000 | [diff] [blame] | 1447 | |
| 1448 | // Select the variable vectors first, then perform the binop: |
| 1449 | // shuffle (op X, C0), (op Y, C1), M --> op (shuffle X, Y, M), C' |
| 1450 | // shuffle (op C0, X), (op C1, Y), M --> op C', (shuffle X, Y, M) |
Sanjay Patel | 5bd3664 | 2018-07-09 13:21:46 +0000 | [diff] [blame] | 1451 | V = Builder.CreateShuffleVector(X, Y, Mask); |
Sanjay Patel | da66753 | 2018-06-29 13:44:06 +0000 | [diff] [blame] | 1452 | } |
| 1453 | |
Sanjay Patel | da66753 | 2018-06-29 13:44:06 +0000 | [diff] [blame] | 1454 | Instruction *NewBO = ConstantsAreOp1 ? BinaryOperator::Create(BOpc, V, NewC) : |
| 1455 | BinaryOperator::Create(BOpc, NewC, V); |
Sanjay Patel | a76b700 | 2018-06-21 20:15:09 +0000 | [diff] [blame] | 1456 | |
Sanjay Patel | 5bd3664 | 2018-07-09 13:21:46 +0000 | [diff] [blame] | 1457 | // Flags are intersected from the 2 source binops. But there are 2 exceptions: |
| 1458 | // 1. If we changed an opcode, poison conditions might have changed. |
| 1459 | // 2. If the shuffle had undef mask elements, the new binop might have undefs |
Sanjay Patel | c8d9d81 | 2018-07-10 16:09:49 +0000 | [diff] [blame] | 1460 | // where the original code did not. But if we already made a safe constant, |
| 1461 | // then there's no danger. |
Sanjay Patel | a76b700 | 2018-06-21 20:15:09 +0000 | [diff] [blame] | 1462 | NewBO->copyIRFlags(B0); |
| 1463 | NewBO->andIRFlags(B1); |
Sanjay Patel | 57bda36 | 2018-06-28 17:48:04 +0000 | [diff] [blame] | 1464 | if (DropNSW) |
| 1465 | NewBO->setHasNoSignedWrap(false); |
Sanjay Patel | c8d9d81 | 2018-07-10 16:09:49 +0000 | [diff] [blame] | 1466 | if (Mask->containsUndefElement() && !MightCreatePoisonOrUB) |
Sanjay Patel | 5bd3664 | 2018-07-09 13:21:46 +0000 | [diff] [blame] | 1467 | NewBO->dropPoisonGeneratingFlags(); |
Sanjay Patel | a76b700 | 2018-06-21 20:15:09 +0000 | [diff] [blame] | 1468 | return NewBO; |
| 1469 | } |
| 1470 | |
Sanjay Patel | c1416b6 | 2018-09-07 21:03:34 +0000 | [diff] [blame] | 1471 | /// Match a shuffle-select-shuffle pattern where the shuffles are widening and |
| 1472 | /// narrowing (concatenating with undef and extracting back to the original |
| 1473 | /// length). This allows replacing the wide select with a narrow select. |
Sanjay Patel | 88194df | 2018-10-09 15:29:26 +0000 | [diff] [blame] | 1474 | static Instruction *narrowVectorSelect(ShuffleVectorInst &Shuf, |
| 1475 | InstCombiner::BuilderTy &Builder) { |
Sanjay Patel | c1416b6 | 2018-09-07 21:03:34 +0000 | [diff] [blame] | 1476 | // This must be a narrowing identity shuffle. It extracts the 1st N elements |
| 1477 | // of the 1st vector operand of a shuffle. |
| 1478 | if (!match(Shuf.getOperand(1), m_Undef()) || !Shuf.isIdentityWithExtract()) |
| 1479 | return nullptr; |
| 1480 | |
| 1481 | // The vector being shuffled must be a vector select that we can eliminate. |
| 1482 | // TODO: The one-use requirement could be eased if X and/or Y are constants. |
| 1483 | Value *Cond, *X, *Y; |
| 1484 | if (!match(Shuf.getOperand(0), |
| 1485 | m_OneUse(m_Select(m_Value(Cond), m_Value(X), m_Value(Y))))) |
| 1486 | return nullptr; |
| 1487 | |
| 1488 | // We need a narrow condition value. It must be extended with undef elements |
| 1489 | // and have the same number of elements as this shuffle. |
| 1490 | unsigned NarrowNumElts = Shuf.getType()->getVectorNumElements(); |
| 1491 | Value *NarrowCond; |
| 1492 | if (!match(Cond, m_OneUse(m_ShuffleVector(m_Value(NarrowCond), m_Undef(), |
| 1493 | m_Constant()))) || |
| 1494 | NarrowCond->getType()->getVectorNumElements() != NarrowNumElts || |
| 1495 | !cast<ShuffleVectorInst>(Cond)->isIdentityWithPadding()) |
| 1496 | return nullptr; |
| 1497 | |
| 1498 | // shuf (sel (shuf NarrowCond, undef, WideMask), X, Y), undef, NarrowMask) --> |
| 1499 | // sel NarrowCond, (shuf X, undef, NarrowMask), (shuf Y, undef, NarrowMask) |
| 1500 | Value *Undef = UndefValue::get(X->getType()); |
| 1501 | Value *NarrowX = Builder.CreateShuffleVector(X, Undef, Shuf.getMask()); |
| 1502 | Value *NarrowY = Builder.CreateShuffleVector(Y, Undef, Shuf.getMask()); |
| 1503 | return SelectInst::Create(NarrowCond, NarrowX, NarrowY); |
| 1504 | } |
| 1505 | |
Sanjay Patel | 7181146 | 2018-10-14 15:25:06 +0000 | [diff] [blame] | 1506 | /// Try to combine 2 shuffles into 1 shuffle by concatenating a shuffle mask. |
| 1507 | static Instruction *foldIdentityExtractShuffle(ShuffleVectorInst &Shuf) { |
| 1508 | Value *Op0 = Shuf.getOperand(0), *Op1 = Shuf.getOperand(1); |
| 1509 | if (!Shuf.isIdentityWithExtract() || !isa<UndefValue>(Op1)) |
| 1510 | return nullptr; |
| 1511 | |
| 1512 | Value *X, *Y; |
| 1513 | Constant *Mask; |
| 1514 | if (!match(Op0, m_ShuffleVector(m_Value(X), m_Value(Y), m_Constant(Mask)))) |
| 1515 | return nullptr; |
| 1516 | |
Sanjay Patel | cddb1e5 | 2019-02-05 22:58:45 +0000 | [diff] [blame] | 1517 | // Be conservative with shuffle transforms. If we can't kill the 1st shuffle, |
| 1518 | // then combining may result in worse codegen. |
| 1519 | if (!Op0->hasOneUse()) |
| 1520 | return nullptr; |
| 1521 | |
Sanjay Patel | 7181146 | 2018-10-14 15:25:06 +0000 | [diff] [blame] | 1522 | // We are extracting a subvector from a shuffle. Remove excess elements from |
| 1523 | // the 1st shuffle mask to eliminate the extract. |
| 1524 | // |
| 1525 | // This transform is conservatively limited to identity extracts because we do |
| 1526 | // not allow arbitrary shuffle mask creation as a target-independent transform |
| 1527 | // (because we can't guarantee that will lower efficiently). |
| 1528 | // |
| 1529 | // If the extracting shuffle has an undef mask element, it transfers to the |
| 1530 | // new shuffle mask. Otherwise, copy the original mask element. Example: |
| 1531 | // shuf (shuf X, Y, <C0, C1, C2, undef, C4>), undef, <0, undef, 2, 3> --> |
| 1532 | // shuf X, Y, <C0, undef, C2, undef> |
| 1533 | unsigned NumElts = Shuf.getType()->getVectorNumElements(); |
| 1534 | SmallVector<Constant *, 16> NewMask(NumElts); |
| 1535 | assert(NumElts < Mask->getType()->getVectorNumElements() && |
| 1536 | "Identity with extract must have less elements than its inputs"); |
| 1537 | |
| 1538 | for (unsigned i = 0; i != NumElts; ++i) { |
| 1539 | Constant *ExtractMaskElt = Shuf.getMask()->getAggregateElement(i); |
| 1540 | Constant *MaskElt = Mask->getAggregateElement(i); |
| 1541 | NewMask[i] = isa<UndefValue>(ExtractMaskElt) ? ExtractMaskElt : MaskElt; |
| 1542 | } |
| 1543 | return new ShuffleVectorInst(X, Y, ConstantVector::get(NewMask)); |
| 1544 | } |
| 1545 | |
Sanjay Patel | b12e410 | 2018-10-30 15:26:39 +0000 | [diff] [blame] | 1546 | /// Try to replace a shuffle with an insertelement. |
| 1547 | static Instruction *foldShuffleWithInsert(ShuffleVectorInst &Shuf) { |
| 1548 | Value *V0 = Shuf.getOperand(0), *V1 = Shuf.getOperand(1); |
| 1549 | SmallVector<int, 16> Mask = Shuf.getShuffleMask(); |
| 1550 | |
| 1551 | // The shuffle must not change vector sizes. |
| 1552 | // TODO: This restriction could be removed if the insert has only one use |
| 1553 | // (because the transform would require a new length-changing shuffle). |
| 1554 | int NumElts = Mask.size(); |
| 1555 | if (NumElts != (int)(V0->getType()->getVectorNumElements())) |
| 1556 | return nullptr; |
| 1557 | |
| 1558 | // shuffle (insert ?, Scalar, IndexC), V1, Mask --> insert V1, Scalar, IndexC' |
| 1559 | auto isShufflingScalarIntoOp1 = [&](Value *&Scalar, ConstantInt *&IndexC) { |
| 1560 | // We need an insertelement with a constant index. |
| 1561 | if (!match(V0, m_InsertElement(m_Value(), m_Value(Scalar), |
| 1562 | m_ConstantInt(IndexC)))) |
| 1563 | return false; |
| 1564 | |
| 1565 | // Test the shuffle mask to see if it splices the inserted scalar into the |
| 1566 | // operand 1 vector of the shuffle. |
| 1567 | int NewInsIndex = -1; |
| 1568 | for (int i = 0; i != NumElts; ++i) { |
| 1569 | // Ignore undef mask elements. |
| 1570 | if (Mask[i] == -1) |
| 1571 | continue; |
| 1572 | |
| 1573 | // The shuffle takes elements of operand 1 without lane changes. |
| 1574 | if (Mask[i] == NumElts + i) |
| 1575 | continue; |
| 1576 | |
| 1577 | // The shuffle must choose the inserted scalar exactly once. |
| 1578 | if (NewInsIndex != -1 || Mask[i] != IndexC->getSExtValue()) |
| 1579 | return false; |
| 1580 | |
| 1581 | // The shuffle is placing the inserted scalar into element i. |
| 1582 | NewInsIndex = i; |
| 1583 | } |
| 1584 | |
| 1585 | assert(NewInsIndex != -1 && "Did not fold shuffle with unused operand?"); |
| 1586 | |
| 1587 | // Index is updated to the potentially translated insertion lane. |
| 1588 | IndexC = ConstantInt::get(IndexC->getType(), NewInsIndex); |
| 1589 | return true; |
| 1590 | }; |
| 1591 | |
| 1592 | // If the shuffle is unnecessary, insert the scalar operand directly into |
| 1593 | // operand 1 of the shuffle. Example: |
| 1594 | // shuffle (insert ?, S, 1), V1, <1, 5, 6, 7> --> insert V1, S, 0 |
| 1595 | Value *Scalar; |
| 1596 | ConstantInt *IndexC; |
| 1597 | if (isShufflingScalarIntoOp1(Scalar, IndexC)) |
| 1598 | return InsertElementInst::Create(V1, Scalar, IndexC); |
| 1599 | |
| 1600 | // Try again after commuting shuffle. Example: |
| 1601 | // shuffle V0, (insert ?, S, 0), <0, 1, 2, 4> --> |
| 1602 | // shuffle (insert ?, S, 0), V0, <4, 5, 6, 0> --> insert V0, S, 3 |
| 1603 | std::swap(V0, V1); |
| 1604 | ShuffleVectorInst::commuteShuffleMask(Mask, NumElts); |
| 1605 | if (isShufflingScalarIntoOp1(Scalar, IndexC)) |
| 1606 | return InsertElementInst::Create(V1, Scalar, IndexC); |
| 1607 | |
| 1608 | return nullptr; |
| 1609 | } |
| 1610 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 1611 | Instruction *InstCombiner::visitShuffleVectorInst(ShuffleVectorInst &SVI) { |
| 1612 | Value *LHS = SVI.getOperand(0); |
| 1613 | Value *RHS = SVI.getOperand(1); |
Craig Topper | a420562 | 2017-06-09 03:21:29 +0000 | [diff] [blame] | 1614 | if (auto *V = SimplifyShuffleVectorInst( |
| 1615 | LHS, RHS, SVI.getMask(), SVI.getType(), SQ.getWithInstruction(&SVI))) |
Zvi Rackover | 82bf48d | 2017-04-04 04:47:57 +0000 | [diff] [blame] | 1616 | return replaceInstUsesWith(SVI, V); |
| 1617 | |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 1618 | // Canonicalize shuffle(x ,x,mask) -> shuffle(x, undef,mask') |
| 1619 | // Canonicalize shuffle(undef,x,mask) -> shuffle(x, undef,mask'). |
Sanjay Patel | 3f4d1b4 | 2019-03-29 16:49:38 +0000 | [diff] [blame] | 1620 | unsigned VWidth = SVI.getType()->getVectorNumElements(); |
| 1621 | unsigned LHSWidth = LHS->getType()->getVectorNumElements(); |
| 1622 | SmallVector<int, 16> Mask = SVI.getShuffleMask(); |
| 1623 | Type *Int32Ty = Type::getInt32Ty(SVI.getContext()); |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 1624 | if (LHS == RHS || isa<UndefValue>(LHS)) { |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 1625 | // Remap any references to RHS to use LHS. |
Chris Lattner | 0256be9 | 2012-01-27 03:08:05 +0000 | [diff] [blame] | 1626 | SmallVector<Constant*, 16> Elts; |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1627 | for (unsigned i = 0, e = LHSWidth; i != VWidth; ++i) { |
Chris Lattner | 0256be9 | 2012-01-27 03:08:05 +0000 | [diff] [blame] | 1628 | if (Mask[i] < 0) { |
JF Bastien | d52c990 | 2015-02-25 22:30:51 +0000 | [diff] [blame] | 1629 | Elts.push_back(UndefValue::get(Int32Ty)); |
Chris Lattner | 0256be9 | 2012-01-27 03:08:05 +0000 | [diff] [blame] | 1630 | continue; |
| 1631 | } |
| 1632 | |
| 1633 | if ((Mask[i] >= (int)e && isa<UndefValue>(RHS)) || |
| 1634 | (Mask[i] < (int)e && isa<UndefValue>(LHS))) { |
| 1635 | Mask[i] = -1; // Turn into undef. |
JF Bastien | d52c990 | 2015-02-25 22:30:51 +0000 | [diff] [blame] | 1636 | Elts.push_back(UndefValue::get(Int32Ty)); |
Chris Lattner | 0256be9 | 2012-01-27 03:08:05 +0000 | [diff] [blame] | 1637 | } else { |
| 1638 | Mask[i] = Mask[i] % e; // Force to LHS. |
JF Bastien | d52c990 | 2015-02-25 22:30:51 +0000 | [diff] [blame] | 1639 | Elts.push_back(ConstantInt::get(Int32Ty, Mask[i])); |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 1640 | } |
| 1641 | } |
| 1642 | SVI.setOperand(0, SVI.getOperand(1)); |
| 1643 | SVI.setOperand(1, UndefValue::get(RHS->getType())); |
| 1644 | SVI.setOperand(2, ConstantVector::get(Elts)); |
Sanjay Patel | 3f4d1b4 | 2019-03-29 16:49:38 +0000 | [diff] [blame] | 1645 | return &SVI; |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 1646 | } |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 1647 | |
Sanjay Patel | 3f4d1b4 | 2019-03-29 16:49:38 +0000 | [diff] [blame] | 1648 | if (Instruction *I = foldSelectShuffle(SVI, Builder, DL)) |
| 1649 | return I; |
| 1650 | |
| 1651 | if (Instruction *I = narrowVectorSelect(SVI, Builder)) |
| 1652 | return I; |
| 1653 | |
| 1654 | APInt UndefElts(VWidth, 0); |
| 1655 | APInt AllOnesEltMask(APInt::getAllOnesValue(VWidth)); |
| 1656 | if (Value *V = SimplifyDemandedVectorElts(&SVI, AllOnesEltMask, UndefElts)) { |
| 1657 | if (V != &SVI) |
| 1658 | return replaceInstUsesWith(SVI, V); |
| 1659 | return &SVI; |
| 1660 | } |
| 1661 | |
| 1662 | if (Instruction *I = foldIdentityExtractShuffle(SVI)) |
| 1663 | return I; |
| 1664 | |
| 1665 | // This transform has the potential to lose undef knowledge, so it is |
| 1666 | // intentionally placed after SimplifyDemandedVectorElts(). |
| 1667 | if (Instruction *I = foldShuffleWithInsert(SVI)) |
| 1668 | return I; |
| 1669 | |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1670 | if (VWidth == LHSWidth) { |
| 1671 | // Analyze the shuffle, are the LHS or RHS and identity shuffles? |
Serge Pavlov | 9ef66a8 | 2014-05-11 08:46:12 +0000 | [diff] [blame] | 1672 | bool isLHSID, isRHSID; |
Sanjay Patel | 431e114 | 2015-11-17 17:24:08 +0000 | [diff] [blame] | 1673 | recognizeIdentityMask(Mask, isLHSID, isRHSID); |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1674 | |
| 1675 | // Eliminate identity shuffles. |
Sanjay Patel | 4b19880 | 2016-02-01 22:23:39 +0000 | [diff] [blame] | 1676 | if (isLHSID) return replaceInstUsesWith(SVI, LHS); |
| 1677 | if (isRHSID) return replaceInstUsesWith(SVI, RHS); |
Eric Christopher | 51edc7b | 2010-08-17 22:55:27 +0000 | [diff] [blame] | 1678 | } |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 1679 | |
Sanjay Patel | 26c119a | 2018-09-30 13:50:42 +0000 | [diff] [blame] | 1680 | if (isa<UndefValue>(RHS) && canEvaluateShuffled(LHS, Mask)) { |
Sanjay Patel | 54d31ef | 2018-09-29 15:05:24 +0000 | [diff] [blame] | 1681 | Value *V = evaluateInDifferentElementOrder(LHS, Mask); |
Sanjay Patel | 4b19880 | 2016-02-01 22:23:39 +0000 | [diff] [blame] | 1682 | return replaceInstUsesWith(SVI, V); |
Nick Lewycky | a2b7720 | 2013-05-31 00:59:42 +0000 | [diff] [blame] | 1683 | } |
| 1684 | |
JF Bastien | d52c990 | 2015-02-25 22:30:51 +0000 | [diff] [blame] | 1685 | // SROA generates shuffle+bitcast when the extracted sub-vector is bitcast to |
| 1686 | // a non-vector type. We can instead bitcast the original vector followed by |
| 1687 | // an extract of the desired element: |
| 1688 | // |
| 1689 | // %sroa = shufflevector <16 x i8> %in, <16 x i8> undef, |
| 1690 | // <4 x i32> <i32 0, i32 1, i32 2, i32 3> |
| 1691 | // %1 = bitcast <4 x i8> %sroa to i32 |
| 1692 | // Becomes: |
| 1693 | // %bc = bitcast <16 x i8> %in to <4 x i32> |
| 1694 | // %ext = extractelement <4 x i32> %bc, i32 0 |
| 1695 | // |
| 1696 | // If the shuffle is extracting a contiguous range of values from the input |
| 1697 | // vector then each use which is a bitcast of the extracted size can be |
| 1698 | // replaced. This will work if the vector types are compatible, and the begin |
| 1699 | // index is aligned to a value in the casted vector type. If the begin index |
| 1700 | // isn't aligned then we can shuffle the original vector (keeping the same |
| 1701 | // vector type) before extracting. |
| 1702 | // |
| 1703 | // This code will bail out if the target type is fundamentally incompatible |
| 1704 | // with vectors of the source type. |
| 1705 | // |
| 1706 | // Example of <16 x i8>, target type i32: |
| 1707 | // Index range [4,8): v-----------v Will work. |
| 1708 | // +--+--+--+--+--+--+--+--+--+--+--+--+--+--+--+--+ |
| 1709 | // <16 x i8>: | | | | | | | | | | | | | | | | | |
| 1710 | // <4 x i32>: | | | | | |
| 1711 | // +-----------+-----------+-----------+-----------+ |
| 1712 | // Index range [6,10): ^-----------^ Needs an extra shuffle. |
| 1713 | // Target type i40: ^--------------^ Won't work, bail. |
Sanjay Patel | 3f4d1b4 | 2019-03-29 16:49:38 +0000 | [diff] [blame] | 1714 | bool MadeChange = false; |
JF Bastien | d52c990 | 2015-02-25 22:30:51 +0000 | [diff] [blame] | 1715 | if (isShuffleExtractingFromLHS(SVI, Mask)) { |
| 1716 | Value *V = LHS; |
| 1717 | unsigned MaskElems = Mask.size(); |
JF Bastien | d52c990 | 2015-02-25 22:30:51 +0000 | [diff] [blame] | 1718 | VectorType *SrcTy = cast<VectorType>(V->getType()); |
| 1719 | unsigned VecBitWidth = SrcTy->getBitWidth(); |
David Majnemer | 98cfe2b | 2015-04-03 20:18:40 +0000 | [diff] [blame] | 1720 | unsigned SrcElemBitWidth = DL.getTypeSizeInBits(SrcTy->getElementType()); |
JF Bastien | d52c990 | 2015-02-25 22:30:51 +0000 | [diff] [blame] | 1721 | assert(SrcElemBitWidth && "vector elements must have a bitwidth"); |
| 1722 | unsigned SrcNumElems = SrcTy->getNumElements(); |
| 1723 | SmallVector<BitCastInst *, 8> BCs; |
| 1724 | DenseMap<Type *, Value *> NewBCs; |
| 1725 | for (User *U : SVI.users()) |
| 1726 | if (BitCastInst *BC = dyn_cast<BitCastInst>(U)) |
| 1727 | if (!BC->use_empty()) |
| 1728 | // Only visit bitcasts that weren't previously handled. |
| 1729 | BCs.push_back(BC); |
| 1730 | for (BitCastInst *BC : BCs) { |
Eugene Leviant | 958fcd7 | 2017-02-17 07:36:03 +0000 | [diff] [blame] | 1731 | unsigned BegIdx = Mask.front(); |
JF Bastien | d52c990 | 2015-02-25 22:30:51 +0000 | [diff] [blame] | 1732 | Type *TgtTy = BC->getDestTy(); |
David Majnemer | 98cfe2b | 2015-04-03 20:18:40 +0000 | [diff] [blame] | 1733 | unsigned TgtElemBitWidth = DL.getTypeSizeInBits(TgtTy); |
JF Bastien | d52c990 | 2015-02-25 22:30:51 +0000 | [diff] [blame] | 1734 | if (!TgtElemBitWidth) |
| 1735 | continue; |
| 1736 | unsigned TgtNumElems = VecBitWidth / TgtElemBitWidth; |
| 1737 | bool VecBitWidthsEqual = VecBitWidth == TgtNumElems * TgtElemBitWidth; |
| 1738 | bool BegIsAligned = 0 == ((SrcElemBitWidth * BegIdx) % TgtElemBitWidth); |
| 1739 | if (!VecBitWidthsEqual) |
| 1740 | continue; |
| 1741 | if (!VectorType::isValidElementType(TgtTy)) |
| 1742 | continue; |
| 1743 | VectorType *CastSrcTy = VectorType::get(TgtTy, TgtNumElems); |
| 1744 | if (!BegIsAligned) { |
| 1745 | // Shuffle the input so [0,NumElements) contains the output, and |
| 1746 | // [NumElems,SrcNumElems) is undef. |
| 1747 | SmallVector<Constant *, 16> ShuffleMask(SrcNumElems, |
| 1748 | UndefValue::get(Int32Ty)); |
| 1749 | for (unsigned I = 0, E = MaskElems, Idx = BegIdx; I != E; ++Idx, ++I) |
| 1750 | ShuffleMask[I] = ConstantInt::get(Int32Ty, Idx); |
Craig Topper | bb4069e | 2017-07-07 23:16:26 +0000 | [diff] [blame] | 1751 | V = Builder.CreateShuffleVector(V, UndefValue::get(V->getType()), |
| 1752 | ConstantVector::get(ShuffleMask), |
| 1753 | SVI.getName() + ".extract"); |
JF Bastien | d52c990 | 2015-02-25 22:30:51 +0000 | [diff] [blame] | 1754 | BegIdx = 0; |
| 1755 | } |
| 1756 | unsigned SrcElemsPerTgtElem = TgtElemBitWidth / SrcElemBitWidth; |
| 1757 | assert(SrcElemsPerTgtElem); |
| 1758 | BegIdx /= SrcElemsPerTgtElem; |
| 1759 | bool BCAlreadyExists = NewBCs.find(CastSrcTy) != NewBCs.end(); |
| 1760 | auto *NewBC = |
| 1761 | BCAlreadyExists |
| 1762 | ? NewBCs[CastSrcTy] |
Craig Topper | bb4069e | 2017-07-07 23:16:26 +0000 | [diff] [blame] | 1763 | : Builder.CreateBitCast(V, CastSrcTy, SVI.getName() + ".bc"); |
JF Bastien | d52c990 | 2015-02-25 22:30:51 +0000 | [diff] [blame] | 1764 | if (!BCAlreadyExists) |
| 1765 | NewBCs[CastSrcTy] = NewBC; |
Craig Topper | bb4069e | 2017-07-07 23:16:26 +0000 | [diff] [blame] | 1766 | auto *Ext = Builder.CreateExtractElement( |
JF Bastien | d52c990 | 2015-02-25 22:30:51 +0000 | [diff] [blame] | 1767 | NewBC, ConstantInt::get(Int32Ty, BegIdx), SVI.getName() + ".extract"); |
| 1768 | // The shufflevector isn't being replaced: the bitcast that used it |
| 1769 | // is. InstCombine will visit the newly-created instructions. |
Sanjay Patel | 4b19880 | 2016-02-01 22:23:39 +0000 | [diff] [blame] | 1770 | replaceInstUsesWith(*BC, Ext); |
JF Bastien | d52c990 | 2015-02-25 22:30:51 +0000 | [diff] [blame] | 1771 | MadeChange = true; |
| 1772 | } |
| 1773 | } |
| 1774 | |
Eric Christopher | 51edc7b | 2010-08-17 22:55:27 +0000 | [diff] [blame] | 1775 | // If the LHS is a shufflevector itself, see if we can combine it with this |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1776 | // one without producing an unusual shuffle. |
| 1777 | // Cases that might be simplified: |
| 1778 | // 1. |
| 1779 | // x1=shuffle(v1,v2,mask1) |
| 1780 | // x=shuffle(x1,undef,mask) |
| 1781 | // ==> |
| 1782 | // x=shuffle(v1,undef,newMask) |
| 1783 | // newMask[i] = (mask[i] < x1.size()) ? mask1[mask[i]] : -1 |
| 1784 | // 2. |
| 1785 | // x1=shuffle(v1,undef,mask1) |
| 1786 | // x=shuffle(x1,x2,mask) |
| 1787 | // where v1.size() == mask1.size() |
| 1788 | // ==> |
| 1789 | // x=shuffle(v1,x2,newMask) |
| 1790 | // newMask[i] = (mask[i] < x1.size()) ? mask1[mask[i]] : mask[i] |
| 1791 | // 3. |
| 1792 | // x2=shuffle(v2,undef,mask2) |
| 1793 | // x=shuffle(x1,x2,mask) |
| 1794 | // where v2.size() == mask2.size() |
| 1795 | // ==> |
| 1796 | // x=shuffle(x1,v2,newMask) |
| 1797 | // newMask[i] = (mask[i] < x1.size()) |
| 1798 | // ? mask[i] : mask2[mask[i]-x1.size()]+x1.size() |
| 1799 | // 4. |
| 1800 | // x1=shuffle(v1,undef,mask1) |
| 1801 | // x2=shuffle(v2,undef,mask2) |
| 1802 | // x=shuffle(x1,x2,mask) |
| 1803 | // where v1.size() == v2.size() |
| 1804 | // ==> |
| 1805 | // x=shuffle(v1,v2,newMask) |
| 1806 | // newMask[i] = (mask[i] < x1.size()) |
| 1807 | // ? mask1[mask[i]] : mask2[mask[i]-x1.size()]+v1.size() |
| 1808 | // |
| 1809 | // Here we are really conservative: |
Eric Christopher | 51edc7b | 2010-08-17 22:55:27 +0000 | [diff] [blame] | 1810 | // we are absolutely afraid of producing a shuffle mask not in the input |
| 1811 | // program, because the code gen may not be smart enough to turn a merged |
| 1812 | // shuffle into two specific shuffles: it may produce worse code. As such, |
Jim Grosbach | d11584a | 2013-05-01 00:25:27 +0000 | [diff] [blame] | 1813 | // we only merge two shuffles if the result is either a splat or one of the |
| 1814 | // input shuffle masks. In this case, merging the shuffles just removes |
| 1815 | // one instruction, which we know is safe. This is good for things like |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1816 | // turning: (splat(splat)) -> splat, or |
| 1817 | // merge(V[0..n], V[n+1..2n]) -> V[0..2n] |
| 1818 | ShuffleVectorInst* LHSShuffle = dyn_cast<ShuffleVectorInst>(LHS); |
| 1819 | ShuffleVectorInst* RHSShuffle = dyn_cast<ShuffleVectorInst>(RHS); |
| 1820 | if (LHSShuffle) |
| 1821 | if (!isa<UndefValue>(LHSShuffle->getOperand(1)) && !isa<UndefValue>(RHS)) |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1822 | LHSShuffle = nullptr; |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1823 | if (RHSShuffle) |
| 1824 | if (!isa<UndefValue>(RHSShuffle->getOperand(1))) |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1825 | RHSShuffle = nullptr; |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1826 | if (!LHSShuffle && !RHSShuffle) |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1827 | return MadeChange ? &SVI : nullptr; |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1828 | |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1829 | Value* LHSOp0 = nullptr; |
| 1830 | Value* LHSOp1 = nullptr; |
| 1831 | Value* RHSOp0 = nullptr; |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1832 | unsigned LHSOp0Width = 0; |
| 1833 | unsigned RHSOp0Width = 0; |
| 1834 | if (LHSShuffle) { |
| 1835 | LHSOp0 = LHSShuffle->getOperand(0); |
| 1836 | LHSOp1 = LHSShuffle->getOperand(1); |
Craig Topper | 17b5568 | 2016-12-29 07:03:18 +0000 | [diff] [blame] | 1837 | LHSOp0Width = LHSOp0->getType()->getVectorNumElements(); |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1838 | } |
| 1839 | if (RHSShuffle) { |
| 1840 | RHSOp0 = RHSShuffle->getOperand(0); |
Craig Topper | 17b5568 | 2016-12-29 07:03:18 +0000 | [diff] [blame] | 1841 | RHSOp0Width = RHSOp0->getType()->getVectorNumElements(); |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1842 | } |
| 1843 | Value* newLHS = LHS; |
| 1844 | Value* newRHS = RHS; |
| 1845 | if (LHSShuffle) { |
| 1846 | // case 1 |
Eric Christopher | 51edc7b | 2010-08-17 22:55:27 +0000 | [diff] [blame] | 1847 | if (isa<UndefValue>(RHS)) { |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1848 | newLHS = LHSOp0; |
| 1849 | newRHS = LHSOp1; |
| 1850 | } |
| 1851 | // case 2 or 4 |
| 1852 | else if (LHSOp0Width == LHSWidth) { |
| 1853 | newLHS = LHSOp0; |
| 1854 | } |
| 1855 | } |
| 1856 | // case 3 or 4 |
| 1857 | if (RHSShuffle && RHSOp0Width == LHSWidth) { |
| 1858 | newRHS = RHSOp0; |
| 1859 | } |
| 1860 | // case 4 |
| 1861 | if (LHSOp0 == RHSOp0) { |
| 1862 | newLHS = LHSOp0; |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1863 | newRHS = nullptr; |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1864 | } |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 1865 | |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1866 | if (newLHS == LHS && newRHS == RHS) |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1867 | return MadeChange ? &SVI : nullptr; |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 1868 | |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1869 | SmallVector<int, 16> LHSMask; |
| 1870 | SmallVector<int, 16> RHSMask; |
Chris Lattner | 8326bd8 | 2012-01-26 00:42:34 +0000 | [diff] [blame] | 1871 | if (newLHS != LHS) |
| 1872 | LHSMask = LHSShuffle->getShuffleMask(); |
| 1873 | if (RHSShuffle && newRHS != RHS) |
| 1874 | RHSMask = RHSShuffle->getShuffleMask(); |
| 1875 | |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1876 | unsigned newLHSWidth = (newLHS != LHS) ? LHSOp0Width : LHSWidth; |
| 1877 | SmallVector<int, 16> newMask; |
| 1878 | bool isSplat = true; |
| 1879 | int SplatElt = -1; |
| 1880 | // Create a new mask for the new ShuffleVectorInst so that the new |
| 1881 | // ShuffleVectorInst is equivalent to the original one. |
| 1882 | for (unsigned i = 0; i < VWidth; ++i) { |
| 1883 | int eltMask; |
Craig Topper | 45d9f4b | 2013-01-18 05:30:07 +0000 | [diff] [blame] | 1884 | if (Mask[i] < 0) { |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1885 | // This element is an undef value. |
| 1886 | eltMask = -1; |
| 1887 | } else if (Mask[i] < (int)LHSWidth) { |
| 1888 | // This element is from left hand side vector operand. |
Craig Topper | 2ea22b0 | 2013-01-18 05:09:16 +0000 | [diff] [blame] | 1889 | // |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1890 | // If LHS is going to be replaced (case 1, 2, or 4), calculate the |
| 1891 | // new mask value for the element. |
| 1892 | if (newLHS != LHS) { |
| 1893 | eltMask = LHSMask[Mask[i]]; |
| 1894 | // If the value selected is an undef value, explicitly specify it |
| 1895 | // with a -1 mask value. |
| 1896 | if (eltMask >= (int)LHSOp0Width && isa<UndefValue>(LHSOp1)) |
| 1897 | eltMask = -1; |
Craig Topper | 2ea22b0 | 2013-01-18 05:09:16 +0000 | [diff] [blame] | 1898 | } else |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1899 | eltMask = Mask[i]; |
| 1900 | } else { |
| 1901 | // This element is from right hand side vector operand |
| 1902 | // |
| 1903 | // If the value selected is an undef value, explicitly specify it |
| 1904 | // with a -1 mask value. (case 1) |
| 1905 | if (isa<UndefValue>(RHS)) |
| 1906 | eltMask = -1; |
| 1907 | // If RHS is going to be replaced (case 3 or 4), calculate the |
| 1908 | // new mask value for the element. |
| 1909 | else if (newRHS != RHS) { |
| 1910 | eltMask = RHSMask[Mask[i]-LHSWidth]; |
| 1911 | // If the value selected is an undef value, explicitly specify it |
| 1912 | // with a -1 mask value. |
| 1913 | if (eltMask >= (int)RHSOp0Width) { |
| 1914 | assert(isa<UndefValue>(RHSShuffle->getOperand(1)) |
| 1915 | && "should have been check above"); |
| 1916 | eltMask = -1; |
Nate Begeman | 2a0ca3e9 | 2010-08-13 00:17:53 +0000 | [diff] [blame] | 1917 | } |
Craig Topper | 2ea22b0 | 2013-01-18 05:09:16 +0000 | [diff] [blame] | 1918 | } else |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1919 | eltMask = Mask[i]-LHSWidth; |
| 1920 | |
| 1921 | // If LHS's width is changed, shift the mask value accordingly. |
Eugene Zelenko | 7f0f9bc | 2017-10-24 21:24:53 +0000 | [diff] [blame] | 1922 | // If newRHS == nullptr, i.e. LHSOp0 == RHSOp0, we want to remap any |
Michael Gottesman | 02a1141 | 2012-10-16 21:29:38 +0000 | [diff] [blame] | 1923 | // references from RHSOp0 to LHSOp0, so we don't need to shift the mask. |
| 1924 | // If newRHS == newLHS, we want to remap any references from newRHS to |
| 1925 | // newLHS so that we can properly identify splats that may occur due to |
Alp Toker | cb40291 | 2014-01-24 17:20:08 +0000 | [diff] [blame] | 1926 | // obfuscation across the two vectors. |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1927 | if (eltMask >= 0 && newRHS != nullptr && newLHS != newRHS) |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1928 | eltMask += newLHSWidth; |
Nate Begeman | 2a0ca3e9 | 2010-08-13 00:17:53 +0000 | [diff] [blame] | 1929 | } |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1930 | |
| 1931 | // Check if this could still be a splat. |
| 1932 | if (eltMask >= 0) { |
| 1933 | if (SplatElt >= 0 && SplatElt != eltMask) |
| 1934 | isSplat = false; |
| 1935 | SplatElt = eltMask; |
| 1936 | } |
| 1937 | |
| 1938 | newMask.push_back(eltMask); |
| 1939 | } |
| 1940 | |
| 1941 | // If the result mask is equal to one of the original shuffle masks, |
Jim Grosbach | d11584a | 2013-05-01 00:25:27 +0000 | [diff] [blame] | 1942 | // or is a splat, do the replacement. |
| 1943 | if (isSplat || newMask == LHSMask || newMask == RHSMask || newMask == Mask) { |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1944 | SmallVector<Constant*, 16> Elts; |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1945 | for (unsigned i = 0, e = newMask.size(); i != e; ++i) { |
| 1946 | if (newMask[i] < 0) { |
| 1947 | Elts.push_back(UndefValue::get(Int32Ty)); |
| 1948 | } else { |
| 1949 | Elts.push_back(ConstantInt::get(Int32Ty, newMask[i])); |
| 1950 | } |
| 1951 | } |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1952 | if (!newRHS) |
Eli Friedman | ce81827 | 2011-10-21 19:06:29 +0000 | [diff] [blame] | 1953 | newRHS = UndefValue::get(newLHS->getType()); |
| 1954 | return new ShuffleVectorInst(newLHS, newRHS, ConstantVector::get(Elts)); |
Nate Begeman | 2a0ca3e9 | 2010-08-13 00:17:53 +0000 | [diff] [blame] | 1955 | } |
Bob Wilson | 8ecf98b | 2010-10-29 22:20:43 +0000 | [diff] [blame] | 1956 | |
Serge Pavlov | 9ef66a8 | 2014-05-11 08:46:12 +0000 | [diff] [blame] | 1957 | // If the result mask is an identity, replace uses of this instruction with |
| 1958 | // corresponding argument. |
Serge Pavlov | b575ee8 | 2014-05-13 06:07:21 +0000 | [diff] [blame] | 1959 | bool isLHSID, isRHSID; |
Sanjay Patel | 431e114 | 2015-11-17 17:24:08 +0000 | [diff] [blame] | 1960 | recognizeIdentityMask(newMask, isLHSID, isRHSID); |
Sanjay Patel | 4b19880 | 2016-02-01 22:23:39 +0000 | [diff] [blame] | 1961 | if (isLHSID && VWidth == LHSOp0Width) return replaceInstUsesWith(SVI, newLHS); |
| 1962 | if (isRHSID && VWidth == RHSOp0Width) return replaceInstUsesWith(SVI, newRHS); |
Serge Pavlov | 9ef66a8 | 2014-05-11 08:46:12 +0000 | [diff] [blame] | 1963 | |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1964 | return MadeChange ? &SVI : nullptr; |
Chris Lattner | ec97a90 | 2010-01-05 05:36:20 +0000 | [diff] [blame] | 1965 | } |