Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 1 | //===- InstructionCombining.cpp - Combine multiple instructions -----------===// |
Chris Lattner | ca08125 | 2001-12-14 16:52:21 +0000 | [diff] [blame] | 2 | // |
| 3 | // InstructionCombining - Combine instructions to form fewer, simple |
Chris Lattner | 99f48c6 | 2002-09-02 04:59:56 +0000 | [diff] [blame] | 4 | // instructions. This pass does not modify the CFG This pass is where algebraic |
| 5 | // simplification happens. |
Chris Lattner | ca08125 | 2001-12-14 16:52:21 +0000 | [diff] [blame] | 6 | // |
| 7 | // This pass combines things like: |
| 8 | // %Y = add int 1, %X |
| 9 | // %Z = add int 1, %Y |
| 10 | // into: |
| 11 | // %Z = add int 2, %X |
| 12 | // |
| 13 | // This is a simple worklist driven algorithm. |
| 14 | // |
| 15 | //===----------------------------------------------------------------------===// |
| 16 | |
Chris Lattner | b4cfa7f | 2002-05-07 20:03:00 +0000 | [diff] [blame] | 17 | #include "llvm/Transforms/Scalar.h" |
Chris Lattner | 9b55e5a | 2002-05-07 18:12:18 +0000 | [diff] [blame] | 18 | #include "llvm/Transforms/Utils/BasicBlockUtils.h" |
Chris Lattner | ae7a0d3 | 2002-08-02 19:29:35 +0000 | [diff] [blame] | 19 | #include "llvm/Transforms/Utils/Local.h" |
Chris Lattner | 65b529f | 2002-04-08 20:18:09 +0000 | [diff] [blame] | 20 | #include "llvm/ConstantHandling.h" |
Chris Lattner | ca08125 | 2001-12-14 16:52:21 +0000 | [diff] [blame] | 21 | #include "llvm/iMemory.h" |
Chris Lattner | 3a60d04 | 2002-04-15 19:45:29 +0000 | [diff] [blame] | 22 | #include "llvm/iOther.h" |
Chris Lattner | bbbdd85 | 2002-05-06 18:06:38 +0000 | [diff] [blame] | 23 | #include "llvm/iPHINode.h" |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 24 | #include "llvm/iOperators.h" |
Chris Lattner | 04805fa | 2002-02-26 21:46:54 +0000 | [diff] [blame] | 25 | #include "llvm/Pass.h" |
Chris Lattner | 1085bdf | 2002-11-04 16:18:53 +0000 | [diff] [blame] | 26 | #include "llvm/DerivedTypes.h" |
Chris Lattner | 60a6591 | 2002-02-12 21:07:25 +0000 | [diff] [blame] | 27 | #include "llvm/Support/InstIterator.h" |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 28 | #include "llvm/Support/InstVisitor.h" |
Chris Lattner | bf3a099 | 2002-10-01 22:38:41 +0000 | [diff] [blame] | 29 | #include "Support/Statistic.h" |
Chris Lattner | 053c093 | 2002-05-14 15:24:07 +0000 | [diff] [blame] | 30 | #include <algorithm> |
Chris Lattner | ca08125 | 2001-12-14 16:52:21 +0000 | [diff] [blame] | 31 | |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 32 | namespace { |
Chris Lattner | bf3a099 | 2002-10-01 22:38:41 +0000 | [diff] [blame] | 33 | Statistic<> NumCombined ("instcombine", "Number of insts combined"); |
| 34 | Statistic<> NumConstProp("instcombine", "Number of constant folds"); |
| 35 | Statistic<> NumDeadInst ("instcombine", "Number of dead inst eliminated"); |
| 36 | |
Chris Lattner | c8e6654 | 2002-04-27 06:56:12 +0000 | [diff] [blame] | 37 | class InstCombiner : public FunctionPass, |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 38 | public InstVisitor<InstCombiner, Instruction*> { |
| 39 | // Worklist of all of the instructions that need to be simplified. |
| 40 | std::vector<Instruction*> WorkList; |
| 41 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 42 | void AddUsesToWorkList(Instruction &I) { |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 43 | // The instruction was simplified, add all users of the instruction to |
| 44 | // the work lists because they might get more simplified now... |
| 45 | // |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 46 | for (Value::use_iterator UI = I.use_begin(), UE = I.use_end(); |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 47 | UI != UE; ++UI) |
| 48 | WorkList.push_back(cast<Instruction>(*UI)); |
| 49 | } |
| 50 | |
Chris Lattner | 99f48c6 | 2002-09-02 04:59:56 +0000 | [diff] [blame] | 51 | // removeFromWorkList - remove all instances of I from the worklist. |
| 52 | void removeFromWorkList(Instruction *I); |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 53 | public: |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 54 | virtual bool runOnFunction(Function &F); |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 55 | |
Chris Lattner | f12cc84 | 2002-04-28 21:27:06 +0000 | [diff] [blame] | 56 | virtual void getAnalysisUsage(AnalysisUsage &AU) const { |
Chris Lattner | 820d971 | 2002-10-21 20:00:28 +0000 | [diff] [blame] | 57 | AU.setPreservesCFG(); |
Chris Lattner | f12cc84 | 2002-04-28 21:27:06 +0000 | [diff] [blame] | 58 | } |
| 59 | |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 60 | // Visitation implementation - Implement instruction combining for different |
| 61 | // instruction types. The semantics are as follows: |
| 62 | // Return Value: |
| 63 | // null - No change was made |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 64 | // I - Change was made, I is still valid, I may be dead though |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 65 | // otherwise - Change was made, replace I with returned instruction |
| 66 | // |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 67 | Instruction *visitAdd(BinaryOperator &I); |
| 68 | Instruction *visitSub(BinaryOperator &I); |
| 69 | Instruction *visitMul(BinaryOperator &I); |
| 70 | Instruction *visitDiv(BinaryOperator &I); |
| 71 | Instruction *visitRem(BinaryOperator &I); |
| 72 | Instruction *visitAnd(BinaryOperator &I); |
| 73 | Instruction *visitOr (BinaryOperator &I); |
| 74 | Instruction *visitXor(BinaryOperator &I); |
| 75 | Instruction *visitSetCondInst(BinaryOperator &I); |
| 76 | Instruction *visitShiftInst(Instruction &I); |
| 77 | Instruction *visitCastInst(CastInst &CI); |
| 78 | Instruction *visitPHINode(PHINode &PN); |
| 79 | Instruction *visitGetElementPtrInst(GetElementPtrInst &GEP); |
Chris Lattner | 1085bdf | 2002-11-04 16:18:53 +0000 | [diff] [blame] | 80 | Instruction *visitAllocationInst(AllocationInst &AI); |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 81 | |
| 82 | // visitInstruction - Specify what to return for unhandled instructions... |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 83 | Instruction *visitInstruction(Instruction &I) { return 0; } |
Chris Lattner | 6d14f2a | 2002-08-09 23:47:40 +0000 | [diff] [blame] | 84 | |
| 85 | // InsertNewInstBefore - insert an instruction New before instruction Old |
| 86 | // in the program. Add the new instruction to the worklist. |
| 87 | // |
| 88 | void InsertNewInstBefore(Instruction *New, Instruction &Old) { |
Chris Lattner | 65217ff | 2002-08-23 18:32:43 +0000 | [diff] [blame] | 89 | assert(New && New->getParent() == 0 && |
| 90 | "New instruction already inserted into a basic block!"); |
Chris Lattner | 6d14f2a | 2002-08-09 23:47:40 +0000 | [diff] [blame] | 91 | BasicBlock *BB = Old.getParent(); |
| 92 | BB->getInstList().insert(&Old, New); // Insert inst |
| 93 | WorkList.push_back(New); // Add to worklist |
| 94 | } |
| 95 | |
| 96 | // ReplaceInstUsesWith - This method is to be used when an instruction is |
| 97 | // found to be dead, replacable with another preexisting expression. Here |
| 98 | // we add all uses of I to the worklist, replace all uses of I with the new |
| 99 | // value, then return I, so that the inst combiner will know that I was |
| 100 | // modified. |
| 101 | // |
| 102 | Instruction *ReplaceInstUsesWith(Instruction &I, Value *V) { |
| 103 | AddUsesToWorkList(I); // Add all modified instrs to worklist |
| 104 | I.replaceAllUsesWith(V); |
| 105 | return &I; |
| 106 | } |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 107 | }; |
Chris Lattner | b28b680 | 2002-07-23 18:06:35 +0000 | [diff] [blame] | 108 | |
Chris Lattner | c8b7092 | 2002-07-26 21:12:46 +0000 | [diff] [blame] | 109 | RegisterOpt<InstCombiner> X("instcombine", "Combine redundant instructions"); |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 110 | } |
| 111 | |
| 112 | |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 113 | // Make sure that this instruction has a constant on the right hand side if it |
| 114 | // has any constant arguments. If not, fix it an return true. |
| 115 | // |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 116 | static bool SimplifyBinOp(BinaryOperator &I) { |
| 117 | if (isa<Constant>(I.getOperand(0)) && !isa<Constant>(I.getOperand(1))) |
| 118 | return !I.swapOperands(); |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 119 | return false; |
| 120 | } |
Chris Lattner | ca08125 | 2001-12-14 16:52:21 +0000 | [diff] [blame] | 121 | |
Chris Lattner | 9fa53de | 2002-05-06 16:49:18 +0000 | [diff] [blame] | 122 | // dyn_castNegInst - Given a 'sub' instruction, return the RHS of the |
| 123 | // instruction if the LHS is a constant zero (which is the 'negate' form). |
| 124 | // |
| 125 | static inline Value *dyn_castNegInst(Value *V) { |
| 126 | Instruction *I = dyn_cast<Instruction>(V); |
| 127 | if (!I || I->getOpcode() != Instruction::Sub) return 0; |
| 128 | |
| 129 | if (I->getOperand(0) == Constant::getNullValue(I->getType())) |
| 130 | return I->getOperand(1); |
| 131 | return 0; |
| 132 | } |
| 133 | |
Chris Lattner | 31ae863 | 2002-08-14 17:51:49 +0000 | [diff] [blame] | 134 | static inline Value *dyn_castNotInst(Value *V) { |
| 135 | Instruction *I = dyn_cast<Instruction>(V); |
| 136 | if (!I || I->getOpcode() != Instruction::Xor) return 0; |
| 137 | |
| 138 | if (ConstantIntegral *CI = dyn_cast<ConstantIntegral>(I->getOperand(1))) |
| 139 | if (CI->isAllOnesValue()) |
| 140 | return I->getOperand(0); |
| 141 | return 0; |
| 142 | } |
| 143 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 144 | Instruction *InstCombiner::visitAdd(BinaryOperator &I) { |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 145 | bool Changed = SimplifyBinOp(I); |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 146 | Value *LHS = I.getOperand(0), *RHS = I.getOperand(1); |
Chris Lattner | 9fa53de | 2002-05-06 16:49:18 +0000 | [diff] [blame] | 147 | |
| 148 | // Eliminate 'add int %X, 0' |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 149 | if (RHS == Constant::getNullValue(I.getType())) |
| 150 | return ReplaceInstUsesWith(I, LHS); |
Chris Lattner | 9fa53de | 2002-05-06 16:49:18 +0000 | [diff] [blame] | 151 | |
Chris Lattner | 147e975 | 2002-05-08 22:46:53 +0000 | [diff] [blame] | 152 | // -A + B --> B - A |
Chris Lattner | 9fa53de | 2002-05-06 16:49:18 +0000 | [diff] [blame] | 153 | if (Value *V = dyn_castNegInst(LHS)) |
Chris Lattner | 147e975 | 2002-05-08 22:46:53 +0000 | [diff] [blame] | 154 | return BinaryOperator::create(Instruction::Sub, RHS, V); |
Chris Lattner | 9fa53de | 2002-05-06 16:49:18 +0000 | [diff] [blame] | 155 | |
| 156 | // A + -B --> A - B |
| 157 | if (Value *V = dyn_castNegInst(RHS)) |
Chris Lattner | 147e975 | 2002-05-08 22:46:53 +0000 | [diff] [blame] | 158 | return BinaryOperator::create(Instruction::Sub, LHS, V); |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 159 | |
| 160 | // Simplify add instructions with a constant RHS... |
Chris Lattner | 9fa53de | 2002-05-06 16:49:18 +0000 | [diff] [blame] | 161 | if (Constant *Op2 = dyn_cast<Constant>(RHS)) { |
| 162 | if (BinaryOperator *ILHS = dyn_cast<BinaryOperator>(LHS)) { |
| 163 | if (ILHS->getOpcode() == Instruction::Add && |
| 164 | isa<Constant>(ILHS->getOperand(1))) { |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 165 | // Fold: |
| 166 | // %Y = add int %X, 1 |
| 167 | // %Z = add int %Y, 1 |
| 168 | // into: |
| 169 | // %Z = add int %X, 2 |
| 170 | // |
Chris Lattner | 9fa53de | 2002-05-06 16:49:18 +0000 | [diff] [blame] | 171 | if (Constant *Val = *Op2 + *cast<Constant>(ILHS->getOperand(1))) { |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 172 | I.setOperand(0, ILHS->getOperand(0)); |
| 173 | I.setOperand(1, Val); |
| 174 | return &I; |
Chris Lattner | 04805fa | 2002-02-26 21:46:54 +0000 | [diff] [blame] | 175 | } |
Chris Lattner | ca08125 | 2001-12-14 16:52:21 +0000 | [diff] [blame] | 176 | } |
| 177 | } |
Chris Lattner | ca08125 | 2001-12-14 16:52:21 +0000 | [diff] [blame] | 178 | } |
| 179 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 180 | return Changed ? &I : 0; |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 181 | } |
| 182 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 183 | Instruction *InstCombiner::visitSub(BinaryOperator &I) { |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 184 | Value *Op0 = I.getOperand(0), *Op1 = I.getOperand(1); |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 185 | |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 186 | if (Op0 == Op1) // sub X, X -> 0 |
| 187 | return ReplaceInstUsesWith(I, Constant::getNullValue(I.getType())); |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 188 | |
| 189 | // If this is a subtract instruction with a constant RHS, convert it to an add |
| 190 | // instruction of a negative constant |
| 191 | // |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 192 | if (Constant *Op2 = dyn_cast<Constant>(Op1)) |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 193 | if (Constant *RHS = *Constant::getNullValue(I.getType()) - *Op2) // 0 - RHS |
| 194 | return BinaryOperator::create(Instruction::Add, Op0, RHS, I.getName()); |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 195 | |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 196 | // If this is a 'B = x-(-A)', change to B = x+A... |
Chris Lattner | 147e975 | 2002-05-08 22:46:53 +0000 | [diff] [blame] | 197 | if (Value *V = dyn_castNegInst(Op1)) |
| 198 | return BinaryOperator::create(Instruction::Add, Op0, V); |
Chris Lattner | 9fa53de | 2002-05-06 16:49:18 +0000 | [diff] [blame] | 199 | |
Chris Lattner | ad3c495 | 2002-05-09 01:29:19 +0000 | [diff] [blame] | 200 | // Replace (x - (y - z)) with (x + (z - y)) if the (y - z) subexpression is |
| 201 | // not used by anyone else... |
| 202 | // |
| 203 | if (BinaryOperator *Op1I = dyn_cast<BinaryOperator>(Op1)) |
Chris Lattner | 170ed7b | 2002-05-14 16:44:07 +0000 | [diff] [blame] | 204 | if (Op1I->use_size() == 1 && Op1I->getOpcode() == Instruction::Sub) { |
Chris Lattner | ad3c495 | 2002-05-09 01:29:19 +0000 | [diff] [blame] | 205 | // Swap the two operands of the subexpr... |
| 206 | Value *IIOp0 = Op1I->getOperand(0), *IIOp1 = Op1I->getOperand(1); |
| 207 | Op1I->setOperand(0, IIOp1); |
| 208 | Op1I->setOperand(1, IIOp0); |
| 209 | |
| 210 | // Create the new top level add instruction... |
| 211 | return BinaryOperator::create(Instruction::Add, Op0, Op1); |
| 212 | } |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 213 | return 0; |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 214 | } |
| 215 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 216 | Instruction *InstCombiner::visitMul(BinaryOperator &I) { |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 217 | bool Changed = SimplifyBinOp(I); |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 218 | Value *Op1 = I.getOperand(0); |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 219 | |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 220 | // Simplify mul instructions with a constant RHS... |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 221 | if (Constant *Op2 = dyn_cast<Constant>(I.getOperand(1))) { |
Chris Lattner | b0b412e | 2002-09-03 01:08:28 +0000 | [diff] [blame] | 222 | if (I.getType()->isInteger() && cast<ConstantInt>(Op2)->equalsInt(1)) |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 223 | return ReplaceInstUsesWith(I, Op1); // Eliminate 'mul int %X, 1' |
Chris Lattner | 31ba129 | 2002-04-29 22:24:47 +0000 | [diff] [blame] | 224 | |
Chris Lattner | b0b412e | 2002-09-03 01:08:28 +0000 | [diff] [blame] | 225 | if (I.getType()->isInteger() && cast<ConstantInt>(Op2)->equalsInt(2)) |
Chris Lattner | 31ba129 | 2002-04-29 22:24:47 +0000 | [diff] [blame] | 226 | // Convert 'mul int %X, 2' to 'add int %X, %X' |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 227 | return BinaryOperator::create(Instruction::Add, Op1, Op1, I.getName()); |
Chris Lattner | 31ba129 | 2002-04-29 22:24:47 +0000 | [diff] [blame] | 228 | |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 229 | if (Op2->isNullValue()) |
| 230 | return ReplaceInstUsesWith(I, Op2); // Eliminate 'mul int %X, 0' |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 231 | } |
| 232 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 233 | return Changed ? &I : 0; |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 234 | } |
| 235 | |
| 236 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 237 | Instruction *InstCombiner::visitDiv(BinaryOperator &I) { |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 238 | // div X, 1 == X |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 239 | if (ConstantInt *RHS = dyn_cast<ConstantInt>(I.getOperand(1))) |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 240 | if (RHS->equalsInt(1)) |
| 241 | return ReplaceInstUsesWith(I, I.getOperand(0)); |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 242 | return 0; |
| 243 | } |
| 244 | |
| 245 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 246 | Instruction *InstCombiner::visitRem(BinaryOperator &I) { |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 247 | // rem X, 1 == 0 |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 248 | if (ConstantInt *RHS = dyn_cast<ConstantInt>(I.getOperand(1))) |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 249 | if (RHS->equalsInt(1)) |
| 250 | return ReplaceInstUsesWith(I, Constant::getNullValue(I.getType())); |
| 251 | |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 252 | return 0; |
| 253 | } |
| 254 | |
Chris Lattner | 6d14f2a | 2002-08-09 23:47:40 +0000 | [diff] [blame] | 255 | // isMaxValueMinusOne - return true if this is Max-1 |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 256 | static bool isMaxValueMinusOne(const ConstantInt *C) { |
Chris Lattner | 6d14f2a | 2002-08-09 23:47:40 +0000 | [diff] [blame] | 257 | if (const ConstantUInt *CU = dyn_cast<ConstantUInt>(C)) { |
| 258 | // Calculate -1 casted to the right type... |
| 259 | unsigned TypeBits = C->getType()->getPrimitiveSize()*8; |
| 260 | uint64_t Val = ~0ULL; // All ones |
| 261 | Val >>= 64-TypeBits; // Shift out unwanted 1 bits... |
| 262 | return CU->getValue() == Val-1; |
| 263 | } |
| 264 | |
| 265 | const ConstantSInt *CS = cast<ConstantSInt>(C); |
| 266 | |
| 267 | // Calculate 0111111111..11111 |
| 268 | unsigned TypeBits = C->getType()->getPrimitiveSize()*8; |
| 269 | int64_t Val = INT64_MAX; // All ones |
| 270 | Val >>= 64-TypeBits; // Shift out unwanted 1 bits... |
| 271 | return CS->getValue() == Val-1; |
| 272 | } |
| 273 | |
| 274 | // isMinValuePlusOne - return true if this is Min+1 |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 275 | static bool isMinValuePlusOne(const ConstantInt *C) { |
Chris Lattner | 6d14f2a | 2002-08-09 23:47:40 +0000 | [diff] [blame] | 276 | if (const ConstantUInt *CU = dyn_cast<ConstantUInt>(C)) |
| 277 | return CU->getValue() == 1; |
| 278 | |
| 279 | const ConstantSInt *CS = cast<ConstantSInt>(C); |
| 280 | |
| 281 | // Calculate 1111111111000000000000 |
| 282 | unsigned TypeBits = C->getType()->getPrimitiveSize()*8; |
| 283 | int64_t Val = -1; // All ones |
| 284 | Val <<= TypeBits-1; // Shift over to the right spot |
| 285 | return CS->getValue() == Val+1; |
| 286 | } |
| 287 | |
| 288 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 289 | Instruction *InstCombiner::visitAnd(BinaryOperator &I) { |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 290 | bool Changed = SimplifyBinOp(I); |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 291 | Value *Op0 = I.getOperand(0), *Op1 = I.getOperand(1); |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 292 | |
| 293 | // and X, X = X and X, 0 == 0 |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 294 | if (Op0 == Op1 || Op1 == Constant::getNullValue(I.getType())) |
| 295 | return ReplaceInstUsesWith(I, Op1); |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 296 | |
| 297 | // and X, -1 == X |
Chris Lattner | 8328263 | 2002-08-13 17:50:24 +0000 | [diff] [blame] | 298 | if (ConstantIntegral *RHS = dyn_cast<ConstantIntegral>(Op1)) |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 299 | if (RHS->isAllOnesValue()) |
| 300 | return ReplaceInstUsesWith(I, Op0); |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 301 | |
Chris Lattner | 65217ff | 2002-08-23 18:32:43 +0000 | [diff] [blame] | 302 | // and (not A), (not B) == not (or A, B) |
| 303 | if (Op0->use_size() == 1 && Op1->use_size() == 1) |
| 304 | if (Value *A = dyn_castNotInst(Op0)) |
| 305 | if (Value *B = dyn_castNotInst(Op1)) { |
| 306 | Instruction *Or = BinaryOperator::create(Instruction::Or, A, B, |
| 307 | I.getName()+".demorgan"); |
| 308 | InsertNewInstBefore(Or, I); |
| 309 | return BinaryOperator::createNot(Or, I.getName()); |
| 310 | } |
| 311 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 312 | return Changed ? &I : 0; |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 313 | } |
| 314 | |
| 315 | |
| 316 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 317 | Instruction *InstCombiner::visitOr(BinaryOperator &I) { |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 318 | bool Changed = SimplifyBinOp(I); |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 319 | Value *Op0 = I.getOperand(0), *Op1 = I.getOperand(1); |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 320 | |
| 321 | // or X, X = X or X, 0 == X |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 322 | if (Op0 == Op1 || Op1 == Constant::getNullValue(I.getType())) |
| 323 | return ReplaceInstUsesWith(I, Op0); |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 324 | |
| 325 | // or X, -1 == -1 |
Chris Lattner | 8328263 | 2002-08-13 17:50:24 +0000 | [diff] [blame] | 326 | if (ConstantIntegral *RHS = dyn_cast<ConstantIntegral>(Op1)) |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 327 | if (RHS->isAllOnesValue()) |
| 328 | return ReplaceInstUsesWith(I, Op1); |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 329 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 330 | return Changed ? &I : 0; |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 331 | } |
| 332 | |
| 333 | |
| 334 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 335 | Instruction *InstCombiner::visitXor(BinaryOperator &I) { |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 336 | bool Changed = SimplifyBinOp(I); |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 337 | Value *Op0 = I.getOperand(0), *Op1 = I.getOperand(1); |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 338 | |
| 339 | // xor X, X = 0 |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 340 | if (Op0 == Op1) |
| 341 | return ReplaceInstUsesWith(I, Constant::getNullValue(I.getType())); |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 342 | |
Chris Lattner | 8328263 | 2002-08-13 17:50:24 +0000 | [diff] [blame] | 343 | if (ConstantIntegral *Op1C = dyn_cast<ConstantIntegral>(Op1)) { |
Chris Lattner | 6d14f2a | 2002-08-09 23:47:40 +0000 | [diff] [blame] | 344 | // xor X, 0 == X |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 345 | if (Op1C->isNullValue()) |
| 346 | return ReplaceInstUsesWith(I, Op0); |
Chris Lattner | 6d14f2a | 2002-08-09 23:47:40 +0000 | [diff] [blame] | 347 | |
Chris Lattner | b8d6e40 | 2002-08-20 18:24:26 +0000 | [diff] [blame] | 348 | // Is this a "NOT" instruction? |
| 349 | if (Op1C->isAllOnesValue()) { |
| 350 | // xor (xor X, -1), -1 = not (not X) = X |
Chris Lattner | 31ae863 | 2002-08-14 17:51:49 +0000 | [diff] [blame] | 351 | if (Value *X = dyn_castNotInst(Op0)) |
| 352 | return ReplaceInstUsesWith(I, X); |
Chris Lattner | b8d6e40 | 2002-08-20 18:24:26 +0000 | [diff] [blame] | 353 | |
| 354 | // xor (setcc A, B), true = not (setcc A, B) = setncc A, B |
| 355 | if (SetCondInst *SCI = dyn_cast<SetCondInst>(Op0)) |
| 356 | if (SCI->use_size() == 1) |
| 357 | return new SetCondInst(SCI->getInverseCondition(), |
| 358 | SCI->getOperand(0), SCI->getOperand(1)); |
| 359 | } |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 360 | } |
| 361 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 362 | return Changed ? &I : 0; |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 363 | } |
| 364 | |
Chris Lattner | 6d14f2a | 2002-08-09 23:47:40 +0000 | [diff] [blame] | 365 | // AddOne, SubOne - Add or subtract a constant one from an integer constant... |
| 366 | static Constant *AddOne(ConstantInt *C) { |
| 367 | Constant *Result = *C + *ConstantInt::get(C->getType(), 1); |
| 368 | assert(Result && "Constant folding integer addition failed!"); |
| 369 | return Result; |
| 370 | } |
| 371 | static Constant *SubOne(ConstantInt *C) { |
| 372 | Constant *Result = *C - *ConstantInt::get(C->getType(), 1); |
| 373 | assert(Result && "Constant folding integer addition failed!"); |
| 374 | return Result; |
| 375 | } |
| 376 | |
Chris Lattner | 1fc23f3 | 2002-05-09 20:11:54 +0000 | [diff] [blame] | 377 | // isTrueWhenEqual - Return true if the specified setcondinst instruction is |
| 378 | // true when both operands are equal... |
| 379 | // |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 380 | static bool isTrueWhenEqual(Instruction &I) { |
| 381 | return I.getOpcode() == Instruction::SetEQ || |
| 382 | I.getOpcode() == Instruction::SetGE || |
| 383 | I.getOpcode() == Instruction::SetLE; |
Chris Lattner | 1fc23f3 | 2002-05-09 20:11:54 +0000 | [diff] [blame] | 384 | } |
| 385 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 386 | Instruction *InstCombiner::visitSetCondInst(BinaryOperator &I) { |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 387 | bool Changed = SimplifyBinOp(I); |
Chris Lattner | 6d14f2a | 2002-08-09 23:47:40 +0000 | [diff] [blame] | 388 | Value *Op0 = I.getOperand(0), *Op1 = I.getOperand(1); |
| 389 | const Type *Ty = Op0->getType(); |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 390 | |
| 391 | // setcc X, X |
Chris Lattner | 6d14f2a | 2002-08-09 23:47:40 +0000 | [diff] [blame] | 392 | if (Op0 == Op1) |
| 393 | return ReplaceInstUsesWith(I, ConstantBool::get(isTrueWhenEqual(I))); |
Chris Lattner | 1fc23f3 | 2002-05-09 20:11:54 +0000 | [diff] [blame] | 394 | |
| 395 | // setcc <global*>, 0 - Global value addresses are never null! |
Chris Lattner | 6d14f2a | 2002-08-09 23:47:40 +0000 | [diff] [blame] | 396 | if (isa<GlobalValue>(Op0) && isa<ConstantPointerNull>(Op1)) |
| 397 | return ReplaceInstUsesWith(I, ConstantBool::get(!isTrueWhenEqual(I))); |
| 398 | |
| 399 | // setcc's with boolean values can always be turned into bitwise operations |
| 400 | if (Ty == Type::BoolTy) { |
| 401 | // If this is <, >, or !=, we can change this into a simple xor instruction |
| 402 | if (!isTrueWhenEqual(I)) |
| 403 | return BinaryOperator::create(Instruction::Xor, Op0, Op1, I.getName()); |
| 404 | |
| 405 | // Otherwise we need to make a temporary intermediate instruction and insert |
| 406 | // it into the instruction stream. This is what we are after: |
| 407 | // |
| 408 | // seteq bool %A, %B -> ~(A^B) |
| 409 | // setle bool %A, %B -> ~A | B |
| 410 | // setge bool %A, %B -> A | ~B |
| 411 | // |
| 412 | if (I.getOpcode() == Instruction::SetEQ) { // seteq case |
| 413 | Instruction *Xor = BinaryOperator::create(Instruction::Xor, Op0, Op1, |
| 414 | I.getName()+"tmp"); |
| 415 | InsertNewInstBefore(Xor, I); |
Chris Lattner | 31ae863 | 2002-08-14 17:51:49 +0000 | [diff] [blame] | 416 | return BinaryOperator::createNot(Xor, I.getName()); |
Chris Lattner | 6d14f2a | 2002-08-09 23:47:40 +0000 | [diff] [blame] | 417 | } |
| 418 | |
| 419 | // Handle the setXe cases... |
| 420 | assert(I.getOpcode() == Instruction::SetGE || |
| 421 | I.getOpcode() == Instruction::SetLE); |
| 422 | |
| 423 | if (I.getOpcode() == Instruction::SetGE) |
| 424 | std::swap(Op0, Op1); // Change setge -> setle |
| 425 | |
| 426 | // Now we just have the SetLE case. |
Chris Lattner | 31ae863 | 2002-08-14 17:51:49 +0000 | [diff] [blame] | 427 | Instruction *Not = BinaryOperator::createNot(Op0, I.getName()+"tmp"); |
Chris Lattner | 6d14f2a | 2002-08-09 23:47:40 +0000 | [diff] [blame] | 428 | InsertNewInstBefore(Not, I); |
| 429 | return BinaryOperator::create(Instruction::Or, Not, Op1, I.getName()); |
| 430 | } |
| 431 | |
| 432 | // Check to see if we are doing one of many comparisons against constant |
| 433 | // integers at the end of their ranges... |
| 434 | // |
| 435 | if (ConstantInt *CI = dyn_cast<ConstantInt>(Op1)) { |
| 436 | // Check to see if we are comparing against the minimum or maximum value... |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 437 | if (CI->isMinValue()) { |
Chris Lattner | 6d14f2a | 2002-08-09 23:47:40 +0000 | [diff] [blame] | 438 | if (I.getOpcode() == Instruction::SetLT) // A < MIN -> FALSE |
| 439 | return ReplaceInstUsesWith(I, ConstantBool::False); |
| 440 | if (I.getOpcode() == Instruction::SetGE) // A >= MIN -> TRUE |
| 441 | return ReplaceInstUsesWith(I, ConstantBool::True); |
| 442 | if (I.getOpcode() == Instruction::SetLE) // A <= MIN -> A == MIN |
| 443 | return BinaryOperator::create(Instruction::SetEQ, Op0,Op1, I.getName()); |
| 444 | if (I.getOpcode() == Instruction::SetGT) // A > MIN -> A != MIN |
| 445 | return BinaryOperator::create(Instruction::SetNE, Op0,Op1, I.getName()); |
| 446 | |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 447 | } else if (CI->isMaxValue()) { |
Chris Lattner | 6d14f2a | 2002-08-09 23:47:40 +0000 | [diff] [blame] | 448 | if (I.getOpcode() == Instruction::SetGT) // A > MAX -> FALSE |
| 449 | return ReplaceInstUsesWith(I, ConstantBool::False); |
| 450 | if (I.getOpcode() == Instruction::SetLE) // A <= MAX -> TRUE |
| 451 | return ReplaceInstUsesWith(I, ConstantBool::True); |
| 452 | if (I.getOpcode() == Instruction::SetGE) // A >= MAX -> A == MAX |
| 453 | return BinaryOperator::create(Instruction::SetEQ, Op0,Op1, I.getName()); |
| 454 | if (I.getOpcode() == Instruction::SetLT) // A < MAX -> A != MAX |
| 455 | return BinaryOperator::create(Instruction::SetNE, Op0,Op1, I.getName()); |
| 456 | |
| 457 | // Comparing against a value really close to min or max? |
| 458 | } else if (isMinValuePlusOne(CI)) { |
| 459 | if (I.getOpcode() == Instruction::SetLT) // A < MIN+1 -> A == MIN |
| 460 | return BinaryOperator::create(Instruction::SetEQ, Op0, |
| 461 | SubOne(CI), I.getName()); |
| 462 | if (I.getOpcode() == Instruction::SetGE) // A >= MIN-1 -> A != MIN |
| 463 | return BinaryOperator::create(Instruction::SetNE, Op0, |
| 464 | SubOne(CI), I.getName()); |
| 465 | |
| 466 | } else if (isMaxValueMinusOne(CI)) { |
| 467 | if (I.getOpcode() == Instruction::SetGT) // A > MAX-1 -> A == MAX |
| 468 | return BinaryOperator::create(Instruction::SetEQ, Op0, |
| 469 | AddOne(CI), I.getName()); |
| 470 | if (I.getOpcode() == Instruction::SetLE) // A <= MAX-1 -> A != MAX |
| 471 | return BinaryOperator::create(Instruction::SetNE, Op0, |
| 472 | AddOne(CI), I.getName()); |
| 473 | } |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 474 | } |
| 475 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 476 | return Changed ? &I : 0; |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 477 | } |
| 478 | |
| 479 | |
| 480 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 481 | Instruction *InstCombiner::visitShiftInst(Instruction &I) { |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 482 | assert(I.getOperand(1)->getType() == Type::UByteTy); |
| 483 | Value *Op0 = I.getOperand(0), *Op1 = I.getOperand(1); |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 484 | |
| 485 | // shl X, 0 == X and shr X, 0 == X |
| 486 | // shl 0, X == 0 and shr 0, X == 0 |
| 487 | if (Op1 == Constant::getNullValue(Type::UByteTy) || |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 488 | Op0 == Constant::getNullValue(Op0->getType())) |
| 489 | return ReplaceInstUsesWith(I, Op0); |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 490 | |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 491 | // shl uint X, 32 = 0 and shr ubyte Y, 9 = 0, ... just don't eliminate shr of |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 492 | // a signed value. |
| 493 | // |
| 494 | if (ConstantUInt *CUI = dyn_cast<ConstantUInt>(Op1)) { |
Chris Lattner | 55f3d94 | 2002-09-10 23:04:09 +0000 | [diff] [blame] | 495 | if (I.getOpcode() == Instruction::Shr) { |
| 496 | unsigned TypeBits = Op0->getType()->getPrimitiveSize()*8; |
| 497 | if (CUI->getValue() >= TypeBits && !(Op0->getType()->isSigned())) |
| 498 | return ReplaceInstUsesWith(I, Constant::getNullValue(Op0->getType())); |
| 499 | } |
| 500 | |
| 501 | // Check to see if we are shifting left by 1. If so, turn it into an add |
| 502 | // instruction. |
| 503 | if (I.getOpcode() == Instruction::Shl && CUI->equalsInt(1)) |
| 504 | // Convert 'shl int %X, 2' to 'add int %X, %X' |
| 505 | return BinaryOperator::create(Instruction::Add, Op0, Op0, I.getName()); |
| 506 | |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 507 | } |
Chris Lattner | 2e0fb39 | 2002-10-08 16:16:40 +0000 | [diff] [blame] | 508 | |
| 509 | // shr int -1, X = -1 (for any arithmetic shift rights of ~0) |
| 510 | if (ConstantSInt *CSI = dyn_cast<ConstantSInt>(Op0)) |
| 511 | if (I.getOpcode() == Instruction::Shr && CSI->isAllOnesValue()) |
| 512 | return ReplaceInstUsesWith(I, CSI); |
| 513 | |
Chris Lattner | f4cdbf3 | 2002-05-06 16:14:14 +0000 | [diff] [blame] | 514 | return 0; |
| 515 | } |
| 516 | |
| 517 | |
Chris Lattner | 48a44f7 | 2002-05-02 17:06:02 +0000 | [diff] [blame] | 518 | // isEliminableCastOfCast - Return true if it is valid to eliminate the CI |
| 519 | // instruction. |
| 520 | // |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 521 | static inline bool isEliminableCastOfCast(const CastInst &CI, |
Chris Lattner | 48a44f7 | 2002-05-02 17:06:02 +0000 | [diff] [blame] | 522 | const CastInst *CSrc) { |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 523 | assert(CI.getOperand(0) == CSrc); |
Chris Lattner | 48a44f7 | 2002-05-02 17:06:02 +0000 | [diff] [blame] | 524 | const Type *SrcTy = CSrc->getOperand(0)->getType(); |
| 525 | const Type *MidTy = CSrc->getType(); |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 526 | const Type *DstTy = CI.getType(); |
Chris Lattner | 48a44f7 | 2002-05-02 17:06:02 +0000 | [diff] [blame] | 527 | |
Chris Lattner | 650b6da | 2002-08-02 20:00:25 +0000 | [diff] [blame] | 528 | // It is legal to eliminate the instruction if casting A->B->A if the sizes |
| 529 | // are identical and the bits don't get reinterpreted (for example |
Chris Lattner | 0bb7591 | 2002-08-14 23:21:10 +0000 | [diff] [blame] | 530 | // int->float->int would not be allowed) |
Chris Lattner | 650b6da | 2002-08-02 20:00:25 +0000 | [diff] [blame] | 531 | if (SrcTy == DstTy && SrcTy->isLosslesslyConvertableTo(MidTy)) |
| 532 | return true; |
Chris Lattner | 48a44f7 | 2002-05-02 17:06:02 +0000 | [diff] [blame] | 533 | |
| 534 | // Allow free casting and conversion of sizes as long as the sign doesn't |
| 535 | // change... |
Chris Lattner | b0b412e | 2002-09-03 01:08:28 +0000 | [diff] [blame] | 536 | if (SrcTy->isIntegral() && MidTy->isIntegral() && DstTy->isIntegral()) { |
Chris Lattner | 650b6da | 2002-08-02 20:00:25 +0000 | [diff] [blame] | 537 | unsigned SrcSize = SrcTy->getPrimitiveSize(); |
| 538 | unsigned MidSize = MidTy->getPrimitiveSize(); |
| 539 | unsigned DstSize = DstTy->getPrimitiveSize(); |
Chris Lattner | 650b6da | 2002-08-02 20:00:25 +0000 | [diff] [blame] | 540 | |
Chris Lattner | 3732aca | 2002-08-15 16:15:25 +0000 | [diff] [blame] | 541 | // Cases where we are monotonically decreasing the size of the type are |
| 542 | // always ok, regardless of what sign changes are going on. |
| 543 | // |
Chris Lattner | 0bb7591 | 2002-08-14 23:21:10 +0000 | [diff] [blame] | 544 | if (SrcSize >= MidSize && MidSize >= DstSize) |
Chris Lattner | 650b6da | 2002-08-02 20:00:25 +0000 | [diff] [blame] | 545 | return true; |
Chris Lattner | 3732aca | 2002-08-15 16:15:25 +0000 | [diff] [blame] | 546 | |
Chris Lattner | 555518c | 2002-09-23 23:39:43 +0000 | [diff] [blame] | 547 | // Cases where the source and destination type are the same, but the middle |
| 548 | // type is bigger are noops. |
| 549 | // |
| 550 | if (SrcSize == DstSize && MidSize > SrcSize) |
| 551 | return true; |
| 552 | |
Chris Lattner | 3732aca | 2002-08-15 16:15:25 +0000 | [diff] [blame] | 553 | // If we are monotonically growing, things are more complex. |
| 554 | // |
| 555 | if (SrcSize <= MidSize && MidSize <= DstSize) { |
| 556 | // We have eight combinations of signedness to worry about. Here's the |
| 557 | // table: |
| 558 | static const int SignTable[8] = { |
| 559 | // CODE, SrcSigned, MidSigned, DstSigned, Comment |
| 560 | 1, // U U U Always ok |
| 561 | 1, // U U S Always ok |
| 562 | 3, // U S U Ok iff SrcSize != MidSize |
| 563 | 3, // U S S Ok iff SrcSize != MidSize |
| 564 | 0, // S U U Never ok |
| 565 | 2, // S U S Ok iff MidSize == DstSize |
| 566 | 1, // S S U Always ok |
| 567 | 1, // S S S Always ok |
| 568 | }; |
| 569 | |
| 570 | // Choose an action based on the current entry of the signtable that this |
| 571 | // cast of cast refers to... |
| 572 | unsigned Row = SrcTy->isSigned()*4+MidTy->isSigned()*2+DstTy->isSigned(); |
| 573 | switch (SignTable[Row]) { |
| 574 | case 0: return false; // Never ok |
| 575 | case 1: return true; // Always ok |
| 576 | case 2: return MidSize == DstSize; // Ok iff MidSize == DstSize |
| 577 | case 3: // Ok iff SrcSize != MidSize |
| 578 | return SrcSize != MidSize || SrcTy == Type::BoolTy; |
| 579 | default: assert(0 && "Bad entry in sign table!"); |
| 580 | } |
Chris Lattner | 3732aca | 2002-08-15 16:15:25 +0000 | [diff] [blame] | 581 | } |
Chris Lattner | 650b6da | 2002-08-02 20:00:25 +0000 | [diff] [blame] | 582 | } |
Chris Lattner | 48a44f7 | 2002-05-02 17:06:02 +0000 | [diff] [blame] | 583 | |
| 584 | // Otherwise, we cannot succeed. Specifically we do not want to allow things |
| 585 | // like: short -> ushort -> uint, because this can create wrong results if |
| 586 | // the input short is negative! |
| 587 | // |
| 588 | return false; |
| 589 | } |
| 590 | |
| 591 | |
| 592 | // CastInst simplification |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 593 | // |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 594 | Instruction *InstCombiner::visitCastInst(CastInst &CI) { |
Chris Lattner | 48a44f7 | 2002-05-02 17:06:02 +0000 | [diff] [blame] | 595 | // If the user is casting a value to the same type, eliminate this cast |
| 596 | // instruction... |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 597 | if (CI.getType() == CI.getOperand(0)->getType()) |
| 598 | return ReplaceInstUsesWith(CI, CI.getOperand(0)); |
Chris Lattner | 48a44f7 | 2002-05-02 17:06:02 +0000 | [diff] [blame] | 599 | |
Chris Lattner | 48a44f7 | 2002-05-02 17:06:02 +0000 | [diff] [blame] | 600 | // If casting the result of another cast instruction, try to eliminate this |
| 601 | // one! |
| 602 | // |
Chris Lattner | 650b6da | 2002-08-02 20:00:25 +0000 | [diff] [blame] | 603 | if (CastInst *CSrc = dyn_cast<CastInst>(CI.getOperand(0))) { |
Chris Lattner | 48a44f7 | 2002-05-02 17:06:02 +0000 | [diff] [blame] | 604 | if (isEliminableCastOfCast(CI, CSrc)) { |
| 605 | // This instruction now refers directly to the cast's src operand. This |
| 606 | // has a good chance of making CSrc dead. |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 607 | CI.setOperand(0, CSrc->getOperand(0)); |
| 608 | return &CI; |
Chris Lattner | 48a44f7 | 2002-05-02 17:06:02 +0000 | [diff] [blame] | 609 | } |
| 610 | |
Chris Lattner | 650b6da | 2002-08-02 20:00:25 +0000 | [diff] [blame] | 611 | // If this is an A->B->A cast, and we are dealing with integral types, try |
| 612 | // to convert this into a logical 'and' instruction. |
| 613 | // |
| 614 | if (CSrc->getOperand(0)->getType() == CI.getType() && |
Chris Lattner | b0b412e | 2002-09-03 01:08:28 +0000 | [diff] [blame] | 615 | CI.getType()->isInteger() && CSrc->getType()->isInteger() && |
Chris Lattner | 650b6da | 2002-08-02 20:00:25 +0000 | [diff] [blame] | 616 | CI.getType()->isUnsigned() && CSrc->getType()->isUnsigned() && |
| 617 | CSrc->getType()->getPrimitiveSize() < CI.getType()->getPrimitiveSize()){ |
| 618 | assert(CSrc->getType() != Type::ULongTy && |
| 619 | "Cannot have type bigger than ulong!"); |
| 620 | unsigned AndValue = (1U << CSrc->getType()->getPrimitiveSize()*8)-1; |
| 621 | Constant *AndOp = ConstantUInt::get(CI.getType(), AndValue); |
| 622 | return BinaryOperator::create(Instruction::And, CSrc->getOperand(0), |
| 623 | AndOp); |
| 624 | } |
| 625 | } |
| 626 | |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 627 | return 0; |
Chris Lattner | ca08125 | 2001-12-14 16:52:21 +0000 | [diff] [blame] | 628 | } |
| 629 | |
Chris Lattner | 48a44f7 | 2002-05-02 17:06:02 +0000 | [diff] [blame] | 630 | |
Chris Lattner | bbbdd85 | 2002-05-06 18:06:38 +0000 | [diff] [blame] | 631 | // PHINode simplification |
| 632 | // |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 633 | Instruction *InstCombiner::visitPHINode(PHINode &PN) { |
Chris Lattner | bbbdd85 | 2002-05-06 18:06:38 +0000 | [diff] [blame] | 634 | // If the PHI node only has one incoming value, eliminate the PHI node... |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 635 | if (PN.getNumIncomingValues() == 1) |
| 636 | return ReplaceInstUsesWith(PN, PN.getIncomingValue(0)); |
Chris Lattner | 9cd1e66 | 2002-08-20 15:35:35 +0000 | [diff] [blame] | 637 | |
| 638 | // Otherwise if all of the incoming values are the same for the PHI, replace |
| 639 | // the PHI node with the incoming value. |
| 640 | // |
Chris Lattner | f6c0efa | 2002-08-22 20:22:01 +0000 | [diff] [blame] | 641 | Value *InVal = 0; |
| 642 | for (unsigned i = 0, e = PN.getNumIncomingValues(); i != e; ++i) |
| 643 | if (PN.getIncomingValue(i) != &PN) // Not the PHI node itself... |
| 644 | if (InVal && PN.getIncomingValue(i) != InVal) |
| 645 | return 0; // Not the same, bail out. |
| 646 | else |
| 647 | InVal = PN.getIncomingValue(i); |
| 648 | |
| 649 | // The only case that could cause InVal to be null is if we have a PHI node |
| 650 | // that only has entries for itself. In this case, there is no entry into the |
| 651 | // loop, so kill the PHI. |
| 652 | // |
| 653 | if (InVal == 0) InVal = Constant::getNullValue(PN.getType()); |
Chris Lattner | bbbdd85 | 2002-05-06 18:06:38 +0000 | [diff] [blame] | 654 | |
Chris Lattner | 9cd1e66 | 2002-08-20 15:35:35 +0000 | [diff] [blame] | 655 | // All of the incoming values are the same, replace the PHI node now. |
| 656 | return ReplaceInstUsesWith(PN, InVal); |
Chris Lattner | bbbdd85 | 2002-05-06 18:06:38 +0000 | [diff] [blame] | 657 | } |
| 658 | |
Chris Lattner | 48a44f7 | 2002-05-02 17:06:02 +0000 | [diff] [blame] | 659 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 660 | Instruction *InstCombiner::visitGetElementPtrInst(GetElementPtrInst &GEP) { |
Chris Lattner | ae7a0d3 | 2002-08-02 19:29:35 +0000 | [diff] [blame] | 661 | // Is it 'getelementptr %P, uint 0' or 'getelementptr %P' |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 662 | // If so, eliminate the noop. |
Chris Lattner | ae7a0d3 | 2002-08-02 19:29:35 +0000 | [diff] [blame] | 663 | if ((GEP.getNumOperands() == 2 && |
Chris Lattner | 136dab7 | 2002-09-11 01:21:33 +0000 | [diff] [blame] | 664 | GEP.getOperand(1) == Constant::getNullValue(Type::LongTy)) || |
Chris Lattner | e679449 | 2002-08-12 21:17:25 +0000 | [diff] [blame] | 665 | GEP.getNumOperands() == 1) |
| 666 | return ReplaceInstUsesWith(GEP, GEP.getOperand(0)); |
Chris Lattner | 48a44f7 | 2002-05-02 17:06:02 +0000 | [diff] [blame] | 667 | |
Chris Lattner | ae7a0d3 | 2002-08-02 19:29:35 +0000 | [diff] [blame] | 668 | // Combine Indices - If the source pointer to this getelementptr instruction |
| 669 | // is a getelementptr instruction, combine the indices of the two |
| 670 | // getelementptr instructions into a single instruction. |
| 671 | // |
Chris Lattner | c59af1d | 2002-08-17 22:21:59 +0000 | [diff] [blame] | 672 | if (GetElementPtrInst *Src = dyn_cast<GetElementPtrInst>(GEP.getOperand(0))) { |
Chris Lattner | ae7a0d3 | 2002-08-02 19:29:35 +0000 | [diff] [blame] | 673 | std::vector<Value *> Indices; |
Chris Lattner | ca08125 | 2001-12-14 16:52:21 +0000 | [diff] [blame] | 674 | |
Chris Lattner | ae7a0d3 | 2002-08-02 19:29:35 +0000 | [diff] [blame] | 675 | // Can we combine the two pointer arithmetics offsets? |
| 676 | if (Src->getNumOperands() == 2 && isa<Constant>(Src->getOperand(1)) && |
| 677 | isa<Constant>(GEP.getOperand(1))) { |
| 678 | // Replace the index list on this GEP with the index on the getelementptr |
| 679 | Indices.insert(Indices.end(), GEP.idx_begin(), GEP.idx_end()); |
| 680 | Indices[0] = *cast<Constant>(Src->getOperand(1)) + |
| 681 | *cast<Constant>(GEP.getOperand(1)); |
| 682 | assert(Indices[0] != 0 && "Constant folding of uint's failed!?"); |
| 683 | |
Chris Lattner | 5d606a0 | 2002-11-04 16:43:32 +0000 | [diff] [blame^] | 684 | } else if (*GEP.idx_begin() == Constant::getNullValue(Type::LongTy) && |
Chris Lattner | a8339e3 | 2002-09-17 21:05:42 +0000 | [diff] [blame] | 685 | Src->getNumOperands() != 1) { |
Chris Lattner | ae7a0d3 | 2002-08-02 19:29:35 +0000 | [diff] [blame] | 686 | // Otherwise we can do the fold if the first index of the GEP is a zero |
| 687 | Indices.insert(Indices.end(), Src->idx_begin(), Src->idx_end()); |
| 688 | Indices.insert(Indices.end(), GEP.idx_begin()+1, GEP.idx_end()); |
Chris Lattner | 5d606a0 | 2002-11-04 16:43:32 +0000 | [diff] [blame^] | 689 | } else if (Src->getOperand(Src->getNumOperands()-1) == |
| 690 | Constant::getNullValue(Type::LongTy)) { |
| 691 | // If the src gep ends with a constant array index, merge this get into |
| 692 | // it, even if we have a non-zero array index. |
| 693 | Indices.insert(Indices.end(), Src->idx_begin(), Src->idx_end()-1); |
| 694 | Indices.insert(Indices.end(), GEP.idx_begin(), GEP.idx_end()); |
Chris Lattner | ae7a0d3 | 2002-08-02 19:29:35 +0000 | [diff] [blame] | 695 | } |
| 696 | |
| 697 | if (!Indices.empty()) |
| 698 | return new GetElementPtrInst(Src->getOperand(0), Indices, GEP.getName()); |
Chris Lattner | c59af1d | 2002-08-17 22:21:59 +0000 | [diff] [blame] | 699 | |
| 700 | } else if (GlobalValue *GV = dyn_cast<GlobalValue>(GEP.getOperand(0))) { |
| 701 | // GEP of global variable. If all of the indices for this GEP are |
| 702 | // constants, we can promote this to a constexpr instead of an instruction. |
| 703 | |
| 704 | // Scan for nonconstants... |
| 705 | std::vector<Constant*> Indices; |
| 706 | User::op_iterator I = GEP.idx_begin(), E = GEP.idx_end(); |
| 707 | for (; I != E && isa<Constant>(*I); ++I) |
| 708 | Indices.push_back(cast<Constant>(*I)); |
| 709 | |
| 710 | if (I == E) { // If they are all constants... |
| 711 | ConstantExpr *CE = |
| 712 | ConstantExpr::getGetElementPtr(ConstantPointerRef::get(GV), Indices); |
| 713 | |
| 714 | // Replace all uses of the GEP with the new constexpr... |
| 715 | return ReplaceInstUsesWith(GEP, CE); |
| 716 | } |
Chris Lattner | ca08125 | 2001-12-14 16:52:21 +0000 | [diff] [blame] | 717 | } |
| 718 | |
Chris Lattner | ca08125 | 2001-12-14 16:52:21 +0000 | [diff] [blame] | 719 | return 0; |
| 720 | } |
| 721 | |
Chris Lattner | 1085bdf | 2002-11-04 16:18:53 +0000 | [diff] [blame] | 722 | Instruction *InstCombiner::visitAllocationInst(AllocationInst &AI) { |
| 723 | // Convert: malloc Ty, C - where C is a constant != 1 into: malloc [C x Ty], 1 |
| 724 | if (AI.isArrayAllocation()) // Check C != 1 |
| 725 | if (const ConstantUInt *C = dyn_cast<ConstantUInt>(AI.getArraySize())) { |
| 726 | const Type *NewTy = ArrayType::get(AI.getAllocatedType(), C->getValue()); |
| 727 | AllocationInst *New; |
| 728 | |
| 729 | // Create and insert the replacement instruction... |
| 730 | if (isa<MallocInst>(AI)) |
| 731 | New = new MallocInst(NewTy, 0, AI.getName(), &AI); |
| 732 | else if (isa<AllocaInst>(AI)) |
| 733 | New = new AllocaInst(NewTy, 0, AI.getName(), &AI); |
| 734 | |
| 735 | // Scan to the end of the allocation instructions, to skip over a block of |
| 736 | // allocas if possible... |
| 737 | // |
| 738 | BasicBlock::iterator It = New; |
| 739 | while (isa<AllocationInst>(*It)) ++It; |
| 740 | |
| 741 | // Now that I is pointing to the first non-allocation-inst in the block, |
| 742 | // insert our getelementptr instruction... |
| 743 | // |
| 744 | std::vector<Value*> Idx(2, Constant::getNullValue(Type::LongTy)); |
| 745 | Value *V = new GetElementPtrInst(New, Idx, New->getName()+".sub", It); |
| 746 | |
| 747 | // Now make everything use the getelementptr instead of the original |
| 748 | // allocation. |
| 749 | ReplaceInstUsesWith(AI, V); |
| 750 | return &AI; |
| 751 | } |
| 752 | return 0; |
| 753 | } |
| 754 | |
| 755 | |
Chris Lattner | ca08125 | 2001-12-14 16:52:21 +0000 | [diff] [blame] | 756 | |
Chris Lattner | 99f48c6 | 2002-09-02 04:59:56 +0000 | [diff] [blame] | 757 | void InstCombiner::removeFromWorkList(Instruction *I) { |
| 758 | WorkList.erase(std::remove(WorkList.begin(), WorkList.end(), I), |
| 759 | WorkList.end()); |
| 760 | } |
| 761 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 762 | bool InstCombiner::runOnFunction(Function &F) { |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 763 | bool Changed = false; |
Chris Lattner | ca08125 | 2001-12-14 16:52:21 +0000 | [diff] [blame] | 764 | |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 765 | WorkList.insert(WorkList.end(), inst_begin(F), inst_end(F)); |
Chris Lattner | ca08125 | 2001-12-14 16:52:21 +0000 | [diff] [blame] | 766 | |
| 767 | while (!WorkList.empty()) { |
| 768 | Instruction *I = WorkList.back(); // Get an instruction from the worklist |
| 769 | WorkList.pop_back(); |
| 770 | |
Misha Brukman | 632df28 | 2002-10-29 23:06:16 +0000 | [diff] [blame] | 771 | // Check to see if we can DCE or ConstantPropagate the instruction... |
Chris Lattner | 99f48c6 | 2002-09-02 04:59:56 +0000 | [diff] [blame] | 772 | // Check to see if we can DIE the instruction... |
| 773 | if (isInstructionTriviallyDead(I)) { |
| 774 | // Add operands to the worklist... |
| 775 | for (unsigned i = 0, e = I->getNumOperands(); i != e; ++i) |
| 776 | if (Instruction *Op = dyn_cast<Instruction>(I->getOperand(i))) |
| 777 | WorkList.push_back(Op); |
| 778 | |
| 779 | ++NumDeadInst; |
| 780 | BasicBlock::iterator BBI = I; |
| 781 | if (dceInstruction(BBI)) { |
| 782 | removeFromWorkList(I); |
| 783 | continue; |
| 784 | } |
| 785 | } |
| 786 | |
Misha Brukman | 632df28 | 2002-10-29 23:06:16 +0000 | [diff] [blame] | 787 | // Instruction isn't dead, see if we can constant propagate it... |
Chris Lattner | 99f48c6 | 2002-09-02 04:59:56 +0000 | [diff] [blame] | 788 | if (Constant *C = ConstantFoldInstruction(I)) { |
| 789 | // Add operands to the worklist... |
| 790 | for (unsigned i = 0, e = I->getNumOperands(); i != e; ++i) |
| 791 | if (Instruction *Op = dyn_cast<Instruction>(I->getOperand(i))) |
| 792 | WorkList.push_back(Op); |
| 793 | I->replaceAllUsesWith(C); |
| 794 | ++NumConstProp; |
| 795 | BasicBlock::iterator BBI = I; |
| 796 | if (dceInstruction(BBI)) { |
| 797 | removeFromWorkList(I); |
| 798 | continue; |
| 799 | } |
| 800 | } |
| 801 | |
Chris Lattner | ca08125 | 2001-12-14 16:52:21 +0000 | [diff] [blame] | 802 | // Now that we have an instruction, try combining it to simplify it... |
Chris Lattner | ae7a0d3 | 2002-08-02 19:29:35 +0000 | [diff] [blame] | 803 | if (Instruction *Result = visit(*I)) { |
Chris Lattner | 0b18c1d | 2002-05-10 15:38:35 +0000 | [diff] [blame] | 804 | ++NumCombined; |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 805 | // Should we replace the old instruction with a new one? |
Chris Lattner | 053c093 | 2002-05-14 15:24:07 +0000 | [diff] [blame] | 806 | if (Result != I) { |
| 807 | // Instructions can end up on the worklist more than once. Make sure |
| 808 | // we do not process an instruction that has been deleted. |
Chris Lattner | 99f48c6 | 2002-09-02 04:59:56 +0000 | [diff] [blame] | 809 | removeFromWorkList(I); |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 810 | ReplaceInstWithInst(I, Result); |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 811 | } else { |
Chris Lattner | ae7a0d3 | 2002-08-02 19:29:35 +0000 | [diff] [blame] | 812 | BasicBlock::iterator II = I; |
| 813 | |
| 814 | // If the instruction was modified, it's possible that it is now dead. |
| 815 | // if so, remove it. |
| 816 | if (dceInstruction(II)) { |
| 817 | // Instructions may end up in the worklist more than once. Erase them |
| 818 | // all. |
Chris Lattner | 99f48c6 | 2002-09-02 04:59:56 +0000 | [diff] [blame] | 819 | removeFromWorkList(I); |
Chris Lattner | ae7a0d3 | 2002-08-02 19:29:35 +0000 | [diff] [blame] | 820 | Result = 0; |
| 821 | } |
Chris Lattner | 053c093 | 2002-05-14 15:24:07 +0000 | [diff] [blame] | 822 | } |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 823 | |
Chris Lattner | ae7a0d3 | 2002-08-02 19:29:35 +0000 | [diff] [blame] | 824 | if (Result) { |
| 825 | WorkList.push_back(Result); |
| 826 | AddUsesToWorkList(*Result); |
| 827 | } |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 828 | Changed = true; |
Chris Lattner | ca08125 | 2001-12-14 16:52:21 +0000 | [diff] [blame] | 829 | } |
| 830 | } |
| 831 | |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 832 | return Changed; |
Chris Lattner | 04805fa | 2002-02-26 21:46:54 +0000 | [diff] [blame] | 833 | } |
| 834 | |
| 835 | Pass *createInstructionCombiningPass() { |
Chris Lattner | 260ab20 | 2002-04-18 17:39:14 +0000 | [diff] [blame] | 836 | return new InstCombiner(); |
Chris Lattner | 04805fa | 2002-02-26 21:46:54 +0000 | [diff] [blame] | 837 | } |