Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 1 | //===-- IPConstantPropagation.cpp - Propagate constants through calls -----===// |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 2 | // |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
Chris Lattner | 4ee451d | 2007-12-29 20:36:04 +0000 | [diff] [blame] | 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 7 | // |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 8 | //===----------------------------------------------------------------------===// |
| 9 | // |
| 10 | // This pass implements an _extremely_ simple interprocedural constant |
| 11 | // propagation pass. It could certainly be improved in many different ways, |
| 12 | // like using a worklist. This pass makes arguments dead, but does not remove |
| 13 | // them. The existing dead argument elimination pass should be run after this |
| 14 | // to clean up the mess. |
| 15 | // |
| 16 | //===----------------------------------------------------------------------===// |
| 17 | |
Chris Lattner | 86453c5 | 2006-12-19 22:09:18 +0000 | [diff] [blame] | 18 | #define DEBUG_TYPE "ipconstprop" |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 19 | #include "llvm/Transforms/IPO.h" |
Chris Lattner | a990d94 | 2004-11-14 06:10:11 +0000 | [diff] [blame] | 20 | #include "llvm/Constants.h" |
| 21 | #include "llvm/Instructions.h" |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 22 | #include "llvm/Module.h" |
| 23 | #include "llvm/Pass.h" |
Matthijs Kooijman | 8b0fcf3 | 2008-06-18 08:09:27 +0000 | [diff] [blame] | 24 | #include "llvm/Analysis/ValueTracking.h" |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 25 | #include "llvm/Support/CallSite.h" |
Reid Spencer | 551ccae | 2004-09-01 22:55:40 +0000 | [diff] [blame] | 26 | #include "llvm/ADT/Statistic.h" |
Devang Patel | 7db30ba | 2008-03-11 22:24:29 +0000 | [diff] [blame] | 27 | #include "llvm/ADT/SmallVector.h" |
Chris Lattner | 1e2385b | 2003-11-21 21:54:22 +0000 | [diff] [blame] | 28 | using namespace llvm; |
Brian Gaeke | d0fde30 | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 29 | |
Chris Lattner | 86453c5 | 2006-12-19 22:09:18 +0000 | [diff] [blame] | 30 | STATISTIC(NumArgumentsProped, "Number of args turned into constants"); |
| 31 | STATISTIC(NumReturnValProped, "Number of return values turned into constants"); |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 32 | |
Chris Lattner | 86453c5 | 2006-12-19 22:09:18 +0000 | [diff] [blame] | 33 | namespace { |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 34 | /// IPCP - The interprocedural constant propagation pass |
| 35 | /// |
Nick Lewycky | 6726b6d | 2009-10-25 06:33:48 +0000 | [diff] [blame] | 36 | struct IPCP : public ModulePass { |
Nick Lewycky | ecd94c8 | 2007-05-06 13:37:16 +0000 | [diff] [blame] | 37 | static char ID; // Pass identification, replacement for typeid |
Owen Anderson | 081c34b | 2010-10-19 17:21:58 +0000 | [diff] [blame] | 38 | IPCP() : ModulePass(ID) { |
| 39 | initializeIPCPPass(*PassRegistry::getPassRegistry()); |
| 40 | } |
Devang Patel | 794fd75 | 2007-05-01 21:15:47 +0000 | [diff] [blame] | 41 | |
Chris Lattner | b12914b | 2004-09-20 04:48:05 +0000 | [diff] [blame] | 42 | bool runOnModule(Module &M); |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 43 | private: |
Chris Lattner | a990d94 | 2004-11-14 06:10:11 +0000 | [diff] [blame] | 44 | bool PropagateConstantsIntoArguments(Function &F); |
| 45 | bool PropagateConstantReturn(Function &F); |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 46 | }; |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 47 | } |
| 48 | |
Dan Gohman | 844731a | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 49 | char IPCP::ID = 0; |
Owen Anderson | d13db2c | 2010-07-21 22:09:45 +0000 | [diff] [blame] | 50 | INITIALIZE_PASS(IPCP, "ipconstprop", |
Owen Anderson | ce665bd | 2010-10-07 22:25:06 +0000 | [diff] [blame] | 51 | "Interprocedural constant propagation", false, false) |
Dan Gohman | 844731a | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 52 | |
Chris Lattner | b12914b | 2004-09-20 04:48:05 +0000 | [diff] [blame] | 53 | ModulePass *llvm::createIPConstantPropagationPass() { return new IPCP(); } |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 54 | |
Chris Lattner | b12914b | 2004-09-20 04:48:05 +0000 | [diff] [blame] | 55 | bool IPCP::runOnModule(Module &M) { |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 56 | bool Changed = false; |
Chris Lattner | 2e8dfb8 | 2003-10-27 21:09:00 +0000 | [diff] [blame] | 57 | bool LocalChange = true; |
| 58 | |
| 59 | // FIXME: instead of using smart algorithms, we just iterate until we stop |
| 60 | // making changes. |
| 61 | while (LocalChange) { |
| 62 | LocalChange = false; |
| 63 | for (Module::iterator I = M.begin(), E = M.end(); I != E; ++I) |
Reid Spencer | 5cbf985 | 2007-01-30 20:08:39 +0000 | [diff] [blame] | 64 | if (!I->isDeclaration()) { |
Chris Lattner | a990d94 | 2004-11-14 06:10:11 +0000 | [diff] [blame] | 65 | // Delete any klingons. |
| 66 | I->removeDeadConstantUsers(); |
Rafael Espindola | bb46f52 | 2009-01-15 20:18:42 +0000 | [diff] [blame] | 67 | if (I->hasLocalLinkage()) |
Chris Lattner | a990d94 | 2004-11-14 06:10:11 +0000 | [diff] [blame] | 68 | LocalChange |= PropagateConstantsIntoArguments(*I); |
| 69 | Changed |= PropagateConstantReturn(*I); |
| 70 | } |
Chris Lattner | 2e8dfb8 | 2003-10-27 21:09:00 +0000 | [diff] [blame] | 71 | Changed |= LocalChange; |
| 72 | } |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 73 | return Changed; |
| 74 | } |
| 75 | |
Chris Lattner | a990d94 | 2004-11-14 06:10:11 +0000 | [diff] [blame] | 76 | /// PropagateConstantsIntoArguments - Look at all uses of the specified |
| 77 | /// function. If all uses are direct call sites, and all pass a particular |
| 78 | /// constant in for an argument, propagate that constant in as the argument. |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 79 | /// |
Chris Lattner | a990d94 | 2004-11-14 06:10:11 +0000 | [diff] [blame] | 80 | bool IPCP::PropagateConstantsIntoArguments(Function &F) { |
Chris Lattner | 7f8897f | 2006-08-27 22:42:52 +0000 | [diff] [blame] | 81 | if (F.arg_empty() || F.use_empty()) return false; // No arguments? Early exit. |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 82 | |
Chris Lattner | 4af6ad3 | 2008-04-23 06:16:27 +0000 | [diff] [blame] | 83 | // For each argument, keep track of its constant value and whether it is a |
| 84 | // constant or not. The bool is driven to true when found to be non-constant. |
| 85 | SmallVector<std::pair<Constant*, bool>, 16> ArgumentConstants; |
Chris Lattner | e4d5c44 | 2005-03-15 04:54:21 +0000 | [diff] [blame] | 86 | ArgumentConstants.resize(F.arg_size()); |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 87 | |
| 88 | unsigned NumNonconstant = 0; |
Chris Lattner | 4af6ad3 | 2008-04-23 06:16:27 +0000 | [diff] [blame] | 89 | for (Value::use_iterator UI = F.use_begin(), E = F.use_end(); UI != E; ++UI) { |
Gabor Greif | 65b9212 | 2010-07-12 14:14:03 +0000 | [diff] [blame] | 90 | User *U = *UI; |
Chris Lattner | b271004 | 2009-11-01 06:11:53 +0000 | [diff] [blame] | 91 | // Ignore blockaddress uses. |
Gabor Greif | 65b9212 | 2010-07-12 14:14:03 +0000 | [diff] [blame] | 92 | if (isa<BlockAddress>(U)) continue; |
Chris Lattner | b271004 | 2009-11-01 06:11:53 +0000 | [diff] [blame] | 93 | |
Chris Lattner | 4af6ad3 | 2008-04-23 06:16:27 +0000 | [diff] [blame] | 94 | // Used by a non-instruction, or not the callee of a function, do not |
| 95 | // transform. |
Gabor Greif | 65b9212 | 2010-07-12 14:14:03 +0000 | [diff] [blame] | 96 | if (!isa<CallInst>(U) && !isa<InvokeInst>(U)) |
Chris Lattner | 4af6ad3 | 2008-04-23 06:16:27 +0000 | [diff] [blame] | 97 | return false; |
| 98 | |
Gabor Greif | 7d3056b | 2010-07-28 22:50:26 +0000 | [diff] [blame] | 99 | CallSite CS(cast<Instruction>(U)); |
Gabor Greif | edc4d69 | 2009-01-22 21:35:57 +0000 | [diff] [blame] | 100 | if (!CS.isCallee(UI)) |
| 101 | return false; |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 102 | |
Chris Lattner | 4af6ad3 | 2008-04-23 06:16:27 +0000 | [diff] [blame] | 103 | // Check out all of the potentially constant arguments. Note that we don't |
| 104 | // inspect varargs here. |
| 105 | CallSite::arg_iterator AI = CS.arg_begin(); |
| 106 | Function::arg_iterator Arg = F.arg_begin(); |
| 107 | for (unsigned i = 0, e = ArgumentConstants.size(); i != e; |
| 108 | ++i, ++AI, ++Arg) { |
| 109 | |
| 110 | // If this argument is known non-constant, ignore it. |
| 111 | if (ArgumentConstants[i].second) |
| 112 | continue; |
| 113 | |
| 114 | Constant *C = dyn_cast<Constant>(*AI); |
| 115 | if (C && ArgumentConstants[i].first == 0) { |
| 116 | ArgumentConstants[i].first = C; // First constant seen. |
| 117 | } else if (C && ArgumentConstants[i].first == C) { |
| 118 | // Still the constant value we think it is. |
| 119 | } else if (*AI == &*Arg) { |
| 120 | // Ignore recursive calls passing argument down. |
| 121 | } else { |
| 122 | // Argument became non-constant. If all arguments are non-constant now, |
| 123 | // give up on this function. |
| 124 | if (++NumNonconstant == ArgumentConstants.size()) |
| 125 | return false; |
| 126 | ArgumentConstants[i].second = true; |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 127 | } |
| 128 | } |
Chris Lattner | 4af6ad3 | 2008-04-23 06:16:27 +0000 | [diff] [blame] | 129 | } |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 130 | |
| 131 | // If we got to this point, there is a constant argument! |
| 132 | assert(NumNonconstant != ArgumentConstants.size()); |
Chris Lattner | 2e8dfb8 | 2003-10-27 21:09:00 +0000 | [diff] [blame] | 133 | bool MadeChange = false; |
Chris Lattner | 4af6ad3 | 2008-04-23 06:16:27 +0000 | [diff] [blame] | 134 | Function::arg_iterator AI = F.arg_begin(); |
| 135 | for (unsigned i = 0, e = ArgumentConstants.size(); i != e; ++i, ++AI) { |
| 136 | // Do we have a constant argument? |
Torok Edwin | c338499 | 2009-09-24 18:33:42 +0000 | [diff] [blame] | 137 | if (ArgumentConstants[i].second || AI->use_empty() || |
| 138 | (AI->hasByValAttr() && !F.onlyReadsMemory())) |
Chris Lattner | 4af6ad3 | 2008-04-23 06:16:27 +0000 | [diff] [blame] | 139 | continue; |
| 140 | |
| 141 | Value *V = ArgumentConstants[i].first; |
Owen Anderson | 9e9a0d5 | 2009-07-30 23:03:37 +0000 | [diff] [blame] | 142 | if (V == 0) V = UndefValue::get(AI->getType()); |
Chris Lattner | 4af6ad3 | 2008-04-23 06:16:27 +0000 | [diff] [blame] | 143 | AI->replaceAllUsesWith(V); |
| 144 | ++NumArgumentsProped; |
| 145 | MadeChange = true; |
| 146 | } |
Chris Lattner | 2e8dfb8 | 2003-10-27 21:09:00 +0000 | [diff] [blame] | 147 | return MadeChange; |
Chris Lattner | d358e6f | 2003-10-23 16:52:27 +0000 | [diff] [blame] | 148 | } |
Brian Gaeke | d0fde30 | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 149 | |
Chris Lattner | a990d94 | 2004-11-14 06:10:11 +0000 | [diff] [blame] | 150 | |
Matthijs Kooijman | 8b0fcf3 | 2008-06-18 08:09:27 +0000 | [diff] [blame] | 151 | // Check to see if this function returns one or more constants. If so, replace |
| 152 | // all callers that use those return values with the constant value. This will |
| 153 | // leave in the actual return values and instructions, but deadargelim will |
| 154 | // clean that up. |
Matthijs Kooijman | c2afe89 | 2008-06-18 08:30:37 +0000 | [diff] [blame] | 155 | // |
| 156 | // Additionally if a function always returns one of its arguments directly, |
| 157 | // callers will be updated to use the value they pass in directly instead of |
| 158 | // using the return value. |
Chris Lattner | a990d94 | 2004-11-14 06:10:11 +0000 | [diff] [blame] | 159 | bool IPCP::PropagateConstantReturn(Function &F) { |
Nick Lewycky | ae3d802 | 2009-11-23 03:29:18 +0000 | [diff] [blame] | 160 | if (F.getReturnType()->isVoidTy()) |
Chris Lattner | a990d94 | 2004-11-14 06:10:11 +0000 | [diff] [blame] | 161 | return false; // No return value. |
| 162 | |
Chris Lattner | 18d73c2 | 2008-06-09 07:58:07 +0000 | [diff] [blame] | 163 | // If this function could be overridden later in the link stage, we can't |
| 164 | // propagate information about its results into callers. |
Nuno Lopes | 5ed3b89 | 2008-09-29 14:40:32 +0000 | [diff] [blame] | 165 | if (F.mayBeOverridden()) |
Chris Lattner | 18d73c2 | 2008-06-09 07:58:07 +0000 | [diff] [blame] | 166 | return false; |
Owen Anderson | e922c02 | 2009-07-22 00:24:57 +0000 | [diff] [blame] | 167 | |
Chris Lattner | a990d94 | 2004-11-14 06:10:11 +0000 | [diff] [blame] | 168 | // Check to see if this function returns a constant. |
Devang Patel | 7db30ba | 2008-03-11 22:24:29 +0000 | [diff] [blame] | 169 | SmallVector<Value *,4> RetVals; |
Devang Patel | 7db30ba | 2008-03-11 22:24:29 +0000 | [diff] [blame] | 170 | const StructType *STy = dyn_cast<StructType>(F.getReturnType()); |
| 171 | if (STy) |
Matthijs Kooijman | 8b0fcf3 | 2008-06-18 08:09:27 +0000 | [diff] [blame] | 172 | for (unsigned i = 0, e = STy->getNumElements(); i < e; ++i) |
Owen Anderson | 9e9a0d5 | 2009-07-30 23:03:37 +0000 | [diff] [blame] | 173 | RetVals.push_back(UndefValue::get(STy->getElementType(i))); |
Devang Patel | 7db30ba | 2008-03-11 22:24:29 +0000 | [diff] [blame] | 174 | else |
Owen Anderson | 9e9a0d5 | 2009-07-30 23:03:37 +0000 | [diff] [blame] | 175 | RetVals.push_back(UndefValue::get(F.getReturnType())); |
Devang Patel | 7db30ba | 2008-03-11 22:24:29 +0000 | [diff] [blame] | 176 | |
Matthijs Kooijman | 8b0fcf3 | 2008-06-18 08:09:27 +0000 | [diff] [blame] | 177 | unsigned NumNonConstant = 0; |
Chris Lattner | a990d94 | 2004-11-14 06:10:11 +0000 | [diff] [blame] | 178 | for (Function::iterator BB = F.begin(), E = F.end(); BB != E; ++BB) |
Anton Korobeynikov | 07e6e56 | 2008-02-20 11:26:25 +0000 | [diff] [blame] | 179 | if (ReturnInst *RI = dyn_cast<ReturnInst>(BB->getTerminator())) { |
Chris Lattner | e9625c6 | 2008-04-23 05:59:23 +0000 | [diff] [blame] | 180 | for (unsigned i = 0, e = RetVals.size(); i != e; ++i) { |
Matthijs Kooijman | 8b0fcf3 | 2008-06-18 08:09:27 +0000 | [diff] [blame] | 181 | // Already found conflicting return values? |
Chris Lattner | e9625c6 | 2008-04-23 05:59:23 +0000 | [diff] [blame] | 182 | Value *RV = RetVals[i]; |
Matthijs Kooijman | 8b0fcf3 | 2008-06-18 08:09:27 +0000 | [diff] [blame] | 183 | if (!RV) |
| 184 | continue; |
| 185 | |
| 186 | // Find the returned value |
| 187 | Value *V; |
Dan Gohman | 792e1e9 | 2008-10-03 22:21:24 +0000 | [diff] [blame] | 188 | if (!STy) |
Matthijs Kooijman | 8b0fcf3 | 2008-06-18 08:09:27 +0000 | [diff] [blame] | 189 | V = RI->getOperand(i); |
| 190 | else |
Nick Lewycky | ae3d802 | 2009-11-23 03:29:18 +0000 | [diff] [blame] | 191 | V = FindInsertedValue(RI->getOperand(0), i); |
Matthijs Kooijman | 8b0fcf3 | 2008-06-18 08:09:27 +0000 | [diff] [blame] | 192 | |
| 193 | if (V) { |
| 194 | // Ignore undefs, we can change them into anything |
| 195 | if (isa<UndefValue>(V)) |
| 196 | continue; |
| 197 | |
Matthijs Kooijman | c2afe89 | 2008-06-18 08:30:37 +0000 | [diff] [blame] | 198 | // Try to see if all the rets return the same constant or argument. |
| 199 | if (isa<Constant>(V) || isa<Argument>(V)) { |
Matthijs Kooijman | 8b0fcf3 | 2008-06-18 08:09:27 +0000 | [diff] [blame] | 200 | if (isa<UndefValue>(RV)) { |
| 201 | // No value found yet? Try the current one. |
| 202 | RetVals[i] = V; |
| 203 | continue; |
| 204 | } |
| 205 | // Returning the same value? Good. |
| 206 | if (RV == V) |
| 207 | continue; |
| 208 | } |
| 209 | } |
| 210 | // Different or no known return value? Don't propagate this return |
| 211 | // value. |
| 212 | RetVals[i] = 0; |
| 213 | // All values non constant? Stop looking. |
| 214 | if (++NumNonConstant == RetVals.size()) |
| 215 | return false; |
Chris Lattner | a990d94 | 2004-11-14 06:10:11 +0000 | [diff] [blame] | 216 | } |
Anton Korobeynikov | 07e6e56 | 2008-02-20 11:26:25 +0000 | [diff] [blame] | 217 | } |
Chris Lattner | a990d94 | 2004-11-14 06:10:11 +0000 | [diff] [blame] | 218 | |
Matthijs Kooijman | 8b0fcf3 | 2008-06-18 08:09:27 +0000 | [diff] [blame] | 219 | // If we got here, the function returns at least one constant value. Loop |
| 220 | // over all users, replacing any uses of the return value with the returned |
| 221 | // constant. |
Chris Lattner | a990d94 | 2004-11-14 06:10:11 +0000 | [diff] [blame] | 222 | bool MadeChange = false; |
Chris Lattner | e9625c6 | 2008-04-23 05:59:23 +0000 | [diff] [blame] | 223 | for (Value::use_iterator UI = F.use_begin(), E = F.use_end(); UI != E; ++UI) { |
Gabor Greif | 7d3056b | 2010-07-28 22:50:26 +0000 | [diff] [blame] | 224 | CallSite CS(*UI); |
Matthijs Kooijman | f0da203 | 2008-06-19 08:53:24 +0000 | [diff] [blame] | 225 | Instruction* Call = CS.getInstruction(); |
| 226 | |
| 227 | // Not a call instruction or a call instruction that's not calling F |
| 228 | // directly? |
Gabor Greif | edc4d69 | 2009-01-22 21:35:57 +0000 | [diff] [blame] | 229 | if (!Call || !CS.isCallee(UI)) |
Chris Lattner | e9625c6 | 2008-04-23 05:59:23 +0000 | [diff] [blame] | 230 | continue; |
Chris Lattner | e9625c6 | 2008-04-23 05:59:23 +0000 | [diff] [blame] | 231 | |
Matthijs Kooijman | f0da203 | 2008-06-19 08:53:24 +0000 | [diff] [blame] | 232 | // Call result not used? |
Chris Lattner | e9625c6 | 2008-04-23 05:59:23 +0000 | [diff] [blame] | 233 | if (Call->use_empty()) |
| 234 | continue; |
Chris Lattner | a990d94 | 2004-11-14 06:10:11 +0000 | [diff] [blame] | 235 | |
Chris Lattner | e9625c6 | 2008-04-23 05:59:23 +0000 | [diff] [blame] | 236 | MadeChange = true; |
| 237 | |
| 238 | if (STy == 0) { |
Matthijs Kooijman | c2afe89 | 2008-06-18 08:30:37 +0000 | [diff] [blame] | 239 | Value* New = RetVals[0]; |
| 240 | if (Argument *A = dyn_cast<Argument>(New)) |
| 241 | // Was an argument returned? Then find the corresponding argument in |
Matthijs Kooijman | f0da203 | 2008-06-19 08:53:24 +0000 | [diff] [blame] | 242 | // the call instruction and use that. |
| 243 | New = CS.getArgument(A->getArgNo()); |
Matthijs Kooijman | c2afe89 | 2008-06-18 08:30:37 +0000 | [diff] [blame] | 244 | Call->replaceAllUsesWith(New); |
Chris Lattner | e9625c6 | 2008-04-23 05:59:23 +0000 | [diff] [blame] | 245 | continue; |
| 246 | } |
| 247 | |
Matthijs Kooijman | 8b0fcf3 | 2008-06-18 08:09:27 +0000 | [diff] [blame] | 248 | for (Value::use_iterator I = Call->use_begin(), E = Call->use_end(); |
| 249 | I != E;) { |
Jay Foad | 0906b1b | 2009-06-06 17:49:35 +0000 | [diff] [blame] | 250 | Instruction *Ins = cast<Instruction>(*I); |
Matthijs Kooijman | 8b0fcf3 | 2008-06-18 08:09:27 +0000 | [diff] [blame] | 251 | |
| 252 | // Increment now, so we can remove the use |
| 253 | ++I; |
| 254 | |
Matthijs Kooijman | 8b0fcf3 | 2008-06-18 08:09:27 +0000 | [diff] [blame] | 255 | // Find the index of the retval to replace with |
| 256 | int index = -1; |
Dan Gohman | fc74abf | 2008-07-23 00:34:11 +0000 | [diff] [blame] | 257 | if (ExtractValueInst *EV = dyn_cast<ExtractValueInst>(Ins)) |
Matthijs Kooijman | 8b0fcf3 | 2008-06-18 08:09:27 +0000 | [diff] [blame] | 258 | if (EV->hasIndices()) |
| 259 | index = *EV->idx_begin(); |
| 260 | |
| 261 | // If this use uses a specific return value, and we have a replacement, |
| 262 | // replace it. |
| 263 | if (index != -1) { |
| 264 | Value *New = RetVals[index]; |
| 265 | if (New) { |
Matthijs Kooijman | c2afe89 | 2008-06-18 08:30:37 +0000 | [diff] [blame] | 266 | if (Argument *A = dyn_cast<Argument>(New)) |
| 267 | // Was an argument returned? Then find the corresponding argument in |
Matthijs Kooijman | f0da203 | 2008-06-19 08:53:24 +0000 | [diff] [blame] | 268 | // the call instruction and use that. |
| 269 | New = CS.getArgument(A->getArgNo()); |
Matthijs Kooijman | 8b0fcf3 | 2008-06-18 08:09:27 +0000 | [diff] [blame] | 270 | Ins->replaceAllUsesWith(New); |
| 271 | Ins->eraseFromParent(); |
Chris Lattner | 2ffa47b | 2004-12-11 17:00:14 +0000 | [diff] [blame] | 272 | } |
Devang Patel | 488b678 | 2008-03-20 18:30:32 +0000 | [diff] [blame] | 273 | } |
Devang Patel | 7db30ba | 2008-03-11 22:24:29 +0000 | [diff] [blame] | 274 | } |
Chris Lattner | a990d94 | 2004-11-14 06:10:11 +0000 | [diff] [blame] | 275 | } |
| 276 | |
| 277 | if (MadeChange) ++NumReturnValProped; |
Chris Lattner | a990d94 | 2004-11-14 06:10:11 +0000 | [diff] [blame] | 278 | return MadeChange; |
| 279 | } |