Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 1 | //===- Local.cpp - Functions to perform local transformations -------------===// |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 2 | // |
John Criswell | 482202a | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
Chris Lattner | f3ebc3f | 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 | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 7 | // |
John Criswell | 482202a | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 8 | //===----------------------------------------------------------------------===// |
Chris Lattner | 28537df | 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 | |
David Blaikie | 31b98d2 | 2018-06-04 21:23:21 +0000 | [diff] [blame] | 15 | #include "llvm/Transforms/Utils/Local.h" |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 16 | #include "llvm/ADT/APInt.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 17 | #include "llvm/ADT/DenseMap.h" |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 18 | #include "llvm/ADT/DenseMapInfo.h" |
Benjamin Kramer | 2b2cdd7 | 2015-06-18 16:01:00 +0000 | [diff] [blame] | 19 | #include "llvm/ADT/DenseSet.h" |
| 20 | #include "llvm/ADT/Hashing.h" |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 21 | #include "llvm/ADT/None.h" |
| 22 | #include "llvm/ADT/Optional.h" |
Evgeniy Stepanov | 4fbc0d08 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 23 | #include "llvm/ADT/STLExtras.h" |
Fiona Glaser | f74cc40 | 2015-09-28 18:56:07 +0000 | [diff] [blame] | 24 | #include "llvm/ADT/SetVector.h" |
Chandler Carruth | be81023 | 2013-01-02 10:22:59 +0000 | [diff] [blame] | 25 | #include "llvm/ADT/SmallPtrSet.h" |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 26 | #include "llvm/ADT/SmallVector.h" |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 27 | #include "llvm/ADT/Statistic.h" |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 28 | #include "llvm/ADT/TinyPtrVector.h" |
| 29 | #include "llvm/Analysis/ConstantFolding.h" |
David Majnemer | 70497c6 | 2015-12-02 23:06:39 +0000 | [diff] [blame] | 30 | #include "llvm/Analysis/EHPersonalities.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 31 | #include "llvm/Analysis/InstructionSimplify.h" |
David Majnemer | d9833ea | 2016-01-10 07:13:04 +0000 | [diff] [blame] | 32 | #include "llvm/Analysis/LazyValueInfo.h" |
Chandler Carruth | 6bda14b | 2017-06-06 11:49:48 +0000 | [diff] [blame] | 33 | #include "llvm/Analysis/MemoryBuiltins.h" |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 34 | #include "llvm/Analysis/TargetLibraryInfo.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 35 | #include "llvm/Analysis/ValueTracking.h" |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 36 | #include "llvm/BinaryFormat/Dwarf.h" |
| 37 | #include "llvm/IR/Argument.h" |
| 38 | #include "llvm/IR/Attributes.h" |
| 39 | #include "llvm/IR/BasicBlock.h" |
Chandler Carruth | 1305dc3 | 2014-03-04 11:45:46 +0000 | [diff] [blame] | 40 | #include "llvm/IR/CFG.h" |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 41 | #include "llvm/IR/CallSite.h" |
| 42 | #include "llvm/IR/Constant.h" |
Chandler Carruth | 2abb65a | 2017-06-26 03:31:31 +0000 | [diff] [blame] | 43 | #include "llvm/IR/ConstantRange.h" |
Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 44 | #include "llvm/IR/Constants.h" |
Chandler Carruth | 12664a0 | 2014-03-06 00:22:06 +0000 | [diff] [blame] | 45 | #include "llvm/IR/DIBuilder.h" |
Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 46 | #include "llvm/IR/DataLayout.h" |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 47 | #include "llvm/IR/DebugInfoMetadata.h" |
| 48 | #include "llvm/IR/DebugLoc.h" |
Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 49 | #include "llvm/IR/DerivedTypes.h" |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 50 | #include "llvm/IR/DomTreeUpdater.h" |
Chandler Carruth | 5ad5f15 | 2014-01-13 09:26:24 +0000 | [diff] [blame] | 51 | #include "llvm/IR/Dominators.h" |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 52 | #include "llvm/IR/Function.h" |
Chandler Carruth | 03eb0de | 2014-03-04 10:40:04 +0000 | [diff] [blame] | 53 | #include "llvm/IR/GetElementPtrTypeIterator.h" |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 54 | #include "llvm/IR/GlobalObject.h" |
Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 55 | #include "llvm/IR/IRBuilder.h" |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 56 | #include "llvm/IR/InstrTypes.h" |
| 57 | #include "llvm/IR/Instruction.h" |
Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 58 | #include "llvm/IR/Instructions.h" |
| 59 | #include "llvm/IR/IntrinsicInst.h" |
| 60 | #include "llvm/IR/Intrinsics.h" |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 61 | #include "llvm/IR/LLVMContext.h" |
Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 62 | #include "llvm/IR/MDBuilder.h" |
| 63 | #include "llvm/IR/Metadata.h" |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 64 | #include "llvm/IR/Module.h" |
Chandler Carruth | 9fb823b | 2013-01-02 11:36:10 +0000 | [diff] [blame] | 65 | #include "llvm/IR/Operator.h" |
David Majnemer | 9f50625 | 2016-06-25 08:34:38 +0000 | [diff] [blame] | 66 | #include "llvm/IR/PatternMatch.h" |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 67 | #include "llvm/IR/Type.h" |
| 68 | #include "llvm/IR/Use.h" |
| 69 | #include "llvm/IR/User.h" |
| 70 | #include "llvm/IR/Value.h" |
Chandler Carruth | 4220e9c | 2014-03-04 11:17:44 +0000 | [diff] [blame] | 71 | #include "llvm/IR/ValueHandle.h" |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 72 | #include "llvm/Support/Casting.h" |
Chris Lattner | cbd18fc | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 73 | #include "llvm/Support/Debug.h" |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 74 | #include "llvm/Support/ErrorHandling.h" |
Craig Topper | b45eabc | 2017-04-26 16:39:58 +0000 | [diff] [blame] | 75 | #include "llvm/Support/KnownBits.h" |
Chris Lattner | cbd18fc | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 76 | #include "llvm/Support/raw_ostream.h" |
Vedant Kumar | 6bfc869 | 2018-01-25 21:37:05 +0000 | [diff] [blame] | 77 | #include "llvm/Transforms/Utils/ValueMapper.h" |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 78 | #include <algorithm> |
| 79 | #include <cassert> |
| 80 | #include <climits> |
| 81 | #include <cstdint> |
| 82 | #include <iterator> |
| 83 | #include <map> |
| 84 | #include <utility> |
| 85 | |
Chris Lattner | 04efa4b | 2003-12-19 05:56:28 +0000 | [diff] [blame] | 86 | using namespace llvm; |
David Majnemer | 9f50625 | 2016-06-25 08:34:38 +0000 | [diff] [blame] | 87 | using namespace llvm::PatternMatch; |
Brian Gaeke | 960707c | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 88 | |
Chandler Carruth | e96dd89 | 2014-04-21 22:55:11 +0000 | [diff] [blame] | 89 | #define DEBUG_TYPE "local" |
| 90 | |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 91 | STATISTIC(NumRemoved, "Number of unreachable basic blocks removed"); |
| 92 | |
Chris Lattner | 28537df | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 93 | //===----------------------------------------------------------------------===// |
Chris Lattner | c6c481c | 2008-11-27 22:57:53 +0000 | [diff] [blame] | 94 | // Local constant propagation. |
Chris Lattner | 28537df | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 95 | // |
| 96 | |
Frits van Bommel | ad96455 | 2011-05-22 16:24:18 +0000 | [diff] [blame] | 97 | /// ConstantFoldTerminator - If a terminator instruction is predicated on a |
| 98 | /// constant value, convert it into an unconditional branch to the constant |
| 99 | /// destination. This is a nontrivial operation because the successors of this |
| 100 | /// basic block must have their PHI nodes updated. |
| 101 | /// Also calls RecursivelyDeleteTriviallyDeadInstructions() on any branch/switch |
| 102 | /// conditions and indirectbr addresses this might make dead if |
| 103 | /// DeleteDeadConditions is true. |
Benjamin Kramer | 8bcc971 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 104 | bool llvm::ConstantFoldTerminator(BasicBlock *BB, bool DeleteDeadConditions, |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 105 | const TargetLibraryInfo *TLI, |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 106 | DomTreeUpdater *DTU) { |
Chris Lattner | 4b009ad | 2002-05-21 20:04:50 +0000 | [diff] [blame] | 107 | TerminatorInst *T = BB->getTerminator(); |
Devang Patel | 1fabbe9 | 2011-05-18 17:26:46 +0000 | [diff] [blame] | 108 | IRBuilder<> Builder(T); |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 109 | |
Chris Lattner | 28537df | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 110 | // Branch - See if we are conditional jumping on constant |
Davide Italiano | 0512bf5 | 2017-12-31 16:51:50 +0000 | [diff] [blame] | 111 | if (auto *BI = dyn_cast<BranchInst>(T)) { |
Chris Lattner | 28537df | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 112 | if (BI->isUnconditional()) return false; // Can't optimize uncond branch |
Gabor Greif | 97f1720 | 2009-01-30 18:21:13 +0000 | [diff] [blame] | 113 | BasicBlock *Dest1 = BI->getSuccessor(0); |
| 114 | BasicBlock *Dest2 = BI->getSuccessor(1); |
Chris Lattner | 28537df | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 115 | |
Davide Italiano | 0512bf5 | 2017-12-31 16:51:50 +0000 | [diff] [blame] | 116 | if (auto *Cond = dyn_cast<ConstantInt>(BI->getCondition())) { |
Chris Lattner | 28537df | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 117 | // Are we branching on constant? |
| 118 | // YES. Change to unconditional branch... |
Reid Spencer | cddc9df | 2007-01-12 04:24:46 +0000 | [diff] [blame] | 119 | BasicBlock *Destination = Cond->getZExtValue() ? Dest1 : Dest2; |
| 120 | BasicBlock *OldDest = Cond->getZExtValue() ? Dest2 : Dest1; |
Chris Lattner | 28537df | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 121 | |
Chris Lattner | 28537df | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 122 | // Let the basic block know that we are letting go of it. Based on this, |
| 123 | // it will adjust it's PHI nodes. |
Jay Foad | 6a85be2 | 2011-04-19 15:23:29 +0000 | [diff] [blame] | 124 | OldDest->removePredecessor(BB); |
Chris Lattner | 28537df | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 125 | |
Jay Foad | 89afb43 | 2011-01-07 20:25:56 +0000 | [diff] [blame] | 126 | // Replace the conditional branch with an unconditional one. |
Devang Patel | 1fabbe9 | 2011-05-18 17:26:46 +0000 | [diff] [blame] | 127 | Builder.CreateBr(Destination); |
Jay Foad | 89afb43 | 2011-01-07 20:25:56 +0000 | [diff] [blame] | 128 | BI->eraseFromParent(); |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 129 | if (DTU) |
| 130 | DTU->deleteEdgeRelaxed(BB, OldDest); |
Chris Lattner | 28537df | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 131 | return true; |
Chris Lattner | 54a4b84 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 132 | } |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 133 | |
Chris Lattner | 54a4b84 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 134 | if (Dest2 == Dest1) { // Conditional branch to same location? |
Misha Brukman | b1c9317 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 135 | // This branch matches something like this: |
Chris Lattner | 28537df | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 136 | // br bool %cond, label %Dest, label %Dest |
| 137 | // and changes it into: br label %Dest |
| 138 | |
| 139 | // Let the basic block know that we are letting go of one copy of it. |
| 140 | assert(BI->getParent() && "Terminator not inserted in block!"); |
| 141 | Dest1->removePredecessor(BI->getParent()); |
| 142 | |
Jay Foad | 89afb43 | 2011-01-07 20:25:56 +0000 | [diff] [blame] | 143 | // Replace the conditional branch with an unconditional one. |
Devang Patel | 1fabbe9 | 2011-05-18 17:26:46 +0000 | [diff] [blame] | 144 | Builder.CreateBr(Dest1); |
Frits van Bommel | ad96455 | 2011-05-22 16:24:18 +0000 | [diff] [blame] | 145 | Value *Cond = BI->getCondition(); |
Jay Foad | 89afb43 | 2011-01-07 20:25:56 +0000 | [diff] [blame] | 146 | BI->eraseFromParent(); |
Frits van Bommel | ad96455 | 2011-05-22 16:24:18 +0000 | [diff] [blame] | 147 | if (DeleteDeadConditions) |
Benjamin Kramer | 8bcc971 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 148 | RecursivelyDeleteTriviallyDeadInstructions(Cond, TLI); |
Chris Lattner | 28537df | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 149 | return true; |
| 150 | } |
Chris Lattner | 54a4b84 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 151 | return false; |
| 152 | } |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 153 | |
Davide Italiano | 0512bf5 | 2017-12-31 16:51:50 +0000 | [diff] [blame] | 154 | if (auto *SI = dyn_cast<SwitchInst>(T)) { |
Hans Wennborg | 90b827c | 2015-01-26 19:52:24 +0000 | [diff] [blame] | 155 | // If we are switching on a constant, we can convert the switch to an |
| 156 | // unconditional branch. |
Davide Italiano | 0512bf5 | 2017-12-31 16:51:50 +0000 | [diff] [blame] | 157 | auto *CI = dyn_cast<ConstantInt>(SI->getCondition()); |
Hans Wennborg | 90b827c | 2015-01-26 19:52:24 +0000 | [diff] [blame] | 158 | BasicBlock *DefaultDest = SI->getDefaultDest(); |
| 159 | BasicBlock *TheOnlyDest = DefaultDest; |
| 160 | |
| 161 | // If the default is unreachable, ignore it when searching for TheOnlyDest. |
| 162 | if (isa<UnreachableInst>(DefaultDest->getFirstNonPHIOrDbg()) && |
| 163 | SI->getNumCases() > 0) { |
Chandler Carruth | 927d8e6 | 2017-04-12 07:27:28 +0000 | [diff] [blame] | 164 | TheOnlyDest = SI->case_begin()->getCaseSuccessor(); |
Hans Wennborg | 90b827c | 2015-01-26 19:52:24 +0000 | [diff] [blame] | 165 | } |
Chris Lattner | 031340a | 2003-08-17 19:41:53 +0000 | [diff] [blame] | 166 | |
Chris Lattner | 54a4b84 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 167 | // Figure out which case it goes to. |
Chandler Carruth | 0d256c0 | 2017-03-26 02:49:23 +0000 | [diff] [blame] | 168 | for (auto i = SI->case_begin(), e = SI->case_end(); i != e;) { |
Chris Lattner | 821deee | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 169 | // Found case matching a constant operand? |
Chandler Carruth | 927d8e6 | 2017-04-12 07:27:28 +0000 | [diff] [blame] | 170 | if (i->getCaseValue() == CI) { |
| 171 | TheOnlyDest = i->getCaseSuccessor(); |
Chris Lattner | 821deee | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 172 | break; |
| 173 | } |
Chris Lattner | 031340a | 2003-08-17 19:41:53 +0000 | [diff] [blame] | 174 | |
Chris Lattner | c54d608 | 2003-08-23 23:18:19 +0000 | [diff] [blame] | 175 | // Check to see if this branch is going to the same place as the default |
| 176 | // dest. If so, eliminate it as an explicit compare. |
Chandler Carruth | 927d8e6 | 2017-04-12 07:27:28 +0000 | [diff] [blame] | 177 | if (i->getCaseSuccessor() == DefaultDest) { |
Duncan P. N. Exon Smith | de36e80 | 2014-11-11 21:30:22 +0000 | [diff] [blame] | 178 | MDNode *MD = SI->getMetadata(LLVMContext::MD_prof); |
Justin Bogner | a41a7b3 | 2013-12-10 00:13:41 +0000 | [diff] [blame] | 179 | unsigned NCases = SI->getNumCases(); |
| 180 | // Fold the case metadata into the default if there will be any branches |
| 181 | // left, unless the metadata doesn't match the switch. |
| 182 | if (NCases > 1 && MD && MD->getNumOperands() == 2 + NCases) { |
Manman Ren | 49dbe25 | 2012-09-12 17:04:11 +0000 | [diff] [blame] | 183 | // Collect branch weights into a vector. |
| 184 | SmallVector<uint32_t, 8> Weights; |
| 185 | for (unsigned MD_i = 1, MD_e = MD->getNumOperands(); MD_i < MD_e; |
| 186 | ++MD_i) { |
David Majnemer | 9f50625 | 2016-06-25 08:34:38 +0000 | [diff] [blame] | 187 | auto *CI = mdconst::extract<ConstantInt>(MD->getOperand(MD_i)); |
Manman Ren | 49dbe25 | 2012-09-12 17:04:11 +0000 | [diff] [blame] | 188 | Weights.push_back(CI->getValue().getZExtValue()); |
| 189 | } |
| 190 | // Merge weight of this case to the default weight. |
Chandler Carruth | 927d8e6 | 2017-04-12 07:27:28 +0000 | [diff] [blame] | 191 | unsigned idx = i->getCaseIndex(); |
Manman Ren | 49dbe25 | 2012-09-12 17:04:11 +0000 | [diff] [blame] | 192 | Weights[0] += Weights[idx+1]; |
| 193 | // Remove weight for this case. |
| 194 | std::swap(Weights[idx+1], Weights.back()); |
| 195 | Weights.pop_back(); |
| 196 | SI->setMetadata(LLVMContext::MD_prof, |
| 197 | MDBuilder(BB->getContext()). |
| 198 | createBranchWeights(Weights)); |
| 199 | } |
Chris Lattner | 54a4b84 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 200 | // Remove this entry. |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 201 | BasicBlock *ParentBB = SI->getParent(); |
| 202 | DefaultDest->removePredecessor(ParentBB); |
Chandler Carruth | 0d256c0 | 2017-03-26 02:49:23 +0000 | [diff] [blame] | 203 | i = SI->removeCase(i); |
| 204 | e = SI->case_end(); |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 205 | if (DTU) |
| 206 | DTU->deleteEdgeRelaxed(ParentBB, DefaultDest); |
Chris Lattner | c54d608 | 2003-08-23 23:18:19 +0000 | [diff] [blame] | 207 | continue; |
| 208 | } |
| 209 | |
Chris Lattner | 821deee | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 210 | // Otherwise, check to see if the switch only branches to one destination. |
| 211 | // We do this by reseting "TheOnlyDest" to null when we find two non-equal |
| 212 | // destinations. |
Chandler Carruth | 927d8e6 | 2017-04-12 07:27:28 +0000 | [diff] [blame] | 213 | if (i->getCaseSuccessor() != TheOnlyDest) |
| 214 | TheOnlyDest = nullptr; |
Chandler Carruth | 0d256c0 | 2017-03-26 02:49:23 +0000 | [diff] [blame] | 215 | |
| 216 | // Increment this iterator as we haven't removed the case. |
| 217 | ++i; |
Chris Lattner | 031340a | 2003-08-17 19:41:53 +0000 | [diff] [blame] | 218 | } |
| 219 | |
Chris Lattner | 821deee | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 220 | if (CI && !TheOnlyDest) { |
| 221 | // Branching on a constant, but not any of the cases, go to the default |
| 222 | // successor. |
| 223 | TheOnlyDest = SI->getDefaultDest(); |
| 224 | } |
| 225 | |
| 226 | // If we found a single destination that we can fold the switch into, do so |
| 227 | // now. |
| 228 | if (TheOnlyDest) { |
Chris Lattner | 54a4b84 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 229 | // Insert the new branch. |
Devang Patel | 1fabbe9 | 2011-05-18 17:26:46 +0000 | [diff] [blame] | 230 | Builder.CreateBr(TheOnlyDest); |
Chris Lattner | 821deee | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 231 | BasicBlock *BB = SI->getParent(); |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 232 | std::vector <DominatorTree::UpdateType> Updates; |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 233 | if (DTU) |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 234 | Updates.reserve(SI->getNumSuccessors() - 1); |
Chris Lattner | 821deee | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 235 | |
| 236 | // Remove entries from PHI nodes which we no longer branch to... |
Pete Cooper | ebcd748 | 2015-08-06 20:22:46 +0000 | [diff] [blame] | 237 | for (BasicBlock *Succ : SI->successors()) { |
Chris Lattner | 821deee | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 238 | // Found case matching a constant operand? |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 239 | if (Succ == TheOnlyDest) { |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 240 | TheOnlyDest = nullptr; // Don't modify the first branch to TheOnlyDest |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 241 | } else { |
Chris Lattner | 821deee | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 242 | Succ->removePredecessor(BB); |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 243 | if (DTU) |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 244 | Updates.push_back({DominatorTree::Delete, BB, Succ}); |
| 245 | } |
Chris Lattner | 821deee | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 246 | } |
| 247 | |
Chris Lattner | 54a4b84 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 248 | // Delete the old switch. |
Frits van Bommel | ad96455 | 2011-05-22 16:24:18 +0000 | [diff] [blame] | 249 | Value *Cond = SI->getCondition(); |
| 250 | SI->eraseFromParent(); |
| 251 | if (DeleteDeadConditions) |
Benjamin Kramer | 8bcc971 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 252 | RecursivelyDeleteTriviallyDeadInstructions(Cond, TLI); |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 253 | if (DTU) |
| 254 | DTU->applyUpdates(Updates, /*ForceRemoveDuplicates*/ true); |
Chris Lattner | 821deee | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 255 | return true; |
Chris Lattner | 54a4b84 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 256 | } |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 257 | |
Stepan Dyatkovskiy | 513aaa5 | 2012-02-01 07:49:51 +0000 | [diff] [blame] | 258 | if (SI->getNumCases() == 1) { |
Chris Lattner | 821deee | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 259 | // Otherwise, we can fold this switch into a conditional branch |
| 260 | // instruction if it has only one non-default destination. |
Chandler Carruth | 927d8e6 | 2017-04-12 07:27:28 +0000 | [diff] [blame] | 261 | auto FirstCase = *SI->case_begin(); |
Bob Wilson | e407736 | 2013-09-09 19:14:35 +0000 | [diff] [blame] | 262 | Value *Cond = Builder.CreateICmpEQ(SI->getCondition(), |
| 263 | FirstCase.getCaseValue(), "cond"); |
Devang Patel | 1fabbe9 | 2011-05-18 17:26:46 +0000 | [diff] [blame] | 264 | |
Bob Wilson | e407736 | 2013-09-09 19:14:35 +0000 | [diff] [blame] | 265 | // Insert the new branch. |
| 266 | BranchInst *NewBr = Builder.CreateCondBr(Cond, |
| 267 | FirstCase.getCaseSuccessor(), |
| 268 | SI->getDefaultDest()); |
Duncan P. N. Exon Smith | de36e80 | 2014-11-11 21:30:22 +0000 | [diff] [blame] | 269 | MDNode *MD = SI->getMetadata(LLVMContext::MD_prof); |
Bob Wilson | e407736 | 2013-09-09 19:14:35 +0000 | [diff] [blame] | 270 | if (MD && MD->getNumOperands() == 3) { |
Duncan P. N. Exon Smith | 5bf8fef | 2014-12-09 18:38:53 +0000 | [diff] [blame] | 271 | ConstantInt *SICase = |
| 272 | mdconst::dyn_extract<ConstantInt>(MD->getOperand(2)); |
| 273 | ConstantInt *SIDef = |
| 274 | mdconst::dyn_extract<ConstantInt>(MD->getOperand(1)); |
Bob Wilson | e407736 | 2013-09-09 19:14:35 +0000 | [diff] [blame] | 275 | assert(SICase && SIDef); |
| 276 | // The TrueWeight should be the weight for the single case of SI. |
| 277 | NewBr->setMetadata(LLVMContext::MD_prof, |
| 278 | MDBuilder(BB->getContext()). |
| 279 | createBranchWeights(SICase->getValue().getZExtValue(), |
| 280 | SIDef->getValue().getZExtValue())); |
Stepan Dyatkovskiy | 7a50155 | 2012-05-23 08:18:26 +0000 | [diff] [blame] | 281 | } |
Bob Wilson | e407736 | 2013-09-09 19:14:35 +0000 | [diff] [blame] | 282 | |
Chen Li | eafbc9d | 2015-08-07 19:30:12 +0000 | [diff] [blame] | 283 | // Update make.implicit metadata to the newly-created conditional branch. |
| 284 | MDNode *MakeImplicitMD = SI->getMetadata(LLVMContext::MD_make_implicit); |
| 285 | if (MakeImplicitMD) |
| 286 | NewBr->setMetadata(LLVMContext::MD_make_implicit, MakeImplicitMD); |
| 287 | |
Bob Wilson | e407736 | 2013-09-09 19:14:35 +0000 | [diff] [blame] | 288 | // Delete the old switch. |
| 289 | SI->eraseFromParent(); |
| 290 | return true; |
Chris Lattner | 821deee | 2003-08-17 20:21:14 +0000 | [diff] [blame] | 291 | } |
Chris Lattner | 54a4b84 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 292 | return false; |
Chris Lattner | 28537df | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 293 | } |
Chris Lattner | 54a4b84 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 294 | |
Davide Italiano | 0512bf5 | 2017-12-31 16:51:50 +0000 | [diff] [blame] | 295 | if (auto *IBI = dyn_cast<IndirectBrInst>(T)) { |
Chris Lattner | 54a4b84 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 296 | // indirectbr blockaddress(@F, @BB) -> br label @BB |
Davide Italiano | 0512bf5 | 2017-12-31 16:51:50 +0000 | [diff] [blame] | 297 | if (auto *BA = |
Chris Lattner | 54a4b84 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 298 | dyn_cast<BlockAddress>(IBI->getAddress()->stripPointerCasts())) { |
| 299 | BasicBlock *TheOnlyDest = BA->getBasicBlock(); |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 300 | std::vector <DominatorTree::UpdateType> Updates; |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 301 | if (DTU) |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 302 | Updates.reserve(IBI->getNumDestinations() - 1); |
| 303 | |
Chris Lattner | 54a4b84 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 304 | // Insert the new branch. |
Devang Patel | 1fabbe9 | 2011-05-18 17:26:46 +0000 | [diff] [blame] | 305 | Builder.CreateBr(TheOnlyDest); |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 306 | |
Chris Lattner | 54a4b84 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 307 | for (unsigned i = 0, e = IBI->getNumDestinations(); i != e; ++i) { |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 308 | if (IBI->getDestination(i) == TheOnlyDest) { |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 309 | TheOnlyDest = nullptr; |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 310 | } else { |
| 311 | BasicBlock *ParentBB = IBI->getParent(); |
| 312 | BasicBlock *DestBB = IBI->getDestination(i); |
| 313 | DestBB->removePredecessor(ParentBB); |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 314 | if (DTU) |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 315 | Updates.push_back({DominatorTree::Delete, ParentBB, DestBB}); |
| 316 | } |
Chris Lattner | 54a4b84 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 317 | } |
Frits van Bommel | ad96455 | 2011-05-22 16:24:18 +0000 | [diff] [blame] | 318 | Value *Address = IBI->getAddress(); |
Chris Lattner | 54a4b84 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 319 | IBI->eraseFromParent(); |
Frits van Bommel | ad96455 | 2011-05-22 16:24:18 +0000 | [diff] [blame] | 320 | if (DeleteDeadConditions) |
Benjamin Kramer | 8bcc971 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 321 | RecursivelyDeleteTriviallyDeadInstructions(Address, TLI); |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 322 | |
Chris Lattner | 54a4b84 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 323 | // If we didn't find our destination in the IBI successor list, then we |
| 324 | // have undefined behavior. Replace the unconditional branch with an |
| 325 | // 'unreachable' instruction. |
| 326 | if (TheOnlyDest) { |
| 327 | BB->getTerminator()->eraseFromParent(); |
| 328 | new UnreachableInst(BB->getContext(), BB); |
| 329 | } |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 330 | |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 331 | if (DTU) |
| 332 | DTU->applyUpdates(Updates, /*ForceRemoveDuplicates*/ true); |
Chris Lattner | 54a4b84 | 2009-11-01 03:40:38 +0000 | [diff] [blame] | 333 | return true; |
| 334 | } |
| 335 | } |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 336 | |
Chris Lattner | 28537df | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 337 | return false; |
| 338 | } |
| 339 | |
Chris Lattner | 28537df | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 340 | //===----------------------------------------------------------------------===// |
Chris Lattner | 852d6d6 | 2009-11-10 22:26:15 +0000 | [diff] [blame] | 341 | // Local dead code elimination. |
Chris Lattner | 28537df | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 342 | // |
| 343 | |
Chris Lattner | c6c481c | 2008-11-27 22:57:53 +0000 | [diff] [blame] | 344 | /// isInstructionTriviallyDead - Return true if the result produced by the |
| 345 | /// instruction is not used, and the instruction has no side effects. |
| 346 | /// |
Benjamin Kramer | 8bcc971 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 347 | bool llvm::isInstructionTriviallyDead(Instruction *I, |
| 348 | const TargetLibraryInfo *TLI) { |
Daniel Berlin | e3e69e1 | 2017-03-10 00:32:33 +0000 | [diff] [blame] | 349 | if (!I->use_empty()) |
| 350 | return false; |
| 351 | return wouldInstructionBeTriviallyDead(I, TLI); |
| 352 | } |
| 353 | |
| 354 | bool llvm::wouldInstructionBeTriviallyDead(Instruction *I, |
| 355 | const TargetLibraryInfo *TLI) { |
| 356 | if (isa<TerminatorInst>(I)) |
| 357 | return false; |
Jeff Cohen | 5f4ef3c | 2005-07-27 06:12:32 +0000 | [diff] [blame] | 358 | |
David Majnemer | 654e130 | 2015-07-31 17:58:14 +0000 | [diff] [blame] | 359 | // We don't want the landingpad-like instructions removed by anything this |
| 360 | // general. |
| 361 | if (I->isEHPad()) |
Bill Wendling | d9fb470 | 2011-08-15 20:10:51 +0000 | [diff] [blame] | 362 | return false; |
| 363 | |
Devang Patel | c1431e6 | 2011-03-18 23:28:02 +0000 | [diff] [blame] | 364 | // We don't want debug info removed by anything this general, unless |
| 365 | // debug info is empty. |
| 366 | if (DbgDeclareInst *DDI = dyn_cast<DbgDeclareInst>(I)) { |
Nick Lewycky | 99890a2 | 2011-08-02 21:19:27 +0000 | [diff] [blame] | 367 | if (DDI->getAddress()) |
Devang Patel | c1431e6 | 2011-03-18 23:28:02 +0000 | [diff] [blame] | 368 | return false; |
Devang Patel | 17bbd7f | 2011-03-21 22:04:45 +0000 | [diff] [blame] | 369 | return true; |
Nick Lewycky | 99890a2 | 2011-08-02 21:19:27 +0000 | [diff] [blame] | 370 | } |
Devang Patel | 17bbd7f | 2011-03-21 22:04:45 +0000 | [diff] [blame] | 371 | if (DbgValueInst *DVI = dyn_cast<DbgValueInst>(I)) { |
Devang Patel | c1431e6 | 2011-03-18 23:28:02 +0000 | [diff] [blame] | 372 | if (DVI->getValue()) |
| 373 | return false; |
Devang Patel | 17bbd7f | 2011-03-21 22:04:45 +0000 | [diff] [blame] | 374 | return true; |
Devang Patel | c1431e6 | 2011-03-18 23:28:02 +0000 | [diff] [blame] | 375 | } |
Shiva Chen | 2c86455 | 2018-05-09 02:40:45 +0000 | [diff] [blame] | 376 | if (DbgLabelInst *DLI = dyn_cast<DbgLabelInst>(I)) { |
| 377 | if (DLI->getLabel()) |
| 378 | return false; |
| 379 | return true; |
| 380 | } |
Devang Patel | c1431e6 | 2011-03-18 23:28:02 +0000 | [diff] [blame] | 381 | |
Daniel Berlin | e3e69e1 | 2017-03-10 00:32:33 +0000 | [diff] [blame] | 382 | if (!I->mayHaveSideEffects()) |
| 383 | return true; |
Duncan Sands | 1efabaa | 2009-05-06 06:49:50 +0000 | [diff] [blame] | 384 | |
| 385 | // Special case intrinsics that "may have side effects" but can be deleted |
| 386 | // when dead. |
Nick Lewycky | 99890a2 | 2011-08-02 21:19:27 +0000 | [diff] [blame] | 387 | if (IntrinsicInst *II = dyn_cast<IntrinsicInst>(I)) { |
Piotr Padlewski | a26a08c | 2018-05-18 23:52:57 +0000 | [diff] [blame] | 388 | // Safe to delete llvm.stacksave and launder.invariant.group if dead. |
| 389 | if (II->getIntrinsicID() == Intrinsic::stacksave || |
| 390 | II->getIntrinsicID() == Intrinsic::launder_invariant_group) |
Chris Lattner | e966583 | 2007-12-29 00:59:12 +0000 | [diff] [blame] | 391 | return true; |
Nick Lewycky | 99890a2 | 2011-08-02 21:19:27 +0000 | [diff] [blame] | 392 | |
| 393 | // Lifetime intrinsics are dead when their right-hand is undef. |
| 394 | if (II->getIntrinsicID() == Intrinsic::lifetime_start || |
| 395 | II->getIntrinsicID() == Intrinsic::lifetime_end) |
| 396 | return isa<UndefValue>(II->getArgOperand(1)); |
Hal Finkel | 9304691 | 2014-07-25 21:13:35 +0000 | [diff] [blame] | 397 | |
Sanjoy Das | 107aefc | 2016-04-29 22:23:16 +0000 | [diff] [blame] | 398 | // Assumptions are dead if their condition is trivially true. Guards on |
| 399 | // true are operationally no-ops. In the future we can consider more |
| 400 | // sophisticated tradeoffs for guards considering potential for check |
| 401 | // widening, but for now we keep things simple. |
| 402 | if (II->getIntrinsicID() == Intrinsic::assume || |
| 403 | II->getIntrinsicID() == Intrinsic::experimental_guard) { |
Hal Finkel | 9304691 | 2014-07-25 21:13:35 +0000 | [diff] [blame] | 404 | if (ConstantInt *Cond = dyn_cast<ConstantInt>(II->getArgOperand(0))) |
| 405 | return !Cond->isZero(); |
| 406 | |
| 407 | return false; |
| 408 | } |
Nick Lewycky | 99890a2 | 2011-08-02 21:19:27 +0000 | [diff] [blame] | 409 | } |
Nick Lewycky | dd1d3df | 2011-10-24 04:35:36 +0000 | [diff] [blame] | 410 | |
Daniel Berlin | e3e69e1 | 2017-03-10 00:32:33 +0000 | [diff] [blame] | 411 | if (isAllocLikeFn(I, TLI)) |
| 412 | return true; |
Nick Lewycky | dd1d3df | 2011-10-24 04:35:36 +0000 | [diff] [blame] | 413 | |
Benjamin Kramer | 8bcc971 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 414 | if (CallInst *CI = isFreeCall(I, TLI)) |
Nick Lewycky | dd1d3df | 2011-10-24 04:35:36 +0000 | [diff] [blame] | 415 | if (Constant *C = dyn_cast<Constant>(CI->getArgOperand(0))) |
| 416 | return C->isNullValue() || isa<UndefValue>(C); |
| 417 | |
Eli Friedman | b6befc3 | 2016-11-02 20:48:11 +0000 | [diff] [blame] | 418 | if (CallSite CS = CallSite(I)) |
| 419 | if (isMathLibCallNoop(CS, TLI)) |
| 420 | return true; |
| 421 | |
Chris Lattner | a36d525 | 2005-05-06 05:27:34 +0000 | [diff] [blame] | 422 | return false; |
Chris Lattner | 28537df | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 423 | } |
| 424 | |
Chris Lattner | c6c481c | 2008-11-27 22:57:53 +0000 | [diff] [blame] | 425 | /// RecursivelyDeleteTriviallyDeadInstructions - If the specified value is a |
| 426 | /// trivially dead instruction, delete it. If that makes any of its operands |
Dan Gohman | cb99fe9 | 2010-01-05 15:45:31 +0000 | [diff] [blame] | 427 | /// trivially dead, delete them too, recursively. Return true if any |
| 428 | /// instructions were deleted. |
Benjamin Kramer | 8bcc971 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 429 | bool |
| 430 | llvm::RecursivelyDeleteTriviallyDeadInstructions(Value *V, |
| 431 | const TargetLibraryInfo *TLI) { |
Chris Lattner | c6c481c | 2008-11-27 22:57:53 +0000 | [diff] [blame] | 432 | Instruction *I = dyn_cast<Instruction>(V); |
Benjamin Kramer | 8bcc971 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 433 | if (!I || !I->use_empty() || !isInstructionTriviallyDead(I, TLI)) |
Dan Gohman | cb99fe9 | 2010-01-05 15:45:31 +0000 | [diff] [blame] | 434 | return false; |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 435 | |
Chris Lattner | e9f6c35 | 2008-11-28 01:20:46 +0000 | [diff] [blame] | 436 | SmallVector<Instruction*, 16> DeadInsts; |
| 437 | DeadInsts.push_back(I); |
Chandler Carruth | 4cbcbb0 | 2018-05-29 20:15:38 +0000 | [diff] [blame] | 438 | RecursivelyDeleteTriviallyDeadInstructions(DeadInsts, TLI); |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 439 | |
Chandler Carruth | 4cbcbb0 | 2018-05-29 20:15:38 +0000 | [diff] [blame] | 440 | return true; |
| 441 | } |
| 442 | |
| 443 | void llvm::RecursivelyDeleteTriviallyDeadInstructions( |
| 444 | SmallVectorImpl<Instruction *> &DeadInsts, const TargetLibraryInfo *TLI) { |
| 445 | // Process the dead instruction list until empty. |
| 446 | while (!DeadInsts.empty()) { |
| 447 | Instruction &I = *DeadInsts.pop_back_val(); |
| 448 | assert(I.use_empty() && "Instructions with uses are not dead."); |
| 449 | assert(isInstructionTriviallyDead(&I, TLI) && |
| 450 | "Live instruction found in dead worklist!"); |
| 451 | |
| 452 | // Don't lose the debug info while deleting the instructions. |
| 453 | salvageDebugInfo(I); |
Chris Lattner | d4b5ba6 | 2008-11-28 00:58:15 +0000 | [diff] [blame] | 454 | |
Chris Lattner | e9f6c35 | 2008-11-28 01:20:46 +0000 | [diff] [blame] | 455 | // Null out all of the instruction's operands to see if any operand becomes |
| 456 | // dead as we go. |
Chandler Carruth | 4cbcbb0 | 2018-05-29 20:15:38 +0000 | [diff] [blame] | 457 | for (Use &OpU : I.operands()) { |
| 458 | Value *OpV = OpU.get(); |
| 459 | OpU.set(nullptr); |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 460 | |
Chandler Carruth | 4cbcbb0 | 2018-05-29 20:15:38 +0000 | [diff] [blame] | 461 | if (!OpV->use_empty()) |
| 462 | continue; |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 463 | |
Chris Lattner | e9f6c35 | 2008-11-28 01:20:46 +0000 | [diff] [blame] | 464 | // If the operand is an instruction that became dead as we nulled out the |
| 465 | // operand, and if it is 'trivially' dead, delete it in a future loop |
| 466 | // iteration. |
| 467 | if (Instruction *OpI = dyn_cast<Instruction>(OpV)) |
Benjamin Kramer | 8bcc971 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 468 | if (isInstructionTriviallyDead(OpI, TLI)) |
Chris Lattner | e9f6c35 | 2008-11-28 01:20:46 +0000 | [diff] [blame] | 469 | DeadInsts.push_back(OpI); |
| 470 | } |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 471 | |
Chandler Carruth | 4cbcbb0 | 2018-05-29 20:15:38 +0000 | [diff] [blame] | 472 | I.eraseFromParent(); |
| 473 | } |
Chris Lattner | 28537df | 2002-05-07 18:07:59 +0000 | [diff] [blame] | 474 | } |
Chris Lattner | 99d6809 | 2008-11-27 07:43:12 +0000 | [diff] [blame] | 475 | |
Nick Lewycky | c8a1569 | 2011-02-20 08:38:20 +0000 | [diff] [blame] | 476 | /// areAllUsesEqual - Check whether the uses of a value are all the same. |
| 477 | /// This is similar to Instruction::hasOneUse() except this will also return |
Duncan Sands | 6dcd49b | 2011-02-21 16:27:36 +0000 | [diff] [blame] | 478 | /// true when there are no uses or multiple uses that all refer to the same |
| 479 | /// value. |
Nick Lewycky | c8a1569 | 2011-02-20 08:38:20 +0000 | [diff] [blame] | 480 | static bool areAllUsesEqual(Instruction *I) { |
Chandler Carruth | cdf4788 | 2014-03-09 03:16:01 +0000 | [diff] [blame] | 481 | Value::user_iterator UI = I->user_begin(); |
| 482 | Value::user_iterator UE = I->user_end(); |
Nick Lewycky | c8a1569 | 2011-02-20 08:38:20 +0000 | [diff] [blame] | 483 | if (UI == UE) |
Duncan Sands | 6dcd49b | 2011-02-21 16:27:36 +0000 | [diff] [blame] | 484 | return true; |
Nick Lewycky | c8a1569 | 2011-02-20 08:38:20 +0000 | [diff] [blame] | 485 | |
| 486 | User *TheUse = *UI; |
| 487 | for (++UI; UI != UE; ++UI) { |
| 488 | if (*UI != TheUse) |
| 489 | return false; |
| 490 | } |
| 491 | return true; |
| 492 | } |
| 493 | |
Dan Gohman | ff08995 | 2009-05-02 18:29:22 +0000 | [diff] [blame] | 494 | /// RecursivelyDeleteDeadPHINode - If the specified value is an effectively |
| 495 | /// dead PHI node, due to being a def-use chain of single-use nodes that |
| 496 | /// either forms a cycle or is terminated by a trivially dead instruction, |
| 497 | /// delete it. If that makes any of its operands trivially dead, delete them |
Duncan Sands | ecbbf08 | 2011-02-21 17:32:05 +0000 | [diff] [blame] | 498 | /// too, recursively. Return true if a change was made. |
Benjamin Kramer | 8bcc971 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 499 | bool llvm::RecursivelyDeleteDeadPHINode(PHINode *PN, |
| 500 | const TargetLibraryInfo *TLI) { |
Duncan Sands | 6dcd49b | 2011-02-21 16:27:36 +0000 | [diff] [blame] | 501 | SmallPtrSet<Instruction*, 4> Visited; |
| 502 | for (Instruction *I = PN; areAllUsesEqual(I) && !I->mayHaveSideEffects(); |
Chandler Carruth | cdf4788 | 2014-03-09 03:16:01 +0000 | [diff] [blame] | 503 | I = cast<Instruction>(*I->user_begin())) { |
Duncan Sands | 6dcd49b | 2011-02-21 16:27:36 +0000 | [diff] [blame] | 504 | if (I->use_empty()) |
Benjamin Kramer | 8bcc971 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 505 | return RecursivelyDeleteTriviallyDeadInstructions(I, TLI); |
Nick Lewycky | 183c24c | 2011-02-20 18:05:56 +0000 | [diff] [blame] | 506 | |
Duncan Sands | 6dcd49b | 2011-02-21 16:27:36 +0000 | [diff] [blame] | 507 | // If we find an instruction more than once, we're on a cycle that |
Dan Gohman | ff08995 | 2009-05-02 18:29:22 +0000 | [diff] [blame] | 508 | // won't prove fruitful. |
David Blaikie | 70573dc | 2014-11-19 07:49:26 +0000 | [diff] [blame] | 509 | if (!Visited.insert(I).second) { |
Duncan Sands | 6dcd49b | 2011-02-21 16:27:36 +0000 | [diff] [blame] | 510 | // Break the cycle and delete the instruction and its operands. |
| 511 | I->replaceAllUsesWith(UndefValue::get(I->getType())); |
Benjamin Kramer | 8bcc971 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 512 | (void)RecursivelyDeleteTriviallyDeadInstructions(I, TLI); |
Duncan Sands | ecbbf08 | 2011-02-21 17:32:05 +0000 | [diff] [blame] | 513 | return true; |
Duncan Sands | 6dcd49b | 2011-02-21 16:27:36 +0000 | [diff] [blame] | 514 | } |
| 515 | } |
| 516 | return false; |
Dan Gohman | ff08995 | 2009-05-02 18:29:22 +0000 | [diff] [blame] | 517 | } |
Chris Lattner | c6c481c | 2008-11-27 22:57:53 +0000 | [diff] [blame] | 518 | |
Fiona Glaser | f74cc40 | 2015-09-28 18:56:07 +0000 | [diff] [blame] | 519 | static bool |
| 520 | simplifyAndDCEInstruction(Instruction *I, |
| 521 | SmallSetVector<Instruction *, 16> &WorkList, |
| 522 | const DataLayout &DL, |
| 523 | const TargetLibraryInfo *TLI) { |
| 524 | if (isInstructionTriviallyDead(I, TLI)) { |
Vedant Kumar | f69baf6 | 2018-03-02 22:46:48 +0000 | [diff] [blame] | 525 | salvageDebugInfo(*I); |
| 526 | |
Fiona Glaser | f74cc40 | 2015-09-28 18:56:07 +0000 | [diff] [blame] | 527 | // Null out all of the instruction's operands to see if any operand becomes |
| 528 | // dead as we go. |
| 529 | for (unsigned i = 0, e = I->getNumOperands(); i != e; ++i) { |
| 530 | Value *OpV = I->getOperand(i); |
| 531 | I->setOperand(i, nullptr); |
| 532 | |
| 533 | if (!OpV->use_empty() || I == OpV) |
| 534 | continue; |
| 535 | |
| 536 | // If the operand is an instruction that became dead as we nulled out the |
| 537 | // operand, and if it is 'trivially' dead, delete it in a future loop |
| 538 | // iteration. |
| 539 | if (Instruction *OpI = dyn_cast<Instruction>(OpV)) |
| 540 | if (isInstructionTriviallyDead(OpI, TLI)) |
| 541 | WorkList.insert(OpI); |
| 542 | } |
| 543 | |
| 544 | I->eraseFromParent(); |
| 545 | |
| 546 | return true; |
| 547 | } |
| 548 | |
| 549 | if (Value *SimpleV = SimplifyInstruction(I, DL)) { |
| 550 | // Add the users to the worklist. CAREFUL: an instruction can use itself, |
| 551 | // in the case of a phi node. |
David Majnemer | b8da3a2 | 2016-06-25 00:04:10 +0000 | [diff] [blame] | 552 | for (User *U : I->users()) { |
| 553 | if (U != I) { |
Fiona Glaser | f74cc40 | 2015-09-28 18:56:07 +0000 | [diff] [blame] | 554 | WorkList.insert(cast<Instruction>(U)); |
David Majnemer | b8da3a2 | 2016-06-25 00:04:10 +0000 | [diff] [blame] | 555 | } |
| 556 | } |
Fiona Glaser | f74cc40 | 2015-09-28 18:56:07 +0000 | [diff] [blame] | 557 | |
| 558 | // Replace the instruction with its simplified value. |
David Majnemer | b8da3a2 | 2016-06-25 00:04:10 +0000 | [diff] [blame] | 559 | bool Changed = false; |
| 560 | if (!I->use_empty()) { |
| 561 | I->replaceAllUsesWith(SimpleV); |
| 562 | Changed = true; |
| 563 | } |
| 564 | if (isInstructionTriviallyDead(I, TLI)) { |
| 565 | I->eraseFromParent(); |
| 566 | Changed = true; |
| 567 | } |
| 568 | return Changed; |
Fiona Glaser | f74cc40 | 2015-09-28 18:56:07 +0000 | [diff] [blame] | 569 | } |
| 570 | return false; |
| 571 | } |
| 572 | |
Chris Lattner | 7c743f2 | 2010-01-12 19:40:54 +0000 | [diff] [blame] | 573 | /// SimplifyInstructionsInBlock - Scan the specified basic block and try to |
| 574 | /// simplify any instructions in it and recursively delete dead instructions. |
| 575 | /// |
| 576 | /// This returns true if it changed the code, note that it can delete |
| 577 | /// instructions in other blocks as well in this block. |
Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 578 | bool llvm::SimplifyInstructionsInBlock(BasicBlock *BB, |
Benjamin Kramer | 8bcc971 | 2012-08-29 15:32:21 +0000 | [diff] [blame] | 579 | const TargetLibraryInfo *TLI) { |
Chris Lattner | 7c743f2 | 2010-01-12 19:40:54 +0000 | [diff] [blame] | 580 | bool MadeChange = false; |
Fiona Glaser | f74cc40 | 2015-09-28 18:56:07 +0000 | [diff] [blame] | 581 | const DataLayout &DL = BB->getModule()->getDataLayout(); |
Chandler Carruth | 0c72e3f | 2012-03-25 03:29:25 +0000 | [diff] [blame] | 582 | |
| 583 | #ifndef NDEBUG |
| 584 | // In debug builds, ensure that the terminator of the block is never replaced |
| 585 | // or deleted by these simplifications. The idea of simplification is that it |
| 586 | // cannot introduce new instructions, and there is no way to replace the |
| 587 | // terminator of a block without introducing a new instruction. |
Duncan P. N. Exon Smith | 5b4c837 | 2015-10-13 02:39:05 +0000 | [diff] [blame] | 588 | AssertingVH<Instruction> TerminatorVH(&BB->back()); |
Chandler Carruth | 0c72e3f | 2012-03-25 03:29:25 +0000 | [diff] [blame] | 589 | #endif |
| 590 | |
Fiona Glaser | f74cc40 | 2015-09-28 18:56:07 +0000 | [diff] [blame] | 591 | SmallSetVector<Instruction *, 16> WorkList; |
| 592 | // Iterate over the original function, only adding insts to the worklist |
| 593 | // if they actually need to be revisited. This avoids having to pre-init |
| 594 | // the worklist with the entire function's worth of instructions. |
Chad Rosier | 56def25 | 2016-05-21 21:12:06 +0000 | [diff] [blame] | 595 | for (BasicBlock::iterator BI = BB->begin(), E = std::prev(BB->end()); |
| 596 | BI != E;) { |
Chandler Carruth | 17fc6ef | 2012-03-24 23:03:27 +0000 | [diff] [blame] | 597 | assert(!BI->isTerminator()); |
Fiona Glaser | f74cc40 | 2015-09-28 18:56:07 +0000 | [diff] [blame] | 598 | Instruction *I = &*BI; |
| 599 | ++BI; |
Chandler Carruth | cf1b585 | 2012-03-24 21:11:24 +0000 | [diff] [blame] | 600 | |
Fiona Glaser | f74cc40 | 2015-09-28 18:56:07 +0000 | [diff] [blame] | 601 | // We're visiting this instruction now, so make sure it's not in the |
| 602 | // worklist from an earlier visit. |
| 603 | if (!WorkList.count(I)) |
| 604 | MadeChange |= simplifyAndDCEInstruction(I, WorkList, DL, TLI); |
| 605 | } |
Eli Friedman | 17bf492 | 2011-04-02 22:45:17 +0000 | [diff] [blame] | 606 | |
Fiona Glaser | f74cc40 | 2015-09-28 18:56:07 +0000 | [diff] [blame] | 607 | while (!WorkList.empty()) { |
| 608 | Instruction *I = WorkList.pop_back_val(); |
| 609 | MadeChange |= simplifyAndDCEInstruction(I, WorkList, DL, TLI); |
Chris Lattner | 7c743f2 | 2010-01-12 19:40:54 +0000 | [diff] [blame] | 610 | } |
| 611 | return MadeChange; |
| 612 | } |
| 613 | |
Chris Lattner | 99d6809 | 2008-11-27 07:43:12 +0000 | [diff] [blame] | 614 | //===----------------------------------------------------------------------===// |
Chris Lattner | 852d6d6 | 2009-11-10 22:26:15 +0000 | [diff] [blame] | 615 | // Control Flow Graph Restructuring. |
Chris Lattner | 99d6809 | 2008-11-27 07:43:12 +0000 | [diff] [blame] | 616 | // |
| 617 | |
Chris Lattner | 852d6d6 | 2009-11-10 22:26:15 +0000 | [diff] [blame] | 618 | /// RemovePredecessorAndSimplify - Like BasicBlock::removePredecessor, this |
| 619 | /// method is called when we're about to delete Pred as a predecessor of BB. If |
| 620 | /// BB contains any PHI nodes, this drops the entries in the PHI nodes for Pred. |
| 621 | /// |
| 622 | /// Unlike the removePredecessor method, this attempts to simplify uses of PHI |
| 623 | /// nodes that collapse into identity values. For example, if we have: |
| 624 | /// x = phi(1, 0, 0, 0) |
| 625 | /// y = and x, z |
| 626 | /// |
| 627 | /// .. and delete the predecessor corresponding to the '1', this will attempt to |
| 628 | /// recursively fold the and to 0. |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 629 | void llvm::RemovePredecessorAndSimplify(BasicBlock *BB, BasicBlock *Pred, |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 630 | DomTreeUpdater *DTU) { |
Chris Lattner | 852d6d6 | 2009-11-10 22:26:15 +0000 | [diff] [blame] | 631 | // This only adjusts blocks with PHI nodes. |
| 632 | if (!isa<PHINode>(BB->begin())) |
| 633 | return; |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 634 | |
Chris Lattner | 852d6d6 | 2009-11-10 22:26:15 +0000 | [diff] [blame] | 635 | // Remove the entries for Pred from the PHI nodes in BB, but do not simplify |
| 636 | // them down. This will leave us with single entry phi nodes and other phis |
| 637 | // that can be removed. |
| 638 | BB->removePredecessor(Pred, true); |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 639 | |
Sanjoy Das | e6bca0e | 2017-05-01 17:07:49 +0000 | [diff] [blame] | 640 | WeakTrackingVH PhiIt = &BB->front(); |
Chris Lattner | 852d6d6 | 2009-11-10 22:26:15 +0000 | [diff] [blame] | 641 | while (PHINode *PN = dyn_cast<PHINode>(PhiIt)) { |
| 642 | PhiIt = &*++BasicBlock::iterator(cast<Instruction>(PhiIt)); |
Chris Lattner | e41ab07 | 2010-07-15 06:06:04 +0000 | [diff] [blame] | 643 | Value *OldPhiIt = PhiIt; |
Chandler Carruth | cf1b585 | 2012-03-24 21:11:24 +0000 | [diff] [blame] | 644 | |
Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 645 | if (!recursivelySimplifyInstruction(PN)) |
Chandler Carruth | cf1b585 | 2012-03-24 21:11:24 +0000 | [diff] [blame] | 646 | continue; |
| 647 | |
Chris Lattner | 852d6d6 | 2009-11-10 22:26:15 +0000 | [diff] [blame] | 648 | // If recursive simplification ended up deleting the next PHI node we would |
| 649 | // iterate to, then our iterator is invalid, restart scanning from the top |
| 650 | // of the block. |
Chris Lattner | e41ab07 | 2010-07-15 06:06:04 +0000 | [diff] [blame] | 651 | if (PhiIt != OldPhiIt) PhiIt = &BB->front(); |
Chris Lattner | 852d6d6 | 2009-11-10 22:26:15 +0000 | [diff] [blame] | 652 | } |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 653 | if (DTU) |
| 654 | DTU->deleteEdgeRelaxed(Pred, BB); |
Chris Lattner | 852d6d6 | 2009-11-10 22:26:15 +0000 | [diff] [blame] | 655 | } |
| 656 | |
Chris Lattner | 99d6809 | 2008-11-27 07:43:12 +0000 | [diff] [blame] | 657 | /// MergeBasicBlockIntoOnlyPred - DestBB is a block with one predecessor and its |
| 658 | /// predecessor is known to have one successor (DestBB!). Eliminate the edge |
| 659 | /// between them, moving the instructions in the predecessor into DestBB and |
| 660 | /// deleting the predecessor block. |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 661 | void llvm::MergeBasicBlockIntoOnlyPred(BasicBlock *DestBB, |
| 662 | DomTreeUpdater *DTU) { |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 663 | |
Chris Lattner | 99d6809 | 2008-11-27 07:43:12 +0000 | [diff] [blame] | 664 | // If BB has single-entry PHI nodes, fold them. |
| 665 | while (PHINode *PN = dyn_cast<PHINode>(DestBB->begin())) { |
| 666 | Value *NewVal = PN->getIncomingValue(0); |
| 667 | // Replace self referencing PHI with undef, it must be dead. |
Owen Anderson | b292b8c | 2009-07-30 23:03:37 +0000 | [diff] [blame] | 668 | if (NewVal == PN) NewVal = UndefValue::get(PN->getType()); |
Chris Lattner | 99d6809 | 2008-11-27 07:43:12 +0000 | [diff] [blame] | 669 | PN->replaceAllUsesWith(NewVal); |
| 670 | PN->eraseFromParent(); |
| 671 | } |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 672 | |
Chris Lattner | 99d6809 | 2008-11-27 07:43:12 +0000 | [diff] [blame] | 673 | BasicBlock *PredBB = DestBB->getSinglePredecessor(); |
| 674 | assert(PredBB && "Block doesn't have a single predecessor!"); |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 675 | |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 676 | bool ReplaceEntryBB = false; |
| 677 | if (PredBB == &DestBB->getParent()->getEntryBlock()) |
| 678 | ReplaceEntryBB = true; |
| 679 | |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 680 | // DTU updates: Collect all the edges that enter |
| 681 | // PredBB. These dominator edges will be redirected to DestBB. |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 682 | std::vector <DominatorTree::UpdateType> Updates; |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 683 | |
| 684 | if (DTU) { |
Vedant Kumar | e0b5f86 | 2018-05-10 23:01:54 +0000 | [diff] [blame] | 685 | Updates.reserve(1 + (2 * pred_size(PredBB))); |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 686 | Updates.push_back({DominatorTree::Delete, PredBB, DestBB}); |
| 687 | for (auto I = pred_begin(PredBB), E = pred_end(PredBB); I != E; ++I) { |
| 688 | Updates.push_back({DominatorTree::Delete, *I, PredBB}); |
| 689 | // This predecessor of PredBB may already have DestBB as a successor. |
| 690 | if (llvm::find(successors(*I), DestBB) == succ_end(*I)) |
| 691 | Updates.push_back({DominatorTree::Insert, *I, DestBB}); |
| 692 | } |
| 693 | } |
| 694 | |
Chris Lattner | 6fbfe58 | 2010-02-15 20:47:49 +0000 | [diff] [blame] | 695 | // Zap anything that took the address of DestBB. Not doing this will give the |
| 696 | // address an invalid value. |
| 697 | if (DestBB->hasAddressTaken()) { |
| 698 | BlockAddress *BA = BlockAddress::get(DestBB); |
| 699 | Constant *Replacement = |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 700 | ConstantInt::get(Type::getInt32Ty(BA->getContext()), 1); |
Chris Lattner | 6fbfe58 | 2010-02-15 20:47:49 +0000 | [diff] [blame] | 701 | BA->replaceAllUsesWith(ConstantExpr::getIntToPtr(Replacement, |
| 702 | BA->getType())); |
| 703 | BA->destroyConstant(); |
| 704 | } |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 705 | |
Chris Lattner | 99d6809 | 2008-11-27 07:43:12 +0000 | [diff] [blame] | 706 | // Anything that branched to PredBB now branches to DestBB. |
| 707 | PredBB->replaceAllUsesWith(DestBB); |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 708 | |
Jay Foad | 61ea0e4 | 2011-06-23 09:09:15 +0000 | [diff] [blame] | 709 | // Splice all the instructions from PredBB to DestBB. |
| 710 | PredBB->getTerminator()->eraseFromParent(); |
Bill Wendling | 90dd90a | 2013-10-21 04:09:17 +0000 | [diff] [blame] | 711 | DestBB->getInstList().splice(DestBB->begin(), PredBB->getInstList()); |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 712 | new UnreachableInst(PredBB->getContext(), PredBB); |
Jay Foad | 61ea0e4 | 2011-06-23 09:09:15 +0000 | [diff] [blame] | 713 | |
Owen Anderson | a8d1c3e | 2014-07-12 07:12:47 +0000 | [diff] [blame] | 714 | // If the PredBB is the entry block of the function, move DestBB up to |
| 715 | // become the entry block after we erase PredBB. |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 716 | if (ReplaceEntryBB) |
Owen Anderson | a8d1c3e | 2014-07-12 07:12:47 +0000 | [diff] [blame] | 717 | DestBB->moveAfter(PredBB); |
Evandro Menezes | 3701df5 | 2017-09-28 17:24:40 +0000 | [diff] [blame] | 718 | |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 719 | if (DTU) { |
| 720 | assert(PredBB->getInstList().size() == 1 && |
| 721 | isa<UnreachableInst>(PredBB->getTerminator()) && |
| 722 | "The successor list of PredBB isn't empty before " |
| 723 | "applying corresponding DTU updates."); |
| 724 | DTU->applyUpdates(Updates, /*ForceRemoveDuplicates*/ true); |
| 725 | DTU->deleteBB(PredBB); |
| 726 | // Recalculation of DomTree is needed when updating a forward DomTree and |
| 727 | // the Entry BB is replaced. |
| 728 | if (ReplaceEntryBB && DTU->hasDomTree()) { |
| 729 | // The entry block was removed and there is no external interface for |
| 730 | // the dominator tree to be notified of this change. In this corner-case |
| 731 | // we recalculate the entire tree. |
| 732 | DTU->recalculate(*(DestBB->getParent())); |
Balaram Makam | 9ee942f | 2017-10-26 15:04:53 +0000 | [diff] [blame] | 733 | } |
Evandro Menezes | 3701df5 | 2017-09-28 17:24:40 +0000 | [diff] [blame] | 734 | } |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 735 | |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 736 | else { |
| 737 | PredBB->eraseFromParent(); // Nuke BB if DTU is nullptr. |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 738 | } |
Chris Lattner | 99d6809 | 2008-11-27 07:43:12 +0000 | [diff] [blame] | 739 | } |
Devang Patel | caf4485 | 2009-02-10 07:00:59 +0000 | [diff] [blame] | 740 | |
Duncan Sands | e773c08 | 2013-07-11 08:28:20 +0000 | [diff] [blame] | 741 | /// CanMergeValues - Return true if we can choose one of these values to use |
| 742 | /// in place of the other. Note that we will always choose the non-undef |
| 743 | /// value to keep. |
| 744 | static bool CanMergeValues(Value *First, Value *Second) { |
| 745 | return First == Second || isa<UndefValue>(First) || isa<UndefValue>(Second); |
| 746 | } |
| 747 | |
Chris Lattner | cbd18fc | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 748 | /// CanPropagatePredecessorsForPHIs - Return true if we can fold BB, an |
Mark Lacey | a262655 | 2013-08-14 22:11:42 +0000 | [diff] [blame] | 749 | /// almost-empty BB ending in an unconditional branch to Succ, into Succ. |
Chris Lattner | cbd18fc | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 750 | /// |
| 751 | /// Assumption: Succ is the single successor for BB. |
Chris Lattner | cbd18fc | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 752 | static bool CanPropagatePredecessorsForPHIs(BasicBlock *BB, BasicBlock *Succ) { |
| 753 | assert(*succ_begin(BB) == Succ && "Succ is not successor of BB!"); |
| 754 | |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 755 | LLVM_DEBUG(dbgs() << "Looking to fold " << BB->getName() << " into " |
| 756 | << Succ->getName() << "\n"); |
Chris Lattner | cbd18fc | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 757 | // Shortcut, if there is only a single predecessor it must be BB and merging |
| 758 | // is always safe |
| 759 | if (Succ->getSinglePredecessor()) return true; |
| 760 | |
| 761 | // Make a list of the predecessors of BB |
Benjamin Kramer | b5188f1 | 2011-12-06 16:14:29 +0000 | [diff] [blame] | 762 | SmallPtrSet<BasicBlock*, 16> BBPreds(pred_begin(BB), pred_end(BB)); |
Chris Lattner | cbd18fc | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 763 | |
Chris Lattner | cbd18fc | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 764 | // Look at all the phi nodes in Succ, to see if they present a conflict when |
| 765 | // merging these blocks |
| 766 | for (BasicBlock::iterator I = Succ->begin(); isa<PHINode>(I); ++I) { |
| 767 | PHINode *PN = cast<PHINode>(I); |
| 768 | |
| 769 | // If the incoming value from BB is again a PHINode in |
| 770 | // BB which has the same incoming value for *PI as PN does, we can |
| 771 | // merge the phi nodes and then the blocks can still be merged |
| 772 | PHINode *BBPN = dyn_cast<PHINode>(PN->getIncomingValueForBlock(BB)); |
| 773 | if (BBPN && BBPN->getParent() == BB) { |
Benjamin Kramer | b5188f1 | 2011-12-06 16:14:29 +0000 | [diff] [blame] | 774 | for (unsigned PI = 0, PE = PN->getNumIncomingValues(); PI != PE; ++PI) { |
| 775 | BasicBlock *IBB = PN->getIncomingBlock(PI); |
| 776 | if (BBPreds.count(IBB) && |
Duncan Sands | e773c08 | 2013-07-11 08:28:20 +0000 | [diff] [blame] | 777 | !CanMergeValues(BBPN->getIncomingValueForBlock(IBB), |
| 778 | PN->getIncomingValue(PI))) { |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 779 | LLVM_DEBUG(dbgs() |
| 780 | << "Can't fold, phi node " << PN->getName() << " in " |
| 781 | << Succ->getName() << " is conflicting with " |
| 782 | << BBPN->getName() << " with regard to common predecessor " |
| 783 | << IBB->getName() << "\n"); |
Chris Lattner | cbd18fc | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 784 | return false; |
| 785 | } |
| 786 | } |
| 787 | } else { |
| 788 | Value* Val = PN->getIncomingValueForBlock(BB); |
Benjamin Kramer | b5188f1 | 2011-12-06 16:14:29 +0000 | [diff] [blame] | 789 | for (unsigned PI = 0, PE = PN->getNumIncomingValues(); PI != PE; ++PI) { |
Chris Lattner | cbd18fc | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 790 | // See if the incoming value for the common predecessor is equal to the |
| 791 | // one for BB, in which case this phi node will not prevent the merging |
| 792 | // of the block. |
Benjamin Kramer | b5188f1 | 2011-12-06 16:14:29 +0000 | [diff] [blame] | 793 | BasicBlock *IBB = PN->getIncomingBlock(PI); |
Duncan Sands | e773c08 | 2013-07-11 08:28:20 +0000 | [diff] [blame] | 794 | if (BBPreds.count(IBB) && |
| 795 | !CanMergeValues(Val, PN->getIncomingValue(PI))) { |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 796 | LLVM_DEBUG(dbgs() << "Can't fold, phi node " << PN->getName() |
| 797 | << " in " << Succ->getName() |
| 798 | << " is conflicting with regard to common " |
| 799 | << "predecessor " << IBB->getName() << "\n"); |
Chris Lattner | cbd18fc | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 800 | return false; |
| 801 | } |
| 802 | } |
| 803 | } |
| 804 | } |
| 805 | |
| 806 | return true; |
| 807 | } |
| 808 | |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 809 | using PredBlockVector = SmallVector<BasicBlock *, 16>; |
| 810 | using IncomingValueMap = DenseMap<BasicBlock *, Value *>; |
Duncan Sands | e773c08 | 2013-07-11 08:28:20 +0000 | [diff] [blame] | 811 | |
Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 812 | /// Determines the value to use as the phi node input for a block. |
Duncan Sands | e773c08 | 2013-07-11 08:28:20 +0000 | [diff] [blame] | 813 | /// |
| 814 | /// Select between \p OldVal any value that we know flows from \p BB |
| 815 | /// to a particular phi on the basis of which one (if either) is not |
| 816 | /// undef. Update IncomingValues based on the selected value. |
| 817 | /// |
| 818 | /// \param OldVal The value we are considering selecting. |
| 819 | /// \param BB The block that the value flows in from. |
| 820 | /// \param IncomingValues A map from block-to-value for other phi inputs |
| 821 | /// that we have examined. |
| 822 | /// |
| 823 | /// \returns the selected value. |
| 824 | static Value *selectIncomingValueForBlock(Value *OldVal, BasicBlock *BB, |
| 825 | IncomingValueMap &IncomingValues) { |
| 826 | if (!isa<UndefValue>(OldVal)) { |
| 827 | assert((!IncomingValues.count(BB) || |
| 828 | IncomingValues.find(BB)->second == OldVal) && |
| 829 | "Expected OldVal to match incoming value from BB!"); |
| 830 | |
| 831 | IncomingValues.insert(std::make_pair(BB, OldVal)); |
| 832 | return OldVal; |
| 833 | } |
| 834 | |
| 835 | IncomingValueMap::const_iterator It = IncomingValues.find(BB); |
| 836 | if (It != IncomingValues.end()) return It->second; |
| 837 | |
| 838 | return OldVal; |
| 839 | } |
| 840 | |
Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 841 | /// Create a map from block to value for the operands of a |
Duncan Sands | e773c08 | 2013-07-11 08:28:20 +0000 | [diff] [blame] | 842 | /// given phi. |
| 843 | /// |
| 844 | /// Create a map from block to value for each non-undef value flowing |
| 845 | /// into \p PN. |
| 846 | /// |
| 847 | /// \param PN The phi we are collecting the map for. |
| 848 | /// \param IncomingValues [out] The map from block to value for this phi. |
| 849 | static void gatherIncomingValuesToPhi(PHINode *PN, |
| 850 | IncomingValueMap &IncomingValues) { |
| 851 | for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) { |
| 852 | BasicBlock *BB = PN->getIncomingBlock(i); |
| 853 | Value *V = PN->getIncomingValue(i); |
| 854 | |
| 855 | if (!isa<UndefValue>(V)) |
| 856 | IncomingValues.insert(std::make_pair(BB, V)); |
| 857 | } |
| 858 | } |
| 859 | |
Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 860 | /// Replace the incoming undef values to a phi with the values |
Duncan Sands | e773c08 | 2013-07-11 08:28:20 +0000 | [diff] [blame] | 861 | /// from a block-to-value map. |
| 862 | /// |
| 863 | /// \param PN The phi we are replacing the undefs in. |
| 864 | /// \param IncomingValues A map from block to value. |
| 865 | static void replaceUndefValuesInPhi(PHINode *PN, |
| 866 | const IncomingValueMap &IncomingValues) { |
| 867 | for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) { |
| 868 | Value *V = PN->getIncomingValue(i); |
| 869 | |
| 870 | if (!isa<UndefValue>(V)) continue; |
| 871 | |
| 872 | BasicBlock *BB = PN->getIncomingBlock(i); |
| 873 | IncomingValueMap::const_iterator It = IncomingValues.find(BB); |
| 874 | if (It == IncomingValues.end()) continue; |
| 875 | |
| 876 | PN->setIncomingValue(i, It->second); |
| 877 | } |
| 878 | } |
| 879 | |
Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 880 | /// Replace a value flowing from a block to a phi with |
Duncan Sands | e773c08 | 2013-07-11 08:28:20 +0000 | [diff] [blame] | 881 | /// potentially multiple instances of that value flowing from the |
| 882 | /// block's predecessors to the phi. |
| 883 | /// |
| 884 | /// \param BB The block with the value flowing into the phi. |
| 885 | /// \param BBPreds The predecessors of BB. |
| 886 | /// \param PN The phi that we are updating. |
| 887 | static void redirectValuesFromPredecessorsToPhi(BasicBlock *BB, |
| 888 | const PredBlockVector &BBPreds, |
| 889 | PHINode *PN) { |
| 890 | Value *OldVal = PN->removeIncomingValue(BB, false); |
| 891 | assert(OldVal && "No entry in PHI for Pred BB!"); |
| 892 | |
| 893 | IncomingValueMap IncomingValues; |
| 894 | |
| 895 | // We are merging two blocks - BB, and the block containing PN - and |
| 896 | // as a result we need to redirect edges from the predecessors of BB |
| 897 | // to go to the block containing PN, and update PN |
| 898 | // accordingly. Since we allow merging blocks in the case where the |
| 899 | // predecessor and successor blocks both share some predecessors, |
| 900 | // and where some of those common predecessors might have undef |
| 901 | // values flowing into PN, we want to rewrite those values to be |
| 902 | // consistent with the non-undef values. |
| 903 | |
| 904 | gatherIncomingValuesToPhi(PN, IncomingValues); |
| 905 | |
| 906 | // If this incoming value is one of the PHI nodes in BB, the new entries |
| 907 | // in the PHI node are the entries from the old PHI. |
| 908 | if (isa<PHINode>(OldVal) && cast<PHINode>(OldVal)->getParent() == BB) { |
| 909 | PHINode *OldValPN = cast<PHINode>(OldVal); |
| 910 | for (unsigned i = 0, e = OldValPN->getNumIncomingValues(); i != e; ++i) { |
| 911 | // Note that, since we are merging phi nodes and BB and Succ might |
| 912 | // have common predecessors, we could end up with a phi node with |
| 913 | // identical incoming branches. This will be cleaned up later (and |
| 914 | // will trigger asserts if we try to clean it up now, without also |
| 915 | // simplifying the corresponding conditional branch). |
| 916 | BasicBlock *PredBB = OldValPN->getIncomingBlock(i); |
| 917 | Value *PredVal = OldValPN->getIncomingValue(i); |
| 918 | Value *Selected = selectIncomingValueForBlock(PredVal, PredBB, |
| 919 | IncomingValues); |
| 920 | |
| 921 | // And add a new incoming value for this predecessor for the |
| 922 | // newly retargeted branch. |
| 923 | PN->addIncoming(Selected, PredBB); |
| 924 | } |
| 925 | } else { |
| 926 | for (unsigned i = 0, e = BBPreds.size(); i != e; ++i) { |
| 927 | // Update existing incoming values in PN for this |
| 928 | // predecessor of BB. |
| 929 | BasicBlock *PredBB = BBPreds[i]; |
| 930 | Value *Selected = selectIncomingValueForBlock(OldVal, PredBB, |
| 931 | IncomingValues); |
| 932 | |
| 933 | // And add a new incoming value for this predecessor for the |
| 934 | // newly retargeted branch. |
| 935 | PN->addIncoming(Selected, PredBB); |
| 936 | } |
| 937 | } |
| 938 | |
| 939 | replaceUndefValuesInPhi(PN, IncomingValues); |
| 940 | } |
| 941 | |
Chris Lattner | cbd18fc | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 942 | /// TryToSimplifyUncondBranchFromEmptyBlock - BB is known to contain an |
| 943 | /// unconditional branch, and contains no instructions other than PHI nodes, |
Rafael Espindola | b10a0f2 | 2011-06-30 20:14:24 +0000 | [diff] [blame] | 944 | /// potential side-effect free intrinsics and the branch. If possible, |
| 945 | /// eliminate BB by rewriting all the predecessors to branch to the successor |
| 946 | /// block and return true. If we can't transform, return false. |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 947 | bool llvm::TryToSimplifyUncondBranchFromEmptyBlock(BasicBlock *BB, |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 948 | DomTreeUpdater *DTU) { |
Dan Gohman | 4a63fad | 2010-08-14 00:29:42 +0000 | [diff] [blame] | 949 | assert(BB != &BB->getParent()->getEntryBlock() && |
| 950 | "TryToSimplifyUncondBranchFromEmptyBlock called on entry block!"); |
| 951 | |
Chris Lattner | cbd18fc | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 952 | // We can't eliminate infinite loops. |
| 953 | BasicBlock *Succ = cast<BranchInst>(BB->getTerminator())->getSuccessor(0); |
| 954 | if (BB == Succ) return false; |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 955 | |
Reid Kleckner | bca59d2 | 2016-05-02 19:43:22 +0000 | [diff] [blame] | 956 | // Check to see if merging these blocks would cause conflicts for any of the |
| 957 | // phi nodes in BB or Succ. If not, we can safely merge. |
| 958 | if (!CanPropagatePredecessorsForPHIs(BB, Succ)) return false; |
Chris Lattner | cbd18fc | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 959 | |
Reid Kleckner | bca59d2 | 2016-05-02 19:43:22 +0000 | [diff] [blame] | 960 | // Check for cases where Succ has multiple predecessors and a PHI node in BB |
| 961 | // has uses which will not disappear when the PHI nodes are merged. It is |
| 962 | // possible to handle such cases, but difficult: it requires checking whether |
| 963 | // BB dominates Succ, which is non-trivial to calculate in the case where |
| 964 | // Succ has multiple predecessors. Also, it requires checking whether |
| 965 | // constructing the necessary self-referential PHI node doesn't introduce any |
| 966 | // conflicts; this isn't too difficult, but the previous code for doing this |
| 967 | // was incorrect. |
| 968 | // |
| 969 | // Note that if this check finds a live use, BB dominates Succ, so BB is |
| 970 | // something like a loop pre-header (or rarely, a part of an irreducible CFG); |
| 971 | // folding the branch isn't profitable in that case anyway. |
| 972 | if (!Succ->getSinglePredecessor()) { |
| 973 | BasicBlock::iterator BBI = BB->begin(); |
| 974 | while (isa<PHINode>(*BBI)) { |
| 975 | for (Use &U : BBI->uses()) { |
| 976 | if (PHINode* PN = dyn_cast<PHINode>(U.getUser())) { |
| 977 | if (PN->getIncomingBlock(U) != BB) |
Hans Wennborg | b759932 | 2016-05-02 17:22:54 +0000 | [diff] [blame] | 978 | return false; |
Reid Kleckner | bca59d2 | 2016-05-02 19:43:22 +0000 | [diff] [blame] | 979 | } else { |
| 980 | return false; |
Hans Wennborg | b759932 | 2016-05-02 17:22:54 +0000 | [diff] [blame] | 981 | } |
Hans Wennborg | b759932 | 2016-05-02 17:22:54 +0000 | [diff] [blame] | 982 | } |
Reid Kleckner | bca59d2 | 2016-05-02 19:43:22 +0000 | [diff] [blame] | 983 | ++BBI; |
Hans Wennborg | b759932 | 2016-05-02 17:22:54 +0000 | [diff] [blame] | 984 | } |
Hans Wennborg | b759932 | 2016-05-02 17:22:54 +0000 | [diff] [blame] | 985 | } |
Reid Kleckner | bca59d2 | 2016-05-02 19:43:22 +0000 | [diff] [blame] | 986 | |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 987 | LLVM_DEBUG(dbgs() << "Killing Trivial BB: \n" << *BB); |
Reid Kleckner | bca59d2 | 2016-05-02 19:43:22 +0000 | [diff] [blame] | 988 | |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 989 | std::vector<DominatorTree::UpdateType> Updates; |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 990 | if (DTU) { |
Vedant Kumar | e0b5f86 | 2018-05-10 23:01:54 +0000 | [diff] [blame] | 991 | Updates.reserve(1 + (2 * pred_size(BB))); |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 992 | Updates.push_back({DominatorTree::Delete, BB, Succ}); |
| 993 | // All predecessors of BB will be moved to Succ. |
| 994 | for (auto I = pred_begin(BB), E = pred_end(BB); I != E; ++I) { |
| 995 | Updates.push_back({DominatorTree::Delete, *I, BB}); |
| 996 | // This predecessor of BB may already have Succ as a successor. |
| 997 | if (llvm::find(successors(*I), Succ) == succ_end(*I)) |
| 998 | Updates.push_back({DominatorTree::Insert, *I, Succ}); |
| 999 | } |
| 1000 | } |
| 1001 | |
Reid Kleckner | bca59d2 | 2016-05-02 19:43:22 +0000 | [diff] [blame] | 1002 | if (isa<PHINode>(Succ->begin())) { |
| 1003 | // If there is more than one pred of succ, and there are PHI nodes in |
| 1004 | // the successor, then we need to add incoming edges for the PHI nodes |
| 1005 | // |
| 1006 | const PredBlockVector BBPreds(pred_begin(BB), pred_end(BB)); |
| 1007 | |
| 1008 | // Loop over all of the PHI nodes in the successor of BB. |
| 1009 | for (BasicBlock::iterator I = Succ->begin(); isa<PHINode>(I); ++I) { |
| 1010 | PHINode *PN = cast<PHINode>(I); |
| 1011 | |
| 1012 | redirectValuesFromPredecessorsToPhi(BB, BBPreds, PN); |
| 1013 | } |
| 1014 | } |
| 1015 | |
| 1016 | if (Succ->getSinglePredecessor()) { |
| 1017 | // BB is the only predecessor of Succ, so Succ will end up with exactly |
| 1018 | // the same predecessors BB had. |
| 1019 | |
| 1020 | // Copy over any phi, debug or lifetime instruction. |
| 1021 | BB->getTerminator()->eraseFromParent(); |
| 1022 | Succ->getInstList().splice(Succ->getFirstNonPHI()->getIterator(), |
| 1023 | BB->getInstList()); |
| 1024 | } else { |
| 1025 | while (PHINode *PN = dyn_cast<PHINode>(&BB->front())) { |
| 1026 | // We explicitly check for such uses in CanPropagatePredecessorsForPHIs. |
| 1027 | assert(PN->use_empty() && "There shouldn't be any uses here!"); |
| 1028 | PN->eraseFromParent(); |
| 1029 | } |
| 1030 | } |
| 1031 | |
Florian Hahn | 77382be | 2016-11-18 13:12:07 +0000 | [diff] [blame] | 1032 | // If the unconditional branch we replaced contains llvm.loop metadata, we |
| 1033 | // add the metadata to the branch instructions in the predecessors. |
| 1034 | unsigned LoopMDKind = BB->getContext().getMDKindID("llvm.loop"); |
| 1035 | Instruction *TI = BB->getTerminator(); |
Daniel Jasper | 0a51ec2 | 2017-09-30 11:57:19 +0000 | [diff] [blame] | 1036 | if (TI) |
Florian Hahn | 77382be | 2016-11-18 13:12:07 +0000 | [diff] [blame] | 1037 | if (MDNode *LoopMD = TI->getMetadata(LoopMDKind)) |
| 1038 | for (pred_iterator PI = pred_begin(BB), E = pred_end(BB); PI != E; ++PI) { |
| 1039 | BasicBlock *Pred = *PI; |
| 1040 | Pred->getTerminator()->setMetadata(LoopMDKind, LoopMD); |
| 1041 | } |
| 1042 | |
Reid Kleckner | bca59d2 | 2016-05-02 19:43:22 +0000 | [diff] [blame] | 1043 | // Everything that jumped to BB now goes to Succ. |
| 1044 | BB->replaceAllUsesWith(Succ); |
| 1045 | if (!Succ->hasName()) Succ->takeName(BB); |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 1046 | |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 1047 | // Clear the successor list of BB to match updates applying to DTU later. |
| 1048 | if (BB->getTerminator()) |
| 1049 | BB->getInstList().pop_back(); |
| 1050 | new UnreachableInst(BB->getContext(), BB); |
| 1051 | assert(succ_empty(BB) && "The successor list of BB isn't empty before " |
| 1052 | "applying corresponding DTU updates."); |
| 1053 | |
| 1054 | if (DTU) { |
| 1055 | DTU->applyUpdates(Updates, /*ForceRemoveDuplicates*/ true); |
| 1056 | DTU->deleteBB(BB); |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 1057 | } else { |
| 1058 | BB->eraseFromParent(); // Delete the old basic block. |
| 1059 | } |
Reid Kleckner | bca59d2 | 2016-05-02 19:43:22 +0000 | [diff] [blame] | 1060 | return true; |
Chris Lattner | cbd18fc | 2009-11-10 05:59:26 +0000 | [diff] [blame] | 1061 | } |
| 1062 | |
Jim Grosbach | d831ef4 | 2009-12-02 17:06:45 +0000 | [diff] [blame] | 1063 | /// EliminateDuplicatePHINodes - Check for and eliminate duplicate PHI |
| 1064 | /// nodes in this block. This doesn't try to be clever about PHI nodes |
| 1065 | /// which differ only in the order of the incoming values, but instcombine |
| 1066 | /// orders them so it usually won't matter. |
Jim Grosbach | d831ef4 | 2009-12-02 17:06:45 +0000 | [diff] [blame] | 1067 | bool llvm::EliminateDuplicatePHINodes(BasicBlock *BB) { |
Jim Grosbach | d831ef4 | 2009-12-02 17:06:45 +0000 | [diff] [blame] | 1068 | // This implementation doesn't currently consider undef operands |
Nick Lewycky | fa44dc6 | 2011-06-28 03:57:31 +0000 | [diff] [blame] | 1069 | // specially. Theoretically, two phis which are identical except for |
Jim Grosbach | d831ef4 | 2009-12-02 17:06:45 +0000 | [diff] [blame] | 1070 | // one having an undef where the other doesn't could be collapsed. |
| 1071 | |
Benjamin Kramer | 2b2cdd7 | 2015-06-18 16:01:00 +0000 | [diff] [blame] | 1072 | struct PHIDenseMapInfo { |
| 1073 | static PHINode *getEmptyKey() { |
| 1074 | return DenseMapInfo<PHINode *>::getEmptyKey(); |
| 1075 | } |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 1076 | |
Benjamin Kramer | 2b2cdd7 | 2015-06-18 16:01:00 +0000 | [diff] [blame] | 1077 | static PHINode *getTombstoneKey() { |
| 1078 | return DenseMapInfo<PHINode *>::getTombstoneKey(); |
| 1079 | } |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 1080 | |
Benjamin Kramer | 2b2cdd7 | 2015-06-18 16:01:00 +0000 | [diff] [blame] | 1081 | static unsigned getHashValue(PHINode *PN) { |
| 1082 | // Compute a hash value on the operands. Instcombine will likely have |
| 1083 | // sorted them, which helps expose duplicates, but we have to check all |
| 1084 | // the operands to be safe in case instcombine hasn't run. |
| 1085 | return static_cast<unsigned>(hash_combine( |
| 1086 | hash_combine_range(PN->value_op_begin(), PN->value_op_end()), |
| 1087 | hash_combine_range(PN->block_begin(), PN->block_end()))); |
| 1088 | } |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 1089 | |
Benjamin Kramer | 2b2cdd7 | 2015-06-18 16:01:00 +0000 | [diff] [blame] | 1090 | static bool isEqual(PHINode *LHS, PHINode *RHS) { |
| 1091 | if (LHS == getEmptyKey() || LHS == getTombstoneKey() || |
| 1092 | RHS == getEmptyKey() || RHS == getTombstoneKey()) |
| 1093 | return LHS == RHS; |
| 1094 | return LHS->isIdenticalTo(RHS); |
| 1095 | } |
| 1096 | }; |
Jim Grosbach | d831ef4 | 2009-12-02 17:06:45 +0000 | [diff] [blame] | 1097 | |
Benjamin Kramer | 2b2cdd7 | 2015-06-18 16:01:00 +0000 | [diff] [blame] | 1098 | // Set of unique PHINodes. |
| 1099 | DenseSet<PHINode *, PHIDenseMapInfo> PHISet; |
Jim Grosbach | d831ef4 | 2009-12-02 17:06:45 +0000 | [diff] [blame] | 1100 | |
| 1101 | // Examine each PHI. |
Benjamin Kramer | 2b2cdd7 | 2015-06-18 16:01:00 +0000 | [diff] [blame] | 1102 | bool Changed = false; |
| 1103 | for (auto I = BB->begin(); PHINode *PN = dyn_cast<PHINode>(I++);) { |
| 1104 | auto Inserted = PHISet.insert(PN); |
| 1105 | if (!Inserted.second) { |
| 1106 | // A duplicate. Replace this PHI with its duplicate. |
| 1107 | PN->replaceAllUsesWith(*Inserted.first); |
| 1108 | PN->eraseFromParent(); |
| 1109 | Changed = true; |
Benjamin Kramer | f175e04 | 2015-09-02 19:52:23 +0000 | [diff] [blame] | 1110 | |
| 1111 | // The RAUW can change PHIs that we already visited. Start over from the |
| 1112 | // beginning. |
| 1113 | PHISet.clear(); |
| 1114 | I = BB->begin(); |
Jim Grosbach | d831ef4 | 2009-12-02 17:06:45 +0000 | [diff] [blame] | 1115 | } |
| 1116 | } |
| 1117 | |
| 1118 | return Changed; |
| 1119 | } |
Chris Lattner | 6fcd32e | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 1120 | |
| 1121 | /// enforceKnownAlignment - If the specified pointer points to an object that |
| 1122 | /// we control, modify the object's alignment to PrefAlign. This isn't |
| 1123 | /// often possible though. If alignment is important, a more reliable approach |
| 1124 | /// is to simply align all global variables and allocation instructions to |
| 1125 | /// their preferred alignment from the beginning. |
Benjamin Kramer | 570dd78 | 2010-12-30 22:34:44 +0000 | [diff] [blame] | 1126 | static unsigned enforceKnownAlignment(Value *V, unsigned Align, |
Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 1127 | unsigned PrefAlign, |
| 1128 | const DataLayout &DL) { |
James Y Knight | ac03dca | 2016-01-15 16:33:06 +0000 | [diff] [blame] | 1129 | assert(PrefAlign > Align); |
| 1130 | |
Eli Friedman | 19ace4c | 2011-06-15 21:08:25 +0000 | [diff] [blame] | 1131 | V = V->stripPointerCasts(); |
Chris Lattner | 6fcd32e | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 1132 | |
Eli Friedman | 19ace4c | 2011-06-15 21:08:25 +0000 | [diff] [blame] | 1133 | if (AllocaInst *AI = dyn_cast<AllocaInst>(V)) { |
James Y Knight | ac03dca | 2016-01-15 16:33:06 +0000 | [diff] [blame] | 1134 | // TODO: ideally, computeKnownBits ought to have used |
| 1135 | // AllocaInst::getAlignment() in its computation already, making |
| 1136 | // the below max redundant. But, as it turns out, |
| 1137 | // stripPointerCasts recurses through infinite layers of bitcasts, |
| 1138 | // while computeKnownBits is not allowed to traverse more than 6 |
| 1139 | // levels. |
| 1140 | Align = std::max(AI->getAlignment(), Align); |
| 1141 | if (PrefAlign <= Align) |
| 1142 | return Align; |
| 1143 | |
Lang Hames | de7ab80 | 2011-10-10 23:42:08 +0000 | [diff] [blame] | 1144 | // If the preferred alignment is greater than the natural stack alignment |
| 1145 | // then don't round up. This avoids dynamic stack realignment. |
Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 1146 | if (DL.exceedsNaturalStackAlignment(PrefAlign)) |
Lang Hames | de7ab80 | 2011-10-10 23:42:08 +0000 | [diff] [blame] | 1147 | return Align; |
Chris Lattner | 6fcd32e | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 1148 | AI->setAlignment(PrefAlign); |
| 1149 | return PrefAlign; |
| 1150 | } |
Chris Lattner | 6fcd32e | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 1151 | |
Rafael Espindola | 99e05cf | 2014-05-13 18:45:48 +0000 | [diff] [blame] | 1152 | if (auto *GO = dyn_cast<GlobalObject>(V)) { |
James Y Knight | ac03dca | 2016-01-15 16:33:06 +0000 | [diff] [blame] | 1153 | // TODO: as above, this shouldn't be necessary. |
| 1154 | Align = std::max(GO->getAlignment(), Align); |
| 1155 | if (PrefAlign <= Align) |
| 1156 | return Align; |
| 1157 | |
Chris Lattner | 6fcd32e | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 1158 | // If there is a large requested alignment and we can, bump up the alignment |
Reid Kleckner | 486fa39 | 2015-07-14 00:11:08 +0000 | [diff] [blame] | 1159 | // of the global. If the memory we set aside for the global may not be the |
| 1160 | // memory used by the final program then it is impossible for us to reliably |
| 1161 | // enforce the preferred alignment. |
James Y Knight | ac03dca | 2016-01-15 16:33:06 +0000 | [diff] [blame] | 1162 | if (!GO->canIncreaseAlignment()) |
Rafael Espindola | fc13db4 | 2014-05-09 16:01:06 +0000 | [diff] [blame] | 1163 | return Align; |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 1164 | |
James Y Knight | ac03dca | 2016-01-15 16:33:06 +0000 | [diff] [blame] | 1165 | GO->setAlignment(PrefAlign); |
| 1166 | return PrefAlign; |
Chris Lattner | 6fcd32e | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 1167 | } |
| 1168 | |
| 1169 | return Align; |
| 1170 | } |
| 1171 | |
Chris Lattner | 6fcd32e | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 1172 | unsigned llvm::getOrEnforceKnownAlignment(Value *V, unsigned PrefAlign, |
Mehdi Amini | a28d91d | 2015-03-10 02:37:25 +0000 | [diff] [blame] | 1173 | const DataLayout &DL, |
Hal Finkel | 60db058 | 2014-09-07 18:57:58 +0000 | [diff] [blame] | 1174 | const Instruction *CxtI, |
Daniel Jasper | aec2fa3 | 2016-12-19 08:22:17 +0000 | [diff] [blame] | 1175 | AssumptionCache *AC, |
Hal Finkel | 60db058 | 2014-09-07 18:57:58 +0000 | [diff] [blame] | 1176 | const DominatorTree *DT) { |
Chris Lattner | 6fcd32e | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 1177 | assert(V->getType()->isPointerTy() && |
| 1178 | "getOrEnforceKnownAlignment expects a pointer!"); |
Matt Arsenault | 87dc607 | 2013-08-01 22:42:18 +0000 | [diff] [blame] | 1179 | |
Craig Topper | 8205a1a | 2017-05-24 16:53:07 +0000 | [diff] [blame] | 1180 | KnownBits Known = computeKnownBits(V, DL, 0, AC, CxtI, DT); |
Craig Topper | 8df66c6 | 2017-05-12 17:20:30 +0000 | [diff] [blame] | 1181 | unsigned TrailZ = Known.countMinTrailingZeros(); |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 1182 | |
Matt Arsenault | f64212b | 2013-07-23 22:20:57 +0000 | [diff] [blame] | 1183 | // Avoid trouble with ridiculously large TrailZ values, such as |
Chris Lattner | 6fcd32e | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 1184 | // those computed from a null pointer. |
| 1185 | TrailZ = std::min(TrailZ, unsigned(sizeof(unsigned) * CHAR_BIT - 1)); |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 1186 | |
Craig Topper | 8205a1a | 2017-05-24 16:53:07 +0000 | [diff] [blame] | 1187 | unsigned Align = 1u << std::min(Known.getBitWidth() - 1, TrailZ); |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 1188 | |
Chris Lattner | 6fcd32e | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 1189 | // LLVM doesn't support alignments larger than this currently. |
| 1190 | Align = std::min(Align, +Value::MaximumAlignment); |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 1191 | |
Chris Lattner | 6fcd32e | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 1192 | if (PrefAlign > Align) |
Matt Arsenault | 87dc607 | 2013-08-01 22:42:18 +0000 | [diff] [blame] | 1193 | Align = enforceKnownAlignment(V, Align, PrefAlign, DL); |
Jakub Staszak | 8e1a6e7 | 2013-07-22 23:16:36 +0000 | [diff] [blame] | 1194 | |
Chris Lattner | 6fcd32e | 2010-12-25 20:37:57 +0000 | [diff] [blame] | 1195 | // We don't need to make any adjustment. |
| 1196 | return Align; |
| 1197 | } |
| 1198 | |
Devang Patel | 8c0b16b | 2011-03-17 21:58:19 +0000 | [diff] [blame] | 1199 | ///===---------------------------------------------------------------------===// |
| 1200 | /// Dbg Intrinsic utilities |
| 1201 | /// |
| 1202 | |
Adrian Prantl | 29b9de7 | 2013-04-26 17:48:33 +0000 | [diff] [blame] | 1203 | /// See if there is a dbg.value intrinsic for DIVar before I. |
Adrian Prantl | a5b2a64 | 2016-02-17 20:02:25 +0000 | [diff] [blame] | 1204 | static bool LdStHasDebugValue(DILocalVariable *DIVar, DIExpression *DIExpr, |
| 1205 | Instruction *I) { |
Adrian Prantl | 29b9de7 | 2013-04-26 17:48:33 +0000 | [diff] [blame] | 1206 | // Since we can't guarantee that the original dbg.declare instrinsic |
| 1207 | // is removed by LowerDbgDeclare(), we need to make sure that we are |
| 1208 | // not inserting the same dbg.value intrinsic over and over. |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 1209 | BasicBlock::InstListType::iterator PrevI(I); |
Adrian Prantl | 29b9de7 | 2013-04-26 17:48:33 +0000 | [diff] [blame] | 1210 | if (PrevI != I->getParent()->getInstList().begin()) { |
| 1211 | --PrevI; |
| 1212 | if (DbgValueInst *DVI = dyn_cast<DbgValueInst>(PrevI)) |
| 1213 | if (DVI->getValue() == I->getOperand(0) && |
Adrian Prantl | a5b2a64 | 2016-02-17 20:02:25 +0000 | [diff] [blame] | 1214 | DVI->getVariable() == DIVar && |
| 1215 | DVI->getExpression() == DIExpr) |
Adrian Prantl | 29b9de7 | 2013-04-26 17:48:33 +0000 | [diff] [blame] | 1216 | return true; |
| 1217 | } |
| 1218 | return false; |
| 1219 | } |
| 1220 | |
Keith Walker | ba15989 | 2016-09-22 14:13:25 +0000 | [diff] [blame] | 1221 | /// See if there is a dbg.value intrinsic for DIVar for the PHI node. |
Chandler Carruth | 2abb65a | 2017-06-26 03:31:31 +0000 | [diff] [blame] | 1222 | static bool PhiHasDebugValue(DILocalVariable *DIVar, |
Keith Walker | ba15989 | 2016-09-22 14:13:25 +0000 | [diff] [blame] | 1223 | DIExpression *DIExpr, |
| 1224 | PHINode *APN) { |
| 1225 | // Since we can't guarantee that the original dbg.declare instrinsic |
| 1226 | // is removed by LowerDbgDeclare(), we need to make sure that we are |
| 1227 | // not inserting the same dbg.value intrinsic over and over. |
Adrian Prantl | fa9e84e | 2017-03-16 20:11:54 +0000 | [diff] [blame] | 1228 | SmallVector<DbgValueInst *, 1> DbgValues; |
| 1229 | findDbgValues(DbgValues, APN); |
| 1230 | for (auto *DVI : DbgValues) { |
| 1231 | assert(DVI->getValue() == APN); |
Adrian Prantl | fa9e84e | 2017-03-16 20:11:54 +0000 | [diff] [blame] | 1232 | if ((DVI->getVariable() == DIVar) && (DVI->getExpression() == DIExpr)) |
| 1233 | return true; |
| 1234 | } |
| 1235 | return false; |
Keith Walker | ba15989 | 2016-09-22 14:13:25 +0000 | [diff] [blame] | 1236 | } |
| 1237 | |
Bjorn Pettersson | 428caf9 | 2018-06-15 13:48:55 +0000 | [diff] [blame] | 1238 | /// Check if the alloc size of \p ValTy is large enough to cover the variable |
| 1239 | /// (or fragment of the variable) described by \p DII. |
| 1240 | /// |
| 1241 | /// This is primarily intended as a helper for the different |
| 1242 | /// ConvertDebugDeclareToDebugValue functions. The dbg.declare/dbg.addr that is |
| 1243 | /// converted describes an alloca'd variable, so we need to use the |
| 1244 | /// alloc size of the value when doing the comparison. E.g. an i1 value will be |
| 1245 | /// identified as covering an n-bit fragment, if the store size of i1 is at |
| 1246 | /// least n bits. |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1247 | static bool valueCoversEntireFragment(Type *ValTy, DbgVariableIntrinsic *DII) { |
Bjorn Pettersson | 428caf9 | 2018-06-15 13:48:55 +0000 | [diff] [blame] | 1248 | const DataLayout &DL = DII->getModule()->getDataLayout(); |
| 1249 | uint64_t ValueSize = DL.getTypeAllocSizeInBits(ValTy); |
| 1250 | if (auto FragmentSize = DII->getFragmentSizeInBits()) |
| 1251 | return ValueSize >= *FragmentSize; |
Bjorn Pettersson | 550517b | 2018-06-26 06:17:00 +0000 | [diff] [blame] | 1252 | // We can't always calculate the size of the DI variable (e.g. if it is a |
| 1253 | // VLA). Try to use the size of the alloca that the dbg intrinsic describes |
| 1254 | // intead. |
| 1255 | if (DII->isAddressOfVariable()) |
| 1256 | if (auto *AI = dyn_cast_or_null<AllocaInst>(DII->getVariableLocation())) |
| 1257 | if (auto FragmentSize = AI->getAllocationSizeInBits(DL)) |
| 1258 | return ValueSize >= *FragmentSize; |
| 1259 | // Could not determine size of variable. Conservatively return false. |
Bjorn Pettersson | 428caf9 | 2018-06-15 13:48:55 +0000 | [diff] [blame] | 1260 | return false; |
| 1261 | } |
| 1262 | |
Adrian Prantl | d00333a | 2013-04-26 18:10:50 +0000 | [diff] [blame] | 1263 | /// Inserts a llvm.dbg.value intrinsic before a store to an alloca'd value |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1264 | /// that has an associated llvm.dbg.declare or llvm.dbg.addr intrinsic. |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1265 | void llvm::ConvertDebugDeclareToDebugValue(DbgVariableIntrinsic *DII, |
Devang Patel | 8c0b16b | 2011-03-17 21:58:19 +0000 | [diff] [blame] | 1266 | StoreInst *SI, DIBuilder &Builder) { |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1267 | assert(DII->isAddressOfVariable()); |
| 1268 | auto *DIVar = DII->getVariable(); |
Duncan P. N. Exon Smith | d4a19a3 | 2015-04-21 18:24:23 +0000 | [diff] [blame] | 1269 | assert(DIVar && "Missing variable"); |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1270 | auto *DIExpr = DII->getExpression(); |
David Blaikie | 441cfee | 2017-05-15 21:34:01 +0000 | [diff] [blame] | 1271 | Value *DV = SI->getOperand(0); |
Devang Patel | 8c0b16b | 2011-03-17 21:58:19 +0000 | [diff] [blame] | 1272 | |
Bjorn Pettersson | 428caf9 | 2018-06-15 13:48:55 +0000 | [diff] [blame] | 1273 | if (!valueCoversEntireFragment(SI->getValueOperand()->getType(), DII)) { |
| 1274 | // FIXME: If storing to a part of the variable described by the dbg.declare, |
| 1275 | // then we want to insert a dbg.value for the corresponding fragment. |
| 1276 | LLVM_DEBUG(dbgs() << "Failed to convert dbg.declare to dbg.value: " |
| 1277 | << *DII << '\n'); |
| 1278 | // For now, when there is a store to parts of the variable (but we do not |
| 1279 | // know which part) we insert an dbg.value instrinsic to indicate that we |
| 1280 | // know nothing about the variable's content. |
| 1281 | DV = UndefValue::get(DV->getType()); |
| 1282 | if (!LdStHasDebugValue(DIVar, DIExpr, SI)) |
| 1283 | Builder.insertDbgValueIntrinsic(DV, DIVar, DIExpr, DII->getDebugLoc(), |
| 1284 | SI); |
| 1285 | return; |
| 1286 | } |
| 1287 | |
Devang Patel | 8e60ff1 | 2011-05-16 21:24:05 +0000 | [diff] [blame] | 1288 | // If an argument is zero extended then use argument directly. The ZExt |
| 1289 | // may be zapped by an optimization pass in future. |
Craig Topper | f40110f | 2014-04-25 05:29:35 +0000 | [diff] [blame] | 1290 | Argument *ExtendedArg = nullptr; |
Devang Patel | 8e60ff1 | 2011-05-16 21:24:05 +0000 | [diff] [blame] | 1291 | if (ZExtInst *ZExt = dyn_cast<ZExtInst>(SI->getOperand(0))) |
| 1292 | ExtendedArg = dyn_cast<Argument>(ZExt->getOperand(0)); |
| 1293 | if (SExtInst *SExt = dyn_cast<SExtInst>(SI->getOperand(0))) |
| 1294 | ExtendedArg = dyn_cast<Argument>(SExt->getOperand(0)); |
Keno Fischer | 9aae445 | 2016-01-12 22:46:09 +0000 | [diff] [blame] | 1295 | if (ExtendedArg) { |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1296 | // If this DII was already describing only a fragment of a variable, ensure |
David Blaikie | 441cfee | 2017-05-15 21:34:01 +0000 | [diff] [blame] | 1297 | // that fragment is appropriately narrowed here. |
| 1298 | // But if a fragment wasn't used, describe the value as the original |
| 1299 | // argument (rather than the zext or sext) so that it remains described even |
| 1300 | // if the sext/zext is optimized away. This widens the variable description, |
| 1301 | // leaving it up to the consumer to know how the smaller value may be |
| 1302 | // represented in a larger register. |
| 1303 | if (auto Fragment = DIExpr->getFragmentInfo()) { |
| 1304 | unsigned FragmentOffset = Fragment->OffsetInBits; |
| 1305 | SmallVector<uint64_t, 3> Ops(DIExpr->elements_begin(), |
| 1306 | DIExpr->elements_end() - 3); |
| 1307 | Ops.push_back(dwarf::DW_OP_LLVM_fragment); |
| 1308 | Ops.push_back(FragmentOffset); |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1309 | const DataLayout &DL = DII->getModule()->getDataLayout(); |
David Blaikie | 441cfee | 2017-05-15 21:34:01 +0000 | [diff] [blame] | 1310 | Ops.push_back(DL.getTypeSizeInBits(ExtendedArg->getType())); |
| 1311 | DIExpr = Builder.createExpression(Ops); |
Keno Fischer | 9aae445 | 2016-01-12 22:46:09 +0000 | [diff] [blame] | 1312 | } |
David Blaikie | 441cfee | 2017-05-15 21:34:01 +0000 | [diff] [blame] | 1313 | DV = ExtendedArg; |
| 1314 | } |
| 1315 | if (!LdStHasDebugValue(DIVar, DIExpr, SI)) |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1316 | Builder.insertDbgValueIntrinsic(DV, DIVar, DIExpr, DII->getDebugLoc(), |
David Blaikie | 441cfee | 2017-05-15 21:34:01 +0000 | [diff] [blame] | 1317 | SI); |
Devang Patel | 8c0b16b | 2011-03-17 21:58:19 +0000 | [diff] [blame] | 1318 | } |
| 1319 | |
Adrian Prantl | d00333a | 2013-04-26 18:10:50 +0000 | [diff] [blame] | 1320 | /// Inserts a llvm.dbg.value intrinsic before a load of an alloca'd value |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1321 | /// that has an associated llvm.dbg.declare or llvm.dbg.addr intrinsic. |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1322 | void llvm::ConvertDebugDeclareToDebugValue(DbgVariableIntrinsic *DII, |
Devang Patel | 2c7ee27 | 2011-03-18 23:45:43 +0000 | [diff] [blame] | 1323 | LoadInst *LI, DIBuilder &Builder) { |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1324 | auto *DIVar = DII->getVariable(); |
| 1325 | auto *DIExpr = DII->getExpression(); |
Duncan P. N. Exon Smith | d4a19a3 | 2015-04-21 18:24:23 +0000 | [diff] [blame] | 1326 | assert(DIVar && "Missing variable"); |
Devang Patel | 2c7ee27 | 2011-03-18 23:45:43 +0000 | [diff] [blame] | 1327 | |
Adrian Prantl | a5b2a64 | 2016-02-17 20:02:25 +0000 | [diff] [blame] | 1328 | if (LdStHasDebugValue(DIVar, DIExpr, LI)) |
Keith Walker | ba15989 | 2016-09-22 14:13:25 +0000 | [diff] [blame] | 1329 | return; |
Adrian Prantl | 29b9de7 | 2013-04-26 17:48:33 +0000 | [diff] [blame] | 1330 | |
Bjorn Pettersson | 550517b | 2018-06-26 06:17:00 +0000 | [diff] [blame] | 1331 | if (!valueCoversEntireFragment(LI->getType(), DII)) { |
| 1332 | // FIXME: If only referring to a part of the variable described by the |
| 1333 | // dbg.declare, then we want to insert a dbg.value for the corresponding |
| 1334 | // fragment. |
| 1335 | LLVM_DEBUG(dbgs() << "Failed to convert dbg.declare to dbg.value: " |
| 1336 | << *DII << '\n'); |
| 1337 | return; |
| 1338 | } |
| 1339 | |
Keno Fischer | 00cbf9a | 2015-12-19 02:02:44 +0000 | [diff] [blame] | 1340 | // We are now tracking the loaded value instead of the address. In the |
| 1341 | // future if multi-location support is added to the IR, it might be |
| 1342 | // preferable to keep tracking both the loaded value and the original |
| 1343 | // address in case the alloca can not be elided. |
| 1344 | Instruction *DbgValue = Builder.insertDbgValueIntrinsic( |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1345 | LI, DIVar, DIExpr, DII->getDebugLoc(), (Instruction *)nullptr); |
Keno Fischer | 00cbf9a | 2015-12-19 02:02:44 +0000 | [diff] [blame] | 1346 | DbgValue->insertAfter(LI); |
Keith Walker | ba15989 | 2016-09-22 14:13:25 +0000 | [diff] [blame] | 1347 | } |
| 1348 | |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1349 | /// Inserts a llvm.dbg.value intrinsic after a phi that has an associated |
| 1350 | /// llvm.dbg.declare or llvm.dbg.addr intrinsic. |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1351 | void llvm::ConvertDebugDeclareToDebugValue(DbgVariableIntrinsic *DII, |
Keith Walker | ba15989 | 2016-09-22 14:13:25 +0000 | [diff] [blame] | 1352 | PHINode *APN, DIBuilder &Builder) { |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1353 | auto *DIVar = DII->getVariable(); |
| 1354 | auto *DIExpr = DII->getExpression(); |
Keith Walker | ba15989 | 2016-09-22 14:13:25 +0000 | [diff] [blame] | 1355 | assert(DIVar && "Missing variable"); |
| 1356 | |
| 1357 | if (PhiHasDebugValue(DIVar, DIExpr, APN)) |
| 1358 | return; |
| 1359 | |
Bjorn Pettersson | 550517b | 2018-06-26 06:17:00 +0000 | [diff] [blame] | 1360 | if (!valueCoversEntireFragment(APN->getType(), DII)) { |
| 1361 | // FIXME: If only referring to a part of the variable described by the |
| 1362 | // dbg.declare, then we want to insert a dbg.value for the corresponding |
| 1363 | // fragment. |
| 1364 | LLVM_DEBUG(dbgs() << "Failed to convert dbg.declare to dbg.value: " |
| 1365 | << *DII << '\n'); |
| 1366 | return; |
| 1367 | } |
| 1368 | |
Reid Kleckner | 6481822 | 2016-09-27 18:45:31 +0000 | [diff] [blame] | 1369 | BasicBlock *BB = APN->getParent(); |
Keith Walker | ba15989 | 2016-09-22 14:13:25 +0000 | [diff] [blame] | 1370 | auto InsertionPt = BB->getFirstInsertionPt(); |
Reid Kleckner | 6481822 | 2016-09-27 18:45:31 +0000 | [diff] [blame] | 1371 | |
| 1372 | // The block may be a catchswitch block, which does not have a valid |
| 1373 | // insertion point. |
| 1374 | // FIXME: Insert dbg.value markers in the successors when appropriate. |
| 1375 | if (InsertionPt != BB->end()) |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1376 | Builder.insertDbgValueIntrinsic(APN, DIVar, DIExpr, DII->getDebugLoc(), |
Reid Kleckner | 6481822 | 2016-09-27 18:45:31 +0000 | [diff] [blame] | 1377 | &*InsertionPt); |
Keith Walker | c941252 | 2016-09-19 09:49:30 +0000 | [diff] [blame] | 1378 | } |
| 1379 | |
Adrian Prantl | 232897f | 2014-04-25 23:00:25 +0000 | [diff] [blame] | 1380 | /// Determine whether this alloca is either a VLA or an array. |
| 1381 | static bool isArray(AllocaInst *AI) { |
| 1382 | return AI->isArrayAllocation() || |
| 1383 | AI->getType()->getElementType()->isArrayTy(); |
| 1384 | } |
| 1385 | |
Devang Patel | aad34d8 | 2011-03-17 22:18:16 +0000 | [diff] [blame] | 1386 | /// LowerDbgDeclare - Lowers llvm.dbg.declare intrinsics into appropriate set |
| 1387 | /// of llvm.dbg.value intrinsics. |
| 1388 | bool llvm::LowerDbgDeclare(Function &F) { |
Duncan P. N. Exon Smith | 5bf8fef | 2014-12-09 18:38:53 +0000 | [diff] [blame] | 1389 | DIBuilder DIB(*F.getParent(), /*AllowUnresolved*/ false); |
Devang Patel | aad34d8 | 2011-03-17 22:18:16 +0000 | [diff] [blame] | 1390 | SmallVector<DbgDeclareInst *, 4> Dbgs; |
Adrian Prantl | 79c8e8f | 2014-03-27 23:30:04 +0000 | [diff] [blame] | 1391 | for (auto &FI : F) |
Duncan P. N. Exon Smith | 5b4c837 | 2015-10-13 02:39:05 +0000 | [diff] [blame] | 1392 | for (Instruction &BI : FI) |
| 1393 | if (auto DDI = dyn_cast<DbgDeclareInst>(&BI)) |
Devang Patel | aad34d8 | 2011-03-17 22:18:16 +0000 | [diff] [blame] | 1394 | Dbgs.push_back(DDI); |
Adrian Prantl | 79c8e8f | 2014-03-27 23:30:04 +0000 | [diff] [blame] | 1395 | |
Devang Patel | aad34d8 | 2011-03-17 22:18:16 +0000 | [diff] [blame] | 1396 | if (Dbgs.empty()) |
| 1397 | return false; |
| 1398 | |
Adrian Prantl | 79c8e8f | 2014-03-27 23:30:04 +0000 | [diff] [blame] | 1399 | for (auto &I : Dbgs) { |
| 1400 | DbgDeclareInst *DDI = I; |
Adrian Prantl | 8e10fdb | 2013-11-18 23:04:38 +0000 | [diff] [blame] | 1401 | AllocaInst *AI = dyn_cast_or_null<AllocaInst>(DDI->getAddress()); |
| 1402 | // If this is an alloca for a scalar variable, insert a dbg.value |
| 1403 | // at each load and store to the alloca and erase the dbg.declare. |
Adrian Prantl | 32da889 | 2014-04-25 20:49:25 +0000 | [diff] [blame] | 1404 | // The dbg.values allow tracking a variable even if it is not |
| 1405 | // stored on the stack, while the dbg.declare can only describe |
| 1406 | // the stack slot (and at a lexical-scope granularity). Later |
| 1407 | // passes will attempt to elide the stack slot. |
Adrian Prantl | 5b477be | 2018-03-09 00:45:04 +0000 | [diff] [blame] | 1408 | if (!AI || isArray(AI)) |
| 1409 | continue; |
| 1410 | |
| 1411 | // A volatile load/store means that the alloca can't be elided anyway. |
| 1412 | if (llvm::any_of(AI->users(), [](User *U) -> bool { |
| 1413 | if (LoadInst *LI = dyn_cast<LoadInst>(U)) |
| 1414 | return LI->isVolatile(); |
| 1415 | if (StoreInst *SI = dyn_cast<StoreInst>(U)) |
| 1416 | return SI->isVolatile(); |
| 1417 | return false; |
| 1418 | })) |
| 1419 | continue; |
| 1420 | |
| 1421 | for (auto &AIUse : AI->uses()) { |
| 1422 | User *U = AIUse.getUser(); |
| 1423 | if (StoreInst *SI = dyn_cast<StoreInst>(U)) { |
| 1424 | if (AIUse.getOperandNo() == 1) |
| 1425 | ConvertDebugDeclareToDebugValue(DDI, SI, DIB); |
| 1426 | } else if (LoadInst *LI = dyn_cast<LoadInst>(U)) { |
| 1427 | ConvertDebugDeclareToDebugValue(DDI, LI, DIB); |
| 1428 | } else if (CallInst *CI = dyn_cast<CallInst>(U)) { |
Vedant Kumar | b572f64 | 2018-07-26 20:56:53 +0000 | [diff] [blame] | 1429 | // This is a call by-value or some other instruction that takes a |
| 1430 | // pointer to the variable. Insert a *value* intrinsic that describes |
| 1431 | // the variable by dereferencing the alloca. |
| 1432 | auto *DerefExpr = |
| 1433 | DIExpression::append(DDI->getExpression(), dwarf::DW_OP_deref); |
| 1434 | DIB.insertDbgValueIntrinsic(AI, DDI->getVariable(), DerefExpr, |
| 1435 | DDI->getDebugLoc(), CI); |
Keno Fischer | 1dd319f | 2016-01-14 19:12:27 +0000 | [diff] [blame] | 1436 | } |
Devang Patel | aad34d8 | 2011-03-17 22:18:16 +0000 | [diff] [blame] | 1437 | } |
Adrian Prantl | 5b477be | 2018-03-09 00:45:04 +0000 | [diff] [blame] | 1438 | DDI->eraseFromParent(); |
Devang Patel | aad34d8 | 2011-03-17 22:18:16 +0000 | [diff] [blame] | 1439 | } |
| 1440 | return true; |
| 1441 | } |
Cameron Zwarich | 843bc7d | 2011-05-24 03:10:43 +0000 | [diff] [blame] | 1442 | |
Vedant Kumar | 6bfc869 | 2018-01-25 21:37:05 +0000 | [diff] [blame] | 1443 | /// Propagate dbg.value intrinsics through the newly inserted PHIs. |
| 1444 | void llvm::insertDebugValuesForPHIs(BasicBlock *BB, |
| 1445 | SmallVectorImpl<PHINode *> &InsertedPHIs) { |
| 1446 | assert(BB && "No BasicBlock to clone dbg.value(s) from."); |
| 1447 | if (InsertedPHIs.size() == 0) |
| 1448 | return; |
| 1449 | |
| 1450 | // Map existing PHI nodes to their dbg.values. |
| 1451 | ValueToValueMapTy DbgValueMap; |
| 1452 | for (auto &I : *BB) { |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1453 | if (auto DbgII = dyn_cast<DbgVariableIntrinsic>(&I)) { |
Vedant Kumar | 6bfc869 | 2018-01-25 21:37:05 +0000 | [diff] [blame] | 1454 | if (auto *Loc = dyn_cast_or_null<PHINode>(DbgII->getVariableLocation())) |
| 1455 | DbgValueMap.insert({Loc, DbgII}); |
| 1456 | } |
| 1457 | } |
| 1458 | if (DbgValueMap.size() == 0) |
| 1459 | return; |
| 1460 | |
| 1461 | // Then iterate through the new PHIs and look to see if they use one of the |
| 1462 | // previously mapped PHIs. If so, insert a new dbg.value intrinsic that will |
| 1463 | // propagate the info through the new PHI. |
| 1464 | LLVMContext &C = BB->getContext(); |
| 1465 | for (auto PHI : InsertedPHIs) { |
Matt Davis | 523c656 | 2018-02-23 17:38:27 +0000 | [diff] [blame] | 1466 | BasicBlock *Parent = PHI->getParent(); |
| 1467 | // Avoid inserting an intrinsic into an EH block. |
| 1468 | if (Parent->getFirstNonPHI()->isEHPad()) |
| 1469 | continue; |
| 1470 | auto PhiMAV = MetadataAsValue::get(C, ValueAsMetadata::get(PHI)); |
Vedant Kumar | 6bfc869 | 2018-01-25 21:37:05 +0000 | [diff] [blame] | 1471 | for (auto VI : PHI->operand_values()) { |
| 1472 | auto V = DbgValueMap.find(VI); |
| 1473 | if (V != DbgValueMap.end()) { |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1474 | auto *DbgII = cast<DbgVariableIntrinsic>(V->second); |
Vedant Kumar | 6bfc869 | 2018-01-25 21:37:05 +0000 | [diff] [blame] | 1475 | Instruction *NewDbgII = DbgII->clone(); |
Vedant Kumar | 6bfc869 | 2018-01-25 21:37:05 +0000 | [diff] [blame] | 1476 | NewDbgII->setOperand(0, PhiMAV); |
Vedant Kumar | 6394df9 | 2018-01-25 23:48:29 +0000 | [diff] [blame] | 1477 | auto InsertionPt = Parent->getFirstInsertionPt(); |
| 1478 | assert(InsertionPt != Parent->end() && "Ill-formed basic block"); |
| 1479 | NewDbgII->insertBefore(&*InsertionPt); |
Vedant Kumar | 6bfc869 | 2018-01-25 21:37:05 +0000 | [diff] [blame] | 1480 | } |
| 1481 | } |
| 1482 | } |
| 1483 | } |
| 1484 | |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1485 | /// Finds all intrinsics declaring local variables as living in the memory that |
| 1486 | /// 'V' points to. This may include a mix of dbg.declare and |
| 1487 | /// dbg.addr intrinsics. |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1488 | TinyPtrVector<DbgVariableIntrinsic *> llvm::FindDbgAddrUses(Value *V) { |
Vedant Kumar | de46f65 | 2018-06-26 18:44:52 +0000 | [diff] [blame] | 1489 | // This function is hot. Check whether the value has any metadata to avoid a |
| 1490 | // DenseMap lookup. |
| 1491 | if (!V->isUsedByMetadata()) |
| 1492 | return {}; |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1493 | auto *L = LocalAsMetadata::getIfExists(V); |
| 1494 | if (!L) |
| 1495 | return {}; |
| 1496 | auto *MDV = MetadataAsValue::getIfExists(V->getContext(), L); |
| 1497 | if (!MDV) |
| 1498 | return {}; |
Cameron Zwarich | 843bc7d | 2011-05-24 03:10:43 +0000 | [diff] [blame] | 1499 | |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1500 | TinyPtrVector<DbgVariableIntrinsic *> Declares; |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1501 | for (User *U : MDV->users()) { |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1502 | if (auto *DII = dyn_cast<DbgVariableIntrinsic>(U)) |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1503 | if (DII->isAddressOfVariable()) |
| 1504 | Declares.push_back(DII); |
| 1505 | } |
| 1506 | |
| 1507 | return Declares; |
Cameron Zwarich | 843bc7d | 2011-05-24 03:10:43 +0000 | [diff] [blame] | 1508 | } |
Alexey Samsonov | 3d43b63 | 2012-12-12 14:31:53 +0000 | [diff] [blame] | 1509 | |
Adrian Prantl | fa9e84e | 2017-03-16 20:11:54 +0000 | [diff] [blame] | 1510 | void llvm::findDbgValues(SmallVectorImpl<DbgValueInst *> &DbgValues, Value *V) { |
Vedant Kumar | de46f65 | 2018-06-26 18:44:52 +0000 | [diff] [blame] | 1511 | // This function is hot. Check whether the value has any metadata to avoid a |
| 1512 | // DenseMap lookup. |
| 1513 | if (!V->isUsedByMetadata()) |
| 1514 | return; |
Keith Walker | ba15989 | 2016-09-22 14:13:25 +0000 | [diff] [blame] | 1515 | if (auto *L = LocalAsMetadata::getIfExists(V)) |
| 1516 | if (auto *MDV = MetadataAsValue::getIfExists(V->getContext(), L)) |
| 1517 | for (User *U : MDV->users()) |
| 1518 | if (DbgValueInst *DVI = dyn_cast<DbgValueInst>(U)) |
Adrian Prantl | fa9e84e | 2017-03-16 20:11:54 +0000 | [diff] [blame] | 1519 | DbgValues.push_back(DVI); |
Keith Walker | ba15989 | 2016-09-22 14:13:25 +0000 | [diff] [blame] | 1520 | } |
| 1521 | |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1522 | void llvm::findDbgUsers(SmallVectorImpl<DbgVariableIntrinsic *> &DbgUsers, |
Vedant Kumar | b2ec02b | 2018-01-05 23:27:02 +0000 | [diff] [blame] | 1523 | Value *V) { |
Vedant Kumar | de46f65 | 2018-06-26 18:44:52 +0000 | [diff] [blame] | 1524 | // This function is hot. Check whether the value has any metadata to avoid a |
| 1525 | // DenseMap lookup. |
| 1526 | if (!V->isUsedByMetadata()) |
| 1527 | return; |
Reid Kleckner | 29a5c03 | 2017-11-14 21:49:06 +0000 | [diff] [blame] | 1528 | if (auto *L = LocalAsMetadata::getIfExists(V)) |
| 1529 | if (auto *MDV = MetadataAsValue::getIfExists(V->getContext(), L)) |
| 1530 | for (User *U : MDV->users()) |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1531 | if (DbgVariableIntrinsic *DII = dyn_cast<DbgVariableIntrinsic>(U)) |
Reid Kleckner | 29a5c03 | 2017-11-14 21:49:06 +0000 | [diff] [blame] | 1532 | DbgUsers.push_back(DII); |
| 1533 | } |
| 1534 | |
Evgeniy Stepanov | 42f3b12 | 2015-12-01 00:40:05 +0000 | [diff] [blame] | 1535 | bool llvm::replaceDbgDeclare(Value *Address, Value *NewAddress, |
| 1536 | Instruction *InsertBefore, DIBuilder &Builder, |
Adrian Prantl | d131701 | 2017-12-08 21:58:18 +0000 | [diff] [blame] | 1537 | bool DerefBefore, int Offset, bool DerefAfter) { |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1538 | auto DbgAddrs = FindDbgAddrUses(Address); |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1539 | for (DbgVariableIntrinsic *DII : DbgAddrs) { |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1540 | DebugLoc Loc = DII->getDebugLoc(); |
| 1541 | auto *DIVar = DII->getVariable(); |
| 1542 | auto *DIExpr = DII->getExpression(); |
| 1543 | assert(DIVar && "Missing variable"); |
Adrian Prantl | d131701 | 2017-12-08 21:58:18 +0000 | [diff] [blame] | 1544 | DIExpr = DIExpression::prepend(DIExpr, DerefBefore, Offset, DerefAfter); |
Vedant Kumar | d37e078 | 2018-07-13 22:39:31 +0000 | [diff] [blame] | 1545 | // Insert llvm.dbg.declare immediately before InsertBefore, and remove old |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1546 | // llvm.dbg.declare. |
| 1547 | Builder.insertDeclare(NewAddress, DIVar, DIExpr, Loc, InsertBefore); |
| 1548 | if (DII == InsertBefore) |
Vedant Kumar | d37e078 | 2018-07-13 22:39:31 +0000 | [diff] [blame] | 1549 | InsertBefore = InsertBefore->getNextNode(); |
Reid Kleckner | 0fe506b | 2017-09-21 19:52:03 +0000 | [diff] [blame] | 1550 | DII->eraseFromParent(); |
| 1551 | } |
| 1552 | return !DbgAddrs.empty(); |
Alexey Samsonov | 3d43b63 | 2012-12-12 14:31:53 +0000 | [diff] [blame] | 1553 | } |
Evgeniy Stepanov | 4fbc0d08 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 1554 | |
Evgeniy Stepanov | 42f3b12 | 2015-12-01 00:40:05 +0000 | [diff] [blame] | 1555 | bool llvm::replaceDbgDeclareForAlloca(AllocaInst *AI, Value *NewAllocaAddress, |
Adrian Prantl | d131701 | 2017-12-08 21:58:18 +0000 | [diff] [blame] | 1556 | DIBuilder &Builder, bool DerefBefore, |
| 1557 | int Offset, bool DerefAfter) { |
Evgeniy Stepanov | 42f3b12 | 2015-12-01 00:40:05 +0000 | [diff] [blame] | 1558 | return replaceDbgDeclare(AI, NewAllocaAddress, AI->getNextNode(), Builder, |
Adrian Prantl | d131701 | 2017-12-08 21:58:18 +0000 | [diff] [blame] | 1559 | DerefBefore, Offset, DerefAfter); |
Evgeniy Stepanov | 42f3b12 | 2015-12-01 00:40:05 +0000 | [diff] [blame] | 1560 | } |
| 1561 | |
Evgeniy Stepanov | 72d961a | 2016-06-16 22:34:00 +0000 | [diff] [blame] | 1562 | static void replaceOneDbgValueForAlloca(DbgValueInst *DVI, Value *NewAddress, |
| 1563 | DIBuilder &Builder, int Offset) { |
| 1564 | DebugLoc Loc = DVI->getDebugLoc(); |
| 1565 | auto *DIVar = DVI->getVariable(); |
| 1566 | auto *DIExpr = DVI->getExpression(); |
| 1567 | assert(DIVar && "Missing variable"); |
| 1568 | |
| 1569 | // This is an alloca-based llvm.dbg.value. The first thing it should do with |
| 1570 | // the alloca pointer is dereference it. Otherwise we don't know how to handle |
| 1571 | // it and give up. |
| 1572 | if (!DIExpr || DIExpr->getNumElements() < 1 || |
| 1573 | DIExpr->getElement(0) != dwarf::DW_OP_deref) |
| 1574 | return; |
| 1575 | |
| 1576 | // Insert the offset immediately after the first deref. |
| 1577 | // We could just change the offset argument of dbg.value, but it's unsigned... |
| 1578 | if (Offset) { |
Adrian Prantl | 47ea647 | 2017-03-16 21:14:09 +0000 | [diff] [blame] | 1579 | SmallVector<uint64_t, 4> Ops; |
| 1580 | Ops.push_back(dwarf::DW_OP_deref); |
Andrew Ng | 03e35b6 | 2017-04-28 08:44:30 +0000 | [diff] [blame] | 1581 | DIExpression::appendOffset(Ops, Offset); |
Adrian Prantl | 47ea647 | 2017-03-16 21:14:09 +0000 | [diff] [blame] | 1582 | Ops.append(DIExpr->elements_begin() + 1, DIExpr->elements_end()); |
| 1583 | DIExpr = Builder.createExpression(Ops); |
Evgeniy Stepanov | 72d961a | 2016-06-16 22:34:00 +0000 | [diff] [blame] | 1584 | } |
| 1585 | |
Adrian Prantl | abe0475 | 2017-07-28 20:21:02 +0000 | [diff] [blame] | 1586 | Builder.insertDbgValueIntrinsic(NewAddress, DIVar, DIExpr, Loc, DVI); |
Evgeniy Stepanov | 72d961a | 2016-06-16 22:34:00 +0000 | [diff] [blame] | 1587 | DVI->eraseFromParent(); |
| 1588 | } |
| 1589 | |
| 1590 | void llvm::replaceDbgValueForAlloca(AllocaInst *AI, Value *NewAllocaAddress, |
| 1591 | DIBuilder &Builder, int Offset) { |
| 1592 | if (auto *L = LocalAsMetadata::getIfExists(AI)) |
| 1593 | if (auto *MDV = MetadataAsValue::getIfExists(AI->getContext(), L)) |
| 1594 | for (auto UI = MDV->use_begin(), UE = MDV->use_end(); UI != UE;) { |
| 1595 | Use &U = *UI++; |
| 1596 | if (auto *DVI = dyn_cast<DbgValueInst>(U.getUser())) |
| 1597 | replaceOneDbgValueForAlloca(DVI, NewAllocaAddress, Builder, Offset); |
| 1598 | } |
| 1599 | } |
| 1600 | |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1601 | /// Wrap \p V in a ValueAsMetadata instance. |
| 1602 | static MetadataAsValue *wrapValueInMetadata(LLVMContext &C, Value *V) { |
| 1603 | return MetadataAsValue::get(C, ValueAsMetadata::get(V)); |
| 1604 | } |
| 1605 | |
| 1606 | bool llvm::salvageDebugInfo(Instruction &I) { |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1607 | SmallVector<DbgVariableIntrinsic *, 1> DbgUsers; |
Vedant Kumar | b2ec02b | 2018-01-05 23:27:02 +0000 | [diff] [blame] | 1608 | findDbgUsers(DbgUsers, &I); |
| 1609 | if (DbgUsers.empty()) |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1610 | return false; |
Vedant Kumar | b2ec02b | 2018-01-05 23:27:02 +0000 | [diff] [blame] | 1611 | |
Adrian Prantl | 47ea647 | 2017-03-16 21:14:09 +0000 | [diff] [blame] | 1612 | auto &M = *I.getModule(); |
Vedant Kumar | 044b588 | 2018-02-15 19:13:03 +0000 | [diff] [blame] | 1613 | auto &DL = M.getDataLayout(); |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1614 | auto &Ctx = I.getContext(); |
| 1615 | auto wrapMD = [&](Value *V) { return wrapValueInMetadata(Ctx, V); }; |
Adrian Prantl | 47ea647 | 2017-03-16 21:14:09 +0000 | [diff] [blame] | 1616 | |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1617 | auto doSalvage = [&](DbgVariableIntrinsic *DII, SmallVectorImpl<uint64_t> &Ops) { |
Vedant Kumar | b2ec02b | 2018-01-05 23:27:02 +0000 | [diff] [blame] | 1618 | auto *DIExpr = DII->getExpression(); |
Bjorn Pettersson | 8dd6cf7 | 2018-07-03 11:29:00 +0000 | [diff] [blame] | 1619 | if (!Ops.empty()) { |
| 1620 | // Do not add DW_OP_stack_value for DbgDeclare and DbgAddr, because they |
| 1621 | // are implicitly pointing out the value as a DWARF memory location |
| 1622 | // description. |
| 1623 | bool WithStackValue = isa<DbgValueInst>(DII); |
| 1624 | DIExpr = DIExpression::prependOpcodes(DIExpr, Ops, WithStackValue); |
| 1625 | } |
Vedant Kumar | b2ec02b | 2018-01-05 23:27:02 +0000 | [diff] [blame] | 1626 | DII->setOperand(0, wrapMD(I.getOperand(0))); |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1627 | DII->setOperand(2, MetadataAsValue::get(Ctx, DIExpr)); |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 1628 | LLVM_DEBUG(dbgs() << "SALVAGE: " << *DII << '\n'); |
Adrian Prantl | 182f9fe | 2017-11-06 22:49:39 +0000 | [diff] [blame] | 1629 | }; |
| 1630 | |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1631 | auto applyOffset = [&](DbgVariableIntrinsic *DII, uint64_t Offset) { |
Vedant Kumar | 04386d8 | 2018-02-09 19:19:55 +0000 | [diff] [blame] | 1632 | SmallVector<uint64_t, 8> Ops; |
| 1633 | DIExpression::appendOffset(Ops, Offset); |
| 1634 | doSalvage(DII, Ops); |
| 1635 | }; |
| 1636 | |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1637 | auto applyOps = [&](DbgVariableIntrinsic *DII, |
Vedant Kumar | 04386d8 | 2018-02-09 19:19:55 +0000 | [diff] [blame] | 1638 | std::initializer_list<uint64_t> Opcodes) { |
| 1639 | SmallVector<uint64_t, 8> Ops(Opcodes); |
| 1640 | doSalvage(DII, Ops); |
| 1641 | }; |
| 1642 | |
Vedant Kumar | 388fac5 | 2018-02-13 03:34:23 +0000 | [diff] [blame] | 1643 | if (auto *CI = dyn_cast<CastInst>(&I)) { |
Vedant Kumar | 044b588 | 2018-02-15 19:13:03 +0000 | [diff] [blame] | 1644 | if (!CI->isNoopCast(DL)) |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1645 | return false; |
Vedant Kumar | 388fac5 | 2018-02-13 03:34:23 +0000 | [diff] [blame] | 1646 | |
| 1647 | // No-op casts are irrelevant for debug info. |
| 1648 | MetadataAsValue *CastSrc = wrapMD(I.getOperand(0)); |
Reid Kleckner | 29a5c03 | 2017-11-14 21:49:06 +0000 | [diff] [blame] | 1649 | for (auto *DII : DbgUsers) { |
Vedant Kumar | 388fac5 | 2018-02-13 03:34:23 +0000 | [diff] [blame] | 1650 | DII->setOperand(0, CastSrc); |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 1651 | LLVM_DEBUG(dbgs() << "SALVAGE: " << *DII << '\n'); |
Adrian Prantl | 47ea647 | 2017-03-16 21:14:09 +0000 | [diff] [blame] | 1652 | } |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1653 | return true; |
Adrian Prantl | 47ea647 | 2017-03-16 21:14:09 +0000 | [diff] [blame] | 1654 | } else if (auto *GEP = dyn_cast<GetElementPtrInst>(&I)) { |
Vedant Kumar | b2ec02b | 2018-01-05 23:27:02 +0000 | [diff] [blame] | 1655 | unsigned BitWidth = |
Elena Demikhovsky | 945b7e5 | 2018-02-14 06:58:08 +0000 | [diff] [blame] | 1656 | M.getDataLayout().getIndexSizeInBits(GEP->getPointerAddressSpace()); |
Vedant Kumar | b2ec02b | 2018-01-05 23:27:02 +0000 | [diff] [blame] | 1657 | // Rewrite a constant GEP into a DIExpression. Since we are performing |
| 1658 | // arithmetic to compute the variable's *value* in the DIExpression, we |
| 1659 | // need to mark the expression with a DW_OP_stack_value. |
| 1660 | APInt Offset(BitWidth, 0); |
| 1661 | if (GEP->accumulateConstantOffset(M.getDataLayout(), Offset)) |
| 1662 | for (auto *DII : DbgUsers) |
| 1663 | applyOffset(DII, Offset.getSExtValue()); |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1664 | return true; |
Adrian Prantl | 182f9fe | 2017-11-06 22:49:39 +0000 | [diff] [blame] | 1665 | } else if (auto *BI = dyn_cast<BinaryOperator>(&I)) { |
Vedant Kumar | 044b588 | 2018-02-15 19:13:03 +0000 | [diff] [blame] | 1666 | // Rewrite binary operations with constant integer operands. |
Vedant Kumar | 96b7dc0 | 2018-02-13 01:09:46 +0000 | [diff] [blame] | 1667 | auto *ConstInt = dyn_cast<ConstantInt>(I.getOperand(1)); |
| 1668 | if (!ConstInt || ConstInt->getBitWidth() > 64) |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1669 | return false; |
Vedant Kumar | 96b7dc0 | 2018-02-13 01:09:46 +0000 | [diff] [blame] | 1670 | |
| 1671 | uint64_t Val = ConstInt->getSExtValue(); |
| 1672 | for (auto *DII : DbgUsers) { |
| 1673 | switch (BI->getOpcode()) { |
| 1674 | case Instruction::Add: |
| 1675 | applyOffset(DII, Val); |
| 1676 | break; |
Vedant Kumar | 47b16c4 | 2018-02-13 01:09:47 +0000 | [diff] [blame] | 1677 | case Instruction::Sub: |
| 1678 | applyOffset(DII, -int64_t(Val)); |
| 1679 | break; |
Vedant Kumar | 4011c26 | 2018-02-13 01:09:52 +0000 | [diff] [blame] | 1680 | case Instruction::Mul: |
| 1681 | applyOps(DII, {dwarf::DW_OP_constu, Val, dwarf::DW_OP_mul}); |
| 1682 | break; |
| 1683 | case Instruction::SDiv: |
| 1684 | applyOps(DII, {dwarf::DW_OP_constu, Val, dwarf::DW_OP_div}); |
| 1685 | break; |
| 1686 | case Instruction::SRem: |
| 1687 | applyOps(DII, {dwarf::DW_OP_constu, Val, dwarf::DW_OP_mod}); |
| 1688 | break; |
Vedant Kumar | 96b7dc0 | 2018-02-13 01:09:46 +0000 | [diff] [blame] | 1689 | case Instruction::Or: |
| 1690 | applyOps(DII, {dwarf::DW_OP_constu, Val, dwarf::DW_OP_or}); |
| 1691 | break; |
Petar Jovanovic | 1768957 | 2018-02-14 13:10:35 +0000 | [diff] [blame] | 1692 | case Instruction::And: |
| 1693 | applyOps(DII, {dwarf::DW_OP_constu, Val, dwarf::DW_OP_and}); |
| 1694 | break; |
Vedant Kumar | 96b7dc0 | 2018-02-13 01:09:46 +0000 | [diff] [blame] | 1695 | case Instruction::Xor: |
| 1696 | applyOps(DII, {dwarf::DW_OP_constu, Val, dwarf::DW_OP_xor}); |
| 1697 | break; |
Vedant Kumar | 31ec356 | 2018-02-13 01:09:49 +0000 | [diff] [blame] | 1698 | case Instruction::Shl: |
| 1699 | applyOps(DII, {dwarf::DW_OP_constu, Val, dwarf::DW_OP_shl}); |
| 1700 | break; |
| 1701 | case Instruction::LShr: |
| 1702 | applyOps(DII, {dwarf::DW_OP_constu, Val, dwarf::DW_OP_shr}); |
| 1703 | break; |
| 1704 | case Instruction::AShr: |
| 1705 | applyOps(DII, {dwarf::DW_OP_constu, Val, dwarf::DW_OP_shra}); |
| 1706 | break; |
Vedant Kumar | 96b7dc0 | 2018-02-13 01:09:46 +0000 | [diff] [blame] | 1707 | default: |
| 1708 | // TODO: Salvage constants from each kind of binop we know about. |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1709 | return false; |
Vedant Kumar | 96b7dc0 | 2018-02-13 01:09:46 +0000 | [diff] [blame] | 1710 | } |
| 1711 | } |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1712 | return true; |
Adrian Prantl | 6d80a26 | 2017-03-20 16:39:41 +0000 | [diff] [blame] | 1713 | } else if (isa<LoadInst>(&I)) { |
Vedant Kumar | b2ec02b | 2018-01-05 23:27:02 +0000 | [diff] [blame] | 1714 | MetadataAsValue *AddrMD = wrapMD(I.getOperand(0)); |
| 1715 | for (auto *DII : DbgUsers) { |
Adrian Prantl | 47ea647 | 2017-03-16 21:14:09 +0000 | [diff] [blame] | 1716 | // Rewrite the load into DW_OP_deref. |
Vedant Kumar | b2ec02b | 2018-01-05 23:27:02 +0000 | [diff] [blame] | 1717 | auto *DIExpr = DII->getExpression(); |
Adrian Prantl | 109b236 | 2017-04-28 17:51:05 +0000 | [diff] [blame] | 1718 | DIExpr = DIExpression::prepend(DIExpr, DIExpression::WithDeref); |
Vedant Kumar | b2ec02b | 2018-01-05 23:27:02 +0000 | [diff] [blame] | 1719 | DII->setOperand(0, AddrMD); |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1720 | DII->setOperand(2, MetadataAsValue::get(Ctx, DIExpr)); |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 1721 | LLVM_DEBUG(dbgs() << "SALVAGE: " << *DII << '\n'); |
Adrian Prantl | 47ea647 | 2017-03-16 21:14:09 +0000 | [diff] [blame] | 1722 | } |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1723 | return true; |
Adrian Prantl | 47ea647 | 2017-03-16 21:14:09 +0000 | [diff] [blame] | 1724 | } |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1725 | return false; |
Adrian Prantl | 47ea647 | 2017-03-16 21:14:09 +0000 | [diff] [blame] | 1726 | } |
| 1727 | |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1728 | /// A replacement for a dbg.value expression. |
| 1729 | using DbgValReplacement = Optional<DIExpression *>; |
| 1730 | |
| 1731 | /// Point debug users of \p From to \p To using exprs given by \p RewriteExpr, |
| 1732 | /// possibly moving/deleting users to prevent use-before-def. Returns true if |
| 1733 | /// changes are made. |
| 1734 | static bool rewriteDebugUsers( |
| 1735 | Instruction &From, Value &To, Instruction &DomPoint, DominatorTree &DT, |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1736 | function_ref<DbgValReplacement(DbgVariableIntrinsic &DII)> RewriteExpr) { |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1737 | // Find debug users of From. |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1738 | SmallVector<DbgVariableIntrinsic *, 1> Users; |
Vedant Kumar | 6fa24b0 | 2018-06-20 16:50:25 +0000 | [diff] [blame] | 1739 | findDbgUsers(Users, &From); |
| 1740 | if (Users.empty()) |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1741 | return false; |
Vedant Kumar | 6fa24b0 | 2018-06-20 16:50:25 +0000 | [diff] [blame] | 1742 | |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1743 | // Prevent use-before-def of To. |
| 1744 | bool Changed = false; |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1745 | SmallPtrSet<DbgVariableIntrinsic *, 1> DeleteOrSalvage; |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1746 | if (isa<Instruction>(&To)) { |
| 1747 | bool DomPointAfterFrom = From.getNextNonDebugInstruction() == &DomPoint; |
| 1748 | |
| 1749 | for (auto *DII : Users) { |
| 1750 | // It's common to see a debug user between From and DomPoint. Move it |
| 1751 | // after DomPoint to preserve the variable update without any reordering. |
| 1752 | if (DomPointAfterFrom && DII->getNextNonDebugInstruction() == &DomPoint) { |
| 1753 | LLVM_DEBUG(dbgs() << "MOVE: " << *DII << '\n'); |
| 1754 | DII->moveAfter(&DomPoint); |
| 1755 | Changed = true; |
| 1756 | |
| 1757 | // Users which otherwise aren't dominated by the replacement value must |
| 1758 | // be salvaged or deleted. |
| 1759 | } else if (!DT.dominates(&DomPoint, DII)) { |
| 1760 | DeleteOrSalvage.insert(DII); |
| 1761 | } |
Vedant Kumar | c85ca4c | 2018-06-26 18:44:53 +0000 | [diff] [blame] | 1762 | } |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1763 | } |
| 1764 | |
| 1765 | // Update debug users without use-before-def risk. |
| 1766 | for (auto *DII : Users) { |
| 1767 | if (DeleteOrSalvage.count(DII)) |
| 1768 | continue; |
| 1769 | |
| 1770 | LLVMContext &Ctx = DII->getContext(); |
| 1771 | DbgValReplacement DVR = RewriteExpr(*DII); |
| 1772 | if (!DVR) |
| 1773 | continue; |
| 1774 | |
| 1775 | DII->setOperand(0, wrapValueInMetadata(Ctx, &To)); |
| 1776 | DII->setOperand(2, MetadataAsValue::get(Ctx, *DVR)); |
| 1777 | LLVM_DEBUG(dbgs() << "REWRITE: " << *DII << '\n'); |
| 1778 | Changed = true; |
| 1779 | } |
| 1780 | |
| 1781 | if (!DeleteOrSalvage.empty()) { |
| 1782 | // Try to salvage the remaining debug users. |
| 1783 | Changed |= salvageDebugInfo(From); |
| 1784 | |
| 1785 | // Delete the debug users which weren't salvaged. |
| 1786 | for (auto *DII : DeleteOrSalvage) { |
| 1787 | if (DII->getVariableLocation() == &From) { |
| 1788 | LLVM_DEBUG(dbgs() << "Erased UseBeforeDef: " << *DII << '\n'); |
| 1789 | DII->eraseFromParent(); |
| 1790 | Changed = true; |
| 1791 | } |
| 1792 | } |
| 1793 | } |
| 1794 | |
| 1795 | return Changed; |
Vedant Kumar | c85ca4c | 2018-06-26 18:44:53 +0000 | [diff] [blame] | 1796 | } |
| 1797 | |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1798 | /// Check if a bitcast between a value of type \p FromTy to type \p ToTy would |
| 1799 | /// losslessly preserve the bits and semantics of the value. This predicate is |
| 1800 | /// symmetric, i.e swapping \p FromTy and \p ToTy should give the same result. |
| 1801 | /// |
| 1802 | /// Note that Type::canLosslesslyBitCastTo is not suitable here because it |
| 1803 | /// allows semantically unequivalent bitcasts, such as <2 x i64> -> <4 x i32>, |
| 1804 | /// and also does not allow lossless pointer <-> integer conversions. |
| 1805 | static bool isBitCastSemanticsPreserving(const DataLayout &DL, Type *FromTy, |
| 1806 | Type *ToTy) { |
| 1807 | // Trivially compatible types. |
| 1808 | if (FromTy == ToTy) |
| 1809 | return true; |
| 1810 | |
| 1811 | // Handle compatible pointer <-> integer conversions. |
| 1812 | if (FromTy->isIntOrPtrTy() && ToTy->isIntOrPtrTy()) { |
| 1813 | bool SameSize = DL.getTypeSizeInBits(FromTy) == DL.getTypeSizeInBits(ToTy); |
| 1814 | bool LosslessConversion = !DL.isNonIntegralPointerType(FromTy) && |
| 1815 | !DL.isNonIntegralPointerType(ToTy); |
| 1816 | return SameSize && LosslessConversion; |
| 1817 | } |
| 1818 | |
| 1819 | // TODO: This is not exhaustive. |
| 1820 | return false; |
| 1821 | } |
| 1822 | |
| 1823 | bool llvm::replaceAllDbgUsesWith(Instruction &From, Value &To, |
| 1824 | Instruction &DomPoint, DominatorTree &DT) { |
| 1825 | // Exit early if From has no debug users. |
| 1826 | if (!From.isUsedByMetadata()) |
| 1827 | return false; |
| 1828 | |
| 1829 | assert(&From != &To && "Can't replace something with itself"); |
| 1830 | |
| 1831 | Type *FromTy = From.getType(); |
| 1832 | Type *ToTy = To.getType(); |
| 1833 | |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1834 | auto Identity = [&](DbgVariableIntrinsic &DII) -> DbgValReplacement { |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1835 | return DII.getExpression(); |
| 1836 | }; |
| 1837 | |
| 1838 | // Handle no-op conversions. |
| 1839 | Module &M = *From.getModule(); |
| 1840 | const DataLayout &DL = M.getDataLayout(); |
| 1841 | if (isBitCastSemanticsPreserving(DL, FromTy, ToTy)) |
| 1842 | return rewriteDebugUsers(From, To, DomPoint, DT, Identity); |
| 1843 | |
| 1844 | // Handle integer-to-integer widening and narrowing. |
| 1845 | // FIXME: Use DW_OP_convert when it's available everywhere. |
| 1846 | if (FromTy->isIntegerTy() && ToTy->isIntegerTy()) { |
| 1847 | uint64_t FromBits = FromTy->getPrimitiveSizeInBits(); |
| 1848 | uint64_t ToBits = ToTy->getPrimitiveSizeInBits(); |
| 1849 | assert(FromBits != ToBits && "Unexpected no-op conversion"); |
| 1850 | |
| 1851 | // When the width of the result grows, assume that a debugger will only |
| 1852 | // access the low `FromBits` bits when inspecting the source variable. |
| 1853 | if (FromBits < ToBits) |
| 1854 | return rewriteDebugUsers(From, To, DomPoint, DT, Identity); |
| 1855 | |
| 1856 | // The width of the result has shrunk. Use sign/zero extension to describe |
| 1857 | // the source variable's high bits. |
Hsiangkai Wang | ef72e48 | 2018-08-06 03:59:47 +0000 | [diff] [blame] | 1858 | auto SignOrZeroExt = [&](DbgVariableIntrinsic &DII) -> DbgValReplacement { |
Vedant Kumar | 6379a62 | 2018-07-06 17:32:39 +0000 | [diff] [blame] | 1859 | DILocalVariable *Var = DII.getVariable(); |
| 1860 | |
| 1861 | // Without knowing signedness, sign/zero extension isn't possible. |
| 1862 | auto Signedness = Var->getSignedness(); |
| 1863 | if (!Signedness) |
| 1864 | return None; |
| 1865 | |
| 1866 | bool Signed = *Signedness == DIBasicType::Signedness::Signed; |
| 1867 | |
| 1868 | if (!Signed) { |
| 1869 | // In the unsigned case, assume that a debugger will initialize the |
| 1870 | // high bits to 0 and do a no-op conversion. |
| 1871 | return Identity(DII); |
| 1872 | } else { |
| 1873 | // In the signed case, the high bits are given by sign extension, i.e: |
| 1874 | // (To >> (ToBits - 1)) * ((2 ^ FromBits) - 1) |
| 1875 | // Calculate the high bits and OR them together with the low bits. |
| 1876 | SmallVector<uint64_t, 8> Ops({dwarf::DW_OP_dup, dwarf::DW_OP_constu, |
| 1877 | (ToBits - 1), dwarf::DW_OP_shr, |
| 1878 | dwarf::DW_OP_lit0, dwarf::DW_OP_not, |
| 1879 | dwarf::DW_OP_mul, dwarf::DW_OP_or}); |
| 1880 | return DIExpression::appendToStack(DII.getExpression(), Ops); |
| 1881 | } |
| 1882 | }; |
| 1883 | return rewriteDebugUsers(From, To, DomPoint, DT, SignOrZeroExt); |
| 1884 | } |
| 1885 | |
| 1886 | // TODO: Floating-point conversions, vectors. |
| 1887 | return false; |
Vedant Kumar | 6fa24b0 | 2018-06-20 16:50:25 +0000 | [diff] [blame] | 1888 | } |
| 1889 | |
David Majnemer | 35c46d3 | 2016-01-24 05:26:18 +0000 | [diff] [blame] | 1890 | unsigned llvm::removeAllNonTerminatorAndEHPadInstructions(BasicBlock *BB) { |
| 1891 | unsigned NumDeadInst = 0; |
| 1892 | // Delete the instructions backwards, as it has a reduced likelihood of |
| 1893 | // having to update as many def-use and use-def chains. |
| 1894 | Instruction *EndInst = BB->getTerminator(); // Last not to be deleted. |
Duncan P. N. Exon Smith | e9bc579 | 2016-02-21 20:39:50 +0000 | [diff] [blame] | 1895 | while (EndInst != &BB->front()) { |
David Majnemer | 35c46d3 | 2016-01-24 05:26:18 +0000 | [diff] [blame] | 1896 | // Delete the next to last instruction. |
| 1897 | Instruction *Inst = &*--EndInst->getIterator(); |
| 1898 | if (!Inst->use_empty() && !Inst->getType()->isTokenTy()) |
| 1899 | Inst->replaceAllUsesWith(UndefValue::get(Inst->getType())); |
| 1900 | if (Inst->isEHPad() || Inst->getType()->isTokenTy()) { |
| 1901 | EndInst = Inst; |
| 1902 | continue; |
| 1903 | } |
| 1904 | if (!isa<DbgInfoIntrinsic>(Inst)) |
| 1905 | ++NumDeadInst; |
| 1906 | Inst->eraseFromParent(); |
| 1907 | } |
| 1908 | return NumDeadInst; |
| 1909 | } |
| 1910 | |
Michael Zolotukhin | 5020c99 | 2016-11-18 21:01:12 +0000 | [diff] [blame] | 1911 | unsigned llvm::changeToUnreachable(Instruction *I, bool UseLLVMTrap, |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 1912 | bool PreserveLCSSA, DomTreeUpdater *DTU) { |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1913 | BasicBlock *BB = I->getParent(); |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 1914 | std::vector <DominatorTree::UpdateType> Updates; |
| 1915 | |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1916 | // Loop over all of the successors, removing BB's entry from any PHI |
| 1917 | // nodes. |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 1918 | if (DTU) |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 1919 | Updates.reserve(BB->getTerminator()->getNumSuccessors()); |
| 1920 | for (BasicBlock *Successor : successors(BB)) { |
Michael Zolotukhin | 5020c99 | 2016-11-18 21:01:12 +0000 | [diff] [blame] | 1921 | Successor->removePredecessor(BB, PreserveLCSSA); |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 1922 | if (DTU) |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 1923 | Updates.push_back({DominatorTree::Delete, BB, Successor}); |
| 1924 | } |
David Majnemer | e14e7bc | 2016-06-25 08:19:55 +0000 | [diff] [blame] | 1925 | // Insert a call to llvm.trap right before this. This turns the undefined |
| 1926 | // behavior into a hard fail instead of falling through into random code. |
| 1927 | if (UseLLVMTrap) { |
| 1928 | Function *TrapFn = |
| 1929 | Intrinsic::getDeclaration(BB->getParent()->getParent(), Intrinsic::trap); |
| 1930 | CallInst *CallTrap = CallInst::Create(TrapFn, "", I); |
| 1931 | CallTrap->setDebugLoc(I->getDebugLoc()); |
| 1932 | } |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1933 | new UnreachableInst(I->getContext(), I); |
| 1934 | |
| 1935 | // All instructions after this are dead. |
David Majnemer | 88542a0 | 2016-01-24 06:26:47 +0000 | [diff] [blame] | 1936 | unsigned NumInstrsRemoved = 0; |
Duncan P. N. Exon Smith | 5b4c837 | 2015-10-13 02:39:05 +0000 | [diff] [blame] | 1937 | BasicBlock::iterator BBI = I->getIterator(), BBE = BB->end(); |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1938 | while (BBI != BBE) { |
| 1939 | if (!BBI->use_empty()) |
| 1940 | BBI->replaceAllUsesWith(UndefValue::get(BBI->getType())); |
| 1941 | BB->getInstList().erase(BBI++); |
David Majnemer | 88542a0 | 2016-01-24 06:26:47 +0000 | [diff] [blame] | 1942 | ++NumInstrsRemoved; |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1943 | } |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 1944 | if (DTU) |
| 1945 | DTU->applyUpdates(Updates, /*ForceRemoveDuplicates*/ true); |
David Majnemer | 88542a0 | 2016-01-24 06:26:47 +0000 | [diff] [blame] | 1946 | return NumInstrsRemoved; |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1947 | } |
| 1948 | |
| 1949 | /// changeToCall - Convert the specified invoke into a normal call. |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 1950 | static void changeToCall(InvokeInst *II, DomTreeUpdater *DTU = nullptr) { |
Sanjoy Das | ccd1456 | 2015-12-10 06:39:02 +0000 | [diff] [blame] | 1951 | SmallVector<Value*, 8> Args(II->arg_begin(), II->arg_end()); |
Sanjoy Das | 8a954a0 | 2015-12-08 22:26:08 +0000 | [diff] [blame] | 1952 | SmallVector<OperandBundleDef, 1> OpBundles; |
| 1953 | II->getOperandBundlesAsDefs(OpBundles); |
| 1954 | CallInst *NewCall = CallInst::Create(II->getCalledValue(), Args, OpBundles, |
| 1955 | "", II); |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1956 | NewCall->takeName(II); |
| 1957 | NewCall->setCallingConv(II->getCallingConv()); |
| 1958 | NewCall->setAttributes(II->getAttributes()); |
| 1959 | NewCall->setDebugLoc(II->getDebugLoc()); |
| 1960 | II->replaceAllUsesWith(NewCall); |
| 1961 | |
| 1962 | // Follow the call by a branch to the normal destination. |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 1963 | BasicBlock *NormalDestBB = II->getNormalDest(); |
| 1964 | BranchInst::Create(NormalDestBB, II); |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1965 | |
| 1966 | // Update PHI nodes in the unwind destination |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 1967 | BasicBlock *BB = II->getParent(); |
| 1968 | BasicBlock *UnwindDestBB = II->getUnwindDest(); |
| 1969 | UnwindDestBB->removePredecessor(BB); |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1970 | II->eraseFromParent(); |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 1971 | if (DTU) |
| 1972 | DTU->deleteEdgeRelaxed(BB, UnwindDestBB); |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 1973 | } |
| 1974 | |
Kuba Brecka | ddfdba3 | 2016-11-14 21:41:13 +0000 | [diff] [blame] | 1975 | BasicBlock *llvm::changeToInvokeAndSplitBasicBlock(CallInst *CI, |
| 1976 | BasicBlock *UnwindEdge) { |
| 1977 | BasicBlock *BB = CI->getParent(); |
| 1978 | |
| 1979 | // Convert this function call into an invoke instruction. First, split the |
| 1980 | // basic block. |
| 1981 | BasicBlock *Split = |
| 1982 | BB->splitBasicBlock(CI->getIterator(), CI->getName() + ".noexc"); |
| 1983 | |
| 1984 | // Delete the unconditional branch inserted by splitBasicBlock |
| 1985 | BB->getInstList().pop_back(); |
| 1986 | |
| 1987 | // Create the new invoke instruction. |
| 1988 | SmallVector<Value *, 8> InvokeArgs(CI->arg_begin(), CI->arg_end()); |
| 1989 | SmallVector<OperandBundleDef, 1> OpBundles; |
| 1990 | |
| 1991 | CI->getOperandBundlesAsDefs(OpBundles); |
| 1992 | |
| 1993 | // Note: we're round tripping operand bundles through memory here, and that |
| 1994 | // can potentially be avoided with a cleverer API design that we do not have |
| 1995 | // as of this time. |
| 1996 | |
| 1997 | InvokeInst *II = InvokeInst::Create(CI->getCalledValue(), Split, UnwindEdge, |
| 1998 | InvokeArgs, OpBundles, CI->getName(), BB); |
| 1999 | II->setDebugLoc(CI->getDebugLoc()); |
| 2000 | II->setCallingConv(CI->getCallingConv()); |
| 2001 | II->setAttributes(CI->getAttributes()); |
| 2002 | |
| 2003 | // Make sure that anything using the call now uses the invoke! This also |
Sanjoy Das | e6bca0e | 2017-05-01 17:07:49 +0000 | [diff] [blame] | 2004 | // updates the CallGraph if present, because it uses a WeakTrackingVH. |
Kuba Brecka | ddfdba3 | 2016-11-14 21:41:13 +0000 | [diff] [blame] | 2005 | CI->replaceAllUsesWith(II); |
| 2006 | |
| 2007 | // Delete the original call |
| 2008 | Split->getInstList().pop_front(); |
| 2009 | return Split; |
| 2010 | } |
| 2011 | |
David Majnemer | 7fddecc | 2015-06-17 20:52:32 +0000 | [diff] [blame] | 2012 | static bool markAliveBlocks(Function &F, |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2013 | SmallPtrSetImpl<BasicBlock *> &Reachable, |
| 2014 | DomTreeUpdater *DTU = nullptr) { |
Evgeniy Stepanov | 4fbc0d08 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 2015 | SmallVector<BasicBlock*, 128> Worklist; |
Duncan P. N. Exon Smith | 5b4c837 | 2015-10-13 02:39:05 +0000 | [diff] [blame] | 2016 | BasicBlock *BB = &F.front(); |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2017 | Worklist.push_back(BB); |
| 2018 | Reachable.insert(BB); |
| 2019 | bool Changed = false; |
Evgeniy Stepanov | 4fbc0d08 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 2020 | do { |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2021 | BB = Worklist.pop_back_val(); |
| 2022 | |
| 2023 | // Do a quick scan of the basic block, turning any obviously unreachable |
| 2024 | // instructions into LLVM unreachable insts. The instruction combining pass |
| 2025 | // canonicalizes unreachable insts into stores to null or undef. |
David Majnemer | 9f50625 | 2016-06-25 08:34:38 +0000 | [diff] [blame] | 2026 | for (Instruction &I : *BB) { |
Xin Tong | 074ccf3 | 2018-07-18 18:40:45 +0000 | [diff] [blame] | 2027 | if (auto *CI = dyn_cast<CallInst>(&I)) { |
| 2028 | Value *Callee = CI->getCalledValue(); |
| 2029 | // Handle intrinsic calls. |
| 2030 | if (Function *F = dyn_cast<Function>(Callee)) { |
| 2031 | auto IntrinsicID = F->getIntrinsicID(); |
| 2032 | // Assumptions that are known to be false are equivalent to |
| 2033 | // unreachable. Also, if the condition is undefined, then we make the |
| 2034 | // choice most beneficial to the optimizer, and choose that to also be |
| 2035 | // unreachable. |
| 2036 | if (IntrinsicID == Intrinsic::assume) { |
| 2037 | if (match(CI->getArgOperand(0), m_CombineOr(m_Zero(), m_Undef()))) { |
| 2038 | // Don't insert a call to llvm.trap right before the unreachable. |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2039 | changeToUnreachable(CI, false, false, DTU); |
Sanjoy Das | 54a3a00 | 2016-04-21 05:09:12 +0000 | [diff] [blame] | 2040 | Changed = true; |
| 2041 | break; |
| 2042 | } |
Xin Tong | 074ccf3 | 2018-07-18 18:40:45 +0000 | [diff] [blame] | 2043 | } else if (IntrinsicID == Intrinsic::experimental_guard) { |
| 2044 | // A call to the guard intrinsic bails out of the current |
| 2045 | // compilation unit if the predicate passed to it is false. If the |
| 2046 | // predicate is a constant false, then we know the guard will bail |
| 2047 | // out of the current compile unconditionally, so all code following |
| 2048 | // it is dead. |
| 2049 | // |
| 2050 | // Note: unlike in llvm.assume, it is not "obviously profitable" for |
| 2051 | // guards to treat `undef` as `false` since a guard on `undef` can |
| 2052 | // still be useful for widening. |
| 2053 | if (match(CI->getArgOperand(0), m_Zero())) |
| 2054 | if (!isa<UnreachableInst>(CI->getNextNode())) { |
| 2055 | changeToUnreachable(CI->getNextNode(), /*UseLLVMTrap=*/false, |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2056 | false, DTU); |
Xin Tong | 074ccf3 | 2018-07-18 18:40:45 +0000 | [diff] [blame] | 2057 | Changed = true; |
| 2058 | break; |
| 2059 | } |
| 2060 | } |
| 2061 | } else if ((isa<ConstantPointerNull>(Callee) && |
| 2062 | !NullPointerIsDefined(CI->getFunction())) || |
| 2063 | isa<UndefValue>(Callee)) { |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2064 | changeToUnreachable(CI, /*UseLLVMTrap=*/false, false, DTU); |
David Majnemer | 1fea77c | 2016-06-25 07:37:27 +0000 | [diff] [blame] | 2065 | Changed = true; |
| 2066 | break; |
| 2067 | } |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2068 | if (CI->doesNotReturn()) { |
| 2069 | // If we found a call to a no-return function, insert an unreachable |
| 2070 | // instruction after it. Make sure there isn't *already* one there |
| 2071 | // though. |
David Majnemer | 9f50625 | 2016-06-25 08:34:38 +0000 | [diff] [blame] | 2072 | if (!isa<UnreachableInst>(CI->getNextNode())) { |
David Majnemer | e14e7bc | 2016-06-25 08:19:55 +0000 | [diff] [blame] | 2073 | // Don't insert a call to llvm.trap right before the unreachable. |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2074 | changeToUnreachable(CI->getNextNode(), false, false, DTU); |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2075 | Changed = true; |
| 2076 | } |
| 2077 | break; |
| 2078 | } |
Xin Tong | 074ccf3 | 2018-07-18 18:40:45 +0000 | [diff] [blame] | 2079 | } else if (auto *SI = dyn_cast<StoreInst>(&I)) { |
| 2080 | // Store to undef and store to null are undefined and used to signal |
| 2081 | // that they should be changed to unreachable by passes that can't |
| 2082 | // modify the CFG. |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2083 | |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2084 | // Don't touch volatile stores. |
| 2085 | if (SI->isVolatile()) continue; |
| 2086 | |
| 2087 | Value *Ptr = SI->getOperand(1); |
| 2088 | |
| 2089 | if (isa<UndefValue>(Ptr) || |
| 2090 | (isa<ConstantPointerNull>(Ptr) && |
Manoj Gupta | 77eeac3 | 2018-07-09 22:27:23 +0000 | [diff] [blame] | 2091 | !NullPointerIsDefined(SI->getFunction(), |
| 2092 | SI->getPointerAddressSpace()))) { |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2093 | changeToUnreachable(SI, true, false, DTU); |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2094 | Changed = true; |
| 2095 | break; |
| 2096 | } |
| 2097 | } |
| 2098 | } |
| 2099 | |
David Majnemer | 2fa8651 | 2016-01-05 06:27:50 +0000 | [diff] [blame] | 2100 | TerminatorInst *Terminator = BB->getTerminator(); |
| 2101 | if (auto *II = dyn_cast<InvokeInst>(Terminator)) { |
| 2102 | // Turn invokes that call 'nounwind' functions into ordinary calls. |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2103 | Value *Callee = II->getCalledValue(); |
Manoj Gupta | 77eeac3 | 2018-07-09 22:27:23 +0000 | [diff] [blame] | 2104 | if ((isa<ConstantPointerNull>(Callee) && |
| 2105 | !NullPointerIsDefined(BB->getParent())) || |
| 2106 | isa<UndefValue>(Callee)) { |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2107 | changeToUnreachable(II, true, false, DTU); |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2108 | Changed = true; |
David Majnemer | 7fddecc | 2015-06-17 20:52:32 +0000 | [diff] [blame] | 2109 | } else if (II->doesNotThrow() && canSimplifyInvokeNoUnwind(&F)) { |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2110 | if (II->use_empty() && II->onlyReadsMemory()) { |
| 2111 | // jump to the normal destination branch. |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 2112 | BasicBlock *NormalDestBB = II->getNormalDest(); |
| 2113 | BasicBlock *UnwindDestBB = II->getUnwindDest(); |
| 2114 | BranchInst::Create(NormalDestBB, II); |
| 2115 | UnwindDestBB->removePredecessor(II->getParent()); |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2116 | II->eraseFromParent(); |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2117 | if (DTU) |
| 2118 | DTU->deleteEdgeRelaxed(BB, UnwindDestBB); |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2119 | } else |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2120 | changeToCall(II, DTU); |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2121 | Changed = true; |
| 2122 | } |
David Majnemer | 2fa8651 | 2016-01-05 06:27:50 +0000 | [diff] [blame] | 2123 | } else if (auto *CatchSwitch = dyn_cast<CatchSwitchInst>(Terminator)) { |
| 2124 | // Remove catchpads which cannot be reached. |
David Majnemer | 59eb733 | 2016-01-05 07:42:17 +0000 | [diff] [blame] | 2125 | struct CatchPadDenseMapInfo { |
| 2126 | static CatchPadInst *getEmptyKey() { |
| 2127 | return DenseMapInfo<CatchPadInst *>::getEmptyKey(); |
| 2128 | } |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 2129 | |
David Majnemer | 59eb733 | 2016-01-05 07:42:17 +0000 | [diff] [blame] | 2130 | static CatchPadInst *getTombstoneKey() { |
| 2131 | return DenseMapInfo<CatchPadInst *>::getTombstoneKey(); |
| 2132 | } |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 2133 | |
David Majnemer | 59eb733 | 2016-01-05 07:42:17 +0000 | [diff] [blame] | 2134 | static unsigned getHashValue(CatchPadInst *CatchPad) { |
| 2135 | return static_cast<unsigned>(hash_combine_range( |
| 2136 | CatchPad->value_op_begin(), CatchPad->value_op_end())); |
| 2137 | } |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 2138 | |
David Majnemer | 59eb733 | 2016-01-05 07:42:17 +0000 | [diff] [blame] | 2139 | static bool isEqual(CatchPadInst *LHS, CatchPadInst *RHS) { |
| 2140 | if (LHS == getEmptyKey() || LHS == getTombstoneKey() || |
| 2141 | RHS == getEmptyKey() || RHS == getTombstoneKey()) |
| 2142 | return LHS == RHS; |
| 2143 | return LHS->isIdenticalTo(RHS); |
| 2144 | } |
| 2145 | }; |
| 2146 | |
| 2147 | // Set of unique CatchPads. |
| 2148 | SmallDenseMap<CatchPadInst *, detail::DenseSetEmpty, 4, |
| 2149 | CatchPadDenseMapInfo, detail::DenseSetPair<CatchPadInst *>> |
| 2150 | HandlerSet; |
| 2151 | detail::DenseSetEmpty Empty; |
David Majnemer | 2fa8651 | 2016-01-05 06:27:50 +0000 | [diff] [blame] | 2152 | for (CatchSwitchInst::handler_iterator I = CatchSwitch->handler_begin(), |
| 2153 | E = CatchSwitch->handler_end(); |
| 2154 | I != E; ++I) { |
| 2155 | BasicBlock *HandlerBB = *I; |
David Majnemer | 59eb733 | 2016-01-05 07:42:17 +0000 | [diff] [blame] | 2156 | auto *CatchPad = cast<CatchPadInst>(HandlerBB->getFirstNonPHI()); |
| 2157 | if (!HandlerSet.insert({CatchPad, Empty}).second) { |
David Majnemer | 2fa8651 | 2016-01-05 06:27:50 +0000 | [diff] [blame] | 2158 | CatchSwitch->removeHandler(I); |
| 2159 | --I; |
| 2160 | --E; |
| 2161 | Changed = true; |
| 2162 | } |
| 2163 | } |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2164 | } |
| 2165 | |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2166 | Changed |= ConstantFoldTerminator(BB, true, nullptr, DTU); |
David Majnemer | 9f50625 | 2016-06-25 08:34:38 +0000 | [diff] [blame] | 2167 | for (BasicBlock *Successor : successors(BB)) |
| 2168 | if (Reachable.insert(Successor).second) |
| 2169 | Worklist.push_back(Successor); |
Evgeniy Stepanov | 4fbc0d08 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 2170 | } while (!Worklist.empty()); |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2171 | return Changed; |
| 2172 | } |
Evgeniy Stepanov | 4fbc0d08 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 2173 | |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2174 | void llvm::removeUnwindEdge(BasicBlock *BB, DomTreeUpdater *DTU) { |
Joseph Tremoulet | 09af67a | 2015-09-27 01:47:46 +0000 | [diff] [blame] | 2175 | TerminatorInst *TI = BB->getTerminator(); |
| 2176 | |
| 2177 | if (auto *II = dyn_cast<InvokeInst>(TI)) { |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2178 | changeToCall(II, DTU); |
Joseph Tremoulet | 09af67a | 2015-09-27 01:47:46 +0000 | [diff] [blame] | 2179 | return; |
| 2180 | } |
| 2181 | |
| 2182 | TerminatorInst *NewTI; |
| 2183 | BasicBlock *UnwindDest; |
| 2184 | |
| 2185 | if (auto *CRI = dyn_cast<CleanupReturnInst>(TI)) { |
| 2186 | NewTI = CleanupReturnInst::Create(CRI->getCleanupPad(), nullptr, CRI); |
| 2187 | UnwindDest = CRI->getUnwindDest(); |
David Majnemer | 8a1c45d | 2015-12-12 05:38:55 +0000 | [diff] [blame] | 2188 | } else if (auto *CatchSwitch = dyn_cast<CatchSwitchInst>(TI)) { |
| 2189 | auto *NewCatchSwitch = CatchSwitchInst::Create( |
| 2190 | CatchSwitch->getParentPad(), nullptr, CatchSwitch->getNumHandlers(), |
| 2191 | CatchSwitch->getName(), CatchSwitch); |
| 2192 | for (BasicBlock *PadBB : CatchSwitch->handlers()) |
| 2193 | NewCatchSwitch->addHandler(PadBB); |
| 2194 | |
| 2195 | NewTI = NewCatchSwitch; |
| 2196 | UnwindDest = CatchSwitch->getUnwindDest(); |
Joseph Tremoulet | 09af67a | 2015-09-27 01:47:46 +0000 | [diff] [blame] | 2197 | } else { |
| 2198 | llvm_unreachable("Could not find unwind successor"); |
| 2199 | } |
| 2200 | |
| 2201 | NewTI->takeName(TI); |
| 2202 | NewTI->setDebugLoc(TI->getDebugLoc()); |
| 2203 | UnwindDest->removePredecessor(BB); |
David Majnemer | 8a1c45d | 2015-12-12 05:38:55 +0000 | [diff] [blame] | 2204 | TI->replaceAllUsesWith(NewTI); |
Joseph Tremoulet | 09af67a | 2015-09-27 01:47:46 +0000 | [diff] [blame] | 2205 | TI->eraseFromParent(); |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2206 | if (DTU) |
| 2207 | DTU->deleteEdgeRelaxed(BB, UnwindDest); |
Joseph Tremoulet | 09af67a | 2015-09-27 01:47:46 +0000 | [diff] [blame] | 2208 | } |
| 2209 | |
Davide Italiano | 4eb210b | 2017-07-07 18:54:14 +0000 | [diff] [blame] | 2210 | /// removeUnreachableBlocks - Remove blocks that are not reachable, even |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2211 | /// if they are in a dead cycle. Return true if a change was made, false |
Davide Italiano | 4eb210b | 2017-07-07 18:54:14 +0000 | [diff] [blame] | 2212 | /// otherwise. If `LVI` is passed, this function preserves LazyValueInfo |
| 2213 | /// after modifying the CFG. |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 2214 | bool llvm::removeUnreachableBlocks(Function &F, LazyValueInfo *LVI, |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2215 | DomTreeUpdater *DTU) { |
Matthias Braun | b30f2f51 | 2016-01-30 01:24:31 +0000 | [diff] [blame] | 2216 | SmallPtrSet<BasicBlock*, 16> Reachable; |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2217 | bool Changed = markAliveBlocks(F, Reachable, DTU); |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2218 | |
| 2219 | // If there are unreachable blocks in the CFG... |
Evgeniy Stepanov | 4fbc0d08 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 2220 | if (Reachable.size() == F.size()) |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2221 | return Changed; |
Evgeniy Stepanov | 4fbc0d08 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 2222 | |
| 2223 | assert(Reachable.size() < F.size()); |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2224 | NumRemoved += F.size()-Reachable.size(); |
| 2225 | |
| 2226 | // Loop over all of the basic blocks that are not reachable, dropping all of |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2227 | // their internal references. Update DTU and LVI if available. |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 2228 | std::vector <DominatorTree::UpdateType> Updates; |
| 2229 | for (Function::iterator I = ++F.begin(), E = F.end(); I != E; ++I) { |
| 2230 | auto *BB = &*I; |
| 2231 | if (Reachable.count(BB)) |
Evgeniy Stepanov | 4fbc0d08 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 2232 | continue; |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 2233 | for (BasicBlock *Successor : successors(BB)) { |
Daniel Jasper | 0a51ec2 | 2017-09-30 11:57:19 +0000 | [diff] [blame] | 2234 | if (Reachable.count(Successor)) |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 2235 | Successor->removePredecessor(BB); |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2236 | if (DTU) |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 2237 | Updates.push_back({DominatorTree::Delete, BB, Successor}); |
| 2238 | } |
David Majnemer | d9833ea | 2016-01-10 07:13:04 +0000 | [diff] [blame] | 2239 | if (LVI) |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 2240 | LVI->eraseBlock(BB); |
Peter Collingbourne | 8d642de | 2013-08-12 22:38:43 +0000 | [diff] [blame] | 2241 | BB->dropAllReferences(); |
Evgeniy Stepanov | 4fbc0d08 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 2242 | } |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2243 | SmallVector<BasicBlock *, 8> ToDeleteBBs; |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 2244 | for (Function::iterator I = ++F.begin(); I != F.end();) { |
| 2245 | auto *BB = &*I; |
| 2246 | if (Reachable.count(BB)) { |
Reid Kleckner | cd78ddc | 2018-01-04 23:23:46 +0000 | [diff] [blame] | 2247 | ++I; |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 2248 | continue; |
| 2249 | } |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2250 | if (DTU) { |
| 2251 | ToDeleteBBs.push_back(BB); |
| 2252 | |
| 2253 | // Remove the TerminatorInst of BB to clear the successor list of BB. |
| 2254 | if (BB->getTerminator()) |
| 2255 | BB->getInstList().pop_back(); |
| 2256 | new UnreachableInst(BB->getContext(), BB); |
| 2257 | assert(succ_empty(BB) && "The successor list of BB isn't empty before " |
| 2258 | "applying corresponding DTU updates."); |
Brian M. Rzycki | 9b7ae23 | 2018-01-12 21:06:48 +0000 | [diff] [blame] | 2259 | ++I; |
| 2260 | } else { |
| 2261 | I = F.getBasicBlockList().erase(I); |
| 2262 | } |
| 2263 | } |
Evgeniy Stepanov | 2a066af | 2013-03-22 08:43:04 +0000 | [diff] [blame] | 2264 | |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2265 | if (DTU) { |
| 2266 | DTU->applyUpdates(Updates, /*ForceRemoveDuplicates*/ true); |
Chijun Sima | 5304843 | 2018-08-03 12:45:29 +0000 | [diff] [blame] | 2267 | bool Deleted = false; |
| 2268 | for (auto *BB : ToDeleteBBs) { |
| 2269 | if (DTU->isBBPendingDeletion(BB)) |
| 2270 | --NumRemoved; |
| 2271 | else |
| 2272 | Deleted = true; |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2273 | DTU->deleteBB(BB); |
Chijun Sima | 5304843 | 2018-08-03 12:45:29 +0000 | [diff] [blame] | 2274 | } |
| 2275 | if (!Deleted) |
| 2276 | return false; |
Chijun Sima | 21a8b60 | 2018-08-03 05:08:17 +0000 | [diff] [blame] | 2277 | } |
Evgeniy Stepanov | 4fbc0d08 | 2012-12-21 11:18:49 +0000 | [diff] [blame] | 2278 | return true; |
| 2279 | } |
Rafael Espindola | ea46c32 | 2014-08-15 15:46:38 +0000 | [diff] [blame] | 2280 | |
Piotr Padlewski | dc9b2cf | 2015-10-02 22:12:22 +0000 | [diff] [blame] | 2281 | void llvm::combineMetadata(Instruction *K, const Instruction *J, |
Florian Hahn | 950576b | 2018-08-07 15:36:11 +0000 | [diff] [blame^] | 2282 | ArrayRef<unsigned> KnownIDs, bool DoesKMove) { |
Duncan P. N. Exon Smith | de36e80 | 2014-11-11 21:30:22 +0000 | [diff] [blame] | 2283 | SmallVector<std::pair<unsigned, MDNode *>, 4> Metadata; |
Adrian Prantl | cbdfdb7 | 2015-08-20 22:00:30 +0000 | [diff] [blame] | 2284 | K->dropUnknownNonDebugMetadata(KnownIDs); |
Rafael Espindola | ea46c32 | 2014-08-15 15:46:38 +0000 | [diff] [blame] | 2285 | K->getAllMetadataOtherThanDebugLoc(Metadata); |
David Majnemer | 6f014d3 | 2016-07-25 02:21:19 +0000 | [diff] [blame] | 2286 | for (const auto &MD : Metadata) { |
| 2287 | unsigned Kind = MD.first; |
Duncan P. N. Exon Smith | de36e80 | 2014-11-11 21:30:22 +0000 | [diff] [blame] | 2288 | MDNode *JMD = J->getMetadata(Kind); |
David Majnemer | 6f014d3 | 2016-07-25 02:21:19 +0000 | [diff] [blame] | 2289 | MDNode *KMD = MD.second; |
Rafael Espindola | ea46c32 | 2014-08-15 15:46:38 +0000 | [diff] [blame] | 2290 | |
| 2291 | switch (Kind) { |
| 2292 | default: |
| 2293 | K->setMetadata(Kind, nullptr); // Remove unknown metadata |
| 2294 | break; |
| 2295 | case LLVMContext::MD_dbg: |
| 2296 | llvm_unreachable("getAllMetadataOtherThanDebugLoc returned a MD_dbg"); |
| 2297 | case LLVMContext::MD_tbaa: |
| 2298 | K->setMetadata(Kind, MDNode::getMostGenericTBAA(JMD, KMD)); |
| 2299 | break; |
| 2300 | case LLVMContext::MD_alias_scope: |
Bjorn Steinbrink | 5ec7522 | 2015-02-08 17:07:14 +0000 | [diff] [blame] | 2301 | K->setMetadata(Kind, MDNode::getMostGenericAliasScope(JMD, KMD)); |
| 2302 | break; |
Rafael Espindola | ea46c32 | 2014-08-15 15:46:38 +0000 | [diff] [blame] | 2303 | case LLVMContext::MD_noalias: |
Hal Finkel | e4c0c16 | 2016-04-26 02:06:06 +0000 | [diff] [blame] | 2304 | case LLVMContext::MD_mem_parallel_loop_access: |
Rafael Espindola | ea46c32 | 2014-08-15 15:46:38 +0000 | [diff] [blame] | 2305 | K->setMetadata(Kind, MDNode::intersect(JMD, KMD)); |
| 2306 | break; |
| 2307 | case LLVMContext::MD_range: |
| 2308 | K->setMetadata(Kind, MDNode::getMostGenericRange(JMD, KMD)); |
| 2309 | break; |
| 2310 | case LLVMContext::MD_fpmath: |
| 2311 | K->setMetadata(Kind, MDNode::getMostGenericFPMath(JMD, KMD)); |
| 2312 | break; |
| 2313 | case LLVMContext::MD_invariant_load: |
| 2314 | // Only set the !invariant.load if it is present in both instructions. |
| 2315 | K->setMetadata(Kind, JMD); |
| 2316 | break; |
Philip Reames | d7c2136 | 2014-10-21 21:02:19 +0000 | [diff] [blame] | 2317 | case LLVMContext::MD_nonnull: |
Florian Hahn | 950576b | 2018-08-07 15:36:11 +0000 | [diff] [blame^] | 2318 | // If K does move, keep nonull if it is present in both instructions. |
| 2319 | if (DoesKMove) |
| 2320 | K->setMetadata(Kind, JMD); |
Philip Reames | d7c2136 | 2014-10-21 21:02:19 +0000 | [diff] [blame] | 2321 | break; |
Piotr Padlewski | dc9b2cf | 2015-10-02 22:12:22 +0000 | [diff] [blame] | 2322 | case LLVMContext::MD_invariant_group: |
| 2323 | // Preserve !invariant.group in K. |
| 2324 | break; |
Artur Pilipenko | 5c5011d | 2015-11-02 17:53:51 +0000 | [diff] [blame] | 2325 | case LLVMContext::MD_align: |
Chandler Carruth | 2abb65a | 2017-06-26 03:31:31 +0000 | [diff] [blame] | 2326 | K->setMetadata(Kind, |
Artur Pilipenko | 5c5011d | 2015-11-02 17:53:51 +0000 | [diff] [blame] | 2327 | MDNode::getMostGenericAlignmentOrDereferenceable(JMD, KMD)); |
| 2328 | break; |
| 2329 | case LLVMContext::MD_dereferenceable: |
| 2330 | case LLVMContext::MD_dereferenceable_or_null: |
Chandler Carruth | 2abb65a | 2017-06-26 03:31:31 +0000 | [diff] [blame] | 2331 | K->setMetadata(Kind, |
Artur Pilipenko | 5c5011d | 2015-11-02 17:53:51 +0000 | [diff] [blame] | 2332 | MDNode::getMostGenericAlignmentOrDereferenceable(JMD, KMD)); |
| 2333 | break; |
Rafael Espindola | ea46c32 | 2014-08-15 15:46:38 +0000 | [diff] [blame] | 2334 | } |
| 2335 | } |
Piotr Padlewski | dc9b2cf | 2015-10-02 22:12:22 +0000 | [diff] [blame] | 2336 | // Set !invariant.group from J if J has it. If both instructions have it |
| 2337 | // then we will just pick it from J - even when they are different. |
| 2338 | // Also make sure that K is load or store - f.e. combining bitcast with load |
| 2339 | // could produce bitcast with invariant.group metadata, which is invalid. |
| 2340 | // FIXME: we should try to preserve both invariant.group md if they are |
| 2341 | // different, but right now instruction can only have one invariant.group. |
| 2342 | if (auto *JMD = J->getMetadata(LLVMContext::MD_invariant_group)) |
| 2343 | if (isa<LoadInst>(K) || isa<StoreInst>(K)) |
| 2344 | K->setMetadata(LLVMContext::MD_invariant_group, JMD); |
Rafael Espindola | ea46c32 | 2014-08-15 15:46:38 +0000 | [diff] [blame] | 2345 | } |
Philip Reames | 7c78ef7 | 2015-05-22 23:53:24 +0000 | [diff] [blame] | 2346 | |
Eli Friedman | 02419a9 | 2016-08-08 04:10:22 +0000 | [diff] [blame] | 2347 | void llvm::combineMetadataForCSE(Instruction *K, const Instruction *J) { |
| 2348 | unsigned KnownIDs[] = { |
| 2349 | LLVMContext::MD_tbaa, LLVMContext::MD_alias_scope, |
| 2350 | LLVMContext::MD_noalias, LLVMContext::MD_range, |
| 2351 | LLVMContext::MD_invariant_load, LLVMContext::MD_nonnull, |
| 2352 | LLVMContext::MD_invariant_group, LLVMContext::MD_align, |
| 2353 | LLVMContext::MD_dereferenceable, |
| 2354 | LLVMContext::MD_dereferenceable_or_null}; |
| 2355 | combineMetadata(K, J, KnownIDs); |
| 2356 | } |
| 2357 | |
Florian Hahn | 39bbe17 | 2018-08-07 13:27:33 +0000 | [diff] [blame] | 2358 | void llvm::patchReplacementInstruction(Instruction *I, Value *Repl) { |
| 2359 | auto *ReplInst = dyn_cast<Instruction>(Repl); |
| 2360 | if (!ReplInst) |
| 2361 | return; |
| 2362 | |
| 2363 | // Patch the replacement so that it is not more restrictive than the value |
| 2364 | // being replaced. |
| 2365 | // Note that if 'I' is a load being replaced by some operation, |
| 2366 | // for example, by an arithmetic operation, then andIRFlags() |
| 2367 | // would just erase all math flags from the original arithmetic |
| 2368 | // operation, which is clearly not wanted and not needed. |
| 2369 | if (!isa<LoadInst>(I)) |
| 2370 | ReplInst->andIRFlags(I); |
| 2371 | |
| 2372 | // FIXME: If both the original and replacement value are part of the |
| 2373 | // same control-flow region (meaning that the execution of one |
| 2374 | // guarantees the execution of the other), then we can combine the |
| 2375 | // noalias scopes here and do better than the general conservative |
| 2376 | // answer used in combineMetadata(). |
| 2377 | |
| 2378 | // In general, GVN unifies expressions over different control-flow |
| 2379 | // regions, and so we need a conservative combination of the noalias |
| 2380 | // scopes. |
| 2381 | static const unsigned KnownIDs[] = { |
| 2382 | LLVMContext::MD_tbaa, LLVMContext::MD_alias_scope, |
| 2383 | LLVMContext::MD_noalias, LLVMContext::MD_range, |
| 2384 | LLVMContext::MD_fpmath, LLVMContext::MD_invariant_load, |
Florian Hahn | 950576b | 2018-08-07 15:36:11 +0000 | [diff] [blame^] | 2385 | LLVMContext::MD_invariant_group, LLVMContext::MD_nonnull}; |
| 2386 | combineMetadata(ReplInst, I, KnownIDs, false); |
Florian Hahn | 39bbe17 | 2018-08-07 13:27:33 +0000 | [diff] [blame] | 2387 | } |
| 2388 | |
Piotr Padlewski | d979c1f | 2017-05-09 19:39:44 +0000 | [diff] [blame] | 2389 | template <typename RootType, typename DominatesFn> |
| 2390 | static unsigned replaceDominatedUsesWith(Value *From, Value *To, |
| 2391 | const RootType &Root, |
| 2392 | const DominatesFn &Dominates) { |
Piotr Padlewski | 28ffcbe | 2015-09-02 19:59:59 +0000 | [diff] [blame] | 2393 | assert(From->getType() == To->getType()); |
| 2394 | |
| 2395 | unsigned Count = 0; |
| 2396 | for (Value::use_iterator UI = From->use_begin(), UE = From->use_end(); |
| 2397 | UI != UE;) { |
| 2398 | Use &U = *UI++; |
Piotr Padlewski | d979c1f | 2017-05-09 19:39:44 +0000 | [diff] [blame] | 2399 | if (!Dominates(Root, U)) |
| 2400 | continue; |
| 2401 | U.set(To); |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 2402 | LLVM_DEBUG(dbgs() << "Replace dominated use of '" << From->getName() |
| 2403 | << "' as " << *To << " in " << *U << "\n"); |
Piotr Padlewski | d979c1f | 2017-05-09 19:39:44 +0000 | [diff] [blame] | 2404 | ++Count; |
Piotr Padlewski | 28ffcbe | 2015-09-02 19:59:59 +0000 | [diff] [blame] | 2405 | } |
| 2406 | return Count; |
| 2407 | } |
Sanjoy Das | c21a05a | 2015-10-08 23:18:30 +0000 | [diff] [blame] | 2408 | |
Anna Thomas | c07d554 | 2017-05-23 13:36:25 +0000 | [diff] [blame] | 2409 | unsigned llvm::replaceNonLocalUsesWith(Instruction *From, Value *To) { |
| 2410 | assert(From->getType() == To->getType()); |
| 2411 | auto *BB = From->getParent(); |
| 2412 | unsigned Count = 0; |
| 2413 | |
| 2414 | for (Value::use_iterator UI = From->use_begin(), UE = From->use_end(); |
| 2415 | UI != UE;) { |
| 2416 | Use &U = *UI++; |
| 2417 | auto *I = cast<Instruction>(U.getUser()); |
| 2418 | if (I->getParent() == BB) |
| 2419 | continue; |
| 2420 | U.set(To); |
| 2421 | ++Count; |
| 2422 | } |
| 2423 | return Count; |
| 2424 | } |
| 2425 | |
Piotr Padlewski | d979c1f | 2017-05-09 19:39:44 +0000 | [diff] [blame] | 2426 | unsigned llvm::replaceDominatedUsesWith(Value *From, Value *To, |
| 2427 | DominatorTree &DT, |
| 2428 | const BasicBlockEdge &Root) { |
| 2429 | auto Dominates = [&DT](const BasicBlockEdge &Root, const Use &U) { |
| 2430 | return DT.dominates(Root, U); |
| 2431 | }; |
| 2432 | return ::replaceDominatedUsesWith(From, To, Root, Dominates); |
| 2433 | } |
| 2434 | |
| 2435 | unsigned llvm::replaceDominatedUsesWith(Value *From, Value *To, |
| 2436 | DominatorTree &DT, |
| 2437 | const BasicBlock *BB) { |
| 2438 | auto ProperlyDominates = [&DT](const BasicBlock *BB, const Use &U) { |
| 2439 | auto *I = cast<Instruction>(U.getUser())->getParent(); |
| 2440 | return DT.properlyDominates(BB, I); |
| 2441 | }; |
| 2442 | return ::replaceDominatedUsesWith(From, To, BB, ProperlyDominates); |
| 2443 | } |
| 2444 | |
Daniel Neilson | 2574d7c | 2017-07-27 16:49:39 +0000 | [diff] [blame] | 2445 | bool llvm::callsGCLeafFunction(ImmutableCallSite CS, |
| 2446 | const TargetLibraryInfo &TLI) { |
Sanjoy Das | c21a05a | 2015-10-08 23:18:30 +0000 | [diff] [blame] | 2447 | // Check if the function is specifically marked as a gc leaf function. |
Manuel Jacob | 3eedd11 | 2016-01-05 23:59:08 +0000 | [diff] [blame] | 2448 | if (CS.hasFnAttr("gc-leaf-function")) |
| 2449 | return true; |
Sanjoy Das | d4c7833 | 2016-03-25 20:12:13 +0000 | [diff] [blame] | 2450 | if (const Function *F = CS.getCalledFunction()) { |
| 2451 | if (F->hasFnAttribute("gc-leaf-function")) |
| 2452 | return true; |
| 2453 | |
| 2454 | if (auto IID = F->getIntrinsicID()) |
| 2455 | // Most LLVM intrinsics do not take safepoints. |
| 2456 | return IID != Intrinsic::experimental_gc_statepoint && |
| 2457 | IID != Intrinsic::experimental_deoptimize; |
| 2458 | } |
Sanjoy Das | c21a05a | 2015-10-08 23:18:30 +0000 | [diff] [blame] | 2459 | |
Daniel Neilson | 2574d7c | 2017-07-27 16:49:39 +0000 | [diff] [blame] | 2460 | // Lib calls can be materialized by some passes, and won't be |
| 2461 | // marked as 'gc-leaf-function.' All available Libcalls are |
| 2462 | // GC-leaf. |
| 2463 | LibFunc LF; |
| 2464 | if (TLI.getLibFunc(CS, LF)) { |
| 2465 | return TLI.has(LF); |
| 2466 | } |
| 2467 | |
Sanjoy Das | c21a05a | 2015-10-08 23:18:30 +0000 | [diff] [blame] | 2468 | return false; |
| 2469 | } |
James Molloy | f01488e | 2016-01-15 09:20:19 +0000 | [diff] [blame] | 2470 | |
Chandler Carruth | 2abb65a | 2017-06-26 03:31:31 +0000 | [diff] [blame] | 2471 | void llvm::copyNonnullMetadata(const LoadInst &OldLI, MDNode *N, |
| 2472 | LoadInst &NewLI) { |
| 2473 | auto *NewTy = NewLI.getType(); |
| 2474 | |
| 2475 | // This only directly applies if the new type is also a pointer. |
| 2476 | if (NewTy->isPointerTy()) { |
| 2477 | NewLI.setMetadata(LLVMContext::MD_nonnull, N); |
| 2478 | return; |
| 2479 | } |
| 2480 | |
| 2481 | // The only other translation we can do is to integral loads with !range |
| 2482 | // metadata. |
| 2483 | if (!NewTy->isIntegerTy()) |
| 2484 | return; |
| 2485 | |
| 2486 | MDBuilder MDB(NewLI.getContext()); |
| 2487 | const Value *Ptr = OldLI.getPointerOperand(); |
| 2488 | auto *ITy = cast<IntegerType>(NewTy); |
| 2489 | auto *NullInt = ConstantExpr::getPtrToInt( |
| 2490 | ConstantPointerNull::get(cast<PointerType>(Ptr->getType())), ITy); |
| 2491 | auto *NonNullInt = ConstantExpr::getAdd(NullInt, ConstantInt::get(ITy, 1)); |
| 2492 | NewLI.setMetadata(LLVMContext::MD_range, |
| 2493 | MDB.createRange(NonNullInt, NullInt)); |
| 2494 | } |
| 2495 | |
| 2496 | void llvm::copyRangeMetadata(const DataLayout &DL, const LoadInst &OldLI, |
| 2497 | MDNode *N, LoadInst &NewLI) { |
| 2498 | auto *NewTy = NewLI.getType(); |
| 2499 | |
Rafael Espindola | c06f55e | 2017-11-28 01:25:38 +0000 | [diff] [blame] | 2500 | // Give up unless it is converted to a pointer where there is a single very |
| 2501 | // valuable mapping we can do reliably. |
| 2502 | // FIXME: It would be nice to propagate this in more ways, but the type |
| 2503 | // conversions make it hard. |
| 2504 | if (!NewTy->isPointerTy()) |
Chandler Carruth | 2abb65a | 2017-06-26 03:31:31 +0000 | [diff] [blame] | 2505 | return; |
| 2506 | |
Elena Demikhovsky | 945b7e5 | 2018-02-14 06:58:08 +0000 | [diff] [blame] | 2507 | unsigned BitWidth = DL.getIndexTypeSizeInBits(NewTy); |
Rafael Espindola | c06f55e | 2017-11-28 01:25:38 +0000 | [diff] [blame] | 2508 | if (!getConstantRangeFromMetadata(*N).contains(APInt(BitWidth, 0))) { |
| 2509 | MDNode *NN = MDNode::get(OldLI.getContext(), None); |
| 2510 | NewLI.setMetadata(LLVMContext::MD_nonnull, NN); |
Chandler Carruth | 2abb65a | 2017-06-26 03:31:31 +0000 | [diff] [blame] | 2511 | } |
| 2512 | } |
| 2513 | |
Benjamin Kramer | b7d3311 | 2016-08-06 11:13:10 +0000 | [diff] [blame] | 2514 | namespace { |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 2515 | |
James Molloy | f01488e | 2016-01-15 09:20:19 +0000 | [diff] [blame] | 2516 | /// A potential constituent of a bitreverse or bswap expression. See |
| 2517 | /// collectBitParts for a fuller explanation. |
| 2518 | struct BitPart { |
| 2519 | BitPart(Value *P, unsigned BW) : Provider(P) { |
| 2520 | Provenance.resize(BW); |
| 2521 | } |
| 2522 | |
| 2523 | /// The Value that this is a bitreverse/bswap of. |
| 2524 | Value *Provider; |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 2525 | |
James Molloy | f01488e | 2016-01-15 09:20:19 +0000 | [diff] [blame] | 2526 | /// The "provenance" of each bit. Provenance[A] = B means that bit A |
| 2527 | /// in Provider becomes bit B in the result of this expression. |
| 2528 | SmallVector<int8_t, 32> Provenance; // int8_t means max size is i128. |
| 2529 | |
| 2530 | enum { Unset = -1 }; |
| 2531 | }; |
Eugene Zelenko | 6cadde7 | 2017-10-17 21:27:42 +0000 | [diff] [blame] | 2532 | |
Benjamin Kramer | b7d3311 | 2016-08-06 11:13:10 +0000 | [diff] [blame] | 2533 | } // end anonymous namespace |
James Molloy | f01488e | 2016-01-15 09:20:19 +0000 | [diff] [blame] | 2534 | |
| 2535 | /// Analyze the specified subexpression and see if it is capable of providing |
| 2536 | /// pieces of a bswap or bitreverse. The subexpression provides a potential |
| 2537 | /// piece of a bswap or bitreverse if it can be proven that each non-zero bit in |
| 2538 | /// the output of the expression came from a corresponding bit in some other |
| 2539 | /// value. This function is recursive, and the end result is a mapping of |
| 2540 | /// bitnumber to bitnumber. It is the caller's responsibility to validate that |
| 2541 | /// the bitnumber to bitnumber mapping is correct for a bswap or bitreverse. |
| 2542 | /// |
| 2543 | /// For example, if the current subexpression if "(shl i32 %X, 24)" then we know |
| 2544 | /// that the expression deposits the low byte of %X into the high byte of the |
| 2545 | /// result and that all other bits are zero. This expression is accepted and a |
| 2546 | /// BitPart is returned with Provider set to %X and Provenance[24-31] set to |
| 2547 | /// [0-7]. |
| 2548 | /// |
| 2549 | /// To avoid revisiting values, the BitPart results are memoized into the |
| 2550 | /// provided map. To avoid unnecessary copying of BitParts, BitParts are |
| 2551 | /// constructed in-place in the \c BPS map. Because of this \c BPS needs to |
| 2552 | /// store BitParts objects, not pointers. As we need the concept of a nullptr |
| 2553 | /// BitParts (Value has been analyzed and the analysis failed), we an Optional |
| 2554 | /// type instead to provide the same functionality. |
| 2555 | /// |
| 2556 | /// Because we pass around references into \c BPS, we must use a container that |
| 2557 | /// does not invalidate internal references (std::map instead of DenseMap). |
James Molloy | f01488e | 2016-01-15 09:20:19 +0000 | [diff] [blame] | 2558 | static const Optional<BitPart> & |
| 2559 | collectBitParts(Value *V, bool MatchBSwaps, bool MatchBitReversals, |
| 2560 | std::map<Value *, Optional<BitPart>> &BPS) { |
| 2561 | auto I = BPS.find(V); |
| 2562 | if (I != BPS.end()) |
| 2563 | return I->second; |
| 2564 | |
| 2565 | auto &Result = BPS[V] = None; |
| 2566 | auto BitWidth = cast<IntegerType>(V->getType())->getBitWidth(); |
| 2567 | |
| 2568 | if (Instruction *I = dyn_cast<Instruction>(V)) { |
| 2569 | // If this is an or instruction, it may be an inner node of the bswap. |
| 2570 | if (I->getOpcode() == Instruction::Or) { |
| 2571 | auto &A = collectBitParts(I->getOperand(0), MatchBSwaps, |
| 2572 | MatchBitReversals, BPS); |
| 2573 | auto &B = collectBitParts(I->getOperand(1), MatchBSwaps, |
| 2574 | MatchBitReversals, BPS); |
| 2575 | if (!A || !B) |
| 2576 | return Result; |
| 2577 | |
| 2578 | // Try and merge the two together. |
| 2579 | if (!A->Provider || A->Provider != B->Provider) |
| 2580 | return Result; |
| 2581 | |
| 2582 | Result = BitPart(A->Provider, BitWidth); |
| 2583 | for (unsigned i = 0; i < A->Provenance.size(); ++i) { |
| 2584 | if (A->Provenance[i] != BitPart::Unset && |
| 2585 | B->Provenance[i] != BitPart::Unset && |
| 2586 | A->Provenance[i] != B->Provenance[i]) |
| 2587 | return Result = None; |
| 2588 | |
| 2589 | if (A->Provenance[i] == BitPart::Unset) |
| 2590 | Result->Provenance[i] = B->Provenance[i]; |
| 2591 | else |
| 2592 | Result->Provenance[i] = A->Provenance[i]; |
| 2593 | } |
| 2594 | |
| 2595 | return Result; |
| 2596 | } |
| 2597 | |
| 2598 | // If this is a logical shift by a constant, recurse then shift the result. |
| 2599 | if (I->isLogicalShift() && isa<ConstantInt>(I->getOperand(1))) { |
| 2600 | unsigned BitShift = |
| 2601 | cast<ConstantInt>(I->getOperand(1))->getLimitedValue(~0U); |
| 2602 | // Ensure the shift amount is defined. |
| 2603 | if (BitShift > BitWidth) |
| 2604 | return Result; |
| 2605 | |
| 2606 | auto &Res = collectBitParts(I->getOperand(0), MatchBSwaps, |
| 2607 | MatchBitReversals, BPS); |
| 2608 | if (!Res) |
| 2609 | return Result; |
| 2610 | Result = Res; |
| 2611 | |
| 2612 | // Perform the "shift" on BitProvenance. |
| 2613 | auto &P = Result->Provenance; |
| 2614 | if (I->getOpcode() == Instruction::Shl) { |
| 2615 | P.erase(std::prev(P.end(), BitShift), P.end()); |
| 2616 | P.insert(P.begin(), BitShift, BitPart::Unset); |
| 2617 | } else { |
| 2618 | P.erase(P.begin(), std::next(P.begin(), BitShift)); |
| 2619 | P.insert(P.end(), BitShift, BitPart::Unset); |
| 2620 | } |
| 2621 | |
| 2622 | return Result; |
| 2623 | } |
| 2624 | |
| 2625 | // If this is a logical 'and' with a mask that clears bits, recurse then |
| 2626 | // unset the appropriate bits. |
| 2627 | if (I->getOpcode() == Instruction::And && |
| 2628 | isa<ConstantInt>(I->getOperand(1))) { |
| 2629 | APInt Bit(I->getType()->getPrimitiveSizeInBits(), 1); |
| 2630 | const APInt &AndMask = cast<ConstantInt>(I->getOperand(1))->getValue(); |
| 2631 | |
| 2632 | // Check that the mask allows a multiple of 8 bits for a bswap, for an |
| 2633 | // early exit. |
| 2634 | unsigned NumMaskedBits = AndMask.countPopulation(); |
| 2635 | if (!MatchBitReversals && NumMaskedBits % 8 != 0) |
| 2636 | return Result; |
Chandler Carruth | 2abb65a | 2017-06-26 03:31:31 +0000 | [diff] [blame] | 2637 | |
James Molloy | f01488e | 2016-01-15 09:20:19 +0000 | [diff] [blame] | 2638 | auto &Res = collectBitParts(I->getOperand(0), MatchBSwaps, |
| 2639 | MatchBitReversals, BPS); |
| 2640 | if (!Res) |
| 2641 | return Result; |
| 2642 | Result = Res; |
| 2643 | |
| 2644 | for (unsigned i = 0; i < BitWidth; ++i, Bit <<= 1) |
| 2645 | // If the AndMask is zero for this bit, clear the bit. |
| 2646 | if ((AndMask & Bit) == 0) |
| 2647 | Result->Provenance[i] = BitPart::Unset; |
Chad Rosier | e5819e2 | 2016-05-26 14:58:51 +0000 | [diff] [blame] | 2648 | return Result; |
| 2649 | } |
James Molloy | f01488e | 2016-01-15 09:20:19 +0000 | [diff] [blame] | 2650 | |
Chad Rosier | e5819e2 | 2016-05-26 14:58:51 +0000 | [diff] [blame] | 2651 | // If this is a zext instruction zero extend the result. |
| 2652 | if (I->getOpcode() == Instruction::ZExt) { |
| 2653 | auto &Res = collectBitParts(I->getOperand(0), MatchBSwaps, |
| 2654 | MatchBitReversals, BPS); |
| 2655 | if (!Res) |
| 2656 | return Result; |
| 2657 | |
| 2658 | Result = BitPart(Res->Provider, BitWidth); |
| 2659 | auto NarrowBitWidth = |
| 2660 | cast<IntegerType>(cast<ZExtInst>(I)->getSrcTy())->getBitWidth(); |
| 2661 | for (unsigned i = 0; i < NarrowBitWidth; ++i) |
| 2662 | Result->Provenance[i] = Res->Provenance[i]; |
| 2663 | for (unsigned i = NarrowBitWidth; i < BitWidth; ++i) |
| 2664 | Result->Provenance[i] = BitPart::Unset; |
James Molloy | f01488e | 2016-01-15 09:20:19 +0000 | [diff] [blame] | 2665 | return Result; |
| 2666 | } |
| 2667 | } |
| 2668 | |
| 2669 | // Okay, we got to something that isn't a shift, 'or' or 'and'. This must be |
| 2670 | // the input value to the bswap/bitreverse. |
| 2671 | Result = BitPart(V, BitWidth); |
| 2672 | for (unsigned i = 0; i < BitWidth; ++i) |
| 2673 | Result->Provenance[i] = i; |
| 2674 | return Result; |
| 2675 | } |
| 2676 | |
| 2677 | static bool bitTransformIsCorrectForBSwap(unsigned From, unsigned To, |
| 2678 | unsigned BitWidth) { |
| 2679 | if (From % 8 != To % 8) |
| 2680 | return false; |
| 2681 | // Convert from bit indices to byte indices and check for a byte reversal. |
| 2682 | From >>= 3; |
| 2683 | To >>= 3; |
| 2684 | BitWidth >>= 3; |
| 2685 | return From == BitWidth - To - 1; |
| 2686 | } |
| 2687 | |
| 2688 | static bool bitTransformIsCorrectForBitReverse(unsigned From, unsigned To, |
| 2689 | unsigned BitWidth) { |
| 2690 | return From == BitWidth - To - 1; |
| 2691 | } |
| 2692 | |
Chad Rosier | a00df49 | 2016-05-25 16:22:14 +0000 | [diff] [blame] | 2693 | bool llvm::recognizeBSwapOrBitReverseIdiom( |
James Molloy | f01488e | 2016-01-15 09:20:19 +0000 | [diff] [blame] | 2694 | Instruction *I, bool MatchBSwaps, bool MatchBitReversals, |
| 2695 | SmallVectorImpl<Instruction *> &InsertedInsts) { |
| 2696 | if (Operator::getOpcode(I) != Instruction::Or) |
| 2697 | return false; |
| 2698 | if (!MatchBSwaps && !MatchBitReversals) |
| 2699 | return false; |
| 2700 | IntegerType *ITy = dyn_cast<IntegerType>(I->getType()); |
| 2701 | if (!ITy || ITy->getBitWidth() > 128) |
| 2702 | return false; // Can't do vectors or integers > 128 bits. |
| 2703 | unsigned BW = ITy->getBitWidth(); |
| 2704 | |
Chad Rosier | e5819e2 | 2016-05-26 14:58:51 +0000 | [diff] [blame] | 2705 | unsigned DemandedBW = BW; |
| 2706 | IntegerType *DemandedTy = ITy; |
| 2707 | if (I->hasOneUse()) { |
| 2708 | if (TruncInst *Trunc = dyn_cast<TruncInst>(I->user_back())) { |
| 2709 | DemandedTy = cast<IntegerType>(Trunc->getType()); |
| 2710 | DemandedBW = DemandedTy->getBitWidth(); |
| 2711 | } |
| 2712 | } |
| 2713 | |
James Molloy | f01488e | 2016-01-15 09:20:19 +0000 | [diff] [blame] | 2714 | // Try to find all the pieces corresponding to the bswap. |
| 2715 | std::map<Value *, Optional<BitPart>> BPS; |
| 2716 | auto Res = collectBitParts(I, MatchBSwaps, MatchBitReversals, BPS); |
| 2717 | if (!Res) |
| 2718 | return false; |
| 2719 | auto &BitProvenance = Res->Provenance; |
| 2720 | |
| 2721 | // Now, is the bit permutation correct for a bswap or a bitreverse? We can |
| 2722 | // only byteswap values with an even number of bytes. |
Chad Rosier | e5819e2 | 2016-05-26 14:58:51 +0000 | [diff] [blame] | 2723 | bool OKForBSwap = DemandedBW % 16 == 0, OKForBitReverse = true; |
| 2724 | for (unsigned i = 0; i < DemandedBW; ++i) { |
| 2725 | OKForBSwap &= |
| 2726 | bitTransformIsCorrectForBSwap(BitProvenance[i], i, DemandedBW); |
James Molloy | f01488e | 2016-01-15 09:20:19 +0000 | [diff] [blame] | 2727 | OKForBitReverse &= |
Chad Rosier | e5819e2 | 2016-05-26 14:58:51 +0000 | [diff] [blame] | 2728 | bitTransformIsCorrectForBitReverse(BitProvenance[i], i, DemandedBW); |
James Molloy | f01488e | 2016-01-15 09:20:19 +0000 | [diff] [blame] | 2729 | } |
| 2730 | |
| 2731 | Intrinsic::ID Intrin; |
| 2732 | if (OKForBSwap && MatchBSwaps) |
| 2733 | Intrin = Intrinsic::bswap; |
| 2734 | else if (OKForBitReverse && MatchBitReversals) |
| 2735 | Intrin = Intrinsic::bitreverse; |
| 2736 | else |
| 2737 | return false; |
| 2738 | |
Chad Rosier | e5819e2 | 2016-05-26 14:58:51 +0000 | [diff] [blame] | 2739 | if (ITy != DemandedTy) { |
| 2740 | Function *F = Intrinsic::getDeclaration(I->getModule(), Intrin, DemandedTy); |
| 2741 | Value *Provider = Res->Provider; |
| 2742 | IntegerType *ProviderTy = cast<IntegerType>(Provider->getType()); |
| 2743 | // We may need to truncate the provider. |
| 2744 | if (DemandedTy != ProviderTy) { |
| 2745 | auto *Trunc = CastInst::Create(Instruction::Trunc, Provider, DemandedTy, |
| 2746 | "trunc", I); |
| 2747 | InsertedInsts.push_back(Trunc); |
| 2748 | Provider = Trunc; |
| 2749 | } |
| 2750 | auto *CI = CallInst::Create(F, Provider, "rev", I); |
| 2751 | InsertedInsts.push_back(CI); |
| 2752 | auto *ExtInst = CastInst::Create(Instruction::ZExt, CI, ITy, "zext", I); |
| 2753 | InsertedInsts.push_back(ExtInst); |
| 2754 | return true; |
| 2755 | } |
| 2756 | |
James Molloy | f01488e | 2016-01-15 09:20:19 +0000 | [diff] [blame] | 2757 | Function *F = Intrinsic::getDeclaration(I->getModule(), Intrin, ITy); |
| 2758 | InsertedInsts.push_back(CallInst::Create(F, Res->Provider, "rev", I)); |
| 2759 | return true; |
| 2760 | } |
Marcin Koscielnicki | 3feda22 | 2016-06-18 10:10:37 +0000 | [diff] [blame] | 2761 | |
| 2762 | // CodeGen has special handling for some string functions that may replace |
| 2763 | // them with target-specific intrinsics. Since that'd skip our interceptors |
| 2764 | // in ASan/MSan/TSan/DFSan, and thus make us miss some memory accesses, |
| 2765 | // we mark affected calls as NoBuiltin, which will disable optimization |
| 2766 | // in CodeGen. |
Evgeniy Stepanov | d240a88 | 2016-07-28 23:45:15 +0000 | [diff] [blame] | 2767 | void llvm::maybeMarkSanitizerLibraryCallNoBuiltin( |
| 2768 | CallInst *CI, const TargetLibraryInfo *TLI) { |
Marcin Koscielnicki | 3feda22 | 2016-06-18 10:10:37 +0000 | [diff] [blame] | 2769 | Function *F = CI->getCalledFunction(); |
David L. Jones | d21529f | 2017-01-23 23:16:46 +0000 | [diff] [blame] | 2770 | LibFunc Func; |
Evgeniy Stepanov | d240a88 | 2016-07-28 23:45:15 +0000 | [diff] [blame] | 2771 | if (F && !F->hasLocalLinkage() && F->hasName() && |
| 2772 | TLI->getLibFunc(F->getName(), Func) && TLI->hasOptimizedCodeGen(Func) && |
| 2773 | !F->doesNotAccessMemory()) |
Reid Kleckner | b518054 | 2017-03-21 16:57:19 +0000 | [diff] [blame] | 2774 | CI->addAttribute(AttributeList::FunctionIndex, Attribute::NoBuiltin); |
Marcin Koscielnicki | 3feda22 | 2016-06-18 10:10:37 +0000 | [diff] [blame] | 2775 | } |
James Molloy | a929063 | 2017-05-25 12:51:11 +0000 | [diff] [blame] | 2776 | |
| 2777 | bool llvm::canReplaceOperandWithVariable(const Instruction *I, unsigned OpIdx) { |
| 2778 | // We can't have a PHI with a metadata type. |
| 2779 | if (I->getOperand(OpIdx)->getType()->isMetadataTy()) |
| 2780 | return false; |
| 2781 | |
| 2782 | // Early exit. |
| 2783 | if (!isa<Constant>(I->getOperand(OpIdx))) |
| 2784 | return true; |
| 2785 | |
| 2786 | switch (I->getOpcode()) { |
| 2787 | default: |
| 2788 | return true; |
| 2789 | case Instruction::Call: |
| 2790 | case Instruction::Invoke: |
Leo Li | 93abd7d | 2017-07-10 20:45:34 +0000 | [diff] [blame] | 2791 | // Can't handle inline asm. Skip it. |
| 2792 | if (isa<InlineAsm>(ImmutableCallSite(I).getCalledValue())) |
| 2793 | return false; |
James Molloy | a929063 | 2017-05-25 12:51:11 +0000 | [diff] [blame] | 2794 | // Many arithmetic intrinsics have no issue taking a |
| 2795 | // variable, however it's hard to distingish these from |
| 2796 | // specials such as @llvm.frameaddress that require a constant. |
| 2797 | if (isa<IntrinsicInst>(I)) |
| 2798 | return false; |
| 2799 | |
| 2800 | // Constant bundle operands may need to retain their constant-ness for |
| 2801 | // correctness. |
| 2802 | if (ImmutableCallSite(I).isBundleOperand(OpIdx)) |
| 2803 | return false; |
| 2804 | return true; |
| 2805 | case Instruction::ShuffleVector: |
| 2806 | // Shufflevector masks are constant. |
| 2807 | return OpIdx != 2; |
Leo Li | 5499b1b | 2017-07-06 18:47:05 +0000 | [diff] [blame] | 2808 | case Instruction::Switch: |
James Molloy | a929063 | 2017-05-25 12:51:11 +0000 | [diff] [blame] | 2809 | case Instruction::ExtractValue: |
James Molloy | a929063 | 2017-05-25 12:51:11 +0000 | [diff] [blame] | 2810 | // All operands apart from the first are constant. |
| 2811 | return OpIdx == 0; |
Leo Li | 5499b1b | 2017-07-06 18:47:05 +0000 | [diff] [blame] | 2812 | case Instruction::InsertValue: |
| 2813 | // All operands apart from the first and the second are constant. |
| 2814 | return OpIdx < 2; |
James Molloy | a929063 | 2017-05-25 12:51:11 +0000 | [diff] [blame] | 2815 | case Instruction::Alloca: |
Leo Li | 5499b1b | 2017-07-06 18:47:05 +0000 | [diff] [blame] | 2816 | // Static allocas (constant size in the entry block) are handled by |
| 2817 | // prologue/epilogue insertion so they're free anyway. We definitely don't |
| 2818 | // want to make them non-constant. |
Craig Topper | 781aa18 | 2018-05-05 01:57:00 +0000 | [diff] [blame] | 2819 | return !cast<AllocaInst>(I)->isStaticAlloca(); |
James Molloy | a929063 | 2017-05-25 12:51:11 +0000 | [diff] [blame] | 2820 | case Instruction::GetElementPtr: |
| 2821 | if (OpIdx == 0) |
| 2822 | return true; |
| 2823 | gep_type_iterator It = gep_type_begin(I); |
| 2824 | for (auto E = std::next(It, OpIdx); It != E; ++It) |
| 2825 | if (It.isStruct()) |
| 2826 | return false; |
| 2827 | return true; |
| 2828 | } |
| 2829 | } |