Implement: -A*-B == A*B
[oota-llvm.git] / lib / Transforms / Scalar / InstructionCombining.cpp
1 //===- InstructionCombining.cpp - Combine multiple instructions -----------===//
2 //
3 // InstructionCombining - Combine instructions to form fewer, simple
4 // instructions.  This pass does not modify the CFG This pass is where algebraic
5 // simplification happens.
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
17 #include "llvm/Transforms/Scalar.h"
18 #include "llvm/Transforms/Utils/BasicBlockUtils.h"
19 #include "llvm/Transforms/Utils/Local.h"
20 #include "llvm/ConstantHandling.h"
21 #include "llvm/iMemory.h"
22 #include "llvm/iOther.h"
23 #include "llvm/iPHINode.h"
24 #include "llvm/iOperators.h"
25 #include "llvm/Pass.h"
26 #include "llvm/DerivedTypes.h"
27 #include "llvm/Support/InstIterator.h"
28 #include "llvm/Support/InstVisitor.h"
29 #include "Support/Statistic.h"
30 #include <algorithm>
31
32 namespace {
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
37   class InstCombiner : public FunctionPass,
38                        public InstVisitor<InstCombiner, Instruction*> {
39     // Worklist of all of the instructions that need to be simplified.
40     std::vector<Instruction*> WorkList;
41
42     void AddUsesToWorkList(Instruction &I) {
43       // The instruction was simplified, add all users of the instruction to
44       // the work lists because they might get more simplified now...
45       //
46       for (Value::use_iterator UI = I.use_begin(), UE = I.use_end();
47            UI != UE; ++UI)
48         WorkList.push_back(cast<Instruction>(*UI));
49     }
50
51     // removeFromWorkList - remove all instances of I from the worklist.
52     void removeFromWorkList(Instruction *I);
53   public:
54     virtual bool runOnFunction(Function &F);
55
56     virtual void getAnalysisUsage(AnalysisUsage &AU) const {
57       AU.setPreservesCFG();
58     }
59
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
64     //     I          - Change was made, I is still valid, I may be dead though
65     //   otherwise    - Change was made, replace I with returned instruction
66     //   
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(ShiftInst &I);
77     Instruction *visitCastInst(CastInst &CI);
78     Instruction *visitPHINode(PHINode &PN);
79     Instruction *visitGetElementPtrInst(GetElementPtrInst &GEP);
80     Instruction *visitAllocationInst(AllocationInst &AI);
81
82     // visitInstruction - Specify what to return for unhandled instructions...
83     Instruction *visitInstruction(Instruction &I) { return 0; }
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) {
89       assert(New && New->getParent() == 0 &&
90              "New instruction already inserted into a basic block!");
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     }
107   };
108
109   RegisterOpt<InstCombiner> X("instcombine", "Combine redundant instructions");
110 }
111
112 // getComplexity:  Assign a complexity or rank value to LLVM Values...
113 //   0 -> Constant, 1 -> Other, 2 -> Argument, 2 -> Unary, 3 -> OtherInst
114 static unsigned getComplexity(Value *V) {
115   if (isa<Instruction>(V)) {
116     if (BinaryOperator::isNeg(V) || BinaryOperator::isNot(V))
117       return 2;
118     return 3;
119   }
120   if (isa<Argument>(V)) return 2;
121   return isa<Constant>(V) ? 0 : 1;
122 }
123
124 // SimplifyCommutative - This performs a few simplifications for commutative
125 // operators:
126 //
127 //  1. Order operands such that they are listed from right (least complex) to
128 //     left (most complex).  This puts constants before unary operators before
129 //     binary operators.
130 //
131 //  2. Handle the case of (op (op V, C1), C2), changing it to:
132 //     (op V, (op C1, C2))
133 //
134 static bool SimplifyCommutative(BinaryOperator &I) {
135   bool Changed = false;
136   if (getComplexity(I.getOperand(0)) < getComplexity(I.getOperand(1)))
137     Changed = !I.swapOperands();
138   
139   if (!I.isAssociative()) return Changed;
140   Instruction::BinaryOps Opcode = I.getOpcode();
141   if (BinaryOperator *Op = dyn_cast<BinaryOperator>(I.getOperand(0))) {
142     if (Op->getOpcode() == Opcode && isa<Constant>(I.getOperand(1)) &&
143         isa<Constant>(Op->getOperand(1))) {
144       Instruction *New = BinaryOperator::create(I.getOpcode(), I.getOperand(1),
145                                                 Op->getOperand(1));
146       Constant *Folded = ConstantFoldInstruction(New);
147       delete New;
148       assert(Folded && "Couldn't constant fold commutative operand?");
149       I.setOperand(0, Op->getOperand(0));
150       I.setOperand(1, Folded);
151       return true;
152     }
153   }
154   return Changed;
155 }
156
157 // dyn_castNegVal - Given a 'sub' instruction, return the RHS of the instruction
158 // if the LHS is a constant zero (which is the 'negate' form).
159 //
160 static inline Value *dyn_castNegVal(Value *V) {
161   if (BinaryOperator::isNeg(V))
162     return BinaryOperator::getNegArgument(cast<BinaryOperator>(V));
163
164   // Constants can be considered to be negated values...
165   if (Constant *C = dyn_cast<Constant>(V)) {
166     Constant *NC = *Constant::getNullValue(V->getType()) - *C;
167     assert(NC && "Couldn't constant fold a subtract!");
168     return NC;
169   }
170   return 0;
171 }
172
173 static inline Value *dyn_castNotVal(Value *V) {
174   if (BinaryOperator::isNot(V))
175     return BinaryOperator::getNotArgument(cast<BinaryOperator>(V));
176
177   // Constants can be considered to be not'ed values...
178   if (ConstantIntegral *C = dyn_cast<ConstantIntegral>(V)) {
179     Constant *NC = *ConstantIntegral::getAllOnesValue(C->getType()) ^ *C;
180     assert(NC && "Couldn't constant fold an exclusive or!");
181     return NC;
182   }
183   return 0;
184 }
185
186 static bool isOnlyUse(Value *V) {
187   return V->use_size() == 1 || isa<Constant>(V);
188 }
189
190
191 // Log2 - Calculate the log base 2 for the specified value if it is exactly a
192 // power of 2.
193 static unsigned Log2(uint64_t Val) {
194   assert(Val > 1 && "Values 0 and 1 should be handled elsewhere!");
195   unsigned Count = 0;
196   while (Val != 1) {
197     if (Val & 1) return 0;    // Multiple bits set?
198     Val >>= 1;
199     ++Count;
200   }
201   return Count;
202 }
203
204 static inline Value *dyn_castFoldableMul(Value *V) {
205   if (V->use_size() == 1 && V->getType()->isInteger())
206     if (Instruction *I = dyn_cast<Instruction>(V))
207       if (I->getOpcode() == Instruction::Mul)
208         if (isa<Constant>(I->getOperand(1)))
209           return I->getOperand(0);
210   return 0;
211 }
212
213
214 Instruction *InstCombiner::visitAdd(BinaryOperator &I) {
215   bool Changed = SimplifyCommutative(I);
216   Value *LHS = I.getOperand(0), *RHS = I.getOperand(1);
217
218   // Eliminate 'add int %X, 0'
219   if (RHS == Constant::getNullValue(I.getType()))
220     return ReplaceInstUsesWith(I, LHS);
221
222   // -A + B  -->  B - A
223   if (Value *V = dyn_castNegVal(LHS))
224     return BinaryOperator::create(Instruction::Sub, RHS, V);
225
226   // A + -B  -->  A - B
227   if (!isa<Constant>(RHS))
228     if (Value *V = dyn_castNegVal(RHS))
229       return BinaryOperator::create(Instruction::Sub, LHS, V);
230
231   // X*C + X --> X * (C+1)
232   if (dyn_castFoldableMul(LHS) == RHS) {
233     Constant *CP1 = *cast<Constant>(cast<Instruction>(LHS)->getOperand(1)) +
234                     *ConstantInt::get(I.getType(), 1);
235     assert(CP1 && "Couldn't constant fold C + 1?");
236     return BinaryOperator::create(Instruction::Mul, RHS, CP1);
237   }
238
239   // X + X*C --> X * (C+1)
240   if (dyn_castFoldableMul(RHS) == LHS) {
241     Constant *CP1 = *cast<Constant>(cast<Instruction>(RHS)->getOperand(1)) +
242                     *ConstantInt::get(I.getType(), 1);
243     assert(CP1 && "Couldn't constant fold C + 1?");
244     return BinaryOperator::create(Instruction::Mul, LHS, CP1);
245   }
246
247   return Changed ? &I : 0;
248 }
249
250 Instruction *InstCombiner::visitSub(BinaryOperator &I) {
251   Value *Op0 = I.getOperand(0), *Op1 = I.getOperand(1);
252
253   if (Op0 == Op1)         // sub X, X  -> 0
254     return ReplaceInstUsesWith(I, Constant::getNullValue(I.getType()));
255
256   // If this is a 'B = x-(-A)', change to B = x+A...
257   if (Value *V = dyn_castNegVal(Op1))
258     return BinaryOperator::create(Instruction::Add, Op0, V);
259
260   // Replace (-1 - A) with (~A)...
261   if (ConstantInt *C = dyn_cast<ConstantInt>(Op0))
262     if (C->isAllOnesValue())
263       return BinaryOperator::createNot(Op1);
264
265   if (BinaryOperator *Op1I = dyn_cast<BinaryOperator>(Op1))
266     if (Op1I->use_size() == 1) {
267       // Replace (x - (y - z)) with (x + (z - y)) if the (y - z) subexpression
268       // is not used by anyone else...
269       //
270       if (Op1I->getOpcode() == Instruction::Sub) {
271         // Swap the two operands of the subexpr...
272         Value *IIOp0 = Op1I->getOperand(0), *IIOp1 = Op1I->getOperand(1);
273         Op1I->setOperand(0, IIOp1);
274         Op1I->setOperand(1, IIOp0);
275         
276         // Create the new top level add instruction...
277         return BinaryOperator::create(Instruction::Add, Op0, Op1);
278       }
279
280       // Replace (A - (A & B)) with (A & ~B) if this is the only use of (A&B)...
281       //
282       if (Op1I->getOpcode() == Instruction::And &&
283           (Op1I->getOperand(0) == Op0 || Op1I->getOperand(1) == Op0)) {
284         Value *OtherOp = Op1I->getOperand(Op1I->getOperand(0) == Op0);
285
286         Instruction *NewNot = BinaryOperator::createNot(OtherOp, "B.not", &I);
287         return BinaryOperator::create(Instruction::And, Op0, NewNot);
288       }
289
290       // X - X*C --> X * (1-C)
291       if (dyn_castFoldableMul(Op1I) == Op0) {
292         Constant *CP1 = *ConstantInt::get(I.getType(), 1) -
293                         *cast<Constant>(cast<Instruction>(Op1)->getOperand(1));
294         assert(CP1 && "Couldn't constant fold 1-C?");
295         return BinaryOperator::create(Instruction::Mul, Op0, CP1);
296       }
297     }
298
299   // X*C - X --> X * (C-1)
300   if (dyn_castFoldableMul(Op0) == Op1) {
301     Constant *CP1 = *cast<Constant>(cast<Instruction>(Op0)->getOperand(1)) -
302                     *ConstantInt::get(I.getType(), 1);
303     assert(CP1 && "Couldn't constant fold C - 1?");
304     return BinaryOperator::create(Instruction::Mul, Op1, CP1);
305   }
306
307   return 0;
308 }
309
310 Instruction *InstCombiner::visitMul(BinaryOperator &I) {
311   bool Changed = SimplifyCommutative(I);
312   Value *Op0 = I.getOperand(0);
313
314   // Simplify mul instructions with a constant RHS...
315   if (Constant *Op1 = dyn_cast<Constant>(I.getOperand(1))) {
316     if (ConstantInt *CI = dyn_cast<ConstantInt>(Op1)) {
317       const Type *Ty = CI->getType();
318       uint64_t Val = Ty->isSigned() ?
319                           (uint64_t)cast<ConstantSInt>(CI)->getValue() : 
320                                     cast<ConstantUInt>(CI)->getValue();
321       switch (Val) {
322       case 0:
323         return ReplaceInstUsesWith(I, Op1);  // Eliminate 'mul double %X, 0'
324       case 1:
325         return ReplaceInstUsesWith(I, Op0);  // Eliminate 'mul int %X, 1'
326       case 2:                     // Convert 'mul int %X, 2' to 'add int %X, %X'
327         return BinaryOperator::create(Instruction::Add, Op0, Op0, I.getName());
328       }
329
330       if (uint64_t C = Log2(Val))            // Replace X*(2^C) with X << C
331         return new ShiftInst(Instruction::Shl, Op0,
332                              ConstantUInt::get(Type::UByteTy, C));
333     } else {
334       ConstantFP *Op1F = cast<ConstantFP>(Op1);
335       if (Op1F->isNullValue())
336         return ReplaceInstUsesWith(I, Op1);
337
338       // "In IEEE floating point, x*1 is not equivalent to x for nans.  However,
339       // ANSI says we can drop signals, so we can do this anyway." (from GCC)
340       if (Op1F->getValue() == 1.0)
341         return ReplaceInstUsesWith(I, Op0);  // Eliminate 'mul double %X, 1.0'
342     }
343   }
344
345   if (Value *Op0v = dyn_castNegVal(Op0))     // -X * -Y = X*Y
346     if (Value *Op1v = dyn_castNegVal(I.getOperand(1)))
347       return BinaryOperator::create(Instruction::Mul, Op0v, Op1v);
348
349   return Changed ? &I : 0;
350 }
351
352 Instruction *InstCombiner::visitDiv(BinaryOperator &I) {
353   // div X, 1 == X
354   if (ConstantInt *RHS = dyn_cast<ConstantInt>(I.getOperand(1))) {
355     if (RHS->equalsInt(1))
356       return ReplaceInstUsesWith(I, I.getOperand(0));
357
358     // Check to see if this is an unsigned division with an exact power of 2,
359     // if so, convert to a right shift.
360     if (ConstantUInt *C = dyn_cast<ConstantUInt>(RHS))
361       if (uint64_t Val = C->getValue())    // Don't break X / 0
362         if (uint64_t C = Log2(Val))
363           return new ShiftInst(Instruction::Shr, I.getOperand(0),
364                                ConstantUInt::get(Type::UByteTy, C));
365   }
366
367   // 0 / X == 0, we don't need to preserve faults!
368   if (ConstantInt *LHS = dyn_cast<ConstantInt>(I.getOperand(0)))
369     if (LHS->equalsInt(0))
370       return ReplaceInstUsesWith(I, Constant::getNullValue(I.getType()));
371
372   return 0;
373 }
374
375
376 Instruction *InstCombiner::visitRem(BinaryOperator &I) {
377   if (ConstantInt *RHS = dyn_cast<ConstantInt>(I.getOperand(1))) {
378     if (RHS->equalsInt(1))  // X % 1 == 0
379       return ReplaceInstUsesWith(I, Constant::getNullValue(I.getType()));
380
381     // Check to see if this is an unsigned remainder with an exact power of 2,
382     // if so, convert to a bitwise and.
383     if (ConstantUInt *C = dyn_cast<ConstantUInt>(RHS))
384       if (uint64_t Val = C->getValue())    // Don't break X % 0 (divide by zero)
385         if (Log2(Val))
386           return BinaryOperator::create(Instruction::And, I.getOperand(0),
387                                         ConstantUInt::get(I.getType(), Val-1));
388   }
389
390   // 0 % X == 0, we don't need to preserve faults!
391   if (ConstantInt *LHS = dyn_cast<ConstantInt>(I.getOperand(0)))
392     if (LHS->equalsInt(0))
393       return ReplaceInstUsesWith(I, Constant::getNullValue(I.getType()));
394
395   return 0;
396 }
397
398 // isMaxValueMinusOne - return true if this is Max-1
399 static bool isMaxValueMinusOne(const ConstantInt *C) {
400   if (const ConstantUInt *CU = dyn_cast<ConstantUInt>(C)) {
401     // Calculate -1 casted to the right type...
402     unsigned TypeBits = C->getType()->getPrimitiveSize()*8;
403     uint64_t Val = ~0ULL;                // All ones
404     Val >>= 64-TypeBits;                 // Shift out unwanted 1 bits...
405     return CU->getValue() == Val-1;
406   }
407
408   const ConstantSInt *CS = cast<ConstantSInt>(C);
409   
410   // Calculate 0111111111..11111
411   unsigned TypeBits = C->getType()->getPrimitiveSize()*8;
412   int64_t Val = INT64_MAX;             // All ones
413   Val >>= 64-TypeBits;                 // Shift out unwanted 1 bits...
414   return CS->getValue() == Val-1;
415 }
416
417 // isMinValuePlusOne - return true if this is Min+1
418 static bool isMinValuePlusOne(const ConstantInt *C) {
419   if (const ConstantUInt *CU = dyn_cast<ConstantUInt>(C))
420     return CU->getValue() == 1;
421
422   const ConstantSInt *CS = cast<ConstantSInt>(C);
423   
424   // Calculate 1111111111000000000000 
425   unsigned TypeBits = C->getType()->getPrimitiveSize()*8;
426   int64_t Val = -1;                    // All ones
427   Val <<= TypeBits-1;                  // Shift over to the right spot
428   return CS->getValue() == Val+1;
429 }
430
431
432 Instruction *InstCombiner::visitAnd(BinaryOperator &I) {
433   bool Changed = SimplifyCommutative(I);
434   Value *Op0 = I.getOperand(0), *Op1 = I.getOperand(1);
435
436   // and X, X = X   and X, 0 == 0
437   if (Op0 == Op1 || Op1 == Constant::getNullValue(I.getType()))
438     return ReplaceInstUsesWith(I, Op1);
439
440   // and X, -1 == X
441   if (ConstantIntegral *RHS = dyn_cast<ConstantIntegral>(Op1))
442     if (RHS->isAllOnesValue())
443       return ReplaceInstUsesWith(I, Op0);
444
445   Value *Op0NotVal = dyn_castNotVal(Op0);
446   Value *Op1NotVal = dyn_castNotVal(Op1);
447
448   // (~A & ~B) == (~(A | B)) - Demorgan's Law
449   if (Op0NotVal && Op1NotVal && isOnlyUse(Op0) && isOnlyUse(Op1)) {
450     Instruction *Or = BinaryOperator::create(Instruction::Or, Op0NotVal,
451                                              Op1NotVal,I.getName()+".demorgan",
452                                              &I);
453     WorkList.push_back(Or);
454     return BinaryOperator::createNot(Or);
455   }
456
457   if (Op0NotVal == Op1 || Op1NotVal == Op0)  // A & ~A  == ~A & A == 0
458     return ReplaceInstUsesWith(I, Constant::getNullValue(I.getType()));
459
460   return Changed ? &I : 0;
461 }
462
463
464
465 Instruction *InstCombiner::visitOr(BinaryOperator &I) {
466   bool Changed = SimplifyCommutative(I);
467   Value *Op0 = I.getOperand(0), *Op1 = I.getOperand(1);
468
469   // or X, X = X   or X, 0 == X
470   if (Op0 == Op1 || Op1 == Constant::getNullValue(I.getType()))
471     return ReplaceInstUsesWith(I, Op0);
472
473   // or X, -1 == -1
474   if (ConstantIntegral *RHS = dyn_cast<ConstantIntegral>(Op1))
475     if (RHS->isAllOnesValue())
476       return ReplaceInstUsesWith(I, Op1);
477
478   Value *Op0NotVal = dyn_castNotVal(Op0);
479   Value *Op1NotVal = dyn_castNotVal(Op1);
480
481   if (Op1 == Op0NotVal)   // ~A | A == -1
482     return ReplaceInstUsesWith(I, 
483                                ConstantIntegral::getAllOnesValue(I.getType()));
484
485   if (Op0 == Op1NotVal)   // A | ~A == -1
486     return ReplaceInstUsesWith(I, 
487                                ConstantIntegral::getAllOnesValue(I.getType()));
488
489   // (~A | ~B) == (~(A & B)) - Demorgan's Law
490   if (Op0NotVal && Op1NotVal && isOnlyUse(Op0) && isOnlyUse(Op1)) {
491     Instruction *And = BinaryOperator::create(Instruction::And, Op0NotVal,
492                                               Op1NotVal,I.getName()+".demorgan",
493                                               &I);
494     WorkList.push_back(And);
495     return BinaryOperator::createNot(And);
496   }
497
498   return Changed ? &I : 0;
499 }
500
501
502
503 Instruction *InstCombiner::visitXor(BinaryOperator &I) {
504   bool Changed = SimplifyCommutative(I);
505   Value *Op0 = I.getOperand(0), *Op1 = I.getOperand(1);
506
507   // xor X, X = 0
508   if (Op0 == Op1)
509     return ReplaceInstUsesWith(I, Constant::getNullValue(I.getType()));
510
511   if (ConstantIntegral *Op1C = dyn_cast<ConstantIntegral>(Op1)) {
512     // xor X, 0 == X
513     if (Op1C->isNullValue())
514       return ReplaceInstUsesWith(I, Op0);
515
516     // Is this a "NOT" instruction?
517     if (Op1C->isAllOnesValue()) {
518       // xor (xor X, -1), -1 = not (not X) = X
519       if (Value *X = dyn_castNotVal(Op0))
520         return ReplaceInstUsesWith(I, X);
521
522       // xor (setcc A, B), true = not (setcc A, B) = setncc A, B
523       if (SetCondInst *SCI = dyn_cast<SetCondInst>(Op0))
524         if (SCI->use_size() == 1)
525           return new SetCondInst(SCI->getInverseCondition(),
526                                  SCI->getOperand(0), SCI->getOperand(1));
527     }
528   }
529
530   if (Value *X = dyn_castNotVal(Op0))   // ~A ^ A == -1
531     if (X == Op1)
532       return ReplaceInstUsesWith(I,
533                                 ConstantIntegral::getAllOnesValue(I.getType()));
534
535   if (Value *X = dyn_castNotVal(Op1))   // A ^ ~A == -1
536     if (X == Op0)
537       return ReplaceInstUsesWith(I,
538                                 ConstantIntegral::getAllOnesValue(I.getType()));
539
540
541
542   if (Instruction *Op1I = dyn_cast<Instruction>(Op1))
543     if (Op1I->getOpcode() == Instruction::Or)
544       if (Op1I->getOperand(0) == Op0) {              // B^(B|A) == (A|B)^B
545         cast<BinaryOperator>(Op1I)->swapOperands();
546         I.swapOperands();
547         std::swap(Op0, Op1);
548       } else if (Op1I->getOperand(1) == Op0) {       // B^(A|B) == (A|B)^B
549         I.swapOperands();
550         std::swap(Op0, Op1);
551       }
552
553   if (Instruction *Op0I = dyn_cast<Instruction>(Op0))
554     if (Op0I->getOpcode() == Instruction::Or && Op0I->use_size() == 1) {
555       if (Op0I->getOperand(0) == Op1)                // (B|A)^B == (A|B)^B
556         cast<BinaryOperator>(Op0I)->swapOperands();
557       if (Op0I->getOperand(1) == Op1) {              // (A|B)^B == A & ~B
558         Value *NotB = BinaryOperator::createNot(Op1, Op1->getName()+".not", &I);
559         WorkList.push_back(cast<Instruction>(NotB));
560         return BinaryOperator::create(Instruction::And, Op0I->getOperand(0),
561                                       NotB);
562       }
563     }
564
565   return Changed ? &I : 0;
566 }
567
568 // AddOne, SubOne - Add or subtract a constant one from an integer constant...
569 static Constant *AddOne(ConstantInt *C) {
570   Constant *Result = *C + *ConstantInt::get(C->getType(), 1);
571   assert(Result && "Constant folding integer addition failed!");
572   return Result;
573 }
574 static Constant *SubOne(ConstantInt *C) {
575   Constant *Result = *C - *ConstantInt::get(C->getType(), 1);
576   assert(Result && "Constant folding integer addition failed!");
577   return Result;
578 }
579
580 // isTrueWhenEqual - Return true if the specified setcondinst instruction is
581 // true when both operands are equal...
582 //
583 static bool isTrueWhenEqual(Instruction &I) {
584   return I.getOpcode() == Instruction::SetEQ ||
585          I.getOpcode() == Instruction::SetGE ||
586          I.getOpcode() == Instruction::SetLE;
587 }
588
589 Instruction *InstCombiner::visitSetCondInst(BinaryOperator &I) {
590   bool Changed = SimplifyCommutative(I);
591   Value *Op0 = I.getOperand(0), *Op1 = I.getOperand(1);
592   const Type *Ty = Op0->getType();
593
594   // setcc X, X
595   if (Op0 == Op1)
596     return ReplaceInstUsesWith(I, ConstantBool::get(isTrueWhenEqual(I)));
597
598   // setcc <global*>, 0 - Global value addresses are never null!
599   if (isa<GlobalValue>(Op0) && isa<ConstantPointerNull>(Op1))
600     return ReplaceInstUsesWith(I, ConstantBool::get(!isTrueWhenEqual(I)));
601
602   // setcc's with boolean values can always be turned into bitwise operations
603   if (Ty == Type::BoolTy) {
604     // If this is <, >, or !=, we can change this into a simple xor instruction
605     if (!isTrueWhenEqual(I))
606       return BinaryOperator::create(Instruction::Xor, Op0, Op1, I.getName());
607
608     // Otherwise we need to make a temporary intermediate instruction and insert
609     // it into the instruction stream.  This is what we are after:
610     //
611     //  seteq bool %A, %B -> ~(A^B)
612     //  setle bool %A, %B -> ~A | B
613     //  setge bool %A, %B -> A | ~B
614     //
615     if (I.getOpcode() == Instruction::SetEQ) {  // seteq case
616       Instruction *Xor = BinaryOperator::create(Instruction::Xor, Op0, Op1,
617                                                 I.getName()+"tmp");
618       InsertNewInstBefore(Xor, I);
619       return BinaryOperator::createNot(Xor, I.getName());
620     }
621
622     // Handle the setXe cases...
623     assert(I.getOpcode() == Instruction::SetGE ||
624            I.getOpcode() == Instruction::SetLE);
625
626     if (I.getOpcode() == Instruction::SetGE)
627       std::swap(Op0, Op1);                   // Change setge -> setle
628
629     // Now we just have the SetLE case.
630     Instruction *Not = BinaryOperator::createNot(Op0, I.getName()+"tmp");
631     InsertNewInstBefore(Not, I);
632     return BinaryOperator::create(Instruction::Or, Not, Op1, I.getName());
633   }
634
635   // Check to see if we are doing one of many comparisons against constant
636   // integers at the end of their ranges...
637   //
638   if (ConstantInt *CI = dyn_cast<ConstantInt>(Op1)) {
639     // Check to see if we are comparing against the minimum or maximum value...
640     if (CI->isMinValue()) {
641       if (I.getOpcode() == Instruction::SetLT)       // A < MIN -> FALSE
642         return ReplaceInstUsesWith(I, ConstantBool::False);
643       if (I.getOpcode() == Instruction::SetGE)       // A >= MIN -> TRUE
644         return ReplaceInstUsesWith(I, ConstantBool::True);
645       if (I.getOpcode() == Instruction::SetLE)       // A <= MIN -> A == MIN
646         return BinaryOperator::create(Instruction::SetEQ, Op0,Op1, I.getName());
647       if (I.getOpcode() == Instruction::SetGT)       // A > MIN -> A != MIN
648         return BinaryOperator::create(Instruction::SetNE, Op0,Op1, I.getName());
649
650     } else if (CI->isMaxValue()) {
651       if (I.getOpcode() == Instruction::SetGT)       // A > MAX -> FALSE
652         return ReplaceInstUsesWith(I, ConstantBool::False);
653       if (I.getOpcode() == Instruction::SetLE)       // A <= MAX -> TRUE
654         return ReplaceInstUsesWith(I, ConstantBool::True);
655       if (I.getOpcode() == Instruction::SetGE)       // A >= MAX -> A == MAX
656         return BinaryOperator::create(Instruction::SetEQ, Op0,Op1, I.getName());
657       if (I.getOpcode() == Instruction::SetLT)       // A < MAX -> A != MAX
658         return BinaryOperator::create(Instruction::SetNE, Op0,Op1, I.getName());
659
660       // Comparing against a value really close to min or max?
661     } else if (isMinValuePlusOne(CI)) {
662       if (I.getOpcode() == Instruction::SetLT)       // A < MIN+1 -> A == MIN
663         return BinaryOperator::create(Instruction::SetEQ, Op0,
664                                       SubOne(CI), I.getName());
665       if (I.getOpcode() == Instruction::SetGE)       // A >= MIN-1 -> A != MIN
666         return BinaryOperator::create(Instruction::SetNE, Op0,
667                                       SubOne(CI), I.getName());
668
669     } else if (isMaxValueMinusOne(CI)) {
670       if (I.getOpcode() == Instruction::SetGT)       // A > MAX-1 -> A == MAX
671         return BinaryOperator::create(Instruction::SetEQ, Op0,
672                                       AddOne(CI), I.getName());
673       if (I.getOpcode() == Instruction::SetLE)       // A <= MAX-1 -> A != MAX
674         return BinaryOperator::create(Instruction::SetNE, Op0,
675                                       AddOne(CI), I.getName());
676     }
677   }
678
679   return Changed ? &I : 0;
680 }
681
682
683
684 Instruction *InstCombiner::visitShiftInst(ShiftInst &I) {
685   assert(I.getOperand(1)->getType() == Type::UByteTy);
686   Value *Op0 = I.getOperand(0), *Op1 = I.getOperand(1);
687
688   // shl X, 0 == X and shr X, 0 == X
689   // shl 0, X == 0 and shr 0, X == 0
690   if (Op1 == Constant::getNullValue(Type::UByteTy) ||
691       Op0 == Constant::getNullValue(Op0->getType()))
692     return ReplaceInstUsesWith(I, Op0);
693
694   // If this is a shift of a shift, see if we can fold the two together...
695   if (ShiftInst *Op0SI = dyn_cast<ShiftInst>(Op0)) {
696     if (isa<Constant>(Op1) && isa<Constant>(Op0SI->getOperand(1))) {
697       ConstantUInt *ShiftAmt1C = cast<ConstantUInt>(Op0SI->getOperand(1));
698       unsigned ShiftAmt1 = ShiftAmt1C->getValue();
699       unsigned ShiftAmt2 = cast<ConstantUInt>(Op1)->getValue();
700
701       // Check for (A << c1) << c2   and   (A >> c1) >> c2
702       if (I.getOpcode() == Op0SI->getOpcode()) {
703         unsigned Amt = ShiftAmt1+ShiftAmt2;   // Fold into one big shift...
704         return new ShiftInst(I.getOpcode(), Op0SI->getOperand(0),
705                              ConstantUInt::get(Type::UByteTy, Amt));
706       }
707
708       if (I.getType()->isUnsigned()) { // Check for (A << c1) >> c2 or visaversa
709         // Calculate bitmask for what gets shifted off the edge...
710         Constant *C = ConstantIntegral::getAllOnesValue(I.getType());
711         if (I.getOpcode() == Instruction::Shr)
712           C = *C >> *ShiftAmt1C;
713         else
714           C = *C << *ShiftAmt1C;
715         assert(C && "Couldn't constant fold shift expression?");
716           
717         Instruction *Mask =
718           BinaryOperator::create(Instruction::And, Op0SI->getOperand(0),
719                                  C, Op0SI->getOperand(0)->getName()+".mask",&I);
720         WorkList.push_back(Mask);
721           
722         // Figure out what flavor of shift we should use...
723         if (ShiftAmt1 == ShiftAmt2)
724           return ReplaceInstUsesWith(I, Mask);  // (A << c) >> c  === A & c2
725         else if (ShiftAmt1 < ShiftAmt2) {
726           return new ShiftInst(I.getOpcode(), Mask,
727                          ConstantUInt::get(Type::UByteTy, ShiftAmt2-ShiftAmt1));
728         } else {
729           return new ShiftInst(Op0SI->getOpcode(), Mask,
730                          ConstantUInt::get(Type::UByteTy, ShiftAmt1-ShiftAmt2));
731         }
732       }
733     }
734   }
735
736   // shl uint X, 32 = 0 and shr ubyte Y, 9 = 0, ... just don't eliminate shr of
737   // a signed value.
738   //
739   if (ConstantUInt *CUI = dyn_cast<ConstantUInt>(Op1)) {
740     unsigned TypeBits = Op0->getType()->getPrimitiveSize()*8;
741     if (CUI->getValue() >= TypeBits &&
742         (!Op0->getType()->isSigned() || I.getOpcode() == Instruction::Shl))
743       return ReplaceInstUsesWith(I, Constant::getNullValue(Op0->getType()));
744
745     // Check to see if we are shifting left by 1.  If so, turn it into an add
746     // instruction.
747     if (I.getOpcode() == Instruction::Shl && CUI->equalsInt(1))
748       // Convert 'shl int %X, 1' to 'add int %X, %X'
749       return BinaryOperator::create(Instruction::Add, Op0, Op0, I.getName());
750
751   }
752
753   // shr int -1, X = -1   (for any arithmetic shift rights of ~0)
754   if (ConstantSInt *CSI = dyn_cast<ConstantSInt>(Op0))
755     if (I.getOpcode() == Instruction::Shr && CSI->isAllOnesValue())
756       return ReplaceInstUsesWith(I, CSI);
757   
758   return 0;
759 }
760
761
762 // isEliminableCastOfCast - Return true if it is valid to eliminate the CI
763 // instruction.
764 //
765 static inline bool isEliminableCastOfCast(const CastInst &CI,
766                                           const CastInst *CSrc) {
767   assert(CI.getOperand(0) == CSrc);
768   const Type *SrcTy = CSrc->getOperand(0)->getType();
769   const Type *MidTy = CSrc->getType();
770   const Type *DstTy = CI.getType();
771
772   // It is legal to eliminate the instruction if casting A->B->A if the sizes
773   // are identical and the bits don't get reinterpreted (for example 
774   // int->float->int would not be allowed)
775   if (SrcTy == DstTy && SrcTy->isLosslesslyConvertableTo(MidTy))
776     return true;
777
778   // Allow free casting and conversion of sizes as long as the sign doesn't
779   // change...
780   if (SrcTy->isIntegral() && MidTy->isIntegral() && DstTy->isIntegral()) {
781     unsigned SrcSize = SrcTy->getPrimitiveSize();
782     unsigned MidSize = MidTy->getPrimitiveSize();
783     unsigned DstSize = DstTy->getPrimitiveSize();
784
785     // Cases where we are monotonically decreasing the size of the type are
786     // always ok, regardless of what sign changes are going on.
787     //
788     if (SrcSize >= MidSize && MidSize >= DstSize)
789       return true;
790
791     // Cases where the source and destination type are the same, but the middle
792     // type is bigger are noops.
793     //
794     if (SrcSize == DstSize && MidSize > SrcSize)
795       return true;
796
797     // If we are monotonically growing, things are more complex.
798     //
799     if (SrcSize <= MidSize && MidSize <= DstSize) {
800       // We have eight combinations of signedness to worry about. Here's the
801       // table:
802       static const int SignTable[8] = {
803         // CODE, SrcSigned, MidSigned, DstSigned, Comment
804         1,     //   U          U          U       Always ok
805         1,     //   U          U          S       Always ok
806         3,     //   U          S          U       Ok iff SrcSize != MidSize
807         3,     //   U          S          S       Ok iff SrcSize != MidSize
808         0,     //   S          U          U       Never ok
809         2,     //   S          U          S       Ok iff MidSize == DstSize
810         1,     //   S          S          U       Always ok
811         1,     //   S          S          S       Always ok
812       };
813
814       // Choose an action based on the current entry of the signtable that this
815       // cast of cast refers to...
816       unsigned Row = SrcTy->isSigned()*4+MidTy->isSigned()*2+DstTy->isSigned();
817       switch (SignTable[Row]) {
818       case 0: return false;              // Never ok
819       case 1: return true;               // Always ok
820       case 2: return MidSize == DstSize; // Ok iff MidSize == DstSize
821       case 3:                            // Ok iff SrcSize != MidSize
822         return SrcSize != MidSize || SrcTy == Type::BoolTy;
823       default: assert(0 && "Bad entry in sign table!");
824       }
825     }
826   }
827
828   // Otherwise, we cannot succeed.  Specifically we do not want to allow things
829   // like:  short -> ushort -> uint, because this can create wrong results if
830   // the input short is negative!
831   //
832   return false;
833 }
834
835
836 // CastInst simplification
837 //
838 Instruction *InstCombiner::visitCastInst(CastInst &CI) {
839   // If the user is casting a value to the same type, eliminate this cast
840   // instruction...
841   if (CI.getType() == CI.getOperand(0)->getType())
842     return ReplaceInstUsesWith(CI, CI.getOperand(0));
843
844   // If casting the result of another cast instruction, try to eliminate this
845   // one!
846   //
847   if (CastInst *CSrc = dyn_cast<CastInst>(CI.getOperand(0))) {
848     if (isEliminableCastOfCast(CI, CSrc)) {
849       // This instruction now refers directly to the cast's src operand.  This
850       // has a good chance of making CSrc dead.
851       CI.setOperand(0, CSrc->getOperand(0));
852       return &CI;
853     }
854
855     // If this is an A->B->A cast, and we are dealing with integral types, try
856     // to convert this into a logical 'and' instruction.
857     //
858     if (CSrc->getOperand(0)->getType() == CI.getType() &&
859         CI.getType()->isInteger() && CSrc->getType()->isInteger() &&
860         CI.getType()->isUnsigned() && CSrc->getType()->isUnsigned() &&
861         CSrc->getType()->getPrimitiveSize() < CI.getType()->getPrimitiveSize()){
862       assert(CSrc->getType() != Type::ULongTy &&
863              "Cannot have type bigger than ulong!");
864       unsigned AndValue = (1U << CSrc->getType()->getPrimitiveSize()*8)-1;
865       Constant *AndOp = ConstantUInt::get(CI.getType(), AndValue);
866       return BinaryOperator::create(Instruction::And, CSrc->getOperand(0),
867                                     AndOp);
868     }
869   }
870
871   return 0;
872 }
873
874
875 // PHINode simplification
876 //
877 Instruction *InstCombiner::visitPHINode(PHINode &PN) {
878   // If the PHI node only has one incoming value, eliminate the PHI node...
879   if (PN.getNumIncomingValues() == 1)
880     return ReplaceInstUsesWith(PN, PN.getIncomingValue(0));
881   
882   // Otherwise if all of the incoming values are the same for the PHI, replace
883   // the PHI node with the incoming value.
884   //
885   Value *InVal = 0;
886   for (unsigned i = 0, e = PN.getNumIncomingValues(); i != e; ++i)
887     if (PN.getIncomingValue(i) != &PN)  // Not the PHI node itself...
888       if (InVal && PN.getIncomingValue(i) != InVal)
889         return 0;  // Not the same, bail out.
890       else
891         InVal = PN.getIncomingValue(i);
892
893   // The only case that could cause InVal to be null is if we have a PHI node
894   // that only has entries for itself.  In this case, there is no entry into the
895   // loop, so kill the PHI.
896   //
897   if (InVal == 0) InVal = Constant::getNullValue(PN.getType());
898
899   // All of the incoming values are the same, replace the PHI node now.
900   return ReplaceInstUsesWith(PN, InVal);
901 }
902
903
904 Instruction *InstCombiner::visitGetElementPtrInst(GetElementPtrInst &GEP) {
905   // Is it 'getelementptr %P, uint 0'  or 'getelementptr %P'
906   // If so, eliminate the noop.
907   if ((GEP.getNumOperands() == 2 &&
908        GEP.getOperand(1) == Constant::getNullValue(Type::LongTy)) ||
909       GEP.getNumOperands() == 1)
910     return ReplaceInstUsesWith(GEP, GEP.getOperand(0));
911
912   // Combine Indices - If the source pointer to this getelementptr instruction
913   // is a getelementptr instruction, combine the indices of the two
914   // getelementptr instructions into a single instruction.
915   //
916   if (GetElementPtrInst *Src = dyn_cast<GetElementPtrInst>(GEP.getOperand(0))) {
917     std::vector<Value *> Indices;
918   
919     // Can we combine the two pointer arithmetics offsets?
920      if (Src->getNumOperands() == 2 && isa<Constant>(Src->getOperand(1)) &&
921          isa<Constant>(GEP.getOperand(1))) {
922       // Replace: gep (gep %P, long C1), long C2, ...
923       // With:    gep %P, long (C1+C2), ...
924       Value *Sum = *cast<Constant>(Src->getOperand(1)) +
925                    *cast<Constant>(GEP.getOperand(1));
926       assert(Sum && "Constant folding of longs failed!?");
927       GEP.setOperand(0, Src->getOperand(0));
928       GEP.setOperand(1, Sum);
929       AddUsesToWorkList(*Src);   // Reduce use count of Src
930       return &GEP;
931     } else if (Src->getNumOperands() == 2 && Src->use_size() == 1) {
932       // Replace: gep (gep %P, long B), long A, ...
933       // With:    T = long A+B; gep %P, T, ...
934       //
935       Value *Sum = BinaryOperator::create(Instruction::Add, Src->getOperand(1),
936                                           GEP.getOperand(1),
937                                           Src->getName()+".sum", &GEP);
938       GEP.setOperand(0, Src->getOperand(0));
939       GEP.setOperand(1, Sum);
940       WorkList.push_back(cast<Instruction>(Sum));
941       return &GEP;
942     } else if (*GEP.idx_begin() == Constant::getNullValue(Type::LongTy) &&
943                Src->getNumOperands() != 1) { 
944       // Otherwise we can do the fold if the first index of the GEP is a zero
945       Indices.insert(Indices.end(), Src->idx_begin(), Src->idx_end());
946       Indices.insert(Indices.end(), GEP.idx_begin()+1, GEP.idx_end());
947     } else if (Src->getOperand(Src->getNumOperands()-1) == 
948                Constant::getNullValue(Type::LongTy)) {
949       // If the src gep ends with a constant array index, merge this get into
950       // it, even if we have a non-zero array index.
951       Indices.insert(Indices.end(), Src->idx_begin(), Src->idx_end()-1);
952       Indices.insert(Indices.end(), GEP.idx_begin(), GEP.idx_end());
953     }
954
955     if (!Indices.empty())
956       return new GetElementPtrInst(Src->getOperand(0), Indices, GEP.getName());
957
958   } else if (GlobalValue *GV = dyn_cast<GlobalValue>(GEP.getOperand(0))) {
959     // GEP of global variable.  If all of the indices for this GEP are
960     // constants, we can promote this to a constexpr instead of an instruction.
961
962     // Scan for nonconstants...
963     std::vector<Constant*> Indices;
964     User::op_iterator I = GEP.idx_begin(), E = GEP.idx_end();
965     for (; I != E && isa<Constant>(*I); ++I)
966       Indices.push_back(cast<Constant>(*I));
967
968     if (I == E) {  // If they are all constants...
969       ConstantExpr *CE =
970         ConstantExpr::getGetElementPtr(ConstantPointerRef::get(GV), Indices);
971
972       // Replace all uses of the GEP with the new constexpr...
973       return ReplaceInstUsesWith(GEP, CE);
974     }
975   }
976
977   return 0;
978 }
979
980 Instruction *InstCombiner::visitAllocationInst(AllocationInst &AI) {
981   // Convert: malloc Ty, C - where C is a constant != 1 into: malloc [C x Ty], 1
982   if (AI.isArrayAllocation())    // Check C != 1
983     if (const ConstantUInt *C = dyn_cast<ConstantUInt>(AI.getArraySize())) {
984       const Type *NewTy = ArrayType::get(AI.getAllocatedType(), C->getValue());
985       AllocationInst *New = 0;
986
987       // Create and insert the replacement instruction...
988       if (isa<MallocInst>(AI))
989         New = new MallocInst(NewTy, 0, AI.getName(), &AI);
990       else {
991         assert(isa<AllocaInst>(AI) && "Unknown type of allocation inst!");
992         New = new AllocaInst(NewTy, 0, AI.getName(), &AI);
993       }
994       
995       // Scan to the end of the allocation instructions, to skip over a block of
996       // allocas if possible...
997       //
998       BasicBlock::iterator It = New;
999       while (isa<AllocationInst>(*It)) ++It;
1000
1001       // Now that I is pointing to the first non-allocation-inst in the block,
1002       // insert our getelementptr instruction...
1003       //
1004       std::vector<Value*> Idx(2, Constant::getNullValue(Type::LongTy));
1005       Value *V = new GetElementPtrInst(New, Idx, New->getName()+".sub", It);
1006
1007       // Now make everything use the getelementptr instead of the original
1008       // allocation.
1009       ReplaceInstUsesWith(AI, V);
1010       return &AI;
1011     }
1012   return 0;
1013 }
1014
1015
1016
1017 void InstCombiner::removeFromWorkList(Instruction *I) {
1018   WorkList.erase(std::remove(WorkList.begin(), WorkList.end(), I),
1019                  WorkList.end());
1020 }
1021
1022 bool InstCombiner::runOnFunction(Function &F) {
1023   bool Changed = false;
1024
1025   WorkList.insert(WorkList.end(), inst_begin(F), inst_end(F));
1026
1027   while (!WorkList.empty()) {
1028     Instruction *I = WorkList.back();  // Get an instruction from the worklist
1029     WorkList.pop_back();
1030
1031     // Check to see if we can DCE or ConstantPropagate the instruction...
1032     // Check to see if we can DIE the instruction...
1033     if (isInstructionTriviallyDead(I)) {
1034       // Add operands to the worklist...
1035       for (unsigned i = 0, e = I->getNumOperands(); i != e; ++i)
1036         if (Instruction *Op = dyn_cast<Instruction>(I->getOperand(i)))
1037           WorkList.push_back(Op);
1038
1039       ++NumDeadInst;
1040       BasicBlock::iterator BBI = I;
1041       if (dceInstruction(BBI)) {
1042         removeFromWorkList(I);
1043         continue;
1044       }
1045     } 
1046
1047     // Instruction isn't dead, see if we can constant propagate it...
1048     if (Constant *C = ConstantFoldInstruction(I)) {
1049       // Add operands to the worklist...
1050       for (unsigned i = 0, e = I->getNumOperands(); i != e; ++i)
1051         if (Instruction *Op = dyn_cast<Instruction>(I->getOperand(i)))
1052           WorkList.push_back(Op);
1053       ReplaceInstUsesWith(*I, C);
1054
1055       ++NumConstProp;
1056       BasicBlock::iterator BBI = I;
1057       if (dceInstruction(BBI)) {
1058         removeFromWorkList(I);
1059         continue;
1060       }
1061     }
1062     
1063     // Now that we have an instruction, try combining it to simplify it...
1064     if (Instruction *Result = visit(*I)) {
1065       ++NumCombined;
1066       // Should we replace the old instruction with a new one?
1067       if (Result != I) {
1068         // Instructions can end up on the worklist more than once.  Make sure
1069         // we do not process an instruction that has been deleted.
1070         removeFromWorkList(I);
1071         ReplaceInstWithInst(I, Result);
1072       } else {
1073         BasicBlock::iterator II = I;
1074
1075         // If the instruction was modified, it's possible that it is now dead.
1076         // if so, remove it.
1077         if (dceInstruction(II)) {
1078           // Instructions may end up in the worklist more than once.  Erase them
1079           // all.
1080           removeFromWorkList(I);
1081           Result = 0;
1082         }
1083       }
1084
1085       if (Result) {
1086         WorkList.push_back(Result);
1087         AddUsesToWorkList(*Result);
1088       }
1089       Changed = true;
1090     }
1091   }
1092
1093   return Changed;
1094 }
1095
1096 Pass *createInstructionCombiningPass() {
1097   return new InstCombiner();
1098 }