Chris Lattner | 2240d2b | 2003-09-20 05:03:31 +0000 | [diff] [blame] | 1 | //===- TailRecursionElimination.cpp - Eliminate Tail Calls ----------------===// |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 2 | // |
John Criswell | b576c94 | 2003-10-20 19:43:21 +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 | // |
John Criswell | b576c94 | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 8 | //===----------------------------------------------------------------------===// |
Chris Lattner | 2240d2b | 2003-09-20 05:03:31 +0000 | [diff] [blame] | 9 | // |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 10 | // This file transforms calls of the current function (self recursion) followed |
| 11 | // by a return instruction with a branch to the entry of the function, creating |
| 12 | // a loop. This pass also implements the following extensions to the basic |
| 13 | // algorithm: |
Chris Lattner | 2240d2b | 2003-09-20 05:03:31 +0000 | [diff] [blame] | 14 | // |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 15 | // 1. Trivial instructions between the call and return do not prevent the |
| 16 | // transformation from taking place, though currently the analysis cannot |
| 17 | // support moving any really useful instructions (only dead ones). |
Chris Lattner | 543d622 | 2003-12-08 23:19:26 +0000 | [diff] [blame] | 18 | // 2. This pass transforms functions that are prevented from being tail |
Duncan Sands | 24080a9 | 2010-07-10 20:31:42 +0000 | [diff] [blame] | 19 | // recursive by an associative and commutative expression to use an |
| 20 | // accumulator variable, thus compiling the typical naive factorial or |
| 21 | // 'fib' implementation into efficient code. |
Chris Lattner | d64152a | 2003-12-14 23:57:39 +0000 | [diff] [blame] | 22 | // 3. TRE is performed if the function returns void, if the return |
| 23 | // returns the result returned by the call, or if the function returns a |
| 24 | // run-time constant on all exits from the function. It is possible, though |
| 25 | // unlikely, that the return returns something else (like constant 0), and |
| 26 | // can still be TRE'd. It can be TRE'd if ALL OTHER return instructions in |
| 27 | // the function return the exact same value. |
Nick Lewycky | 0cade26 | 2009-11-07 07:10:01 +0000 | [diff] [blame] | 28 | // 4. If it can prove that callees do not access their caller stack frame, |
Chris Lattner | 7f78f21 | 2005-05-09 23:51:13 +0000 | [diff] [blame] | 29 | // they are marked as eligible for tail call elimination (by the code |
| 30 | // generator). |
Chris Lattner | 2240d2b | 2003-09-20 05:03:31 +0000 | [diff] [blame] | 31 | // |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 32 | // There are several improvements that could be made: |
| 33 | // |
| 34 | // 1. If the function has any alloca instructions, these instructions will be |
| 35 | // moved out of the entry block of the function, causing them to be |
| 36 | // evaluated each time through the tail recursion. Safely keeping allocas |
| 37 | // in the entry block requires analysis to proves that the tail-called |
| 38 | // function does not read or write the stack object. |
Chris Lattner | 2240d2b | 2003-09-20 05:03:31 +0000 | [diff] [blame] | 39 | // 2. Tail recursion is only performed if the call immediately preceeds the |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 40 | // return instruction. It's possible that there could be a jump between |
| 41 | // the call and the return. |
Chris Lattner | d64152a | 2003-12-14 23:57:39 +0000 | [diff] [blame] | 42 | // 3. There can be intervening operations between the call and the return that |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 43 | // prevent the TRE from occurring. For example, there could be GEP's and |
| 44 | // stores to memory that will not be read or written by the call. This |
| 45 | // requires some substantial analysis (such as with DSA) to prove safe to |
| 46 | // move ahead of the call, but doing so could allow many more TREs to be |
| 47 | // performed, for example in TreeAdd/TreeAlloc from the treeadd benchmark. |
Chris Lattner | 7f78f21 | 2005-05-09 23:51:13 +0000 | [diff] [blame] | 48 | // 4. The algorithm we use to detect if callees access their caller stack |
| 49 | // frames is very primitive. |
Chris Lattner | 2240d2b | 2003-09-20 05:03:31 +0000 | [diff] [blame] | 50 | // |
| 51 | //===----------------------------------------------------------------------===// |
| 52 | |
Chris Lattner | 0e5f499 | 2006-12-19 21:40:18 +0000 | [diff] [blame] | 53 | #define DEBUG_TYPE "tailcallelim" |
Chris Lattner | 3fc6ef1 | 2003-09-20 05:14:13 +0000 | [diff] [blame] | 54 | #include "llvm/Transforms/Scalar.h" |
Chris Lattner | 6a35b40 | 2009-06-19 04:22:16 +0000 | [diff] [blame] | 55 | #include "llvm/Transforms/Utils/Local.h" |
Chris Lattner | ce869ee | 2005-08-07 04:27:41 +0000 | [diff] [blame] | 56 | #include "llvm/Constants.h" |
Chris Lattner | 2240d2b | 2003-09-20 05:03:31 +0000 | [diff] [blame] | 57 | #include "llvm/DerivedTypes.h" |
| 58 | #include "llvm/Function.h" |
| 59 | #include "llvm/Instructions.h" |
| 60 | #include "llvm/Pass.h" |
Nick Lewycky | 0cade26 | 2009-11-07 07:10:01 +0000 | [diff] [blame] | 61 | #include "llvm/Analysis/CaptureTracking.h" |
Dan Gohman | ea25b48 | 2010-04-16 15:57:50 +0000 | [diff] [blame] | 62 | #include "llvm/Analysis/InlineCost.h" |
Dan Gohman | dd9344f | 2010-05-28 16:19:17 +0000 | [diff] [blame] | 63 | #include "llvm/Analysis/Loads.h" |
Dan Gohman | ea25b48 | 2010-04-16 15:57:50 +0000 | [diff] [blame] | 64 | #include "llvm/Support/CallSite.h" |
Chris Lattner | 543d622 | 2003-12-08 23:19:26 +0000 | [diff] [blame] | 65 | #include "llvm/Support/CFG.h" |
Reid Spencer | 551ccae | 2004-09-01 22:55:40 +0000 | [diff] [blame] | 66 | #include "llvm/ADT/Statistic.h" |
Chris Lattner | f8485c6 | 2003-11-20 18:25:24 +0000 | [diff] [blame] | 67 | using namespace llvm; |
Brian Gaeke | d0fde30 | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 68 | |
Chris Lattner | 0e5f499 | 2006-12-19 21:40:18 +0000 | [diff] [blame] | 69 | STATISTIC(NumEliminated, "Number of tail calls removed"); |
| 70 | STATISTIC(NumAccumAdded, "Number of accumulators introduced"); |
Chris Lattner | 2240d2b | 2003-09-20 05:03:31 +0000 | [diff] [blame] | 71 | |
Chris Lattner | 0e5f499 | 2006-12-19 21:40:18 +0000 | [diff] [blame] | 72 | namespace { |
Chris Lattner | 3e8b663 | 2009-09-02 06:11:42 +0000 | [diff] [blame] | 73 | struct TailCallElim : public FunctionPass { |
Nick Lewycky | ecd94c8 | 2007-05-06 13:37:16 +0000 | [diff] [blame] | 74 | static char ID; // Pass identification, replacement for typeid |
Dan Gohman | ae73dc1 | 2008-09-04 17:05:41 +0000 | [diff] [blame] | 75 | TailCallElim() : FunctionPass(&ID) {} |
Devang Patel | 794fd75 | 2007-05-01 21:15:47 +0000 | [diff] [blame] | 76 | |
Chris Lattner | 2240d2b | 2003-09-20 05:03:31 +0000 | [diff] [blame] | 77 | virtual bool runOnFunction(Function &F); |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 78 | |
| 79 | private: |
| 80 | bool ProcessReturningBlock(ReturnInst *RI, BasicBlock *&OldEntry, |
Chris Lattner | ce869ee | 2005-08-07 04:27:41 +0000 | [diff] [blame] | 81 | bool &TailCallsAreMarkedTail, |
Nick Lewycky | 0cade26 | 2009-11-07 07:10:01 +0000 | [diff] [blame] | 82 | SmallVector<PHINode*, 8> &ArgumentPHIs, |
Chris Lattner | ce869ee | 2005-08-07 04:27:41 +0000 | [diff] [blame] | 83 | bool CannotTailCallElimCallsMarkedTail); |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 84 | bool CanMoveAboveCall(Instruction *I, CallInst *CI); |
Chris Lattner | 543d622 | 2003-12-08 23:19:26 +0000 | [diff] [blame] | 85 | Value *CanTransformAccumulatorRecursion(Instruction *I, CallInst *CI); |
Chris Lattner | 2240d2b | 2003-09-20 05:03:31 +0000 | [diff] [blame] | 86 | }; |
Chris Lattner | 2240d2b | 2003-09-20 05:03:31 +0000 | [diff] [blame] | 87 | } |
| 88 | |
Dan Gohman | 844731a | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 89 | char TailCallElim::ID = 0; |
| 90 | static RegisterPass<TailCallElim> X("tailcallelim", "Tail Call Elimination"); |
| 91 | |
Brian Gaeke | d0fde30 | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 92 | // Public interface to the TailCallElimination pass |
Chris Lattner | f8485c6 | 2003-11-20 18:25:24 +0000 | [diff] [blame] | 93 | FunctionPass *llvm::createTailCallEliminationPass() { |
| 94 | return new TailCallElim(); |
| 95 | } |
Chris Lattner | 3fc6ef1 | 2003-09-20 05:14:13 +0000 | [diff] [blame] | 96 | |
Nick Lewycky | cb19438 | 2009-11-07 07:42:38 +0000 | [diff] [blame] | 97 | /// AllocaMightEscapeToCalls - Return true if this alloca may be accessed by |
| 98 | /// callees of this function. We only do very simple analysis right now, this |
| 99 | /// could be expanded in the future to use mod/ref information for particular |
| 100 | /// call sites if desired. |
| 101 | static bool AllocaMightEscapeToCalls(AllocaInst *AI) { |
| 102 | // FIXME: do simple 'address taken' analysis. |
| 103 | return true; |
| 104 | } |
| 105 | |
Nick Lewycky | 0cade26 | 2009-11-07 07:10:01 +0000 | [diff] [blame] | 106 | /// CheckForEscapingAllocas - Scan the specified basic block for alloca |
Chris Lattner | 7f78f21 | 2005-05-09 23:51:13 +0000 | [diff] [blame] | 107 | /// instructions. If it contains any that might be accessed by calls, return |
| 108 | /// true. |
Chris Lattner | ce869ee | 2005-08-07 04:27:41 +0000 | [diff] [blame] | 109 | static bool CheckForEscapingAllocas(BasicBlock *BB, |
| 110 | bool &CannotTCETailMarkedCall) { |
| 111 | bool RetVal = false; |
Chris Lattner | 7f78f21 | 2005-05-09 23:51:13 +0000 | [diff] [blame] | 112 | for (BasicBlock::iterator I = BB->begin(), E = BB->end(); I != E; ++I) |
Chris Lattner | ce869ee | 2005-08-07 04:27:41 +0000 | [diff] [blame] | 113 | if (AllocaInst *AI = dyn_cast<AllocaInst>(I)) { |
Nick Lewycky | cb19438 | 2009-11-07 07:42:38 +0000 | [diff] [blame] | 114 | RetVal |= AllocaMightEscapeToCalls(AI); |
Chris Lattner | ce869ee | 2005-08-07 04:27:41 +0000 | [diff] [blame] | 115 | |
| 116 | // If this alloca is in the body of the function, or if it is a variable |
| 117 | // sized allocation, we cannot tail call eliminate calls marked 'tail' |
| 118 | // with this mechanism. |
Dan Gohman | ecb7a77 | 2007-03-22 16:38:57 +0000 | [diff] [blame] | 119 | if (BB != &BB->getParent()->getEntryBlock() || |
Chris Lattner | ce869ee | 2005-08-07 04:27:41 +0000 | [diff] [blame] | 120 | !isa<ConstantInt>(AI->getArraySize())) |
| 121 | CannotTCETailMarkedCall = true; |
| 122 | } |
| 123 | return RetVal; |
Chris Lattner | 7f78f21 | 2005-05-09 23:51:13 +0000 | [diff] [blame] | 124 | } |
| 125 | |
Chris Lattner | 2240d2b | 2003-09-20 05:03:31 +0000 | [diff] [blame] | 126 | bool TailCallElim::runOnFunction(Function &F) { |
| 127 | // If this function is a varargs function, we won't be able to PHI the args |
| 128 | // right, so don't even try to convert it... |
| 129 | if (F.getFunctionType()->isVarArg()) return false; |
| 130 | |
| 131 | BasicBlock *OldEntry = 0; |
Chris Lattner | ce869ee | 2005-08-07 04:27:41 +0000 | [diff] [blame] | 132 | bool TailCallsAreMarkedTail = false; |
Nick Lewycky | 0cade26 | 2009-11-07 07:10:01 +0000 | [diff] [blame] | 133 | SmallVector<PHINode*, 8> ArgumentPHIs; |
Chris Lattner | 2240d2b | 2003-09-20 05:03:31 +0000 | [diff] [blame] | 134 | bool MadeChange = false; |
| 135 | |
Chris Lattner | 7f78f21 | 2005-05-09 23:51:13 +0000 | [diff] [blame] | 136 | bool FunctionContainsEscapingAllocas = false; |
| 137 | |
Chris Lattner | ce869ee | 2005-08-07 04:27:41 +0000 | [diff] [blame] | 138 | // CannotTCETailMarkedCall - If true, we cannot perform TCE on tail calls |
| 139 | // marked with the 'tail' attribute, because doing so would cause the stack |
| 140 | // size to increase (real TCE would deallocate variable sized allocas, TCE |
| 141 | // doesn't). |
| 142 | bool CannotTCETailMarkedCall = false; |
| 143 | |
Chris Lattner | 7f78f21 | 2005-05-09 23:51:13 +0000 | [diff] [blame] | 144 | // Loop over the function, looking for any returning blocks, and keeping track |
| 145 | // of whether this function has any non-trivially used allocas. |
| 146 | for (Function::iterator BB = F.begin(), E = F.end(); BB != E; ++BB) { |
Chris Lattner | ce869ee | 2005-08-07 04:27:41 +0000 | [diff] [blame] | 147 | if (FunctionContainsEscapingAllocas && CannotTCETailMarkedCall) |
| 148 | break; |
Jeff Cohen | 00b16889 | 2005-07-27 06:12:32 +0000 | [diff] [blame] | 149 | |
Chris Lattner | 4c0e4cd | 2005-08-07 07:00:52 +0000 | [diff] [blame] | 150 | FunctionContainsEscapingAllocas |= |
Chris Lattner | ce869ee | 2005-08-07 04:27:41 +0000 | [diff] [blame] | 151 | CheckForEscapingAllocas(BB, CannotTCETailMarkedCall); |
Chris Lattner | 7f78f21 | 2005-05-09 23:51:13 +0000 | [diff] [blame] | 152 | } |
Chris Lattner | 32b1e87 | 2006-10-22 18:42:26 +0000 | [diff] [blame] | 153 | |
| 154 | /// FIXME: The code generator produces really bad code when an 'escaping |
| 155 | /// alloca' is changed from being a static alloca to being a dynamic alloca. |
| 156 | /// Until this is resolved, disable this transformation if that would ever |
| 157 | /// happen. This bug is PR962. |
| 158 | if (FunctionContainsEscapingAllocas) |
| 159 | return false; |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 160 | |
Chris Lattner | ce869ee | 2005-08-07 04:27:41 +0000 | [diff] [blame] | 161 | // Second pass, change any tail calls to loops. |
| 162 | for (Function::iterator BB = F.begin(), E = F.end(); BB != E; ++BB) |
| 163 | if (ReturnInst *Ret = dyn_cast<ReturnInst>(BB->getTerminator())) |
| 164 | MadeChange |= ProcessReturningBlock(Ret, OldEntry, TailCallsAreMarkedTail, |
| 165 | ArgumentPHIs,CannotTCETailMarkedCall); |
| 166 | |
Chris Lattner | cf2f892 | 2003-12-08 23:37:35 +0000 | [diff] [blame] | 167 | // If we eliminated any tail recursions, it's possible that we inserted some |
| 168 | // silly PHI nodes which just merge an initial value (the incoming operand) |
| 169 | // with themselves. Check to see if we did and clean up our mess if so. This |
| 170 | // occurs when a function passes an argument straight through to its tail |
| 171 | // call. |
| 172 | if (!ArgumentPHIs.empty()) { |
Chris Lattner | cf2f892 | 2003-12-08 23:37:35 +0000 | [diff] [blame] | 173 | for (unsigned i = 0, e = ArgumentPHIs.size(); i != e; ++i) { |
| 174 | PHINode *PN = ArgumentPHIs[i]; |
Chris Lattner | cf2f892 | 2003-12-08 23:37:35 +0000 | [diff] [blame] | 175 | |
| 176 | // If the PHI Node is a dynamic constant, replace it with the value it is. |
Chris Lattner | ce869ee | 2005-08-07 04:27:41 +0000 | [diff] [blame] | 177 | if (Value *PNV = PN->hasConstantValue()) { |
| 178 | PN->replaceAllUsesWith(PNV); |
| 179 | PN->eraseFromParent(); |
Chris Lattner | cf2f892 | 2003-12-08 23:37:35 +0000 | [diff] [blame] | 180 | } |
| 181 | } |
| 182 | } |
| 183 | |
Chris Lattner | 7f78f21 | 2005-05-09 23:51:13 +0000 | [diff] [blame] | 184 | // Finally, if this function contains no non-escaping allocas, mark all calls |
| 185 | // in the function as eligible for tail calls (there is no stack memory for |
| 186 | // them to access). |
| 187 | if (!FunctionContainsEscapingAllocas) |
| 188 | for (Function::iterator BB = F.begin(), E = F.end(); BB != E; ++BB) |
| 189 | for (BasicBlock::iterator I = BB->begin(), E = BB->end(); I != E; ++I) |
Evan Cheng | febc816 | 2010-02-03 03:55:59 +0000 | [diff] [blame] | 190 | if (CallInst *CI = dyn_cast<CallInst>(I)) { |
| 191 | CI->setTailCall(); |
| 192 | MadeChange = true; |
| 193 | } |
Chris Lattner | 7f78f21 | 2005-05-09 23:51:13 +0000 | [diff] [blame] | 194 | |
Chris Lattner | 2240d2b | 2003-09-20 05:03:31 +0000 | [diff] [blame] | 195 | return MadeChange; |
| 196 | } |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 197 | |
| 198 | |
Chris Lattner | 543d622 | 2003-12-08 23:19:26 +0000 | [diff] [blame] | 199 | /// CanMoveAboveCall - Return true if it is safe to move the specified |
| 200 | /// instruction from after the call to before the call, assuming that all |
| 201 | /// instructions between the call and this instruction are movable. |
| 202 | /// |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 203 | bool TailCallElim::CanMoveAboveCall(Instruction *I, CallInst *CI) { |
| 204 | // FIXME: We can move load/store/call/free instructions above the call if the |
| 205 | // call does not mod/ref the memory location being processed. |
Chris Lattner | 6a35b40 | 2009-06-19 04:22:16 +0000 | [diff] [blame] | 206 | if (I->mayHaveSideEffects()) // This also handles volatile loads. |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 207 | return false; |
Chris Lattner | 6a35b40 | 2009-06-19 04:22:16 +0000 | [diff] [blame] | 208 | |
Nick Lewycky | 0cade26 | 2009-11-07 07:10:01 +0000 | [diff] [blame] | 209 | if (LoadInst *L = dyn_cast<LoadInst>(I)) { |
Chris Lattner | 6a35b40 | 2009-06-19 04:22:16 +0000 | [diff] [blame] | 210 | // Loads may always be moved above calls without side effects. |
| 211 | if (CI->mayHaveSideEffects()) { |
| 212 | // Non-volatile loads may be moved above a call with side effects if it |
| 213 | // does not write to memory and the load provably won't trap. |
| 214 | // FIXME: Writes to memory only matter if they may alias the pointer |
| 215 | // being loaded from. |
| 216 | if (CI->mayWriteToMemory() || |
Bob Wilson | 49db68f | 2010-01-30 04:42:39 +0000 | [diff] [blame] | 217 | !isSafeToLoadUnconditionally(L->getPointerOperand(), L, |
| 218 | L->getAlignment())) |
Chris Lattner | 6a35b40 | 2009-06-19 04:22:16 +0000 | [diff] [blame] | 219 | return false; |
| 220 | } |
| 221 | } |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 222 | |
| 223 | // Otherwise, if this is a side-effect free instruction, check to make sure |
| 224 | // that it does not use the return value of the call. If it doesn't use the |
| 225 | // return value of the call, it must only use things that are defined before |
| 226 | // the call, or movable instructions between the call and the instruction |
| 227 | // itself. |
| 228 | for (unsigned i = 0, e = I->getNumOperands(); i != e; ++i) |
| 229 | if (I->getOperand(i) == CI) |
| 230 | return false; |
| 231 | return true; |
| 232 | } |
| 233 | |
Chris Lattner | d64152a | 2003-12-14 23:57:39 +0000 | [diff] [blame] | 234 | // isDynamicConstant - Return true if the specified value is the same when the |
| 235 | // return would exit as it was when the initial iteration of the recursive |
| 236 | // function was executed. |
| 237 | // |
| 238 | // We currently handle static constants and arguments that are not modified as |
| 239 | // part of the recursion. |
| 240 | // |
Nick Lewycky | f80fcd0 | 2009-11-07 21:10:15 +0000 | [diff] [blame] | 241 | static bool isDynamicConstant(Value *V, CallInst *CI, ReturnInst *RI) { |
Chris Lattner | d64152a | 2003-12-14 23:57:39 +0000 | [diff] [blame] | 242 | if (isa<Constant>(V)) return true; // Static constants are always dyn consts |
| 243 | |
| 244 | // Check to see if this is an immutable argument, if so, the value |
| 245 | // will be available to initialize the accumulator. |
| 246 | if (Argument *Arg = dyn_cast<Argument>(V)) { |
| 247 | // Figure out which argument number this is... |
| 248 | unsigned ArgNo = 0; |
| 249 | Function *F = CI->getParent()->getParent(); |
Chris Lattner | e4d5c44 | 2005-03-15 04:54:21 +0000 | [diff] [blame] | 250 | for (Function::arg_iterator AI = F->arg_begin(); &*AI != Arg; ++AI) |
Chris Lattner | d64152a | 2003-12-14 23:57:39 +0000 | [diff] [blame] | 251 | ++ArgNo; |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 252 | |
Chris Lattner | d64152a | 2003-12-14 23:57:39 +0000 | [diff] [blame] | 253 | // If we are passing this argument into call as the corresponding |
| 254 | // argument operand, then the argument is dynamically constant. |
| 255 | // Otherwise, we cannot transform this function safely. |
Gabor Greif | de9f545 | 2010-06-24 00:44:01 +0000 | [diff] [blame] | 256 | if (CI->getArgOperand(ArgNo) == Arg) |
Chris Lattner | d64152a | 2003-12-14 23:57:39 +0000 | [diff] [blame] | 257 | return true; |
| 258 | } |
Nick Lewycky | f80fcd0 | 2009-11-07 21:10:15 +0000 | [diff] [blame] | 259 | |
| 260 | // Switch cases are always constant integers. If the value is being switched |
| 261 | // on and the return is only reachable from one of its cases, it's |
| 262 | // effectively constant. |
| 263 | if (BasicBlock *UniquePred = RI->getParent()->getUniquePredecessor()) |
| 264 | if (SwitchInst *SI = dyn_cast<SwitchInst>(UniquePred->getTerminator())) |
| 265 | if (SI->getCondition() == V) |
| 266 | return SI->getDefaultDest() != RI->getParent(); |
| 267 | |
Chris Lattner | d64152a | 2003-12-14 23:57:39 +0000 | [diff] [blame] | 268 | // Not a constant or immutable argument, we can't safely transform. |
| 269 | return false; |
| 270 | } |
| 271 | |
| 272 | // getCommonReturnValue - Check to see if the function containing the specified |
Duncan Sands | d50e9e2 | 2010-06-26 12:53:31 +0000 | [diff] [blame] | 273 | // tail call consistently returns the same runtime-constant value at all exit |
| 274 | // points except for IgnoreRI. If so, return the returned value. |
Chris Lattner | d64152a | 2003-12-14 23:57:39 +0000 | [diff] [blame] | 275 | // |
Duncan Sands | d50e9e2 | 2010-06-26 12:53:31 +0000 | [diff] [blame] | 276 | static Value *getCommonReturnValue(ReturnInst *IgnoreRI, CallInst *CI) { |
| 277 | Function *F = CI->getParent()->getParent(); |
Chris Lattner | d64152a | 2003-12-14 23:57:39 +0000 | [diff] [blame] | 278 | Value *ReturnedValue = 0; |
| 279 | |
| 280 | for (Function::iterator BBI = F->begin(), E = F->end(); BBI != E; ++BBI) |
| 281 | if (ReturnInst *RI = dyn_cast<ReturnInst>(BBI->getTerminator())) |
Duncan Sands | d50e9e2 | 2010-06-26 12:53:31 +0000 | [diff] [blame] | 282 | if (RI != IgnoreRI) { |
Chris Lattner | d64152a | 2003-12-14 23:57:39 +0000 | [diff] [blame] | 283 | Value *RetOp = RI->getOperand(0); |
| 284 | |
| 285 | // We can only perform this transformation if the value returned is |
| 286 | // evaluatable at the start of the initial invocation of the function, |
| 287 | // instead of at the end of the evaluation. |
| 288 | // |
Nick Lewycky | f80fcd0 | 2009-11-07 21:10:15 +0000 | [diff] [blame] | 289 | if (!isDynamicConstant(RetOp, CI, RI)) |
Chris Lattner | d64152a | 2003-12-14 23:57:39 +0000 | [diff] [blame] | 290 | return 0; |
| 291 | |
| 292 | if (ReturnedValue && RetOp != ReturnedValue) |
| 293 | return 0; // Cannot transform if differing values are returned. |
| 294 | ReturnedValue = RetOp; |
| 295 | } |
| 296 | return ReturnedValue; |
| 297 | } |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 298 | |
Chris Lattner | 543d622 | 2003-12-08 23:19:26 +0000 | [diff] [blame] | 299 | /// CanTransformAccumulatorRecursion - If the specified instruction can be |
| 300 | /// transformed using accumulator recursion elimination, return the constant |
| 301 | /// which is the start of the accumulator value. Otherwise return null. |
| 302 | /// |
| 303 | Value *TailCallElim::CanTransformAccumulatorRecursion(Instruction *I, |
| 304 | CallInst *CI) { |
Duncan Sands | 24080a9 | 2010-07-10 20:31:42 +0000 | [diff] [blame] | 305 | if (!I->isAssociative() || !I->isCommutative()) return 0; |
Chris Lattner | 543d622 | 2003-12-08 23:19:26 +0000 | [diff] [blame] | 306 | assert(I->getNumOperands() == 2 && |
Duncan Sands | 24080a9 | 2010-07-10 20:31:42 +0000 | [diff] [blame] | 307 | "Associative/commutative operations should have 2 args!"); |
Chris Lattner | 543d622 | 2003-12-08 23:19:26 +0000 | [diff] [blame] | 308 | |
| 309 | // Exactly one operand should be the result of the call instruction... |
Anton Korobeynikov | 07e6e56 | 2008-02-20 11:26:25 +0000 | [diff] [blame] | 310 | if ((I->getOperand(0) == CI && I->getOperand(1) == CI) || |
| 311 | (I->getOperand(0) != CI && I->getOperand(1) != CI)) |
Chris Lattner | 543d622 | 2003-12-08 23:19:26 +0000 | [diff] [blame] | 312 | return 0; |
| 313 | |
| 314 | // The only user of this instruction we allow is a single return instruction. |
| 315 | if (!I->hasOneUse() || !isa<ReturnInst>(I->use_back())) |
| 316 | return 0; |
| 317 | |
| 318 | // Ok, now we have to check all of the other return instructions in this |
| 319 | // function. If they return non-constants or differing values, then we cannot |
| 320 | // transform the function safely. |
Chris Lattner | d64152a | 2003-12-14 23:57:39 +0000 | [diff] [blame] | 321 | return getCommonReturnValue(cast<ReturnInst>(I->use_back()), CI); |
Chris Lattner | 543d622 | 2003-12-08 23:19:26 +0000 | [diff] [blame] | 322 | } |
| 323 | |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 324 | bool TailCallElim::ProcessReturningBlock(ReturnInst *Ret, BasicBlock *&OldEntry, |
Chris Lattner | ce869ee | 2005-08-07 04:27:41 +0000 | [diff] [blame] | 325 | bool &TailCallsAreMarkedTail, |
Nick Lewycky | 0cade26 | 2009-11-07 07:10:01 +0000 | [diff] [blame] | 326 | SmallVector<PHINode*, 8> &ArgumentPHIs, |
Chris Lattner | ce869ee | 2005-08-07 04:27:41 +0000 | [diff] [blame] | 327 | bool CannotTailCallElimCallsMarkedTail) { |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 328 | BasicBlock *BB = Ret->getParent(); |
| 329 | Function *F = BB->getParent(); |
| 330 | |
| 331 | if (&BB->front() == Ret) // Make sure there is something before the ret... |
| 332 | return false; |
Chris Lattner | 8d9455d | 2007-09-10 20:58:55 +0000 | [diff] [blame] | 333 | |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 334 | // Scan backwards from the return, checking to see if there is a tail call in |
| 335 | // this block. If so, set CI to it. |
| 336 | CallInst *CI; |
| 337 | BasicBlock::iterator BBI = Ret; |
| 338 | while (1) { |
| 339 | CI = dyn_cast<CallInst>(BBI); |
| 340 | if (CI && CI->getCalledFunction() == F) |
| 341 | break; |
| 342 | |
| 343 | if (BBI == BB->begin()) |
| 344 | return false; // Didn't find a potential tail call. |
| 345 | --BBI; |
| 346 | } |
| 347 | |
Chris Lattner | ce869ee | 2005-08-07 04:27:41 +0000 | [diff] [blame] | 348 | // If this call is marked as a tail call, and if there are dynamic allocas in |
| 349 | // the function, we cannot perform this optimization. |
| 350 | if (CI->isTailCall() && CannotTailCallElimCallsMarkedTail) |
| 351 | return false; |
| 352 | |
Dan Gohman | ea25b48 | 2010-04-16 15:57:50 +0000 | [diff] [blame] | 353 | // As a special case, detect code like this: |
| 354 | // double fabs(double f) { return __builtin_fabs(f); } // a 'fabs' call |
| 355 | // and disable this xform in this case, because the code generator will |
| 356 | // lower the call to fabs into inline code. |
| 357 | if (BB == &F->getEntryBlock() && |
| 358 | &BB->front() == CI && &*++BB->begin() == Ret && |
| 359 | callIsSmall(F)) { |
| 360 | // A single-block function with just a call and a return. Check that |
| 361 | // the arguments match. |
| 362 | CallSite::arg_iterator I = CallSite(CI).arg_begin(), |
| 363 | E = CallSite(CI).arg_end(); |
| 364 | Function::arg_iterator FI = F->arg_begin(), |
| 365 | FE = F->arg_end(); |
| 366 | for (; I != E && FI != FE; ++I, ++FI) |
| 367 | if (*I != &*FI) break; |
| 368 | if (I == E && FI == FE) |
| 369 | return false; |
| 370 | } |
| 371 | |
Duncan Sands | 24080a9 | 2010-07-10 20:31:42 +0000 | [diff] [blame] | 372 | // If we are introducing accumulator recursion to eliminate operations after |
| 373 | // the call instruction that are both associative and commutative, the initial |
| 374 | // value for the accumulator is placed in this variable. If this value is set |
| 375 | // then we actually perform accumulator recursion elimination instead of |
Duncan Sands | d0d3ccc | 2010-07-13 15:41:41 +0000 | [diff] [blame^] | 376 | // simple tail recursion elimination. If the operation is an LLVM instruction |
| 377 | // (eg: "add") then it is recorded in AccumulatorRecursionInstr. If not, then |
| 378 | // we are handling the case when the return instruction returns a constant C |
| 379 | // which is different to the constant returned by other return instructions |
| 380 | // (which is recorded in AccumulatorRecursionEliminationInitVal). This is a |
| 381 | // special case of accumulator recursion, the operation being "return C". |
Chris Lattner | 543d622 | 2003-12-08 23:19:26 +0000 | [diff] [blame] | 382 | Value *AccumulatorRecursionEliminationInitVal = 0; |
| 383 | Instruction *AccumulatorRecursionInstr = 0; |
| 384 | |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 385 | // Ok, we found a potential tail call. We can currently only transform the |
| 386 | // tail call if all of the instructions between the call and the return are |
| 387 | // movable to above the call itself, leaving the call next to the return. |
| 388 | // Check that this is the case now. |
| 389 | for (BBI = CI, ++BBI; &*BBI != Ret; ++BBI) |
Chris Lattner | 543d622 | 2003-12-08 23:19:26 +0000 | [diff] [blame] | 390 | if (!CanMoveAboveCall(BBI, CI)) { |
| 391 | // If we can't move the instruction above the call, it might be because it |
Duncan Sands | 24080a9 | 2010-07-10 20:31:42 +0000 | [diff] [blame] | 392 | // is an associative and commutative operation that could be tranformed |
| 393 | // using accumulator recursion elimination. Check to see if this is the |
| 394 | // case, and if so, remember the initial accumulator value for later. |
Chris Lattner | 543d622 | 2003-12-08 23:19:26 +0000 | [diff] [blame] | 395 | if ((AccumulatorRecursionEliminationInitVal = |
| 396 | CanTransformAccumulatorRecursion(BBI, CI))) { |
| 397 | // Yes, this is accumulator recursion. Remember which instruction |
| 398 | // accumulates. |
| 399 | AccumulatorRecursionInstr = BBI; |
| 400 | } else { |
| 401 | return false; // Otherwise, we cannot eliminate the tail recursion! |
| 402 | } |
| 403 | } |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 404 | |
| 405 | // We can only transform call/return pairs that either ignore the return value |
Chris Lattner | d64152a | 2003-12-14 23:57:39 +0000 | [diff] [blame] | 406 | // of the call and return void, ignore the value of the call and return a |
| 407 | // constant, return the value returned by the tail call, or that are being |
| 408 | // accumulator recursion variable eliminated. |
Devang Patel | 826c491 | 2008-03-11 17:33:32 +0000 | [diff] [blame] | 409 | if (Ret->getNumOperands() == 1 && Ret->getReturnValue() != CI && |
Chris Lattner | 3b5f450 | 2005-11-05 08:21:11 +0000 | [diff] [blame] | 410 | !isa<UndefValue>(Ret->getReturnValue()) && |
Chris Lattner | d64152a | 2003-12-14 23:57:39 +0000 | [diff] [blame] | 411 | AccumulatorRecursionEliminationInitVal == 0 && |
Duncan Sands | d0d3ccc | 2010-07-13 15:41:41 +0000 | [diff] [blame^] | 412 | !getCommonReturnValue(0, CI)) { |
| 413 | // One case remains that we are able to handle: the current return |
| 414 | // instruction returns a constant, and all other return instructions |
| 415 | // return a different constant. |
| 416 | if (!isDynamicConstant(Ret->getReturnValue(), CI, Ret)) |
| 417 | return false; // Current return instruction does not return a constant. |
| 418 | // Check that all other return instructions return a common constant. If |
| 419 | // so, record it in AccumulatorRecursionEliminationInitVal. |
| 420 | AccumulatorRecursionEliminationInitVal = getCommonReturnValue(Ret, CI); |
| 421 | if (!AccumulatorRecursionEliminationInitVal) |
| 422 | return false; |
| 423 | } |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 424 | |
| 425 | // OK! We can transform this tail call. If this is the first one found, |
| 426 | // create the new entry block, allowing us to branch back to the old entry. |
| 427 | if (OldEntry == 0) { |
| 428 | OldEntry = &F->getEntryBlock(); |
Owen Anderson | 1d0be15 | 2009-08-13 21:58:54 +0000 | [diff] [blame] | 429 | BasicBlock *NewEntry = BasicBlock::Create(F->getContext(), "", F, OldEntry); |
Chris Lattner | 6934a04 | 2007-02-11 01:23:03 +0000 | [diff] [blame] | 430 | NewEntry->takeName(OldEntry); |
| 431 | OldEntry->setName("tailrecurse"); |
Gabor Greif | 051a950 | 2008-04-06 20:25:17 +0000 | [diff] [blame] | 432 | BranchInst::Create(OldEntry, NewEntry); |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 433 | |
Chris Lattner | ce869ee | 2005-08-07 04:27:41 +0000 | [diff] [blame] | 434 | // If this tail call is marked 'tail' and if there are any allocas in the |
| 435 | // entry block, move them up to the new entry block. |
| 436 | TailCallsAreMarkedTail = CI->isTailCall(); |
| 437 | if (TailCallsAreMarkedTail) |
| 438 | // Move all fixed sized allocas from OldEntry to NewEntry. |
| 439 | for (BasicBlock::iterator OEBI = OldEntry->begin(), E = OldEntry->end(), |
| 440 | NEBI = NewEntry->begin(); OEBI != E; ) |
| 441 | if (AllocaInst *AI = dyn_cast<AllocaInst>(OEBI++)) |
| 442 | if (isa<ConstantInt>(AI->getArraySize())) |
Chris Lattner | 4bc5f80 | 2005-08-08 19:11:57 +0000 | [diff] [blame] | 443 | AI->moveBefore(NEBI); |
Chris Lattner | ce869ee | 2005-08-07 04:27:41 +0000 | [diff] [blame] | 444 | |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 445 | // Now that we have created a new block, which jumps to the entry |
| 446 | // block, insert a PHI node for each argument of the function. |
| 447 | // For now, we initialize each PHI to only have the real arguments |
| 448 | // which are passed in. |
| 449 | Instruction *InsertPos = OldEntry->begin(); |
Chris Lattner | 7f78f21 | 2005-05-09 23:51:13 +0000 | [diff] [blame] | 450 | for (Function::arg_iterator I = F->arg_begin(), E = F->arg_end(); |
| 451 | I != E; ++I) { |
Gabor Greif | b1dbcd8 | 2008-05-15 10:04:30 +0000 | [diff] [blame] | 452 | PHINode *PN = PHINode::Create(I->getType(), |
| 453 | I->getName() + ".tr", InsertPos); |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 454 | I->replaceAllUsesWith(PN); // Everyone use the PHI node now! |
| 455 | PN->addIncoming(I, NewEntry); |
| 456 | ArgumentPHIs.push_back(PN); |
| 457 | } |
| 458 | } |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 459 | |
Chris Lattner | ce869ee | 2005-08-07 04:27:41 +0000 | [diff] [blame] | 460 | // If this function has self recursive calls in the tail position where some |
| 461 | // are marked tail and some are not, only transform one flavor or another. We |
| 462 | // have to choose whether we move allocas in the entry block to the new entry |
| 463 | // block or not, so we can't make a good choice for both. NOTE: We could do |
| 464 | // slightly better here in the case that the function has no entry block |
| 465 | // allocas. |
| 466 | if (TailCallsAreMarkedTail && !CI->isTailCall()) |
| 467 | return false; |
| 468 | |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 469 | // Ok, now that we know we have a pseudo-entry block WITH all of the |
| 470 | // required PHI nodes, add entries into the PHI node for the actual |
| 471 | // parameters passed into the tail-recursive call. |
Gabor Greif | 407014f | 2010-06-24 00:48:48 +0000 | [diff] [blame] | 472 | for (unsigned i = 0, e = CI->getNumArgOperands(); i != e; ++i) |
Gabor Greif | de9f545 | 2010-06-24 00:44:01 +0000 | [diff] [blame] | 473 | ArgumentPHIs[i]->addIncoming(CI->getArgOperand(i), BB); |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 474 | |
Chris Lattner | 543d622 | 2003-12-08 23:19:26 +0000 | [diff] [blame] | 475 | // If we are introducing an accumulator variable to eliminate the recursion, |
| 476 | // do so now. Note that we _know_ that no subsequent tail recursion |
| 477 | // eliminations will happen on this function because of the way the |
| 478 | // accumulator recursion predicate is set up. |
| 479 | // |
| 480 | if (AccumulatorRecursionEliminationInitVal) { |
| 481 | Instruction *AccRecInstr = AccumulatorRecursionInstr; |
| 482 | // Start by inserting a new PHI node for the accumulator. |
Duncan Sands | d0d3ccc | 2010-07-13 15:41:41 +0000 | [diff] [blame^] | 483 | PHINode *AccPN = |
| 484 | PHINode::Create(AccumulatorRecursionEliminationInitVal->getType(), |
| 485 | "accumulator.tr", OldEntry->begin()); |
Chris Lattner | 543d622 | 2003-12-08 23:19:26 +0000 | [diff] [blame] | 486 | |
| 487 | // Loop over all of the predecessors of the tail recursion block. For the |
| 488 | // real entry into the function we seed the PHI with the initial value, |
| 489 | // computed earlier. For any other existing branches to this block (due to |
| 490 | // other tail recursions eliminated) the accumulator is not modified. |
| 491 | // Because we haven't added the branch in the current block to OldEntry yet, |
| 492 | // it will not show up as a predecessor. |
| 493 | for (pred_iterator PI = pred_begin(OldEntry), PE = pred_end(OldEntry); |
| 494 | PI != PE; ++PI) { |
Gabor Greif | a8b9df7 | 2010-07-12 10:36:48 +0000 | [diff] [blame] | 495 | BasicBlock *P = *PI; |
| 496 | if (P == &F->getEntryBlock()) |
| 497 | AccPN->addIncoming(AccumulatorRecursionEliminationInitVal, P); |
Chris Lattner | 543d622 | 2003-12-08 23:19:26 +0000 | [diff] [blame] | 498 | else |
Gabor Greif | a8b9df7 | 2010-07-12 10:36:48 +0000 | [diff] [blame] | 499 | AccPN->addIncoming(AccPN, P); |
Chris Lattner | 543d622 | 2003-12-08 23:19:26 +0000 | [diff] [blame] | 500 | } |
| 501 | |
Duncan Sands | d0d3ccc | 2010-07-13 15:41:41 +0000 | [diff] [blame^] | 502 | if (AccRecInstr) { |
| 503 | // Add an incoming argument for the current block, which is computed by |
| 504 | // our associative and commutative accumulator instruction. |
| 505 | AccPN->addIncoming(AccRecInstr, BB); |
Chris Lattner | 543d622 | 2003-12-08 23:19:26 +0000 | [diff] [blame] | 506 | |
Duncan Sands | d0d3ccc | 2010-07-13 15:41:41 +0000 | [diff] [blame^] | 507 | // Next, rewrite the accumulator recursion instruction so that it does not |
| 508 | // use the result of the call anymore, instead, use the PHI node we just |
| 509 | // inserted. |
| 510 | AccRecInstr->setOperand(AccRecInstr->getOperand(0) != CI, AccPN); |
| 511 | } else { |
| 512 | // Add an incoming argument for the current block, which is just the |
| 513 | // constant returned by the current return instruction. |
| 514 | AccPN->addIncoming(Ret->getReturnValue(), BB); |
| 515 | } |
Chris Lattner | 543d622 | 2003-12-08 23:19:26 +0000 | [diff] [blame] | 516 | |
| 517 | // Finally, rewrite any return instructions in the program to return the PHI |
| 518 | // node instead of the "initval" that they do currently. This loop will |
| 519 | // actually rewrite the return value we are destroying, but that's ok. |
| 520 | for (Function::iterator BBI = F->begin(), E = F->end(); BBI != E; ++BBI) |
| 521 | if (ReturnInst *RI = dyn_cast<ReturnInst>(BBI->getTerminator())) |
| 522 | RI->setOperand(0, AccPN); |
| 523 | ++NumAccumAdded; |
| 524 | } |
| 525 | |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 526 | // Now that all of the PHI nodes are in place, remove the call and |
| 527 | // ret instructions, replacing them with an unconditional branch. |
Gabor Greif | 051a950 | 2008-04-06 20:25:17 +0000 | [diff] [blame] | 528 | BranchInst::Create(OldEntry, Ret); |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 529 | BB->getInstList().erase(Ret); // Remove return. |
| 530 | BB->getInstList().erase(CI); // Remove call. |
Chris Lattner | 543d622 | 2003-12-08 23:19:26 +0000 | [diff] [blame] | 531 | ++NumEliminated; |
Chris Lattner | 7152da3 | 2003-12-08 05:34:54 +0000 | [diff] [blame] | 532 | return true; |
| 533 | } |