Chris Lattner | 4d1e46e | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 1 | //===-- Local.cpp - Functions to perform local transformations ------------===// |
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 | 4d1e46e | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 9 | // |
| 10 | // This family of functions perform various local transformations to the |
| 11 | // program. |
| 12 | // |
| 13 | //===----------------------------------------------------------------------===// |
| 14 | |
| 15 | #include "llvm/Transforms/Utils/Local.h" |
Chandler Carruth | d04a8d4 | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 16 | #include "llvm/ADT/DenseMap.h" |
Evgeniy Stepanov | 3333e66 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 17 | #include "llvm/ADT/STLExtras.h" |
Chandler Carruth | 58a2cbe | 2013-01-02 10:22:59 +0000 | [diff] [blame] | 18 | #include "llvm/ADT/SmallPtrSet.h" |
Peter Collingbourne | 4f96b7e | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 19 | #include "llvm/ADT/Statistic.h" |
Chandler Carruth | d04a8d4 | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 20 | #include "llvm/Analysis/Dominators.h" |
| 21 | #include "llvm/Analysis/InstructionSimplify.h" |
| 22 | #include "llvm/Analysis/MemoryBuiltins.h" |
| 23 | #include "llvm/Analysis/ProfileInfo.h" |
| 24 | #include "llvm/Analysis/ValueTracking.h" |
Chandler Carruth | 06cb8ed | 2012-06-29 12:38:19 +0000 | [diff] [blame] | 25 | #include "llvm/DIBuilder.h" |
Bill Wendling | 0bcbd1d | 2012-06-28 00:05:13 +0000 | [diff] [blame] | 26 | #include "llvm/DebugInfo.h" |
Chandler Carruth | 0b8c9a8 | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 27 | #include "llvm/IR/Constants.h" |
| 28 | #include "llvm/IR/DataLayout.h" |
| 29 | #include "llvm/IR/DerivedTypes.h" |
| 30 | #include "llvm/IR/GlobalAlias.h" |
| 31 | #include "llvm/IR/GlobalVariable.h" |
| 32 | #include "llvm/IR/IRBuilder.h" |
| 33 | #include "llvm/IR/Instructions.h" |
| 34 | #include "llvm/IR/IntrinsicInst.h" |
| 35 | #include "llvm/IR/Intrinsics.h" |
| 36 | #include "llvm/IR/MDBuilder.h" |
| 37 | #include "llvm/IR/Metadata.h" |
| 38 | #include "llvm/IR/Operator.h" |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 39 | #include "llvm/Support/CFG.h" |
| 40 | #include "llvm/Support/Debug.h" |
Chris Lattner | c5f52e6 | 2005-09-26 05:27:10 +0000 | [diff] [blame] | 41 | #include "llvm/Support/GetElementPtrTypeIterator.h" |
| 42 | #include "llvm/Support/MathExtras.h" |
Chris Lattner | 19f2dc4 | 2009-12-29 09:12:29 +0000 | [diff] [blame] | 43 | #include "llvm/Support/ValueHandle.h" |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 44 | #include "llvm/Support/raw_ostream.h" |
Chris Lattner | abbc2dd | 2003-12-19 05:56:28 +0000 | [diff] [blame] | 45 | using namespace llvm; |
Brian Gaeke | d0fde30 | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 46 | |
Peter Collingbourne | 4f96b7e | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 47 | STATISTIC(NumRemoved, "Number of unreachable basic blocks removed"); |
| 48 | |
Chris Lattner | 4d1e46e | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 49 | //===----------------------------------------------------------------------===// |
Chris Lattner | 3481f24 | 2008-11-27 22:57:53 +0000 | [diff] [blame] | 50 | // Local constant propagation. |
Chris Lattner | 4d1e46e | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 51 | // |
| 52 | |
Frits van Bommel | 5649ba7 | 2011-05-22 16:24:18 +0000 | [diff] [blame] | 53 | /// ConstantFoldTerminator - If a terminator instruction is predicated on a |
| 54 | /// constant value, convert it into an unconditional branch to the constant |
| 55 | /// destination. This is a nontrivial operation because the successors of this |
| 56 | /// basic block must have their PHI nodes updated. |
| 57 | /// Also calls RecursivelyDeleteTriviallyDeadInstructions() on any branch/switch |
| 58 | /// conditions and indirectbr addresses this might make dead if |
| 59 | /// DeleteDeadConditions is true. |
Benjamin Kramer | 8e0d1c0 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 60 | bool llvm::ConstantFoldTerminator(BasicBlock *BB, bool DeleteDeadConditions, |
| 61 | const TargetLibraryInfo *TLI) { |
Chris Lattner | 76ae344 | 2002-05-21 20:04:50 +0000 | [diff] [blame] | 62 | TerminatorInst *T = BB->getTerminator(); |
Devang Patel | 62fb355 | 2011-05-18 17:26:46 +0000 | [diff] [blame] | 63 | IRBuilder<> Builder(T); |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 64 | |
Chris Lattner | 4d1e46e | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 65 | // Branch - See if we are conditional jumping on constant |
| 66 | if (BranchInst *BI = dyn_cast<BranchInst>(T)) { |
| 67 | if (BI->isUnconditional()) return false; // Can't optimize uncond branch |
Gabor Greif | c1bb13f | 2009-01-30 18:21:13 +0000 | [diff] [blame] | 68 | BasicBlock *Dest1 = BI->getSuccessor(0); |
| 69 | BasicBlock *Dest2 = BI->getSuccessor(1); |
Chris Lattner | 4d1e46e | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 70 | |
Zhou Sheng | 6b6b6ef | 2007-01-11 12:24:14 +0000 | [diff] [blame] | 71 | if (ConstantInt *Cond = dyn_cast<ConstantInt>(BI->getCondition())) { |
Chris Lattner | 4d1e46e | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 72 | // Are we branching on constant? |
| 73 | // YES. Change to unconditional branch... |
Reid Spencer | 579dca1 | 2007-01-12 04:24:46 +0000 | [diff] [blame] | 74 | BasicBlock *Destination = Cond->getZExtValue() ? Dest1 : Dest2; |
| 75 | BasicBlock *OldDest = Cond->getZExtValue() ? Dest2 : Dest1; |
Chris Lattner | 4d1e46e | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 76 | |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 77 | //cerr << "Function: " << T->getParent()->getParent() |
| 78 | // << "\nRemoving branch from " << T->getParent() |
Chris Lattner | 4d1e46e | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 79 | // << "\n\nTo: " << OldDest << endl; |
| 80 | |
| 81 | // Let the basic block know that we are letting go of it. Based on this, |
| 82 | // it will adjust it's PHI nodes. |
Jay Foad | 1a03902 | 2011-04-19 15:23:29 +0000 | [diff] [blame] | 83 | OldDest->removePredecessor(BB); |
Chris Lattner | 4d1e46e | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 84 | |
Jay Foad | 8f9ffbd | 2011-01-07 20:25:56 +0000 | [diff] [blame] | 85 | // Replace the conditional branch with an unconditional one. |
Devang Patel | 62fb355 | 2011-05-18 17:26:46 +0000 | [diff] [blame] | 86 | Builder.CreateBr(Destination); |
Jay Foad | 8f9ffbd | 2011-01-07 20:25:56 +0000 | [diff] [blame] | 87 | BI->eraseFromParent(); |
Chris Lattner | 4d1e46e | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 88 | return true; |
Chris Lattner | 0a4c678 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 89 | } |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 90 | |
Chris Lattner | 0a4c678 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 91 | if (Dest2 == Dest1) { // Conditional branch to same location? |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 92 | // This branch matches something like this: |
Chris Lattner | 4d1e46e | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 93 | // br bool %cond, label %Dest, label %Dest |
| 94 | // and changes it into: br label %Dest |
| 95 | |
| 96 | // Let the basic block know that we are letting go of one copy of it. |
| 97 | assert(BI->getParent() && "Terminator not inserted in block!"); |
| 98 | Dest1->removePredecessor(BI->getParent()); |
| 99 | |
Jay Foad | 8f9ffbd | 2011-01-07 20:25:56 +0000 | [diff] [blame] | 100 | // Replace the conditional branch with an unconditional one. |
Devang Patel | 62fb355 | 2011-05-18 17:26:46 +0000 | [diff] [blame] | 101 | Builder.CreateBr(Dest1); |
Frits van Bommel | 5649ba7 | 2011-05-22 16:24:18 +0000 | [diff] [blame] | 102 | Value *Cond = BI->getCondition(); |
Jay Foad | 8f9ffbd | 2011-01-07 20:25:56 +0000 | [diff] [blame] | 103 | BI->eraseFromParent(); |
Frits van Bommel | 5649ba7 | 2011-05-22 16:24:18 +0000 | [diff] [blame] | 104 | if (DeleteDeadConditions) |
Benjamin Kramer | 8e0d1c0 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 105 | RecursivelyDeleteTriviallyDeadInstructions(Cond, TLI); |
Chris Lattner | 4d1e46e | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 106 | return true; |
| 107 | } |
Chris Lattner | 0a4c678 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 108 | return false; |
| 109 | } |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 110 | |
Chris Lattner | 0a4c678 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 111 | if (SwitchInst *SI = dyn_cast<SwitchInst>(T)) { |
Chris Lattner | 10b1f5a | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 112 | // If we are switching on a constant, we can convert the switch into a |
| 113 | // single branch instruction! |
| 114 | ConstantInt *CI = dyn_cast<ConstantInt>(SI->getCondition()); |
Stepan Dyatkovskiy | c10fa6c | 2012-03-08 07:06:20 +0000 | [diff] [blame] | 115 | BasicBlock *TheOnlyDest = SI->getDefaultDest(); |
Chris Lattner | 7d6c24c | 2003-08-23 23:18:19 +0000 | [diff] [blame] | 116 | BasicBlock *DefaultDest = TheOnlyDest; |
Chris Lattner | 694e37f | 2003-08-17 19:41:53 +0000 | [diff] [blame] | 117 | |
Chris Lattner | 0a4c678 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 118 | // Figure out which case it goes to. |
Stepan Dyatkovskiy | 3d3abe0 | 2012-03-11 06:09:17 +0000 | [diff] [blame] | 119 | for (SwitchInst::CaseIt i = SI->case_begin(), e = SI->case_end(); |
Stepan Dyatkovskiy | c10fa6c | 2012-03-08 07:06:20 +0000 | [diff] [blame] | 120 | i != e; ++i) { |
Chris Lattner | 10b1f5a | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 121 | // Found case matching a constant operand? |
Stepan Dyatkovskiy | c10fa6c | 2012-03-08 07:06:20 +0000 | [diff] [blame] | 122 | if (i.getCaseValue() == CI) { |
| 123 | TheOnlyDest = i.getCaseSuccessor(); |
Chris Lattner | 10b1f5a | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 124 | break; |
| 125 | } |
Chris Lattner | 694e37f | 2003-08-17 19:41:53 +0000 | [diff] [blame] | 126 | |
Chris Lattner | 7d6c24c | 2003-08-23 23:18:19 +0000 | [diff] [blame] | 127 | // Check to see if this branch is going to the same place as the default |
| 128 | // dest. If so, eliminate it as an explicit compare. |
Stepan Dyatkovskiy | c10fa6c | 2012-03-08 07:06:20 +0000 | [diff] [blame] | 129 | if (i.getCaseSuccessor() == DefaultDest) { |
Manman Ren | ee99c7f | 2012-09-12 17:04:11 +0000 | [diff] [blame] | 130 | MDNode* MD = SI->getMetadata(LLVMContext::MD_prof); |
| 131 | // MD should have 2 + NumCases operands. |
| 132 | if (MD && MD->getNumOperands() == 2 + SI->getNumCases()) { |
| 133 | // Collect branch weights into a vector. |
| 134 | SmallVector<uint32_t, 8> Weights; |
| 135 | for (unsigned MD_i = 1, MD_e = MD->getNumOperands(); MD_i < MD_e; |
| 136 | ++MD_i) { |
| 137 | ConstantInt* CI = dyn_cast<ConstantInt>(MD->getOperand(MD_i)); |
| 138 | assert(CI); |
| 139 | Weights.push_back(CI->getValue().getZExtValue()); |
| 140 | } |
| 141 | // Merge weight of this case to the default weight. |
| 142 | unsigned idx = i.getCaseIndex(); |
| 143 | Weights[0] += Weights[idx+1]; |
| 144 | // Remove weight for this case. |
| 145 | std::swap(Weights[idx+1], Weights.back()); |
| 146 | Weights.pop_back(); |
| 147 | SI->setMetadata(LLVMContext::MD_prof, |
| 148 | MDBuilder(BB->getContext()). |
| 149 | createBranchWeights(Weights)); |
| 150 | } |
Chris Lattner | 0a4c678 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 151 | // Remove this entry. |
Chris Lattner | 7d6c24c | 2003-08-23 23:18:19 +0000 | [diff] [blame] | 152 | DefaultDest->removePredecessor(SI->getParent()); |
| 153 | SI->removeCase(i); |
Stepan Dyatkovskiy | c10fa6c | 2012-03-08 07:06:20 +0000 | [diff] [blame] | 154 | --i; --e; |
Chris Lattner | 7d6c24c | 2003-08-23 23:18:19 +0000 | [diff] [blame] | 155 | continue; |
| 156 | } |
| 157 | |
Chris Lattner | 10b1f5a | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 158 | // Otherwise, check to see if the switch only branches to one destination. |
| 159 | // We do this by reseting "TheOnlyDest" to null when we find two non-equal |
| 160 | // destinations. |
Stepan Dyatkovskiy | c10fa6c | 2012-03-08 07:06:20 +0000 | [diff] [blame] | 161 | if (i.getCaseSuccessor() != TheOnlyDest) TheOnlyDest = 0; |
Chris Lattner | 694e37f | 2003-08-17 19:41:53 +0000 | [diff] [blame] | 162 | } |
| 163 | |
Chris Lattner | 10b1f5a | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 164 | if (CI && !TheOnlyDest) { |
| 165 | // Branching on a constant, but not any of the cases, go to the default |
| 166 | // successor. |
| 167 | TheOnlyDest = SI->getDefaultDest(); |
| 168 | } |
| 169 | |
| 170 | // If we found a single destination that we can fold the switch into, do so |
| 171 | // now. |
| 172 | if (TheOnlyDest) { |
Chris Lattner | 0a4c678 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 173 | // Insert the new branch. |
Devang Patel | 62fb355 | 2011-05-18 17:26:46 +0000 | [diff] [blame] | 174 | Builder.CreateBr(TheOnlyDest); |
Chris Lattner | 10b1f5a | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 175 | BasicBlock *BB = SI->getParent(); |
| 176 | |
| 177 | // Remove entries from PHI nodes which we no longer branch to... |
| 178 | for (unsigned i = 0, e = SI->getNumSuccessors(); i != e; ++i) { |
| 179 | // Found case matching a constant operand? |
| 180 | BasicBlock *Succ = SI->getSuccessor(i); |
| 181 | if (Succ == TheOnlyDest) |
| 182 | TheOnlyDest = 0; // Don't modify the first branch to TheOnlyDest |
| 183 | else |
| 184 | Succ->removePredecessor(BB); |
| 185 | } |
| 186 | |
Chris Lattner | 0a4c678 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 187 | // Delete the old switch. |
Frits van Bommel | 5649ba7 | 2011-05-22 16:24:18 +0000 | [diff] [blame] | 188 | Value *Cond = SI->getCondition(); |
| 189 | SI->eraseFromParent(); |
| 190 | if (DeleteDeadConditions) |
Benjamin Kramer | 8e0d1c0 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 191 | RecursivelyDeleteTriviallyDeadInstructions(Cond, TLI); |
Chris Lattner | 10b1f5a | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 192 | return true; |
Chris Lattner | 0a4c678 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 193 | } |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 194 | |
Stepan Dyatkovskiy | 2447312 | 2012-02-01 07:49:51 +0000 | [diff] [blame] | 195 | if (SI->getNumCases() == 1) { |
Chris Lattner | 10b1f5a | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 196 | // Otherwise, we can fold this switch into a conditional branch |
| 197 | // instruction if it has only one non-default destination. |
Stepan Dyatkovskiy | 3d3abe0 | 2012-03-11 06:09:17 +0000 | [diff] [blame] | 198 | SwitchInst::CaseIt FirstCase = SI->case_begin(); |
Stepan Dyatkovskiy | 47cbc4e | 2012-06-23 10:58:58 +0000 | [diff] [blame] | 199 | IntegersSubset& Case = FirstCase.getCaseValueEx(); |
| 200 | if (Case.isSingleNumber()) { |
Stepan Dyatkovskiy | 484fc93 | 2012-05-28 12:39:09 +0000 | [diff] [blame] | 201 | // FIXME: Currently work with ConstantInt based numbers. |
Stepan Dyatkovskiy | a2067fb | 2012-05-23 08:18:26 +0000 | [diff] [blame] | 202 | Value *Cond = Builder.CreateICmpEQ(SI->getCondition(), |
Stepan Dyatkovskiy | 47cbc4e | 2012-06-23 10:58:58 +0000 | [diff] [blame] | 203 | Case.getSingleNumber(0).toConstantInt(), |
Stepan Dyatkovskiy | 484fc93 | 2012-05-28 12:39:09 +0000 | [diff] [blame] | 204 | "cond"); |
Devang Patel | 62fb355 | 2011-05-18 17:26:46 +0000 | [diff] [blame] | 205 | |
Stepan Dyatkovskiy | a2067fb | 2012-05-23 08:18:26 +0000 | [diff] [blame] | 206 | // Insert the new branch. |
Manman Ren | ad28907 | 2012-09-14 21:53:06 +0000 | [diff] [blame] | 207 | BranchInst *NewBr = Builder.CreateCondBr(Cond, |
| 208 | FirstCase.getCaseSuccessor(), |
| 209 | SI->getDefaultDest()); |
| 210 | MDNode* MD = SI->getMetadata(LLVMContext::MD_prof); |
| 211 | if (MD && MD->getNumOperands() == 3) { |
| 212 | ConstantInt *SICase = dyn_cast<ConstantInt>(MD->getOperand(2)); |
| 213 | ConstantInt *SIDef = dyn_cast<ConstantInt>(MD->getOperand(1)); |
| 214 | assert(SICase && SIDef); |
| 215 | // The TrueWeight should be the weight for the single case of SI. |
| 216 | NewBr->setMetadata(LLVMContext::MD_prof, |
| 217 | MDBuilder(BB->getContext()). |
| 218 | createBranchWeights(SICase->getValue().getZExtValue(), |
| 219 | SIDef->getValue().getZExtValue())); |
| 220 | } |
Chris Lattner | 10b1f5a | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 221 | |
Stepan Dyatkovskiy | a2067fb | 2012-05-23 08:18:26 +0000 | [diff] [blame] | 222 | // Delete the old switch. |
| 223 | SI->eraseFromParent(); |
| 224 | return true; |
Stepan Dyatkovskiy | a2067fb | 2012-05-23 08:18:26 +0000 | [diff] [blame] | 225 | } |
Chris Lattner | 10b1f5a | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 226 | } |
Chris Lattner | 0a4c678 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 227 | return false; |
Chris Lattner | 4d1e46e | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 228 | } |
Chris Lattner | 0a4c678 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 229 | |
| 230 | if (IndirectBrInst *IBI = dyn_cast<IndirectBrInst>(T)) { |
| 231 | // indirectbr blockaddress(@F, @BB) -> br label @BB |
| 232 | if (BlockAddress *BA = |
| 233 | dyn_cast<BlockAddress>(IBI->getAddress()->stripPointerCasts())) { |
| 234 | BasicBlock *TheOnlyDest = BA->getBasicBlock(); |
| 235 | // Insert the new branch. |
Devang Patel | 62fb355 | 2011-05-18 17:26:46 +0000 | [diff] [blame] | 236 | Builder.CreateBr(TheOnlyDest); |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 237 | |
Chris Lattner | 0a4c678 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 238 | for (unsigned i = 0, e = IBI->getNumDestinations(); i != e; ++i) { |
| 239 | if (IBI->getDestination(i) == TheOnlyDest) |
| 240 | TheOnlyDest = 0; |
| 241 | else |
| 242 | IBI->getDestination(i)->removePredecessor(IBI->getParent()); |
| 243 | } |
Frits van Bommel | 5649ba7 | 2011-05-22 16:24:18 +0000 | [diff] [blame] | 244 | Value *Address = IBI->getAddress(); |
Chris Lattner | 0a4c678 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 245 | IBI->eraseFromParent(); |
Frits van Bommel | 5649ba7 | 2011-05-22 16:24:18 +0000 | [diff] [blame] | 246 | if (DeleteDeadConditions) |
Benjamin Kramer | 8e0d1c0 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 247 | RecursivelyDeleteTriviallyDeadInstructions(Address, TLI); |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 248 | |
Chris Lattner | 0a4c678 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 249 | // If we didn't find our destination in the IBI successor list, then we |
| 250 | // have undefined behavior. Replace the unconditional branch with an |
| 251 | // 'unreachable' instruction. |
| 252 | if (TheOnlyDest) { |
| 253 | BB->getTerminator()->eraseFromParent(); |
| 254 | new UnreachableInst(BB->getContext(), BB); |
| 255 | } |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 256 | |
Chris Lattner | 0a4c678 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 257 | return true; |
| 258 | } |
| 259 | } |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 260 | |
Chris Lattner | 4d1e46e | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 261 | return false; |
| 262 | } |
| 263 | |
Chris Lattner | 4d1e46e | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 264 | |
| 265 | //===----------------------------------------------------------------------===// |
Chris Lattner | 40d8c28 | 2009-11-10 22:26:15 +0000 | [diff] [blame] | 266 | // Local dead code elimination. |
Chris Lattner | 4d1e46e | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 267 | // |
| 268 | |
Chris Lattner | 3481f24 | 2008-11-27 22:57:53 +0000 | [diff] [blame] | 269 | /// isInstructionTriviallyDead - Return true if the result produced by the |
| 270 | /// instruction is not used, and the instruction has no side effects. |
| 271 | /// |
Benjamin Kramer | 8e0d1c0 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 272 | bool llvm::isInstructionTriviallyDead(Instruction *I, |
| 273 | const TargetLibraryInfo *TLI) { |
Chris Lattner | ec710c5 | 2005-05-06 05:27:34 +0000 | [diff] [blame] | 274 | if (!I->use_empty() || isa<TerminatorInst>(I)) return false; |
Jeff Cohen | 00b16889 | 2005-07-27 06:12:32 +0000 | [diff] [blame] | 275 | |
Bill Wendling | 187b192 | 2011-08-15 20:10:51 +0000 | [diff] [blame] | 276 | // We don't want the landingpad instruction removed by anything this general. |
| 277 | if (isa<LandingPadInst>(I)) |
| 278 | return false; |
| 279 | |
Devang Patel | 9c5822a | 2011-03-18 23:28:02 +0000 | [diff] [blame] | 280 | // We don't want debug info removed by anything this general, unless |
| 281 | // debug info is empty. |
| 282 | if (DbgDeclareInst *DDI = dyn_cast<DbgDeclareInst>(I)) { |
Nick Lewycky | 3e69c13 | 2011-08-02 21:19:27 +0000 | [diff] [blame] | 283 | if (DDI->getAddress()) |
Devang Patel | 9c5822a | 2011-03-18 23:28:02 +0000 | [diff] [blame] | 284 | return false; |
Devang Patel | b994621 | 2011-03-21 22:04:45 +0000 | [diff] [blame] | 285 | return true; |
Nick Lewycky | 3e69c13 | 2011-08-02 21:19:27 +0000 | [diff] [blame] | 286 | } |
Devang Patel | b994621 | 2011-03-21 22:04:45 +0000 | [diff] [blame] | 287 | if (DbgValueInst *DVI = dyn_cast<DbgValueInst>(I)) { |
Devang Patel | 9c5822a | 2011-03-18 23:28:02 +0000 | [diff] [blame] | 288 | if (DVI->getValue()) |
| 289 | return false; |
Devang Patel | b994621 | 2011-03-21 22:04:45 +0000 | [diff] [blame] | 290 | return true; |
Devang Patel | 9c5822a | 2011-03-18 23:28:02 +0000 | [diff] [blame] | 291 | } |
| 292 | |
Duncan Sands | 7af1c78 | 2009-05-06 06:49:50 +0000 | [diff] [blame] | 293 | if (!I->mayHaveSideEffects()) return true; |
| 294 | |
| 295 | // Special case intrinsics that "may have side effects" but can be deleted |
| 296 | // when dead. |
Nick Lewycky | 3e69c13 | 2011-08-02 21:19:27 +0000 | [diff] [blame] | 297 | if (IntrinsicInst *II = dyn_cast<IntrinsicInst>(I)) { |
Chris Lattner | 741c0ae | 2007-12-29 00:59:12 +0000 | [diff] [blame] | 298 | // Safe to delete llvm.stacksave if dead. |
| 299 | if (II->getIntrinsicID() == Intrinsic::stacksave) |
| 300 | return true; |
Nick Lewycky | 3e69c13 | 2011-08-02 21:19:27 +0000 | [diff] [blame] | 301 | |
| 302 | // Lifetime intrinsics are dead when their right-hand is undef. |
| 303 | if (II->getIntrinsicID() == Intrinsic::lifetime_start || |
| 304 | II->getIntrinsicID() == Intrinsic::lifetime_end) |
| 305 | return isa<UndefValue>(II->getArgOperand(1)); |
| 306 | } |
Nick Lewycky | 4a3935c | 2011-10-24 04:35:36 +0000 | [diff] [blame] | 307 | |
Benjamin Kramer | 8e0d1c0 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 308 | if (isAllocLikeFn(I, TLI)) return true; |
Nick Lewycky | 4a3935c | 2011-10-24 04:35:36 +0000 | [diff] [blame] | 309 | |
Benjamin Kramer | 8e0d1c0 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 310 | if (CallInst *CI = isFreeCall(I, TLI)) |
Nick Lewycky | 4a3935c | 2011-10-24 04:35:36 +0000 | [diff] [blame] | 311 | if (Constant *C = dyn_cast<Constant>(CI->getArgOperand(0))) |
| 312 | return C->isNullValue() || isa<UndefValue>(C); |
| 313 | |
Chris Lattner | ec710c5 | 2005-05-06 05:27:34 +0000 | [diff] [blame] | 314 | return false; |
Chris Lattner | 4d1e46e | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 315 | } |
| 316 | |
Chris Lattner | 3481f24 | 2008-11-27 22:57:53 +0000 | [diff] [blame] | 317 | /// RecursivelyDeleteTriviallyDeadInstructions - If the specified value is a |
| 318 | /// trivially dead instruction, delete it. If that makes any of its operands |
Dan Gohman | 90fe0bd | 2010-01-05 15:45:31 +0000 | [diff] [blame] | 319 | /// trivially dead, delete them too, recursively. Return true if any |
| 320 | /// instructions were deleted. |
Benjamin Kramer | 8e0d1c0 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 321 | bool |
| 322 | llvm::RecursivelyDeleteTriviallyDeadInstructions(Value *V, |
| 323 | const TargetLibraryInfo *TLI) { |
Chris Lattner | 3481f24 | 2008-11-27 22:57:53 +0000 | [diff] [blame] | 324 | Instruction *I = dyn_cast<Instruction>(V); |
Benjamin Kramer | 8e0d1c0 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 325 | if (!I || !I->use_empty() || !isInstructionTriviallyDead(I, TLI)) |
Dan Gohman | 90fe0bd | 2010-01-05 15:45:31 +0000 | [diff] [blame] | 326 | return false; |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 327 | |
Chris Lattner | 7605730 | 2008-11-28 01:20:46 +0000 | [diff] [blame] | 328 | SmallVector<Instruction*, 16> DeadInsts; |
| 329 | DeadInsts.push_back(I); |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 330 | |
Dan Gohman | 321a813 | 2010-01-05 16:27:25 +0000 | [diff] [blame] | 331 | do { |
Dan Gohman | e9d87f4 | 2009-05-06 17:22:41 +0000 | [diff] [blame] | 332 | I = DeadInsts.pop_back_val(); |
Chris Lattner | 2872177 | 2008-11-28 00:58:15 +0000 | [diff] [blame] | 333 | |
Chris Lattner | 7605730 | 2008-11-28 01:20:46 +0000 | [diff] [blame] | 334 | // Null out all of the instruction's operands to see if any operand becomes |
| 335 | // dead as we go. |
| 336 | for (unsigned i = 0, e = I->getNumOperands(); i != e; ++i) { |
| 337 | Value *OpV = I->getOperand(i); |
| 338 | I->setOperand(i, 0); |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 339 | |
Chris Lattner | 7605730 | 2008-11-28 01:20:46 +0000 | [diff] [blame] | 340 | if (!OpV->use_empty()) continue; |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 341 | |
Chris Lattner | 7605730 | 2008-11-28 01:20:46 +0000 | [diff] [blame] | 342 | // If the operand is an instruction that became dead as we nulled out the |
| 343 | // operand, and if it is 'trivially' dead, delete it in a future loop |
| 344 | // iteration. |
| 345 | if (Instruction *OpI = dyn_cast<Instruction>(OpV)) |
Benjamin Kramer | 8e0d1c0 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 346 | if (isInstructionTriviallyDead(OpI, TLI)) |
Chris Lattner | 7605730 | 2008-11-28 01:20:46 +0000 | [diff] [blame] | 347 | DeadInsts.push_back(OpI); |
| 348 | } |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 349 | |
Chris Lattner | 7605730 | 2008-11-28 01:20:46 +0000 | [diff] [blame] | 350 | I->eraseFromParent(); |
Dan Gohman | 321a813 | 2010-01-05 16:27:25 +0000 | [diff] [blame] | 351 | } while (!DeadInsts.empty()); |
Dan Gohman | 90fe0bd | 2010-01-05 15:45:31 +0000 | [diff] [blame] | 352 | |
| 353 | return true; |
Chris Lattner | 4d1e46e | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 354 | } |
Chris Lattner | b29714a | 2008-11-27 07:43:12 +0000 | [diff] [blame] | 355 | |
Nick Lewycky | 1a4021a | 2011-02-20 08:38:20 +0000 | [diff] [blame] | 356 | /// areAllUsesEqual - Check whether the uses of a value are all the same. |
| 357 | /// This is similar to Instruction::hasOneUse() except this will also return |
Duncan Sands | b4098ba | 2011-02-21 16:27:36 +0000 | [diff] [blame] | 358 | /// true when there are no uses or multiple uses that all refer to the same |
| 359 | /// value. |
Nick Lewycky | 1a4021a | 2011-02-20 08:38:20 +0000 | [diff] [blame] | 360 | static bool areAllUsesEqual(Instruction *I) { |
| 361 | Value::use_iterator UI = I->use_begin(); |
| 362 | Value::use_iterator UE = I->use_end(); |
| 363 | if (UI == UE) |
Duncan Sands | b4098ba | 2011-02-21 16:27:36 +0000 | [diff] [blame] | 364 | return true; |
Nick Lewycky | 1a4021a | 2011-02-20 08:38:20 +0000 | [diff] [blame] | 365 | |
| 366 | User *TheUse = *UI; |
| 367 | for (++UI; UI != UE; ++UI) { |
| 368 | if (*UI != TheUse) |
| 369 | return false; |
| 370 | } |
| 371 | return true; |
| 372 | } |
| 373 | |
Dan Gohman | afc36a9 | 2009-05-02 18:29:22 +0000 | [diff] [blame] | 374 | /// RecursivelyDeleteDeadPHINode - If the specified value is an effectively |
| 375 | /// dead PHI node, due to being a def-use chain of single-use nodes that |
| 376 | /// either forms a cycle or is terminated by a trivially dead instruction, |
| 377 | /// delete it. If that makes any of its operands trivially dead, delete them |
Duncan Sands | 2cfbf01 | 2011-02-21 17:32:05 +0000 | [diff] [blame] | 378 | /// too, recursively. Return true if a change was made. |
Benjamin Kramer | 8e0d1c0 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 379 | bool llvm::RecursivelyDeleteDeadPHINode(PHINode *PN, |
| 380 | const TargetLibraryInfo *TLI) { |
Duncan Sands | b4098ba | 2011-02-21 16:27:36 +0000 | [diff] [blame] | 381 | SmallPtrSet<Instruction*, 4> Visited; |
| 382 | for (Instruction *I = PN; areAllUsesEqual(I) && !I->mayHaveSideEffects(); |
| 383 | I = cast<Instruction>(*I->use_begin())) { |
| 384 | if (I->use_empty()) |
Benjamin Kramer | 8e0d1c0 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 385 | return RecursivelyDeleteTriviallyDeadInstructions(I, TLI); |
Nick Lewycky | eff5e69 | 2011-02-20 18:05:56 +0000 | [diff] [blame] | 386 | |
Duncan Sands | b4098ba | 2011-02-21 16:27:36 +0000 | [diff] [blame] | 387 | // If we find an instruction more than once, we're on a cycle that |
Dan Gohman | afc36a9 | 2009-05-02 18:29:22 +0000 | [diff] [blame] | 388 | // won't prove fruitful. |
Duncan Sands | b4098ba | 2011-02-21 16:27:36 +0000 | [diff] [blame] | 389 | if (!Visited.insert(I)) { |
| 390 | // Break the cycle and delete the instruction and its operands. |
| 391 | I->replaceAllUsesWith(UndefValue::get(I->getType())); |
Benjamin Kramer | 8e0d1c0 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 392 | (void)RecursivelyDeleteTriviallyDeadInstructions(I, TLI); |
Duncan Sands | 2cfbf01 | 2011-02-21 17:32:05 +0000 | [diff] [blame] | 393 | return true; |
Duncan Sands | b4098ba | 2011-02-21 16:27:36 +0000 | [diff] [blame] | 394 | } |
| 395 | } |
| 396 | return false; |
Dan Gohman | afc36a9 | 2009-05-02 18:29:22 +0000 | [diff] [blame] | 397 | } |
Chris Lattner | 3481f24 | 2008-11-27 22:57:53 +0000 | [diff] [blame] | 398 | |
Chris Lattner | e234a30 | 2010-01-12 19:40:54 +0000 | [diff] [blame] | 399 | /// SimplifyInstructionsInBlock - Scan the specified basic block and try to |
| 400 | /// simplify any instructions in it and recursively delete dead instructions. |
| 401 | /// |
| 402 | /// This returns true if it changed the code, note that it can delete |
| 403 | /// instructions in other blocks as well in this block. |
Micah Villmow | 3574eca | 2012-10-08 16:38:25 +0000 | [diff] [blame] | 404 | bool llvm::SimplifyInstructionsInBlock(BasicBlock *BB, const DataLayout *TD, |
Benjamin Kramer | 8e0d1c0 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 405 | const TargetLibraryInfo *TLI) { |
Chris Lattner | e234a30 | 2010-01-12 19:40:54 +0000 | [diff] [blame] | 406 | bool MadeChange = false; |
Chandler Carruth | acdae3e | 2012-03-25 03:29:25 +0000 | [diff] [blame] | 407 | |
| 408 | #ifndef NDEBUG |
| 409 | // In debug builds, ensure that the terminator of the block is never replaced |
| 410 | // or deleted by these simplifications. The idea of simplification is that it |
| 411 | // cannot introduce new instructions, and there is no way to replace the |
| 412 | // terminator of a block without introducing a new instruction. |
| 413 | AssertingVH<Instruction> TerminatorVH(--BB->end()); |
| 414 | #endif |
| 415 | |
Chandler Carruth | 858cd1c | 2012-03-24 23:03:27 +0000 | [diff] [blame] | 416 | for (BasicBlock::iterator BI = BB->begin(), E = --BB->end(); BI != E; ) { |
| 417 | assert(!BI->isTerminator()); |
Chris Lattner | e234a30 | 2010-01-12 19:40:54 +0000 | [diff] [blame] | 418 | Instruction *Inst = BI++; |
Chandler Carruth | 6b98054 | 2012-03-24 21:11:24 +0000 | [diff] [blame] | 419 | |
| 420 | WeakVH BIHandle(BI); |
| 421 | if (recursivelySimplifyInstruction(Inst, TD)) { |
Chris Lattner | e234a30 | 2010-01-12 19:40:54 +0000 | [diff] [blame] | 422 | MadeChange = true; |
Chris Lattner | 35a939b | 2010-07-15 06:06:04 +0000 | [diff] [blame] | 423 | if (BIHandle != BI) |
Chris Lattner | e234a30 | 2010-01-12 19:40:54 +0000 | [diff] [blame] | 424 | BI = BB->begin(); |
| 425 | continue; |
| 426 | } |
Eli Friedman | 71ad2c9 | 2011-04-02 22:45:17 +0000 | [diff] [blame] | 427 | |
Benjamin Kramer | 8e0d1c0 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 428 | MadeChange |= RecursivelyDeleteTriviallyDeadInstructions(Inst, TLI); |
Eli Friedman | 71ad2c9 | 2011-04-02 22:45:17 +0000 | [diff] [blame] | 429 | if (BIHandle != BI) |
| 430 | BI = BB->begin(); |
Chris Lattner | e234a30 | 2010-01-12 19:40:54 +0000 | [diff] [blame] | 431 | } |
| 432 | return MadeChange; |
| 433 | } |
| 434 | |
Chris Lattner | b29714a | 2008-11-27 07:43:12 +0000 | [diff] [blame] | 435 | //===----------------------------------------------------------------------===// |
Chris Lattner | 40d8c28 | 2009-11-10 22:26:15 +0000 | [diff] [blame] | 436 | // Control Flow Graph Restructuring. |
Chris Lattner | b29714a | 2008-11-27 07:43:12 +0000 | [diff] [blame] | 437 | // |
| 438 | |
Chris Lattner | 40d8c28 | 2009-11-10 22:26:15 +0000 | [diff] [blame] | 439 | |
| 440 | /// RemovePredecessorAndSimplify - Like BasicBlock::removePredecessor, this |
| 441 | /// method is called when we're about to delete Pred as a predecessor of BB. If |
| 442 | /// BB contains any PHI nodes, this drops the entries in the PHI nodes for Pred. |
| 443 | /// |
| 444 | /// Unlike the removePredecessor method, this attempts to simplify uses of PHI |
| 445 | /// nodes that collapse into identity values. For example, if we have: |
| 446 | /// x = phi(1, 0, 0, 0) |
| 447 | /// y = and x, z |
| 448 | /// |
| 449 | /// .. and delete the predecessor corresponding to the '1', this will attempt to |
| 450 | /// recursively fold the and to 0. |
| 451 | void llvm::RemovePredecessorAndSimplify(BasicBlock *BB, BasicBlock *Pred, |
Micah Villmow | 3574eca | 2012-10-08 16:38:25 +0000 | [diff] [blame] | 452 | DataLayout *TD) { |
Chris Lattner | 40d8c28 | 2009-11-10 22:26:15 +0000 | [diff] [blame] | 453 | // This only adjusts blocks with PHI nodes. |
| 454 | if (!isa<PHINode>(BB->begin())) |
| 455 | return; |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 456 | |
Chris Lattner | 40d8c28 | 2009-11-10 22:26:15 +0000 | [diff] [blame] | 457 | // Remove the entries for Pred from the PHI nodes in BB, but do not simplify |
| 458 | // them down. This will leave us with single entry phi nodes and other phis |
| 459 | // that can be removed. |
| 460 | BB->removePredecessor(Pred, true); |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 461 | |
Chris Lattner | 40d8c28 | 2009-11-10 22:26:15 +0000 | [diff] [blame] | 462 | WeakVH PhiIt = &BB->front(); |
| 463 | while (PHINode *PN = dyn_cast<PHINode>(PhiIt)) { |
| 464 | PhiIt = &*++BasicBlock::iterator(cast<Instruction>(PhiIt)); |
Chris Lattner | 35a939b | 2010-07-15 06:06:04 +0000 | [diff] [blame] | 465 | Value *OldPhiIt = PhiIt; |
Chandler Carruth | 6b98054 | 2012-03-24 21:11:24 +0000 | [diff] [blame] | 466 | |
| 467 | if (!recursivelySimplifyInstruction(PN, TD)) |
| 468 | continue; |
| 469 | |
Chris Lattner | 40d8c28 | 2009-11-10 22:26:15 +0000 | [diff] [blame] | 470 | // If recursive simplification ended up deleting the next PHI node we would |
| 471 | // iterate to, then our iterator is invalid, restart scanning from the top |
| 472 | // of the block. |
Chris Lattner | 35a939b | 2010-07-15 06:06:04 +0000 | [diff] [blame] | 473 | if (PhiIt != OldPhiIt) PhiIt = &BB->front(); |
Chris Lattner | 40d8c28 | 2009-11-10 22:26:15 +0000 | [diff] [blame] | 474 | } |
| 475 | } |
| 476 | |
| 477 | |
Chris Lattner | b29714a | 2008-11-27 07:43:12 +0000 | [diff] [blame] | 478 | /// MergeBasicBlockIntoOnlyPred - DestBB is a block with one predecessor and its |
| 479 | /// predecessor is known to have one successor (DestBB!). Eliminate the edge |
| 480 | /// between them, moving the instructions in the predecessor into DestBB and |
| 481 | /// deleting the predecessor block. |
| 482 | /// |
Andreas Neustifter | ad80981 | 2009-09-16 09:26:52 +0000 | [diff] [blame] | 483 | void llvm::MergeBasicBlockIntoOnlyPred(BasicBlock *DestBB, Pass *P) { |
Chris Lattner | b29714a | 2008-11-27 07:43:12 +0000 | [diff] [blame] | 484 | // If BB has single-entry PHI nodes, fold them. |
| 485 | while (PHINode *PN = dyn_cast<PHINode>(DestBB->begin())) { |
| 486 | Value *NewVal = PN->getIncomingValue(0); |
| 487 | // Replace self referencing PHI with undef, it must be dead. |
Owen Anderson | 9e9a0d5 | 2009-07-30 23:03:37 +0000 | [diff] [blame] | 488 | if (NewVal == PN) NewVal = UndefValue::get(PN->getType()); |
Chris Lattner | b29714a | 2008-11-27 07:43:12 +0000 | [diff] [blame] | 489 | PN->replaceAllUsesWith(NewVal); |
| 490 | PN->eraseFromParent(); |
| 491 | } |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 492 | |
Chris Lattner | b29714a | 2008-11-27 07:43:12 +0000 | [diff] [blame] | 493 | BasicBlock *PredBB = DestBB->getSinglePredecessor(); |
| 494 | assert(PredBB && "Block doesn't have a single predecessor!"); |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 495 | |
Chris Lattner | 37914c8 | 2010-02-15 20:47:49 +0000 | [diff] [blame] | 496 | // Zap anything that took the address of DestBB. Not doing this will give the |
| 497 | // address an invalid value. |
| 498 | if (DestBB->hasAddressTaken()) { |
| 499 | BlockAddress *BA = BlockAddress::get(DestBB); |
| 500 | Constant *Replacement = |
| 501 | ConstantInt::get(llvm::Type::getInt32Ty(BA->getContext()), 1); |
| 502 | BA->replaceAllUsesWith(ConstantExpr::getIntToPtr(Replacement, |
| 503 | BA->getType())); |
| 504 | BA->destroyConstant(); |
| 505 | } |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 506 | |
Chris Lattner | b29714a | 2008-11-27 07:43:12 +0000 | [diff] [blame] | 507 | // Anything that branched to PredBB now branches to DestBB. |
| 508 | PredBB->replaceAllUsesWith(DestBB); |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 509 | |
Jay Foad | 95c3e48 | 2011-06-23 09:09:15 +0000 | [diff] [blame] | 510 | // Splice all the instructions from PredBB to DestBB. |
| 511 | PredBB->getTerminator()->eraseFromParent(); |
| 512 | DestBB->getInstList().splice(DestBB->begin(), PredBB->getInstList()); |
| 513 | |
Andreas Neustifter | ad80981 | 2009-09-16 09:26:52 +0000 | [diff] [blame] | 514 | if (P) { |
Cameron Zwarich | 80f6a50 | 2011-01-08 17:01:52 +0000 | [diff] [blame] | 515 | DominatorTree *DT = P->getAnalysisIfAvailable<DominatorTree>(); |
| 516 | if (DT) { |
| 517 | BasicBlock *PredBBIDom = DT->getNode(PredBB)->getIDom()->getBlock(); |
| 518 | DT->changeImmediateDominator(DestBB, PredBBIDom); |
| 519 | DT->eraseNode(PredBB); |
| 520 | } |
Andreas Neustifter | ad80981 | 2009-09-16 09:26:52 +0000 | [diff] [blame] | 521 | ProfileInfo *PI = P->getAnalysisIfAvailable<ProfileInfo>(); |
| 522 | if (PI) { |
| 523 | PI->replaceAllUses(PredBB, DestBB); |
| 524 | PI->removeEdge(ProfileInfo::getEdge(PredBB, DestBB)); |
| 525 | } |
| 526 | } |
Chris Lattner | b29714a | 2008-11-27 07:43:12 +0000 | [diff] [blame] | 527 | // Nuke BB. |
| 528 | PredBB->eraseFromParent(); |
| 529 | } |
Devang Patel | 4afc90d | 2009-02-10 07:00:59 +0000 | [diff] [blame] | 530 | |
Duncan Sands | c48b55a | 2013-07-11 08:28:20 +0000 | [diff] [blame] | 531 | /// CanMergeValues - Return true if we can choose one of these values to use |
| 532 | /// in place of the other. Note that we will always choose the non-undef |
| 533 | /// value to keep. |
| 534 | static bool CanMergeValues(Value *First, Value *Second) { |
| 535 | return First == Second || isa<UndefValue>(First) || isa<UndefValue>(Second); |
| 536 | } |
| 537 | |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 538 | /// CanPropagatePredecessorsForPHIs - Return true if we can fold BB, an |
Mark Lacey | 1b6e10f | 2013-08-14 22:11:42 +0000 | [diff] [blame^] | 539 | /// almost-empty BB ending in an unconditional branch to Succ, into Succ. |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 540 | /// |
| 541 | /// Assumption: Succ is the single successor for BB. |
| 542 | /// |
| 543 | static bool CanPropagatePredecessorsForPHIs(BasicBlock *BB, BasicBlock *Succ) { |
| 544 | assert(*succ_begin(BB) == Succ && "Succ is not successor of BB!"); |
| 545 | |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 546 | DEBUG(dbgs() << "Looking to fold " << BB->getName() << " into " |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 547 | << Succ->getName() << "\n"); |
| 548 | // Shortcut, if there is only a single predecessor it must be BB and merging |
| 549 | // is always safe |
| 550 | if (Succ->getSinglePredecessor()) return true; |
| 551 | |
| 552 | // Make a list of the predecessors of BB |
Benjamin Kramer | 88c0914 | 2011-12-06 16:14:29 +0000 | [diff] [blame] | 553 | SmallPtrSet<BasicBlock*, 16> BBPreds(pred_begin(BB), pred_end(BB)); |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 554 | |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 555 | // Look at all the phi nodes in Succ, to see if they present a conflict when |
| 556 | // merging these blocks |
| 557 | for (BasicBlock::iterator I = Succ->begin(); isa<PHINode>(I); ++I) { |
| 558 | PHINode *PN = cast<PHINode>(I); |
| 559 | |
| 560 | // If the incoming value from BB is again a PHINode in |
| 561 | // BB which has the same incoming value for *PI as PN does, we can |
| 562 | // merge the phi nodes and then the blocks can still be merged |
| 563 | PHINode *BBPN = dyn_cast<PHINode>(PN->getIncomingValueForBlock(BB)); |
| 564 | if (BBPN && BBPN->getParent() == BB) { |
Benjamin Kramer | 88c0914 | 2011-12-06 16:14:29 +0000 | [diff] [blame] | 565 | for (unsigned PI = 0, PE = PN->getNumIncomingValues(); PI != PE; ++PI) { |
| 566 | BasicBlock *IBB = PN->getIncomingBlock(PI); |
| 567 | if (BBPreds.count(IBB) && |
Duncan Sands | c48b55a | 2013-07-11 08:28:20 +0000 | [diff] [blame] | 568 | !CanMergeValues(BBPN->getIncomingValueForBlock(IBB), |
| 569 | PN->getIncomingValue(PI))) { |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 570 | DEBUG(dbgs() << "Can't fold, phi node " << PN->getName() << " in " |
| 571 | << Succ->getName() << " is conflicting with " |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 572 | << BBPN->getName() << " with regard to common predecessor " |
Benjamin Kramer | 88c0914 | 2011-12-06 16:14:29 +0000 | [diff] [blame] | 573 | << IBB->getName() << "\n"); |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 574 | return false; |
| 575 | } |
| 576 | } |
| 577 | } else { |
| 578 | Value* Val = PN->getIncomingValueForBlock(BB); |
Benjamin Kramer | 88c0914 | 2011-12-06 16:14:29 +0000 | [diff] [blame] | 579 | for (unsigned PI = 0, PE = PN->getNumIncomingValues(); PI != PE; ++PI) { |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 580 | // See if the incoming value for the common predecessor is equal to the |
| 581 | // one for BB, in which case this phi node will not prevent the merging |
| 582 | // of the block. |
Benjamin Kramer | 88c0914 | 2011-12-06 16:14:29 +0000 | [diff] [blame] | 583 | BasicBlock *IBB = PN->getIncomingBlock(PI); |
Duncan Sands | c48b55a | 2013-07-11 08:28:20 +0000 | [diff] [blame] | 584 | if (BBPreds.count(IBB) && |
| 585 | !CanMergeValues(Val, PN->getIncomingValue(PI))) { |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 586 | DEBUG(dbgs() << "Can't fold, phi node " << PN->getName() << " in " |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 587 | << Succ->getName() << " is conflicting with regard to common " |
Benjamin Kramer | 88c0914 | 2011-12-06 16:14:29 +0000 | [diff] [blame] | 588 | << "predecessor " << IBB->getName() << "\n"); |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 589 | return false; |
| 590 | } |
| 591 | } |
| 592 | } |
| 593 | } |
| 594 | |
| 595 | return true; |
| 596 | } |
| 597 | |
Duncan Sands | c48b55a | 2013-07-11 08:28:20 +0000 | [diff] [blame] | 598 | typedef SmallVector<BasicBlock *, 16> PredBlockVector; |
| 599 | typedef DenseMap<BasicBlock *, Value *> IncomingValueMap; |
| 600 | |
| 601 | /// \brief Determines the value to use as the phi node input for a block. |
| 602 | /// |
| 603 | /// Select between \p OldVal any value that we know flows from \p BB |
| 604 | /// to a particular phi on the basis of which one (if either) is not |
| 605 | /// undef. Update IncomingValues based on the selected value. |
| 606 | /// |
| 607 | /// \param OldVal The value we are considering selecting. |
| 608 | /// \param BB The block that the value flows in from. |
| 609 | /// \param IncomingValues A map from block-to-value for other phi inputs |
| 610 | /// that we have examined. |
| 611 | /// |
| 612 | /// \returns the selected value. |
| 613 | static Value *selectIncomingValueForBlock(Value *OldVal, BasicBlock *BB, |
| 614 | IncomingValueMap &IncomingValues) { |
| 615 | if (!isa<UndefValue>(OldVal)) { |
| 616 | assert((!IncomingValues.count(BB) || |
| 617 | IncomingValues.find(BB)->second == OldVal) && |
| 618 | "Expected OldVal to match incoming value from BB!"); |
| 619 | |
| 620 | IncomingValues.insert(std::make_pair(BB, OldVal)); |
| 621 | return OldVal; |
| 622 | } |
| 623 | |
| 624 | IncomingValueMap::const_iterator It = IncomingValues.find(BB); |
| 625 | if (It != IncomingValues.end()) return It->second; |
| 626 | |
| 627 | return OldVal; |
| 628 | } |
| 629 | |
| 630 | /// \brief Create a map from block to value for the operands of a |
| 631 | /// given phi. |
| 632 | /// |
| 633 | /// Create a map from block to value for each non-undef value flowing |
| 634 | /// into \p PN. |
| 635 | /// |
| 636 | /// \param PN The phi we are collecting the map for. |
| 637 | /// \param IncomingValues [out] The map from block to value for this phi. |
| 638 | static void gatherIncomingValuesToPhi(PHINode *PN, |
| 639 | IncomingValueMap &IncomingValues) { |
| 640 | for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) { |
| 641 | BasicBlock *BB = PN->getIncomingBlock(i); |
| 642 | Value *V = PN->getIncomingValue(i); |
| 643 | |
| 644 | if (!isa<UndefValue>(V)) |
| 645 | IncomingValues.insert(std::make_pair(BB, V)); |
| 646 | } |
| 647 | } |
| 648 | |
| 649 | /// \brief Replace the incoming undef values to a phi with the values |
| 650 | /// from a block-to-value map. |
| 651 | /// |
| 652 | /// \param PN The phi we are replacing the undefs in. |
| 653 | /// \param IncomingValues A map from block to value. |
| 654 | static void replaceUndefValuesInPhi(PHINode *PN, |
| 655 | const IncomingValueMap &IncomingValues) { |
| 656 | for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) { |
| 657 | Value *V = PN->getIncomingValue(i); |
| 658 | |
| 659 | if (!isa<UndefValue>(V)) continue; |
| 660 | |
| 661 | BasicBlock *BB = PN->getIncomingBlock(i); |
| 662 | IncomingValueMap::const_iterator It = IncomingValues.find(BB); |
| 663 | if (It == IncomingValues.end()) continue; |
| 664 | |
| 665 | PN->setIncomingValue(i, It->second); |
| 666 | } |
| 667 | } |
| 668 | |
| 669 | /// \brief Replace a value flowing from a block to a phi with |
| 670 | /// potentially multiple instances of that value flowing from the |
| 671 | /// block's predecessors to the phi. |
| 672 | /// |
| 673 | /// \param BB The block with the value flowing into the phi. |
| 674 | /// \param BBPreds The predecessors of BB. |
| 675 | /// \param PN The phi that we are updating. |
| 676 | static void redirectValuesFromPredecessorsToPhi(BasicBlock *BB, |
| 677 | const PredBlockVector &BBPreds, |
| 678 | PHINode *PN) { |
| 679 | Value *OldVal = PN->removeIncomingValue(BB, false); |
| 680 | assert(OldVal && "No entry in PHI for Pred BB!"); |
| 681 | |
| 682 | IncomingValueMap IncomingValues; |
| 683 | |
| 684 | // We are merging two blocks - BB, and the block containing PN - and |
| 685 | // as a result we need to redirect edges from the predecessors of BB |
| 686 | // to go to the block containing PN, and update PN |
| 687 | // accordingly. Since we allow merging blocks in the case where the |
| 688 | // predecessor and successor blocks both share some predecessors, |
| 689 | // and where some of those common predecessors might have undef |
| 690 | // values flowing into PN, we want to rewrite those values to be |
| 691 | // consistent with the non-undef values. |
| 692 | |
| 693 | gatherIncomingValuesToPhi(PN, IncomingValues); |
| 694 | |
| 695 | // If this incoming value is one of the PHI nodes in BB, the new entries |
| 696 | // in the PHI node are the entries from the old PHI. |
| 697 | if (isa<PHINode>(OldVal) && cast<PHINode>(OldVal)->getParent() == BB) { |
| 698 | PHINode *OldValPN = cast<PHINode>(OldVal); |
| 699 | for (unsigned i = 0, e = OldValPN->getNumIncomingValues(); i != e; ++i) { |
| 700 | // Note that, since we are merging phi nodes and BB and Succ might |
| 701 | // have common predecessors, we could end up with a phi node with |
| 702 | // identical incoming branches. This will be cleaned up later (and |
| 703 | // will trigger asserts if we try to clean it up now, without also |
| 704 | // simplifying the corresponding conditional branch). |
| 705 | BasicBlock *PredBB = OldValPN->getIncomingBlock(i); |
| 706 | Value *PredVal = OldValPN->getIncomingValue(i); |
| 707 | Value *Selected = selectIncomingValueForBlock(PredVal, PredBB, |
| 708 | IncomingValues); |
| 709 | |
| 710 | // And add a new incoming value for this predecessor for the |
| 711 | // newly retargeted branch. |
| 712 | PN->addIncoming(Selected, PredBB); |
| 713 | } |
| 714 | } else { |
| 715 | for (unsigned i = 0, e = BBPreds.size(); i != e; ++i) { |
| 716 | // Update existing incoming values in PN for this |
| 717 | // predecessor of BB. |
| 718 | BasicBlock *PredBB = BBPreds[i]; |
| 719 | Value *Selected = selectIncomingValueForBlock(OldVal, PredBB, |
| 720 | IncomingValues); |
| 721 | |
| 722 | // And add a new incoming value for this predecessor for the |
| 723 | // newly retargeted branch. |
| 724 | PN->addIncoming(Selected, PredBB); |
| 725 | } |
| 726 | } |
| 727 | |
| 728 | replaceUndefValuesInPhi(PN, IncomingValues); |
| 729 | } |
| 730 | |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 731 | /// TryToSimplifyUncondBranchFromEmptyBlock - BB is known to contain an |
| 732 | /// unconditional branch, and contains no instructions other than PHI nodes, |
Rafael Espindola | 77a2c37 | 2011-06-30 20:14:24 +0000 | [diff] [blame] | 733 | /// potential side-effect free intrinsics and the branch. If possible, |
| 734 | /// eliminate BB by rewriting all the predecessors to branch to the successor |
| 735 | /// block and return true. If we can't transform, return false. |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 736 | bool llvm::TryToSimplifyUncondBranchFromEmptyBlock(BasicBlock *BB) { |
Dan Gohman | e2c6d13 | 2010-08-14 00:29:42 +0000 | [diff] [blame] | 737 | assert(BB != &BB->getParent()->getEntryBlock() && |
| 738 | "TryToSimplifyUncondBranchFromEmptyBlock called on entry block!"); |
| 739 | |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 740 | // We can't eliminate infinite loops. |
| 741 | BasicBlock *Succ = cast<BranchInst>(BB->getTerminator())->getSuccessor(0); |
| 742 | if (BB == Succ) return false; |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 743 | |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 744 | // Check to see if merging these blocks would cause conflicts for any of the |
| 745 | // phi nodes in BB or Succ. If not, we can safely merge. |
| 746 | if (!CanPropagatePredecessorsForPHIs(BB, Succ)) return false; |
| 747 | |
| 748 | // Check for cases where Succ has multiple predecessors and a PHI node in BB |
| 749 | // has uses which will not disappear when the PHI nodes are merged. It is |
| 750 | // possible to handle such cases, but difficult: it requires checking whether |
| 751 | // BB dominates Succ, which is non-trivial to calculate in the case where |
| 752 | // Succ has multiple predecessors. Also, it requires checking whether |
Alexey Samsonov | 3a19999 | 2012-12-24 08:52:53 +0000 | [diff] [blame] | 753 | // constructing the necessary self-referential PHI node doesn't introduce any |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 754 | // conflicts; this isn't too difficult, but the previous code for doing this |
| 755 | // was incorrect. |
| 756 | // |
| 757 | // Note that if this check finds a live use, BB dominates Succ, so BB is |
| 758 | // something like a loop pre-header (or rarely, a part of an irreducible CFG); |
| 759 | // folding the branch isn't profitable in that case anyway. |
| 760 | if (!Succ->getSinglePredecessor()) { |
| 761 | BasicBlock::iterator BBI = BB->begin(); |
| 762 | while (isa<PHINode>(*BBI)) { |
| 763 | for (Value::use_iterator UI = BBI->use_begin(), E = BBI->use_end(); |
| 764 | UI != E; ++UI) { |
| 765 | if (PHINode* PN = dyn_cast<PHINode>(*UI)) { |
| 766 | if (PN->getIncomingBlock(UI) != BB) |
| 767 | return false; |
| 768 | } else { |
| 769 | return false; |
| 770 | } |
| 771 | } |
| 772 | ++BBI; |
| 773 | } |
| 774 | } |
| 775 | |
David Greene | fae7706 | 2010-01-05 01:26:57 +0000 | [diff] [blame] | 776 | DEBUG(dbgs() << "Killing Trivial BB: \n" << *BB); |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 777 | |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 778 | if (isa<PHINode>(Succ->begin())) { |
| 779 | // If there is more than one pred of succ, and there are PHI nodes in |
| 780 | // the successor, then we need to add incoming edges for the PHI nodes |
| 781 | // |
Duncan Sands | c48b55a | 2013-07-11 08:28:20 +0000 | [diff] [blame] | 782 | const PredBlockVector BBPreds(pred_begin(BB), pred_end(BB)); |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 783 | |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 784 | // Loop over all of the PHI nodes in the successor of BB. |
| 785 | for (BasicBlock::iterator I = Succ->begin(); isa<PHINode>(I); ++I) { |
| 786 | PHINode *PN = cast<PHINode>(I); |
Duncan Sands | c48b55a | 2013-07-11 08:28:20 +0000 | [diff] [blame] | 787 | |
| 788 | redirectValuesFromPredecessorsToPhi(BB, BBPreds, PN); |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 789 | } |
| 790 | } |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 791 | |
Rafael Espindola | 77a2c37 | 2011-06-30 20:14:24 +0000 | [diff] [blame] | 792 | if (Succ->getSinglePredecessor()) { |
| 793 | // BB is the only predecessor of Succ, so Succ will end up with exactly |
| 794 | // the same predecessors BB had. |
| 795 | |
| 796 | // Copy over any phi, debug or lifetime instruction. |
| 797 | BB->getTerminator()->eraseFromParent(); |
| 798 | Succ->getInstList().splice(Succ->getFirstNonPHI(), BB->getInstList()); |
| 799 | } else { |
| 800 | while (PHINode *PN = dyn_cast<PHINode>(&BB->front())) { |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 801 | // We explicitly check for such uses in CanPropagatePredecessorsForPHIs. |
| 802 | assert(PN->use_empty() && "There shouldn't be any uses here!"); |
| 803 | PN->eraseFromParent(); |
| 804 | } |
| 805 | } |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 806 | |
Chris Lattner | dce94d9 | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 807 | // Everything that jumped to BB now goes to Succ. |
| 808 | BB->replaceAllUsesWith(Succ); |
| 809 | if (!Succ->hasName()) Succ->takeName(BB); |
| 810 | BB->eraseFromParent(); // Delete the old basic block. |
| 811 | return true; |
| 812 | } |
| 813 | |
Jim Grosbach | 43a8241 | 2009-12-02 17:06:45 +0000 | [diff] [blame] | 814 | /// EliminateDuplicatePHINodes - Check for and eliminate duplicate PHI |
| 815 | /// nodes in this block. This doesn't try to be clever about PHI nodes |
| 816 | /// which differ only in the order of the incoming values, but instcombine |
| 817 | /// orders them so it usually won't matter. |
| 818 | /// |
| 819 | bool llvm::EliminateDuplicatePHINodes(BasicBlock *BB) { |
| 820 | bool Changed = false; |
| 821 | |
| 822 | // This implementation doesn't currently consider undef operands |
Nick Lewycky | 89991d4 | 2011-06-28 03:57:31 +0000 | [diff] [blame] | 823 | // specially. Theoretically, two phis which are identical except for |
Jim Grosbach | 43a8241 | 2009-12-02 17:06:45 +0000 | [diff] [blame] | 824 | // one having an undef where the other doesn't could be collapsed. |
| 825 | |
| 826 | // Map from PHI hash values to PHI nodes. If multiple PHIs have |
| 827 | // the same hash value, the element is the first PHI in the |
| 828 | // linked list in CollisionMap. |
| 829 | DenseMap<uintptr_t, PHINode *> HashMap; |
| 830 | |
| 831 | // Maintain linked lists of PHI nodes with common hash values. |
| 832 | DenseMap<PHINode *, PHINode *> CollisionMap; |
| 833 | |
| 834 | // Examine each PHI. |
| 835 | for (BasicBlock::iterator I = BB->begin(); |
| 836 | PHINode *PN = dyn_cast<PHINode>(I++); ) { |
| 837 | // Compute a hash value on the operands. Instcombine will likely have sorted |
| 838 | // them, which helps expose duplicates, but we have to check all the |
| 839 | // operands to be safe in case instcombine hasn't run. |
| 840 | uintptr_t Hash = 0; |
Jay Foad | 95c3e48 | 2011-06-23 09:09:15 +0000 | [diff] [blame] | 841 | // This hash algorithm is quite weak as hash functions go, but it seems |
| 842 | // to do a good enough job for this particular purpose, and is very quick. |
Jim Grosbach | 43a8241 | 2009-12-02 17:06:45 +0000 | [diff] [blame] | 843 | for (User::op_iterator I = PN->op_begin(), E = PN->op_end(); I != E; ++I) { |
Jim Grosbach | 43a8241 | 2009-12-02 17:06:45 +0000 | [diff] [blame] | 844 | Hash ^= reinterpret_cast<uintptr_t>(static_cast<Value *>(*I)); |
| 845 | Hash = (Hash << 7) | (Hash >> (sizeof(uintptr_t) * CHAR_BIT - 7)); |
| 846 | } |
Jay Foad | 95c3e48 | 2011-06-23 09:09:15 +0000 | [diff] [blame] | 847 | for (PHINode::block_iterator I = PN->block_begin(), E = PN->block_end(); |
| 848 | I != E; ++I) { |
| 849 | Hash ^= reinterpret_cast<uintptr_t>(static_cast<BasicBlock *>(*I)); |
| 850 | Hash = (Hash << 7) | (Hash >> (sizeof(uintptr_t) * CHAR_BIT - 7)); |
| 851 | } |
Jakob Stoklund Olesen | 2bc2a08 | 2011-03-04 02:48:56 +0000 | [diff] [blame] | 852 | // Avoid colliding with the DenseMap sentinels ~0 and ~0-1. |
| 853 | Hash >>= 1; |
Jim Grosbach | 43a8241 | 2009-12-02 17:06:45 +0000 | [diff] [blame] | 854 | // If we've never seen this hash value before, it's a unique PHI. |
| 855 | std::pair<DenseMap<uintptr_t, PHINode *>::iterator, bool> Pair = |
| 856 | HashMap.insert(std::make_pair(Hash, PN)); |
| 857 | if (Pair.second) continue; |
| 858 | // Otherwise it's either a duplicate or a hash collision. |
| 859 | for (PHINode *OtherPN = Pair.first->second; ; ) { |
| 860 | if (OtherPN->isIdenticalTo(PN)) { |
| 861 | // A duplicate. Replace this PHI with its duplicate. |
| 862 | PN->replaceAllUsesWith(OtherPN); |
| 863 | PN->eraseFromParent(); |
| 864 | Changed = true; |
| 865 | break; |
| 866 | } |
| 867 | // A non-duplicate hash collision. |
| 868 | DenseMap<PHINode *, PHINode *>::iterator I = CollisionMap.find(OtherPN); |
| 869 | if (I == CollisionMap.end()) { |
| 870 | // Set this PHI to be the head of the linked list of colliding PHIs. |
| 871 | PHINode *Old = Pair.first->second; |
| 872 | Pair.first->second = PN; |
| 873 | CollisionMap[PN] = Old; |
| 874 | break; |
| 875 | } |
Benjamin Kramer | d9b0b02 | 2012-06-02 10:20:22 +0000 | [diff] [blame] | 876 | // Proceed to the next PHI in the list. |
Jim Grosbach | 43a8241 | 2009-12-02 17:06:45 +0000 | [diff] [blame] | 877 | OtherPN = I->second; |
| 878 | } |
| 879 | } |
| 880 | |
| 881 | return Changed; |
| 882 | } |
Chris Lattner | 687140c | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 883 | |
| 884 | /// enforceKnownAlignment - If the specified pointer points to an object that |
| 885 | /// we control, modify the object's alignment to PrefAlign. This isn't |
| 886 | /// often possible though. If alignment is important, a more reliable approach |
| 887 | /// is to simply align all global variables and allocation instructions to |
| 888 | /// their preferred alignment from the beginning. |
| 889 | /// |
Benjamin Kramer | 1928236 | 2010-12-30 22:34:44 +0000 | [diff] [blame] | 890 | static unsigned enforceKnownAlignment(Value *V, unsigned Align, |
Micah Villmow | 3574eca | 2012-10-08 16:38:25 +0000 | [diff] [blame] | 891 | unsigned PrefAlign, const DataLayout *TD) { |
Eli Friedman | b53c793 | 2011-06-15 21:08:25 +0000 | [diff] [blame] | 892 | V = V->stripPointerCasts(); |
Chris Lattner | 687140c | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 893 | |
Eli Friedman | b53c793 | 2011-06-15 21:08:25 +0000 | [diff] [blame] | 894 | if (AllocaInst *AI = dyn_cast<AllocaInst>(V)) { |
Lang Hames | bb5b3f3 | 2011-10-10 23:42:08 +0000 | [diff] [blame] | 895 | // If the preferred alignment is greater than the natural stack alignment |
| 896 | // then don't round up. This avoids dynamic stack realignment. |
| 897 | if (TD && TD->exceedsNaturalStackAlignment(PrefAlign)) |
| 898 | return Align; |
Chris Lattner | 687140c | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 899 | // If there is a requested alignment and if this is an alloca, round up. |
| 900 | if (AI->getAlignment() >= PrefAlign) |
| 901 | return AI->getAlignment(); |
| 902 | AI->setAlignment(PrefAlign); |
| 903 | return PrefAlign; |
| 904 | } |
Chris Lattner | 687140c | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 905 | |
| 906 | if (GlobalValue *GV = dyn_cast<GlobalValue>(V)) { |
| 907 | // If there is a large requested alignment and we can, bump up the alignment |
| 908 | // of the global. |
| 909 | if (GV->isDeclaration()) return Align; |
Duncan Sands | d3a38cc | 2011-11-29 18:26:38 +0000 | [diff] [blame] | 910 | // If the memory we set aside for the global may not be the memory used by |
| 911 | // the final program then it is impossible for us to reliably enforce the |
| 912 | // preferred alignment. |
| 913 | if (GV->isWeakForLinker()) return Align; |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 914 | |
Chris Lattner | 687140c | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 915 | if (GV->getAlignment() >= PrefAlign) |
| 916 | return GV->getAlignment(); |
| 917 | // We can only increase the alignment of the global if it has no alignment |
| 918 | // specified or if it is not assigned a section. If it is assigned a |
| 919 | // section, the global could be densely packed with other objects in the |
| 920 | // section, increasing the alignment could cause padding issues. |
| 921 | if (!GV->hasSection() || GV->getAlignment() == 0) |
| 922 | GV->setAlignment(PrefAlign); |
| 923 | return GV->getAlignment(); |
| 924 | } |
| 925 | |
| 926 | return Align; |
| 927 | } |
| 928 | |
| 929 | /// getOrEnforceKnownAlignment - If the specified pointer has an alignment that |
| 930 | /// we can determine, return it, otherwise return 0. If PrefAlign is specified, |
| 931 | /// and it is more than the alignment of the ultimate object, see if we can |
| 932 | /// increase the alignment of the ultimate object, making this check succeed. |
| 933 | unsigned llvm::getOrEnforceKnownAlignment(Value *V, unsigned PrefAlign, |
Matt Arsenault | 186f8f9 | 2013-08-01 22:42:18 +0000 | [diff] [blame] | 934 | const DataLayout *DL) { |
Chris Lattner | 687140c | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 935 | assert(V->getType()->isPointerTy() && |
| 936 | "getOrEnforceKnownAlignment expects a pointer!"); |
Matt Arsenault | 186f8f9 | 2013-08-01 22:42:18 +0000 | [diff] [blame] | 937 | unsigned BitWidth = DL ? DL->getPointerTypeSizeInBits(V->getType()) : 64; |
| 938 | |
Chris Lattner | 687140c | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 939 | APInt KnownZero(BitWidth, 0), KnownOne(BitWidth, 0); |
Matt Arsenault | 186f8f9 | 2013-08-01 22:42:18 +0000 | [diff] [blame] | 940 | ComputeMaskedBits(V, KnownZero, KnownOne, DL); |
Chris Lattner | 687140c | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 941 | unsigned TrailZ = KnownZero.countTrailingOnes(); |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 942 | |
Matt Arsenault | 59a3878 | 2013-07-23 22:20:57 +0000 | [diff] [blame] | 943 | // Avoid trouble with ridiculously large TrailZ values, such as |
Chris Lattner | 687140c | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 944 | // those computed from a null pointer. |
| 945 | TrailZ = std::min(TrailZ, unsigned(sizeof(unsigned) * CHAR_BIT - 1)); |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 946 | |
Chris Lattner | 687140c | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 947 | unsigned Align = 1u << std::min(BitWidth - 1, TrailZ); |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 948 | |
Chris Lattner | 687140c | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 949 | // LLVM doesn't support alignments larger than this currently. |
| 950 | Align = std::min(Align, +Value::MaximumAlignment); |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 951 | |
Chris Lattner | 687140c | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 952 | if (PrefAlign > Align) |
Matt Arsenault | 186f8f9 | 2013-08-01 22:42:18 +0000 | [diff] [blame] | 953 | Align = enforceKnownAlignment(V, Align, PrefAlign, DL); |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 954 | |
Chris Lattner | 687140c | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 955 | // We don't need to make any adjustment. |
| 956 | return Align; |
| 957 | } |
| 958 | |
Devang Patel | 5ee2068 | 2011-03-17 21:58:19 +0000 | [diff] [blame] | 959 | ///===---------------------------------------------------------------------===// |
| 960 | /// Dbg Intrinsic utilities |
| 961 | /// |
| 962 | |
Adrian Prantl | 163da93 | 2013-04-26 17:48:33 +0000 | [diff] [blame] | 963 | /// See if there is a dbg.value intrinsic for DIVar before I. |
| 964 | static bool LdStHasDebugValue(DIVariable &DIVar, Instruction *I) { |
| 965 | // Since we can't guarantee that the original dbg.declare instrinsic |
| 966 | // is removed by LowerDbgDeclare(), we need to make sure that we are |
| 967 | // not inserting the same dbg.value intrinsic over and over. |
| 968 | llvm::BasicBlock::InstListType::iterator PrevI(I); |
| 969 | if (PrevI != I->getParent()->getInstList().begin()) { |
| 970 | --PrevI; |
| 971 | if (DbgValueInst *DVI = dyn_cast<DbgValueInst>(PrevI)) |
| 972 | if (DVI->getValue() == I->getOperand(0) && |
| 973 | DVI->getOffset() == 0 && |
| 974 | DVI->getVariable() == DIVar) |
| 975 | return true; |
| 976 | } |
| 977 | return false; |
| 978 | } |
| 979 | |
Adrian Prantl | 9d5d58a | 2013-04-26 18:10:50 +0000 | [diff] [blame] | 980 | /// Inserts a llvm.dbg.value intrinsic before a store to an alloca'd value |
Devang Patel | 5ee2068 | 2011-03-17 21:58:19 +0000 | [diff] [blame] | 981 | /// that has an associated llvm.dbg.decl intrinsic. |
| 982 | bool llvm::ConvertDebugDeclareToDebugValue(DbgDeclareInst *DDI, |
| 983 | StoreInst *SI, DIBuilder &Builder) { |
| 984 | DIVariable DIVar(DDI->getVariable()); |
Manman Ren | cbafae6 | 2013-06-28 05:43:10 +0000 | [diff] [blame] | 985 | assert((!DIVar || DIVar.isVariable()) && |
| 986 | "Variable in DbgDeclareInst should be either null or a DIVariable."); |
| 987 | if (!DIVar) |
Devang Patel | 5ee2068 | 2011-03-17 21:58:19 +0000 | [diff] [blame] | 988 | return false; |
| 989 | |
Adrian Prantl | 163da93 | 2013-04-26 17:48:33 +0000 | [diff] [blame] | 990 | if (LdStHasDebugValue(DIVar, SI)) |
| 991 | return true; |
| 992 | |
Devang Patel | 227dfdb | 2011-05-16 21:24:05 +0000 | [diff] [blame] | 993 | Instruction *DbgVal = NULL; |
| 994 | // If an argument is zero extended then use argument directly. The ZExt |
| 995 | // may be zapped by an optimization pass in future. |
| 996 | Argument *ExtendedArg = NULL; |
| 997 | if (ZExtInst *ZExt = dyn_cast<ZExtInst>(SI->getOperand(0))) |
| 998 | ExtendedArg = dyn_cast<Argument>(ZExt->getOperand(0)); |
| 999 | if (SExtInst *SExt = dyn_cast<SExtInst>(SI->getOperand(0))) |
| 1000 | ExtendedArg = dyn_cast<Argument>(SExt->getOperand(0)); |
| 1001 | if (ExtendedArg) |
| 1002 | DbgVal = Builder.insertDbgValueIntrinsic(ExtendedArg, 0, DIVar, SI); |
| 1003 | else |
| 1004 | DbgVal = Builder.insertDbgValueIntrinsic(SI->getOperand(0), 0, DIVar, SI); |
| 1005 | |
Devang Patel | 5ee2068 | 2011-03-17 21:58:19 +0000 | [diff] [blame] | 1006 | // Propagate any debug metadata from the store onto the dbg.value. |
| 1007 | DebugLoc SIDL = SI->getDebugLoc(); |
| 1008 | if (!SIDL.isUnknown()) |
| 1009 | DbgVal->setDebugLoc(SIDL); |
| 1010 | // Otherwise propagate debug metadata from dbg.declare. |
| 1011 | else |
| 1012 | DbgVal->setDebugLoc(DDI->getDebugLoc()); |
| 1013 | return true; |
| 1014 | } |
| 1015 | |
Adrian Prantl | 9d5d58a | 2013-04-26 18:10:50 +0000 | [diff] [blame] | 1016 | /// Inserts a llvm.dbg.value intrinsic before a load of an alloca'd value |
Devang Patel | 36fae67 | 2011-03-18 23:45:43 +0000 | [diff] [blame] | 1017 | /// that has an associated llvm.dbg.decl intrinsic. |
| 1018 | bool llvm::ConvertDebugDeclareToDebugValue(DbgDeclareInst *DDI, |
| 1019 | LoadInst *LI, DIBuilder &Builder) { |
| 1020 | DIVariable DIVar(DDI->getVariable()); |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 1021 | assert((!DIVar || DIVar.isVariable()) && |
Manman Ren | cbafae6 | 2013-06-28 05:43:10 +0000 | [diff] [blame] | 1022 | "Variable in DbgDeclareInst should be either null or a DIVariable."); |
| 1023 | if (!DIVar) |
Devang Patel | 36fae67 | 2011-03-18 23:45:43 +0000 | [diff] [blame] | 1024 | return false; |
| 1025 | |
Adrian Prantl | 163da93 | 2013-04-26 17:48:33 +0000 | [diff] [blame] | 1026 | if (LdStHasDebugValue(DIVar, LI)) |
| 1027 | return true; |
| 1028 | |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 1029 | Instruction *DbgVal = |
Devang Patel | 36fae67 | 2011-03-18 23:45:43 +0000 | [diff] [blame] | 1030 | Builder.insertDbgValueIntrinsic(LI->getOperand(0), 0, |
| 1031 | DIVar, LI); |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 1032 | |
Devang Patel | 36fae67 | 2011-03-18 23:45:43 +0000 | [diff] [blame] | 1033 | // Propagate any debug metadata from the store onto the dbg.value. |
| 1034 | DebugLoc LIDL = LI->getDebugLoc(); |
| 1035 | if (!LIDL.isUnknown()) |
| 1036 | DbgVal->setDebugLoc(LIDL); |
| 1037 | // Otherwise propagate debug metadata from dbg.declare. |
| 1038 | else |
| 1039 | DbgVal->setDebugLoc(DDI->getDebugLoc()); |
| 1040 | return true; |
| 1041 | } |
| 1042 | |
Devang Patel | 813c9a0 | 2011-03-17 22:18:16 +0000 | [diff] [blame] | 1043 | /// LowerDbgDeclare - Lowers llvm.dbg.declare intrinsics into appropriate set |
| 1044 | /// of llvm.dbg.value intrinsics. |
| 1045 | bool llvm::LowerDbgDeclare(Function &F) { |
| 1046 | DIBuilder DIB(*F.getParent()); |
| 1047 | SmallVector<DbgDeclareInst *, 4> Dbgs; |
| 1048 | for (Function::iterator FI = F.begin(), FE = F.end(); FI != FE; ++FI) |
| 1049 | for (BasicBlock::iterator BI = FI->begin(), BE = FI->end(); BI != BE; ++BI) { |
| 1050 | if (DbgDeclareInst *DDI = dyn_cast<DbgDeclareInst>(BI)) |
| 1051 | Dbgs.push_back(DDI); |
| 1052 | } |
| 1053 | if (Dbgs.empty()) |
| 1054 | return false; |
| 1055 | |
Craig Topper | 6227d5c | 2013-07-04 01:31:24 +0000 | [diff] [blame] | 1056 | for (SmallVectorImpl<DbgDeclareInst *>::iterator I = Dbgs.begin(), |
Devang Patel | 813c9a0 | 2011-03-17 22:18:16 +0000 | [diff] [blame] | 1057 | E = Dbgs.end(); I != E; ++I) { |
| 1058 | DbgDeclareInst *DDI = *I; |
| 1059 | if (AllocaInst *AI = dyn_cast_or_null<AllocaInst>(DDI->getAddress())) { |
Adrian Prantl | 163da93 | 2013-04-26 17:48:33 +0000 | [diff] [blame] | 1060 | // We only remove the dbg.declare intrinsic if all uses are |
| 1061 | // converted to dbg.value intrinsics. |
Devang Patel | 81ad03c | 2011-04-28 20:32:02 +0000 | [diff] [blame] | 1062 | bool RemoveDDI = true; |
Devang Patel | 813c9a0 | 2011-03-17 22:18:16 +0000 | [diff] [blame] | 1063 | for (Value::use_iterator UI = AI->use_begin(), E = AI->use_end(); |
| 1064 | UI != E; ++UI) |
| 1065 | if (StoreInst *SI = dyn_cast<StoreInst>(*UI)) |
| 1066 | ConvertDebugDeclareToDebugValue(DDI, SI, DIB); |
Devang Patel | 36fae67 | 2011-03-18 23:45:43 +0000 | [diff] [blame] | 1067 | else if (LoadInst *LI = dyn_cast<LoadInst>(*UI)) |
| 1068 | ConvertDebugDeclareToDebugValue(DDI, LI, DIB); |
Devang Patel | 81ad03c | 2011-04-28 20:32:02 +0000 | [diff] [blame] | 1069 | else |
| 1070 | RemoveDDI = false; |
| 1071 | if (RemoveDDI) |
| 1072 | DDI->eraseFromParent(); |
Devang Patel | 813c9a0 | 2011-03-17 22:18:16 +0000 | [diff] [blame] | 1073 | } |
Devang Patel | 813c9a0 | 2011-03-17 22:18:16 +0000 | [diff] [blame] | 1074 | } |
| 1075 | return true; |
| 1076 | } |
Cameron Zwarich | c827939 | 2011-05-24 03:10:43 +0000 | [diff] [blame] | 1077 | |
| 1078 | /// FindAllocaDbgDeclare - Finds the llvm.dbg.declare intrinsic describing the |
| 1079 | /// alloca 'V', if any. |
| 1080 | DbgDeclareInst *llvm::FindAllocaDbgDeclare(Value *V) { |
| 1081 | if (MDNode *DebugNode = MDNode::getIfExists(V->getContext(), V)) |
| 1082 | for (Value::use_iterator UI = DebugNode->use_begin(), |
| 1083 | E = DebugNode->use_end(); UI != E; ++UI) |
| 1084 | if (DbgDeclareInst *DDI = dyn_cast<DbgDeclareInst>(*UI)) |
| 1085 | return DDI; |
| 1086 | |
| 1087 | return 0; |
| 1088 | } |
Alexey Samsonov | 1afbb51 | 2012-12-12 14:31:53 +0000 | [diff] [blame] | 1089 | |
| 1090 | bool llvm::replaceDbgDeclareForAlloca(AllocaInst *AI, Value *NewAllocaAddress, |
| 1091 | DIBuilder &Builder) { |
| 1092 | DbgDeclareInst *DDI = FindAllocaDbgDeclare(AI); |
| 1093 | if (!DDI) |
| 1094 | return false; |
| 1095 | DIVariable DIVar(DDI->getVariable()); |
Jakub Staszak | a18c574 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 1096 | assert((!DIVar || DIVar.isVariable()) && |
Manman Ren | cbafae6 | 2013-06-28 05:43:10 +0000 | [diff] [blame] | 1097 | "Variable in DbgDeclareInst should be either null or a DIVariable."); |
| 1098 | if (!DIVar) |
Alexey Samsonov | 1afbb51 | 2012-12-12 14:31:53 +0000 | [diff] [blame] | 1099 | return false; |
| 1100 | |
| 1101 | // Create a copy of the original DIDescriptor for user variable, appending |
| 1102 | // "deref" operation to a list of address elements, as new llvm.dbg.declare |
| 1103 | // will take a value storing address of the memory for variable, not |
| 1104 | // alloca itself. |
| 1105 | Type *Int64Ty = Type::getInt64Ty(AI->getContext()); |
| 1106 | SmallVector<Value*, 4> NewDIVarAddress; |
| 1107 | if (DIVar.hasComplexAddress()) { |
| 1108 | for (unsigned i = 0, n = DIVar.getNumAddrElements(); i < n; ++i) { |
| 1109 | NewDIVarAddress.push_back( |
| 1110 | ConstantInt::get(Int64Ty, DIVar.getAddrElement(i))); |
| 1111 | } |
| 1112 | } |
| 1113 | NewDIVarAddress.push_back(ConstantInt::get(Int64Ty, DIBuilder::OpDeref)); |
| 1114 | DIVariable NewDIVar = Builder.createComplexVariable( |
| 1115 | DIVar.getTag(), DIVar.getContext(), DIVar.getName(), |
| 1116 | DIVar.getFile(), DIVar.getLineNumber(), DIVar.getType(), |
| 1117 | NewDIVarAddress, DIVar.getArgNumber()); |
| 1118 | |
| 1119 | // Insert llvm.dbg.declare in the same basic block as the original alloca, |
| 1120 | // and remove old llvm.dbg.declare. |
| 1121 | BasicBlock *BB = AI->getParent(); |
| 1122 | Builder.insertDeclare(NewAllocaAddress, NewDIVar, BB); |
| 1123 | DDI->eraseFromParent(); |
| 1124 | return true; |
| 1125 | } |
Evgeniy Stepanov | 3333e66 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 1126 | |
Peter Collingbourne | 4f96b7e | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1127 | /// changeToUnreachable - Insert an unreachable instruction before the specified |
| 1128 | /// instruction, making it and the rest of the code in the block dead. |
| 1129 | static void changeToUnreachable(Instruction *I, bool UseLLVMTrap) { |
| 1130 | BasicBlock *BB = I->getParent(); |
| 1131 | // Loop over all of the successors, removing BB's entry from any PHI |
| 1132 | // nodes. |
| 1133 | for (succ_iterator SI = succ_begin(BB), SE = succ_end(BB); SI != SE; ++SI) |
| 1134 | (*SI)->removePredecessor(BB); |
| 1135 | |
| 1136 | // Insert a call to llvm.trap right before this. This turns the undefined |
| 1137 | // behavior into a hard fail instead of falling through into random code. |
| 1138 | if (UseLLVMTrap) { |
| 1139 | Function *TrapFn = |
| 1140 | Intrinsic::getDeclaration(BB->getParent()->getParent(), Intrinsic::trap); |
| 1141 | CallInst *CallTrap = CallInst::Create(TrapFn, "", I); |
| 1142 | CallTrap->setDebugLoc(I->getDebugLoc()); |
| 1143 | } |
| 1144 | new UnreachableInst(I->getContext(), I); |
| 1145 | |
| 1146 | // All instructions after this are dead. |
| 1147 | BasicBlock::iterator BBI = I, BBE = BB->end(); |
| 1148 | while (BBI != BBE) { |
| 1149 | if (!BBI->use_empty()) |
| 1150 | BBI->replaceAllUsesWith(UndefValue::get(BBI->getType())); |
| 1151 | BB->getInstList().erase(BBI++); |
| 1152 | } |
| 1153 | } |
| 1154 | |
| 1155 | /// changeToCall - Convert the specified invoke into a normal call. |
| 1156 | static void changeToCall(InvokeInst *II) { |
| 1157 | SmallVector<Value*, 8> Args(II->op_begin(), II->op_end() - 3); |
| 1158 | CallInst *NewCall = CallInst::Create(II->getCalledValue(), Args, "", II); |
| 1159 | NewCall->takeName(II); |
| 1160 | NewCall->setCallingConv(II->getCallingConv()); |
| 1161 | NewCall->setAttributes(II->getAttributes()); |
| 1162 | NewCall->setDebugLoc(II->getDebugLoc()); |
| 1163 | II->replaceAllUsesWith(NewCall); |
| 1164 | |
| 1165 | // Follow the call by a branch to the normal destination. |
| 1166 | BranchInst::Create(II->getNormalDest(), II); |
| 1167 | |
| 1168 | // Update PHI nodes in the unwind destination |
| 1169 | II->getUnwindDest()->removePredecessor(II->getParent()); |
| 1170 | II->eraseFromParent(); |
| 1171 | } |
| 1172 | |
| 1173 | static bool markAliveBlocks(BasicBlock *BB, |
| 1174 | SmallPtrSet<BasicBlock*, 128> &Reachable) { |
| 1175 | |
Evgeniy Stepanov | 3333e66 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 1176 | SmallVector<BasicBlock*, 128> Worklist; |
Peter Collingbourne | 4f96b7e | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1177 | Worklist.push_back(BB); |
| 1178 | Reachable.insert(BB); |
| 1179 | bool Changed = false; |
Evgeniy Stepanov | 3333e66 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 1180 | do { |
Peter Collingbourne | 4f96b7e | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1181 | BB = Worklist.pop_back_val(); |
| 1182 | |
| 1183 | // Do a quick scan of the basic block, turning any obviously unreachable |
| 1184 | // instructions into LLVM unreachable insts. The instruction combining pass |
| 1185 | // canonicalizes unreachable insts into stores to null or undef. |
| 1186 | for (BasicBlock::iterator BBI = BB->begin(), E = BB->end(); BBI != E;++BBI){ |
| 1187 | if (CallInst *CI = dyn_cast<CallInst>(BBI)) { |
| 1188 | if (CI->doesNotReturn()) { |
| 1189 | // If we found a call to a no-return function, insert an unreachable |
| 1190 | // instruction after it. Make sure there isn't *already* one there |
| 1191 | // though. |
| 1192 | ++BBI; |
| 1193 | if (!isa<UnreachableInst>(BBI)) { |
| 1194 | // Don't insert a call to llvm.trap right before the unreachable. |
| 1195 | changeToUnreachable(BBI, false); |
| 1196 | Changed = true; |
| 1197 | } |
| 1198 | break; |
| 1199 | } |
| 1200 | } |
| 1201 | |
| 1202 | // Store to undef and store to null are undefined and used to signal that |
| 1203 | // they should be changed to unreachable by passes that can't modify the |
| 1204 | // CFG. |
| 1205 | if (StoreInst *SI = dyn_cast<StoreInst>(BBI)) { |
| 1206 | // Don't touch volatile stores. |
| 1207 | if (SI->isVolatile()) continue; |
| 1208 | |
| 1209 | Value *Ptr = SI->getOperand(1); |
| 1210 | |
| 1211 | if (isa<UndefValue>(Ptr) || |
| 1212 | (isa<ConstantPointerNull>(Ptr) && |
| 1213 | SI->getPointerAddressSpace() == 0)) { |
| 1214 | changeToUnreachable(SI, true); |
| 1215 | Changed = true; |
| 1216 | break; |
| 1217 | } |
| 1218 | } |
| 1219 | } |
| 1220 | |
| 1221 | // Turn invokes that call 'nounwind' functions into ordinary calls. |
| 1222 | if (InvokeInst *II = dyn_cast<InvokeInst>(BB->getTerminator())) { |
| 1223 | Value *Callee = II->getCalledValue(); |
| 1224 | if (isa<ConstantPointerNull>(Callee) || isa<UndefValue>(Callee)) { |
| 1225 | changeToUnreachable(II, true); |
| 1226 | Changed = true; |
| 1227 | } else if (II->doesNotThrow()) { |
| 1228 | if (II->use_empty() && II->onlyReadsMemory()) { |
| 1229 | // jump to the normal destination branch. |
| 1230 | BranchInst::Create(II->getNormalDest(), II); |
| 1231 | II->getUnwindDest()->removePredecessor(II->getParent()); |
| 1232 | II->eraseFromParent(); |
| 1233 | } else |
| 1234 | changeToCall(II); |
| 1235 | Changed = true; |
| 1236 | } |
| 1237 | } |
| 1238 | |
| 1239 | Changed |= ConstantFoldTerminator(BB, true); |
Evgeniy Stepanov | 3333e66 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 1240 | for (succ_iterator SI = succ_begin(BB), SE = succ_end(BB); SI != SE; ++SI) |
| 1241 | if (Reachable.insert(*SI)) |
| 1242 | Worklist.push_back(*SI); |
| 1243 | } while (!Worklist.empty()); |
Peter Collingbourne | 4f96b7e | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1244 | return Changed; |
| 1245 | } |
Evgeniy Stepanov | 3333e66 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 1246 | |
Peter Collingbourne | 4f96b7e | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1247 | /// removeUnreachableBlocksFromFn - Remove blocks that are not reachable, even |
| 1248 | /// if they are in a dead cycle. Return true if a change was made, false |
| 1249 | /// otherwise. |
| 1250 | bool llvm::removeUnreachableBlocks(Function &F) { |
| 1251 | SmallPtrSet<BasicBlock*, 128> Reachable; |
| 1252 | bool Changed = markAliveBlocks(F.begin(), Reachable); |
| 1253 | |
| 1254 | // If there are unreachable blocks in the CFG... |
Evgeniy Stepanov | 3333e66 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 1255 | if (Reachable.size() == F.size()) |
Peter Collingbourne | 4f96b7e | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1256 | return Changed; |
Evgeniy Stepanov | 3333e66 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 1257 | |
| 1258 | assert(Reachable.size() < F.size()); |
Peter Collingbourne | 4f96b7e | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1259 | NumRemoved += F.size()-Reachable.size(); |
| 1260 | |
| 1261 | // Loop over all of the basic blocks that are not reachable, dropping all of |
| 1262 | // their internal references... |
| 1263 | for (Function::iterator BB = ++F.begin(), E = F.end(); BB != E; ++BB) { |
| 1264 | if (Reachable.count(BB)) |
Evgeniy Stepanov | 3333e66 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 1265 | continue; |
| 1266 | |
Peter Collingbourne | 4f96b7e | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1267 | for (succ_iterator SI = succ_begin(BB), SE = succ_end(BB); SI != SE; ++SI) |
Evgeniy Stepanov | 3333e66 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 1268 | if (Reachable.count(*SI)) |
Peter Collingbourne | 4f96b7e | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1269 | (*SI)->removePredecessor(BB); |
| 1270 | BB->dropAllReferences(); |
Evgeniy Stepanov | 3333e66 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 1271 | } |
Evgeniy Stepanov | 7541cd3 | 2013-03-22 08:43:04 +0000 | [diff] [blame] | 1272 | |
Peter Collingbourne | 4f96b7e | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1273 | for (Function::iterator I = ++F.begin(); I != F.end();) |
Evgeniy Stepanov | 7541cd3 | 2013-03-22 08:43:04 +0000 | [diff] [blame] | 1274 | if (!Reachable.count(I)) |
| 1275 | I = F.getBasicBlockList().erase(I); |
| 1276 | else |
| 1277 | ++I; |
| 1278 | |
Evgeniy Stepanov | 3333e66 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 1279 | return true; |
| 1280 | } |