blob: 2def0b54c6d4615310692b26a7570dbe7d7c0575 [file] [log] [blame]
Chris Lattnerb3b60072008-04-20 20:35:01 +00001//===- JumpThreading.cpp - Thread control through conditional blocks ------===//
2//
3// The LLVM Compiler Infrastructure
4//
5// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
9//
Chris Lattner21157222008-04-20 21:13:06 +000010// This file implements the Jump Threading pass.
Chris Lattnerb3b60072008-04-20 20:35:01 +000011//
12//===----------------------------------------------------------------------===//
13
Chris Lattnerb3b60072008-04-20 20:35:01 +000014#include "llvm/Transforms/Scalar.h"
Chandler Carruthed0881b2012-12-03 16:50:05 +000015#include "llvm/ADT/DenseMap.h"
16#include "llvm/ADT/DenseSet.h"
17#include "llvm/ADT/STLExtras.h"
18#include "llvm/ADT/SmallPtrSet.h"
19#include "llvm/ADT/SmallSet.h"
20#include "llvm/ADT/Statistic.h"
James Molloyefbba722015-09-10 10:22:12 +000021#include "llvm/Analysis/GlobalsModRef.h"
Nick Lewycky0b682452013-07-27 01:24:00 +000022#include "llvm/Analysis/CFG.h"
Manman Ren72d44b12015-10-15 14:59:40 +000023#include "llvm/Analysis/BlockFrequencyInfo.h"
24#include "llvm/Analysis/BlockFrequencyInfoImpl.h"
25#include "llvm/Analysis/BranchProbabilityInfo.h"
Owen Anderson92651ec2011-04-14 21:35:50 +000026#include "llvm/Analysis/ConstantFolding.h"
Chris Lattner800aad32009-11-09 23:00:14 +000027#include "llvm/Analysis/InstructionSimplify.h"
Chris Lattnerfde1f8d2009-11-11 02:08:33 +000028#include "llvm/Analysis/LazyValueInfo.h"
Dan Gohman826bdf82010-05-28 16:19:17 +000029#include "llvm/Analysis/Loads.h"
Manman Ren72d44b12015-10-15 14:59:40 +000030#include "llvm/Analysis/LoopInfo.h"
Benjamin Kramer799003b2015-03-23 19:32:43 +000031#include "llvm/Analysis/TargetLibraryInfo.h"
Sanjoy Das13e63a22015-10-28 21:27:08 +000032#include "llvm/Analysis/ValueTracking.h"
Chandler Carruth9fb823b2013-01-02 11:36:10 +000033#include "llvm/IR/DataLayout.h"
34#include "llvm/IR/IntrinsicInst.h"
35#include "llvm/IR/LLVMContext.h"
Manman Ren72d44b12015-10-15 14:59:40 +000036#include "llvm/IR/MDBuilder.h"
Hal Finkelcc39b672014-07-24 12:16:19 +000037#include "llvm/IR/Metadata.h"
Chandler Carruth4220e9c2014-03-04 11:17:44 +000038#include "llvm/IR/ValueHandle.h"
Chandler Carruthed0881b2012-12-03 16:50:05 +000039#include "llvm/Pass.h"
Chris Lattnerb3b60072008-04-20 20:35:01 +000040#include "llvm/Support/CommandLine.h"
Chris Lattner21157222008-04-20 21:13:06 +000041#include "llvm/Support/Debug.h"
Daniel Dunbar9813b0b2009-07-26 07:49:05 +000042#include "llvm/Support/raw_ostream.h"
Chandler Carruthed0881b2012-12-03 16:50:05 +000043#include "llvm/Transforms/Utils/BasicBlockUtils.h"
44#include "llvm/Transforms/Utils/Local.h"
45#include "llvm/Transforms/Utils/SSAUpdater.h"
Manman Ren72d44b12015-10-15 14:59:40 +000046#include <algorithm>
47#include <memory>
Chris Lattnerb3b60072008-04-20 20:35:01 +000048using namespace llvm;
49
Chandler Carruth964daaa2014-04-22 02:55:47 +000050#define DEBUG_TYPE "jump-threading"
51
Chris Lattnerff1c6e32008-04-20 22:39:42 +000052STATISTIC(NumThreads, "Number of jumps threaded");
53STATISTIC(NumFolds, "Number of terminators folded");
Chris Lattner97b14052009-10-11 07:24:57 +000054STATISTIC(NumDupes, "Number of branch blocks duplicated to eliminate phi");
Chris Lattnerb3b60072008-04-20 20:35:01 +000055
Chris Lattner21157222008-04-20 21:13:06 +000056static cl::opt<unsigned>
Michael Liaod1209162014-09-24 04:59:06 +000057BBDuplicateThreshold("jump-threading-threshold",
Chris Lattner21157222008-04-20 21:13:06 +000058 cl::desc("Max block size to duplicate for jump threading"),
59 cl::init(6), cl::Hidden);
60
Sanjoy Das13e63a22015-10-28 21:27:08 +000061static cl::opt<unsigned>
62ImplicationSearchThreshold(
63 "jump-threading-implication-search-threshold",
64 cl::desc("The number of predecessors to search for a stronger "
65 "condition to use to thread over a weaker condition"),
66 cl::init(3), cl::Hidden);
67
Chris Lattnerb3b60072008-04-20 20:35:01 +000068namespace {
Frits van Bommel76244862010-12-05 19:06:41 +000069 // These are at global scope so static functions can use them too.
70 typedef SmallVectorImpl<std::pair<Constant*, BasicBlock*> > PredValueInfo;
71 typedef SmallVector<std::pair<Constant*, BasicBlock*>, 8> PredValueInfoTy;
72
Frits van Bommeld9df6ea2010-12-06 23:36:56 +000073 // This is used to keep track of what kind of constant we're currently hoping
74 // to find.
75 enum ConstantPreference {
76 WantInteger,
77 WantBlockAddress
78 };
79
Chris Lattnere7f0afe2008-05-09 04:43:13 +000080 /// This pass performs 'jump threading', which looks at blocks that have
81 /// multiple predecessors and multiple successors. If one or more of the
82 /// predecessors of the block can be proven to always jump to one of the
83 /// successors, we forward the edge from the predecessor to the successor by
84 /// duplicating the contents of this block.
85 ///
86 /// An example of when this can occur is code like this:
87 ///
88 /// if () { ...
89 /// X = 4;
90 /// }
91 /// if (X < 3) {
92 ///
93 /// In this case, the unconditional branch at the end of the first if can be
94 /// revectored to the false side of the second if.
95 ///
Chris Lattner2dd09db2009-09-02 06:11:42 +000096 class JumpThreading : public FunctionPass {
Chad Rosier43a33062011-12-02 01:26:24 +000097 TargetLibraryInfo *TLI;
Chris Lattnerfde1f8d2009-11-11 02:08:33 +000098 LazyValueInfo *LVI;
Manman Ren72d44b12015-10-15 14:59:40 +000099 std::unique_ptr<BlockFrequencyInfo> BFI;
100 std::unique_ptr<BranchProbabilityInfo> BPI;
101 bool HasProfileData;
Chris Lattnerd579cb12009-05-04 02:28:08 +0000102#ifdef NDEBUG
Benjamin Kramer543762d2016-01-09 18:43:01 +0000103 SmallPtrSet<const BasicBlock *, 16> LoopHeaders;
Chris Lattnerd579cb12009-05-04 02:28:08 +0000104#else
Benjamin Kramer543762d2016-01-09 18:43:01 +0000105 SmallSet<AssertingVH<const BasicBlock>, 16> LoopHeaders;
Chris Lattnerd579cb12009-05-04 02:28:08 +0000106#endif
Owen Anderson3997a072010-08-31 07:36:34 +0000107 DenseSet<std::pair<Value*, BasicBlock*> > RecursionSet;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000108
Michael Liaod1209162014-09-24 04:59:06 +0000109 unsigned BBDupThreshold;
110
Owen Anderson6fdcb1722010-08-31 19:24:27 +0000111 // RAII helper for updating the recursion stack.
112 struct RecursionSetRemover {
113 DenseSet<std::pair<Value*, BasicBlock*> > &TheSet;
114 std::pair<Value*, BasicBlock*> ThePair;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000115
Owen Anderson6fdcb1722010-08-31 19:24:27 +0000116 RecursionSetRemover(DenseSet<std::pair<Value*, BasicBlock*> > &S,
117 std::pair<Value*, BasicBlock*> P)
118 : TheSet(S), ThePair(P) { }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000119
Owen Anderson6fdcb1722010-08-31 19:24:27 +0000120 ~RecursionSetRemover() {
121 TheSet.erase(ThePair);
122 }
123 };
Chris Lattnerb3b60072008-04-20 20:35:01 +0000124 public:
125 static char ID; // Pass identification
Michael Liaod1209162014-09-24 04:59:06 +0000126 JumpThreading(int T = -1) : FunctionPass(ID) {
127 BBDupThreshold = (T == -1) ? BBDuplicateThreshold : unsigned(T);
Owen Anderson6c18d1a2010-10-19 17:21:58 +0000128 initializeJumpThreadingPass(*PassRegistry::getPassRegistry());
129 }
Chris Lattnerb3b60072008-04-20 20:35:01 +0000130
Craig Topper3e4c6972014-03-05 09:10:37 +0000131 bool runOnFunction(Function &F) override;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000132
Craig Topper3e4c6972014-03-05 09:10:37 +0000133 void getAnalysisUsage(AnalysisUsage &AU) const override {
Owen Andersond361aac2010-09-14 20:57:41 +0000134 AU.addRequired<LazyValueInfo>();
135 AU.addPreserved<LazyValueInfo>();
James Molloyefbba722015-09-10 10:22:12 +0000136 AU.addPreserved<GlobalsAAWrapperPass>();
Chandler Carruthb98f63d2015-01-15 10:41:28 +0000137 AU.addRequired<TargetLibraryInfoWrapperPass>();
Chris Lattnerfde1f8d2009-11-11 02:08:33 +0000138 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000139
Manman Ren72d44b12015-10-15 14:59:40 +0000140 void releaseMemory() override {
141 BFI.reset();
142 BPI.reset();
143 }
144
Chris Lattnerfde1f8d2009-11-11 02:08:33 +0000145 void FindLoopHeaders(Function &F);
Chris Lattner240051a2008-11-27 07:20:04 +0000146 bool ProcessBlock(BasicBlock *BB);
Chris Lattner5ff7f562009-11-07 08:05:03 +0000147 bool ThreadEdge(BasicBlock *BB, const SmallVectorImpl<BasicBlock*> &PredBBs,
148 BasicBlock *SuccBB);
Chris Lattner97b14052009-10-11 07:24:57 +0000149 bool DuplicateCondBranchOnPHIIntoPred(BasicBlock *BB,
Chris Lattnereb73bdb2010-01-12 02:07:17 +0000150 const SmallVectorImpl<BasicBlock *> &PredBBs);
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000151
Chris Lattner5ff7f562009-11-07 08:05:03 +0000152 bool ComputeValueKnownInPredecessors(Value *V, BasicBlock *BB,
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000153 PredValueInfo &Result,
Hal Finkel7e184492014-09-07 20:29:59 +0000154 ConstantPreference Preference,
155 Instruction *CxtI = nullptr);
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000156 bool ProcessThreadableEdges(Value *Cond, BasicBlock *BB,
Hal Finkel7e184492014-09-07 20:29:59 +0000157 ConstantPreference Preference,
158 Instruction *CxtI = nullptr);
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000159
Chris Lattner6a19ed02010-01-11 23:41:09 +0000160 bool ProcessBranchOnPHI(PHINode *PN);
Chris Lattnereb73bdb2010-01-12 02:07:17 +0000161 bool ProcessBranchOnXOR(BinaryOperator *BO);
Sanjoy Das13e63a22015-10-28 21:27:08 +0000162 bool ProcessImpliedCondition(BasicBlock *BB);
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000163
Chris Lattner98d89d12008-11-27 05:07:53 +0000164 bool SimplifyPartiallyRedundantLoad(LoadInst *LI);
Benjamin Kramer6a4976d2013-08-07 10:29:38 +0000165 bool TryToUnfoldSelect(CmpInst *CondCmp, BasicBlock *BB);
Haicheng Wua6a32792016-01-08 19:39:39 +0000166 bool TryToUnfoldSelectInCurrBB(BasicBlock *BB);
Manman Ren72d44b12015-10-15 14:59:40 +0000167
168 private:
169 BasicBlock *SplitBlockPreds(BasicBlock *BB, ArrayRef<BasicBlock *> Preds,
170 const char *Suffix);
171 void UpdateBlockFreqAndEdgeWeight(BasicBlock *PredBB, BasicBlock *BB,
172 BasicBlock *NewBB, BasicBlock *SuccBB);
Chris Lattnerb3b60072008-04-20 20:35:01 +0000173 };
Alexander Kornienkof00654e2015-06-23 09:49:53 +0000174}
Chris Lattnerb3b60072008-04-20 20:35:01 +0000175
Dan Gohmand78c4002008-05-13 00:00:25 +0000176char JumpThreading::ID = 0;
Owen Anderson8ac477f2010-10-12 19:48:12 +0000177INITIALIZE_PASS_BEGIN(JumpThreading, "jump-threading",
178 "Jump Threading", false, false)
179INITIALIZE_PASS_DEPENDENCY(LazyValueInfo)
Chandler Carruthb98f63d2015-01-15 10:41:28 +0000180INITIALIZE_PASS_DEPENDENCY(TargetLibraryInfoWrapperPass)
Owen Anderson8ac477f2010-10-12 19:48:12 +0000181INITIALIZE_PASS_END(JumpThreading, "jump-threading",
Owen Andersondf7a4f22010-10-07 22:25:06 +0000182 "Jump Threading", false, false)
Dan Gohmand78c4002008-05-13 00:00:25 +0000183
Chris Lattnerb3b60072008-04-20 20:35:01 +0000184// Public interface to the Jump Threading pass
Michael Liaod1209162014-09-24 04:59:06 +0000185FunctionPass *llvm::createJumpThreadingPass(int Threshold) { return new JumpThreading(Threshold); }
Chris Lattnerb3b60072008-04-20 20:35:01 +0000186
187/// runOnFunction - Top level algorithm.
188///
189bool JumpThreading::runOnFunction(Function &F) {
Paul Robinsonaf4e64d2014-02-06 00:07:05 +0000190 if (skipOptnoneFunction(F))
191 return false;
192
David Greene1efdb452010-01-05 01:27:19 +0000193 DEBUG(dbgs() << "Jump threading on function '" << F.getName() << "'\n");
Chandler Carruthb98f63d2015-01-15 10:41:28 +0000194 TLI = &getAnalysis<TargetLibraryInfoWrapperPass>().getTLI();
Owen Andersond361aac2010-09-14 20:57:41 +0000195 LVI = &getAnalysis<LazyValueInfo>();
Manman Ren72d44b12015-10-15 14:59:40 +0000196 BFI.reset();
197 BPI.reset();
198 // When profile data is available, we need to update edge weights after
199 // successful jump threading, which requires both BPI and BFI being available.
200 HasProfileData = F.getEntryCount().hasValue();
201 if (HasProfileData) {
202 LoopInfo LI{DominatorTree(F)};
203 BPI.reset(new BranchProbabilityInfo(F, LI));
204 BFI.reset(new BlockFrequencyInfo(F, *BPI, LI));
205 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000206
Dinesh Dwivedi8bb5fb02014-06-19 14:11:53 +0000207 // Remove unreachable blocks from function as they may result in infinite
208 // loop. We do threading if we found something profitable. Jump threading a
209 // branch can create other opportunities. If these opportunities form a cycle
David L Kreitzerda700ce2015-09-16 13:27:30 +0000210 // i.e. if any jump threading is undoing previous threading in the path, then
Dinesh Dwivedi8bb5fb02014-06-19 14:11:53 +0000211 // we will loop forever. We take care of this issue by not jump threading for
212 // back edges. This works for normal cases but not for unreachable blocks as
213 // they may have cycle with no back edge.
David Majnemerd9833ea2016-01-10 07:13:04 +0000214 bool EverChanged = false;
215 EverChanged |= removeUnreachableBlocks(F, LVI);
Dinesh Dwivedi657105e2014-06-17 14:34:19 +0000216
Chris Lattnerd579cb12009-05-04 02:28:08 +0000217 FindLoopHeaders(F);
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000218
David Majnemerd9833ea2016-01-10 07:13:04 +0000219 bool Changed;
Benjamin Kramer76e27662010-01-07 13:50:07 +0000220 do {
221 Changed = false;
Chris Lattner595c7272008-12-03 07:48:08 +0000222 for (Function::iterator I = F.begin(), E = F.end(); I != E;) {
Duncan P. N. Exon Smith3a9c9e32015-10-13 18:26:00 +0000223 BasicBlock *BB = &*I;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000224 // Thread all of the branches we can over this block.
Chris Lattner595c7272008-12-03 07:48:08 +0000225 while (ProcessBlock(BB))
Chris Lattnerff1c6e32008-04-20 22:39:42 +0000226 Changed = true;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000227
Chris Lattner595c7272008-12-03 07:48:08 +0000228 ++I;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000229
Chris Lattner595c7272008-12-03 07:48:08 +0000230 // If the block is trivially dead, zap it. This eliminates the successor
231 // edges which simplifies the CFG.
Ramkumar Ramachandra40c3e032015-01-13 03:46:47 +0000232 if (pred_empty(BB) &&
Chris Lattnere5983702008-12-08 22:44:07 +0000233 BB != &BB->getParent()->getEntryBlock()) {
David Greene1efdb452010-01-05 01:27:19 +0000234 DEBUG(dbgs() << " JT: Deleting dead block '" << BB->getName()
Chris Lattner97b14052009-10-11 07:24:57 +0000235 << "' with terminator: " << *BB->getTerminator() << '\n');
Chris Lattnerd579cb12009-05-04 02:28:08 +0000236 LoopHeaders.erase(BB);
Owen Andersond361aac2010-09-14 20:57:41 +0000237 LVI->eraseBlock(BB);
Chris Lattnerfa552d72009-05-04 16:29:24 +0000238 DeleteDeadBlock(BB);
Chris Lattner595c7272008-12-03 07:48:08 +0000239 Changed = true;
Chris Lattner73a58622010-12-13 02:38:13 +0000240 continue;
241 }
Owen Anderson92651ec2011-04-14 21:35:50 +0000242
Chris Lattner73a58622010-12-13 02:38:13 +0000243 BranchInst *BI = dyn_cast<BranchInst>(BB->getTerminator());
Owen Anderson92651ec2011-04-14 21:35:50 +0000244
Chris Lattner73a58622010-12-13 02:38:13 +0000245 // Can't thread an unconditional jump, but if the block is "almost
246 // empty", we can replace uses of it with uses of the successor and make
247 // this dead.
Hyojin Sung4673f102016-03-29 04:08:57 +0000248 // We should not eliminate the loop header either, because eliminating
249 // a loop header might later prevent LoopSimplify from transforming nested
250 // loops into simplified form.
Chris Lattner73a58622010-12-13 02:38:13 +0000251 if (BI && BI->isUnconditional() &&
252 BB != &BB->getParent()->getEntryBlock() &&
Chris Lattner80e7e5a2009-11-10 21:40:01 +0000253 // If the terminator is the only non-phi instruction, try to nuke it.
Hyojin Sung4673f102016-03-29 04:08:57 +0000254 BB->getFirstNonPHIOrDbg()->isTerminator() && !LoopHeaders.count(BB)) {
Chris Lattner73a58622010-12-13 02:38:13 +0000255 // Since TryToSimplifyUncondBranchFromEmptyBlock may delete the
256 // block, we have to make sure it isn't in the LoopHeaders set. We
257 // reinsert afterward if needed.
258 bool ErasedFromLoopHeaders = LoopHeaders.erase(BB);
259 BasicBlock *Succ = BI->getSuccessor(0);
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000260
Chris Lattner73a58622010-12-13 02:38:13 +0000261 // FIXME: It is always conservatively correct to drop the info
262 // for a block even if it doesn't get erased. This isn't totally
263 // awesome, but it allows us to use AssertingVH to prevent nasty
264 // dangling pointer issues within LazyValueInfo.
265 LVI->eraseBlock(BB);
266 if (TryToSimplifyUncondBranchFromEmptyBlock(BB)) {
267 Changed = true;
268 // If we deleted BB and BB was the header of a loop, then the
269 // successor is now the header of the loop.
270 BB = Succ;
Chris Lattner80e7e5a2009-11-10 21:40:01 +0000271 }
Chris Lattner73a58622010-12-13 02:38:13 +0000272
273 if (ErasedFromLoopHeaders)
274 LoopHeaders.insert(BB);
Chris Lattner595c7272008-12-03 07:48:08 +0000275 }
276 }
Chris Lattnerff1c6e32008-04-20 22:39:42 +0000277 EverChanged |= Changed;
Benjamin Kramer76e27662010-01-07 13:50:07 +0000278 } while (Changed);
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000279
Chris Lattnerd579cb12009-05-04 02:28:08 +0000280 LoopHeaders.clear();
Chris Lattnerff1c6e32008-04-20 22:39:42 +0000281 return EverChanged;
Chris Lattnerb3b60072008-04-20 20:35:01 +0000282}
Chris Lattner21157222008-04-20 21:13:06 +0000283
Chris Lattner97b14052009-10-11 07:24:57 +0000284/// getJumpThreadDuplicationCost - Return the cost of duplicating this block to
Nadav Rotem23495312012-12-03 17:34:44 +0000285/// thread across it. Stop scanning the block when passing the threshold.
286static unsigned getJumpThreadDuplicationCost(const BasicBlock *BB,
287 unsigned Threshold) {
Chris Lattner97b14052009-10-11 07:24:57 +0000288 /// Ignore PHI nodes, these will be flattened when duplication happens.
Duncan P. N. Exon Smith3a9c9e32015-10-13 18:26:00 +0000289 BasicBlock::const_iterator I(BB->getFirstNonPHI());
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000290
Chris Lattner3a2ae902009-11-11 00:21:58 +0000291 // FIXME: THREADING will delete values that are just used to compute the
292 // branch, so they shouldn't count against the duplication cost.
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000293
Geoff Berry43dc2852015-12-29 18:10:16 +0000294 unsigned Bonus = 0;
295 const TerminatorInst *BBTerm = BB->getTerminator();
296 // Threading through a switch statement is particularly profitable. If this
297 // block ends in a switch, decrease its cost to make it more likely to happen.
298 if (isa<SwitchInst>(BBTerm))
299 Bonus = 6;
300
301 // The same holds for indirect branches, but slightly more so.
302 if (isa<IndirectBrInst>(BBTerm))
303 Bonus = 8;
304
305 // Bump the threshold up so the early exit from the loop doesn't skip the
306 // terminator-based Size adjustment at the end.
307 Threshold += Bonus;
308
Chris Lattner97b14052009-10-11 07:24:57 +0000309 // Sum up the cost of each instruction until we get to the terminator. Don't
310 // include the terminator because the copy won't include it.
311 unsigned Size = 0;
312 for (; !isa<TerminatorInst>(I); ++I) {
Nadav Rotem23495312012-12-03 17:34:44 +0000313
314 // Stop scanning the block if we've reached the threshold.
315 if (Size > Threshold)
316 return Size;
317
Chris Lattner97b14052009-10-11 07:24:57 +0000318 // Debugger intrinsics don't incur code size.
319 if (isa<DbgInfoIntrinsic>(I)) continue;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000320
Chris Lattner97b14052009-10-11 07:24:57 +0000321 // If this is a pointer->pointer bitcast, it is free.
Duncan Sands19d0b472010-02-16 11:11:14 +0000322 if (isa<BitCastInst>(I) && I->getType()->isPointerTy())
Chris Lattner97b14052009-10-11 07:24:57 +0000323 continue;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000324
David Majnemerb611e3f2015-08-14 05:09:07 +0000325 // Bail out if this instruction gives back a token type, it is not possible
David L Kreitzerda700ce2015-09-16 13:27:30 +0000326 // to duplicate it if it is used outside this BB.
David Majnemerb611e3f2015-08-14 05:09:07 +0000327 if (I->getType()->isTokenTy() && I->isUsedOutsideOfBlock(BB))
328 return ~0U;
329
Chris Lattner97b14052009-10-11 07:24:57 +0000330 // All other instructions count for at least one unit.
331 ++Size;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000332
Chris Lattner97b14052009-10-11 07:24:57 +0000333 // Calls are more expensive. If they are non-intrinsic calls, we model them
334 // as having cost of 4. If they are a non-vector intrinsic, we model them
335 // as having cost of 2 total, and if they are a vector intrinsic, we model
336 // them as having cost 1.
337 if (const CallInst *CI = dyn_cast<CallInst>(I)) {
Jingyue Wue84f6712015-08-31 06:10:27 +0000338 if (CI->cannotDuplicate() || CI->isConvergent())
James Molloy4f6fb952012-12-20 16:04:27 +0000339 // Blocks with NoDuplicate are modelled as having infinite cost, so they
340 // are never duplicated.
341 return ~0U;
342 else if (!isa<IntrinsicInst>(CI))
Chris Lattner97b14052009-10-11 07:24:57 +0000343 Size += 3;
Duncan Sands19d0b472010-02-16 11:11:14 +0000344 else if (!CI->getType()->isVectorTy())
Chris Lattner97b14052009-10-11 07:24:57 +0000345 Size += 1;
346 }
347 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000348
Geoff Berry43dc2852015-12-29 18:10:16 +0000349 return Size > Bonus ? Size - Bonus : 0;
Chris Lattner97b14052009-10-11 07:24:57 +0000350}
351
Chris Lattnerfa552d72009-05-04 16:29:24 +0000352/// FindLoopHeaders - We do not want jump threading to turn proper loop
Chris Lattnerd579cb12009-05-04 02:28:08 +0000353/// structures into irreducible loops. Doing this breaks up the loop nesting
354/// hierarchy and pessimizes later transformations. To prevent this from
355/// happening, we first have to find the loop headers. Here we approximate this
356/// by finding targets of backedges in the CFG.
357///
358/// Note that there definitely are cases when we want to allow threading of
359/// edges across a loop header. For example, threading a jump from outside the
360/// loop (the preheader) to an exit block of the loop is definitely profitable.
361/// It is also almost always profitable to thread backedges from within the loop
362/// to exit blocks, and is often profitable to thread backedges to other blocks
363/// within the loop (forming a nested loop). This simple analysis is not rich
364/// enough to track all of these properties and keep it up-to-date as the CFG
365/// mutates, so we don't allow any of these transformations.
366///
367void JumpThreading::FindLoopHeaders(Function &F) {
368 SmallVector<std::pair<const BasicBlock*,const BasicBlock*>, 32> Edges;
369 FindFunctionBackedges(F, Edges);
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000370
Benjamin Kramer543762d2016-01-09 18:43:01 +0000371 for (const auto &Edge : Edges)
372 LoopHeaders.insert(Edge.second);
Chris Lattnerd579cb12009-05-04 02:28:08 +0000373}
374
Frits van Bommel76244862010-12-05 19:06:41 +0000375/// getKnownConstant - Helper method to determine if we can thread over a
376/// terminator with the given value as its condition, and if so what value to
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000377/// use for that. What kind of value this is depends on whether we want an
378/// integer or a block address, but an undef is always accepted.
Frits van Bommel76244862010-12-05 19:06:41 +0000379/// Returns null if Val is null or not an appropriate constant.
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000380static Constant *getKnownConstant(Value *Val, ConstantPreference Preference) {
Frits van Bommel76244862010-12-05 19:06:41 +0000381 if (!Val)
Craig Topperf40110f2014-04-25 05:29:35 +0000382 return nullptr;
Frits van Bommel76244862010-12-05 19:06:41 +0000383
384 // Undef is "known" enough.
385 if (UndefValue *U = dyn_cast<UndefValue>(Val))
386 return U;
387
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000388 if (Preference == WantBlockAddress)
389 return dyn_cast<BlockAddress>(Val->stripPointerCasts());
390
Frits van Bommel76244862010-12-05 19:06:41 +0000391 return dyn_cast<ConstantInt>(Val);
392}
393
Chris Lattner5ff7f562009-11-07 08:05:03 +0000394/// ComputeValueKnownInPredecessors - Given a basic block BB and a value V, see
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000395/// if we can infer that the value is a known ConstantInt/BlockAddress or undef
396/// in any of our predecessors. If so, return the known list of value and pred
397/// BB in the result vector.
Chris Lattner5ff7f562009-11-07 08:05:03 +0000398///
Chris Lattner5ff7f562009-11-07 08:05:03 +0000399/// This returns true if there were any known values.
400///
Haicheng Wu64d9d7c2016-03-15 23:38:47 +0000401bool JumpThreading::
402ComputeValueKnownInPredecessors(Value *V, BasicBlock *BB, PredValueInfo &Result,
403 ConstantPreference Preference,
404 Instruction *CxtI) {
Owen Anderson6fdcb1722010-08-31 19:24:27 +0000405 // This method walks up use-def chains recursively. Because of this, we could
406 // get into an infinite loop going around loops in the use-def chain. To
407 // prevent this, keep track of what (value, block) pairs we've already visited
408 // and terminate the search if we loop back to them
Owen Anderson3997a072010-08-31 07:36:34 +0000409 if (!RecursionSet.insert(std::make_pair(V, BB)).second)
410 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000411
Owen Anderson6fdcb1722010-08-31 19:24:27 +0000412 // An RAII help to remove this pair from the recursion set once the recursion
413 // stack pops back out again.
414 RecursionSetRemover remover(RecursionSet, std::make_pair(V, BB));
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000415
Frits van Bommel76244862010-12-05 19:06:41 +0000416 // If V is a constant, then it is known in all predecessors.
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000417 if (Constant *KC = getKnownConstant(V, Preference)) {
Benjamin Kramer543762d2016-01-09 18:43:01 +0000418 for (BasicBlock *Pred : predecessors(BB))
419 Result.push_back(std::make_pair(KC, Pred));
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000420
Haicheng Wub35f7722016-02-08 17:00:39 +0000421 return !Result.empty();
Chris Lattner5ff7f562009-11-07 08:05:03 +0000422 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000423
Chris Lattner5ff7f562009-11-07 08:05:03 +0000424 // If V is a non-instruction value, or an instruction in a different block,
425 // then it can't be derived from a PHI.
Haicheng Wu64d9d7c2016-03-15 23:38:47 +0000426 Instruction *I = dyn_cast<Instruction>(V);
Craig Topperf40110f2014-04-25 05:29:35 +0000427 if (!I || I->getParent() != BB) {
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000428
Chris Lattnerfde1f8d2009-11-11 02:08:33 +0000429 // Okay, if this is a live-in value, see if it has a known value at the end
430 // of any of our predecessors.
431 //
432 // FIXME: This should be an edge property, not a block end property.
433 /// TODO: Per PR2563, we could infer value range information about a
434 /// predecessor based on its terminator.
435 //
Owen Andersond361aac2010-09-14 20:57:41 +0000436 // FIXME: change this to use the more-rich 'getPredicateOnEdge' method if
437 // "I" is a non-local compare-with-a-constant instruction. This would be
438 // able to handle value inequalities better, for example if the compare is
439 // "X < 4" and "X < 3" is known true but "X < 4" itself is not available.
440 // Perhaps getConstantOnEdge should be smart enough to do this?
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000441
Benjamin Kramer543762d2016-01-09 18:43:01 +0000442 for (BasicBlock *P : predecessors(BB)) {
Owen Andersond361aac2010-09-14 20:57:41 +0000443 // If the value is known by LazyValueInfo to be a constant in a
444 // predecessor, use that information to try to thread this block.
Hal Finkel7e184492014-09-07 20:29:59 +0000445 Constant *PredCst = LVI->getConstantOnEdge(V, P, BB, CxtI);
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000446 if (Constant *KC = getKnownConstant(PredCst, Preference))
Frits van Bommel76244862010-12-05 19:06:41 +0000447 Result.push_back(std::make_pair(KC, P));
Owen Andersond361aac2010-09-14 20:57:41 +0000448 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000449
Owen Andersond361aac2010-09-14 20:57:41 +0000450 return !Result.empty();
Chris Lattnerfde1f8d2009-11-11 02:08:33 +0000451 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000452
Chris Lattner5ff7f562009-11-07 08:05:03 +0000453 /// If I is a PHI node, then we know the incoming values for any constants.
454 if (PHINode *PN = dyn_cast<PHINode>(I)) {
455 for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) {
456 Value *InVal = PN->getIncomingValue(i);
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000457 if (Constant *KC = getKnownConstant(InVal, Preference)) {
Frits van Bommel76244862010-12-05 19:06:41 +0000458 Result.push_back(std::make_pair(KC, PN->getIncomingBlock(i)));
Owen Andersond361aac2010-09-14 20:57:41 +0000459 } else {
Owen Andersonbd2ecc72010-08-26 17:40:24 +0000460 Constant *CI = LVI->getConstantOnEdge(InVal,
Hal Finkel7e184492014-09-07 20:29:59 +0000461 PN->getIncomingBlock(i),
462 BB, CxtI);
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000463 if (Constant *KC = getKnownConstant(CI, Preference))
464 Result.push_back(std::make_pair(KC, PN->getIncomingBlock(i)));
Chris Lattner5ff7f562009-11-07 08:05:03 +0000465 }
466 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000467
Chris Lattner5ff7f562009-11-07 08:05:03 +0000468 return !Result.empty();
469 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000470
Haicheng Wu78738572016-03-16 04:52:52 +0000471 // Handle Cast instructions. Only see through Cast when the source operand is
472 // PHI or Cmp and the source type is i1 to save the compilation time.
473 if (CastInst *CI = dyn_cast<CastInst>(I)) {
474 Value *Source = CI->getOperand(0);
475 if (!Source->getType()->isIntegerTy(1))
476 return false;
477 if (!isa<PHINode>(Source) && !isa<CmpInst>(Source))
478 return false;
479 ComputeValueKnownInPredecessors(Source, BB, Result, Preference, CxtI);
480 if (Result.empty())
481 return false;
482
483 // Convert the known values.
484 for (auto &R : Result)
485 R.first = ConstantExpr::getCast(CI->getOpcode(), R.first, CI->getType());
486
487 return true;
488 }
489
Frits van Bommel76244862010-12-05 19:06:41 +0000490 PredValueInfoTy LHSVals, RHSVals;
Chris Lattner5ff7f562009-11-07 08:05:03 +0000491
492 // Handle some boolean conditions.
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000493 if (I->getType()->getPrimitiveSizeInBits() == 1) {
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000494 assert(Preference == WantInteger && "One-bit non-integer type?");
Chris Lattner5ff7f562009-11-07 08:05:03 +0000495 // X | true -> true
496 // X & false -> false
497 if (I->getOpcode() == Instruction::Or ||
498 I->getOpcode() == Instruction::And) {
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000499 ComputeValueKnownInPredecessors(I->getOperand(0), BB, LHSVals,
Haicheng Wu64d9d7c2016-03-15 23:38:47 +0000500 WantInteger, CxtI);
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000501 ComputeValueKnownInPredecessors(I->getOperand(1), BB, RHSVals,
Haicheng Wu64d9d7c2016-03-15 23:38:47 +0000502 WantInteger, CxtI);
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000503
Owen Anderson6fdcb1722010-08-31 19:24:27 +0000504 if (LHSVals.empty() && RHSVals.empty())
Chris Lattner5ff7f562009-11-07 08:05:03 +0000505 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000506
Chris Lattner5ff7f562009-11-07 08:05:03 +0000507 ConstantInt *InterestingVal;
508 if (I->getOpcode() == Instruction::Or)
509 InterestingVal = ConstantInt::getTrue(I->getContext());
510 else
511 InterestingVal = ConstantInt::getFalse(I->getContext());
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000512
Chris Lattner3c603022010-08-18 03:14:36 +0000513 SmallPtrSet<BasicBlock*, 4> LHSKnownBBs;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000514
Chris Lattnerd924f632010-02-11 04:40:44 +0000515 // Scan for the sentinel. If we find an undef, force it to the
516 // interesting value: x|undef -> true and x&undef -> false.
Benjamin Kramer543762d2016-01-09 18:43:01 +0000517 for (const auto &LHSVal : LHSVals)
518 if (LHSVal.first == InterestingVal || isa<UndefValue>(LHSVal.first)) {
519 Result.emplace_back(InterestingVal, LHSVal.second);
520 LHSKnownBBs.insert(LHSVal.second);
Chris Lattnerd924f632010-02-11 04:40:44 +0000521 }
Benjamin Kramer543762d2016-01-09 18:43:01 +0000522 for (const auto &RHSVal : RHSVals)
523 if (RHSVal.first == InterestingVal || isa<UndefValue>(RHSVal.first)) {
Chris Lattnerbbc25ff2010-07-12 00:47:34 +0000524 // If we already inferred a value for this block on the LHS, don't
525 // re-add it.
Benjamin Kramer543762d2016-01-09 18:43:01 +0000526 if (!LHSKnownBBs.count(RHSVal.second))
527 Result.emplace_back(InterestingVal, RHSVal.second);
Chris Lattnerd924f632010-02-11 04:40:44 +0000528 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000529
Chris Lattner5ff7f562009-11-07 08:05:03 +0000530 return !Result.empty();
531 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000532
Chris Lattner9518fbb2009-11-10 22:39:16 +0000533 // Handle the NOT form of XOR.
534 if (I->getOpcode() == Instruction::Xor &&
535 isa<ConstantInt>(I->getOperand(1)) &&
536 cast<ConstantInt>(I->getOperand(1))->isOne()) {
Haicheng Wu64d9d7c2016-03-15 23:38:47 +0000537 ComputeValueKnownInPredecessors(I->getOperand(0), BB, Result,
538 WantInteger, CxtI);
Owen Anderson6fdcb1722010-08-31 19:24:27 +0000539 if (Result.empty())
Chris Lattner9518fbb2009-11-10 22:39:16 +0000540 return false;
541
542 // Invert the known values.
Benjamin Kramer543762d2016-01-09 18:43:01 +0000543 for (auto &R : Result)
544 R.first = ConstantExpr::getNot(R.first);
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000545
Chris Lattner9518fbb2009-11-10 22:39:16 +0000546 return true;
547 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000548
Owen Andersonbd2ecc72010-08-26 17:40:24 +0000549 // Try to simplify some other binary operator values.
550 } else if (BinaryOperator *BO = dyn_cast<BinaryOperator>(I)) {
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000551 assert(Preference != WantBlockAddress
552 && "A binary operator creating a block address?");
Owen Anderson3c84ecb2010-08-31 20:26:04 +0000553 if (ConstantInt *CI = dyn_cast<ConstantInt>(BO->getOperand(1))) {
Frits van Bommel76244862010-12-05 19:06:41 +0000554 PredValueInfoTy LHSVals;
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000555 ComputeValueKnownInPredecessors(BO->getOperand(0), BB, LHSVals,
Haicheng Wu64d9d7c2016-03-15 23:38:47 +0000556 WantInteger, CxtI);
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000557
Owen Anderson3997a072010-08-31 07:36:34 +0000558 // Try to use constant folding to simplify the binary operator.
Benjamin Kramer543762d2016-01-09 18:43:01 +0000559 for (const auto &LHSVal : LHSVals) {
560 Constant *V = LHSVal.first;
Owen Anderson3c84ecb2010-08-31 20:26:04 +0000561 Constant *Folded = ConstantExpr::get(BO->getOpcode(), V, CI);
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000562
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000563 if (Constant *KC = getKnownConstant(Folded, WantInteger))
Benjamin Kramer543762d2016-01-09 18:43:01 +0000564 Result.push_back(std::make_pair(KC, LHSVal.second));
Owen Anderson3997a072010-08-31 07:36:34 +0000565 }
Owen Andersonbd2ecc72010-08-26 17:40:24 +0000566 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000567
Owen Anderson3997a072010-08-31 07:36:34 +0000568 return !Result.empty();
Chris Lattner5ff7f562009-11-07 08:05:03 +0000569 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000570
Chris Lattner5ff7f562009-11-07 08:05:03 +0000571 // Handle compare with phi operand, where the PHI is defined in this block.
572 if (CmpInst *Cmp = dyn_cast<CmpInst>(I)) {
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000573 assert(Preference == WantInteger && "Compares only produce integers");
Chris Lattner5ff7f562009-11-07 08:05:03 +0000574 PHINode *PN = dyn_cast<PHINode>(Cmp->getOperand(0));
575 if (PN && PN->getParent() == BB) {
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000576 const DataLayout &DL = PN->getModule()->getDataLayout();
Chris Lattner5ff7f562009-11-07 08:05:03 +0000577 // We can do this simplification if any comparisons fold to true or false.
578 // See if any do.
579 for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) {
580 BasicBlock *PredBB = PN->getIncomingBlock(i);
581 Value *LHS = PN->getIncomingValue(i);
582 Value *RHS = Cmp->getOperand(1)->DoPHITranslation(BB, PredBB);
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000583
Rafael Espindola37dc9e12014-02-21 00:06:31 +0000584 Value *Res = SimplifyCmpInst(Cmp->getPredicate(), LHS, RHS, DL);
Craig Topperf40110f2014-04-25 05:29:35 +0000585 if (!Res) {
Owen Andersond361aac2010-09-14 20:57:41 +0000586 if (!isa<Constant>(RHS))
Chris Lattner5f6b8b22009-11-12 05:24:05 +0000587 continue;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000588
589 LazyValueInfo::Tristate
Chris Lattner5f6b8b22009-11-12 05:24:05 +0000590 ResT = LVI->getPredicateOnEdge(Cmp->getPredicate(), LHS,
Hal Finkel7e184492014-09-07 20:29:59 +0000591 cast<Constant>(RHS), PredBB, BB,
592 CxtI ? CxtI : Cmp);
Chris Lattner5f6b8b22009-11-12 05:24:05 +0000593 if (ResT == LazyValueInfo::Unknown)
594 continue;
595 Res = ConstantInt::get(Type::getInt1Ty(LHS->getContext()), ResT);
596 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000597
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000598 if (Constant *KC = getKnownConstant(Res, WantInteger))
599 Result.push_back(std::make_pair(KC, PredBB));
Chris Lattner5ff7f562009-11-07 08:05:03 +0000600 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000601
Chris Lattner5ff7f562009-11-07 08:05:03 +0000602 return !Result.empty();
603 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000604
Chris Lattner67146692009-11-11 22:31:38 +0000605 // If comparing a live-in value against a constant, see if we know the
606 // live-in value on any predecessors.
Owen Andersond361aac2010-09-14 20:57:41 +0000607 if (isa<Constant>(Cmp->getOperand(1)) && Cmp->getType()->isIntegerTy()) {
Owen Andersonbd2ecc72010-08-26 17:40:24 +0000608 if (!isa<Instruction>(Cmp->getOperand(0)) ||
Owen Andersonc910acb2010-08-30 23:22:36 +0000609 cast<Instruction>(Cmp->getOperand(0))->getParent() != BB) {
Owen Andersonbd2ecc72010-08-26 17:40:24 +0000610 Constant *RHSCst = cast<Constant>(Cmp->getOperand(1));
Gabor Greifa5fa8852010-07-12 14:10:24 +0000611
Benjamin Kramer543762d2016-01-09 18:43:01 +0000612 for (BasicBlock *P : predecessors(BB)) {
Owen Andersonbd2ecc72010-08-26 17:40:24 +0000613 // If the value is known by LazyValueInfo to be a constant in a
614 // predecessor, use that information to try to thread this block.
615 LazyValueInfo::Tristate Res =
616 LVI->getPredicateOnEdge(Cmp->getPredicate(), Cmp->getOperand(0),
Hal Finkel7e184492014-09-07 20:29:59 +0000617 RHSCst, P, BB, CxtI ? CxtI : Cmp);
Owen Andersonbd2ecc72010-08-26 17:40:24 +0000618 if (Res == LazyValueInfo::Unknown)
619 continue;
Chris Lattnerc893c4ed2009-11-12 04:37:50 +0000620
Owen Andersonbd2ecc72010-08-26 17:40:24 +0000621 Constant *ResC = ConstantInt::get(Cmp->getType(), Res);
Frits van Bommel76244862010-12-05 19:06:41 +0000622 Result.push_back(std::make_pair(ResC, P));
Owen Andersonbd2ecc72010-08-26 17:40:24 +0000623 }
624
625 return !Result.empty();
Chris Lattner67146692009-11-11 22:31:38 +0000626 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000627
Owen Anderson3997a072010-08-31 07:36:34 +0000628 // Try to find a constant value for the LHS of a comparison,
Owen Andersonbd2ecc72010-08-26 17:40:24 +0000629 // and evaluate it statically if we can.
Owen Andersonc910acb2010-08-30 23:22:36 +0000630 if (Constant *CmpConst = dyn_cast<Constant>(Cmp->getOperand(1))) {
Frits van Bommel76244862010-12-05 19:06:41 +0000631 PredValueInfoTy LHSVals;
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000632 ComputeValueKnownInPredecessors(I->getOperand(0), BB, LHSVals,
Haicheng Wu64d9d7c2016-03-15 23:38:47 +0000633 WantInteger, CxtI);
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000634
Benjamin Kramer543762d2016-01-09 18:43:01 +0000635 for (const auto &LHSVal : LHSVals) {
636 Constant *V = LHSVal.first;
Owen Anderson3c84ecb2010-08-31 20:26:04 +0000637 Constant *Folded = ConstantExpr::getCompare(Cmp->getPredicate(),
638 V, CmpConst);
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000639 if (Constant *KC = getKnownConstant(Folded, WantInteger))
Benjamin Kramer543762d2016-01-09 18:43:01 +0000640 Result.push_back(std::make_pair(KC, LHSVal.second));
Owen Andersonbd2ecc72010-08-26 17:40:24 +0000641 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000642
Owen Andersonbd2ecc72010-08-26 17:40:24 +0000643 return !Result.empty();
644 }
Chris Lattner67146692009-11-11 22:31:38 +0000645 }
Chris Lattner5ff7f562009-11-07 08:05:03 +0000646 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000647
Frits van Bommel3d180342010-12-15 09:51:20 +0000648 if (SelectInst *SI = dyn_cast<SelectInst>(I)) {
649 // Handle select instructions where at least one operand is a known constant
650 // and we can figure out the condition value for any predecessor block.
651 Constant *TrueVal = getKnownConstant(SI->getTrueValue(), Preference);
652 Constant *FalseVal = getKnownConstant(SI->getFalseValue(), Preference);
653 PredValueInfoTy Conds;
654 if ((TrueVal || FalseVal) &&
655 ComputeValueKnownInPredecessors(SI->getCondition(), BB, Conds,
Haicheng Wu64d9d7c2016-03-15 23:38:47 +0000656 WantInteger, CxtI)) {
Benjamin Kramer543762d2016-01-09 18:43:01 +0000657 for (auto &C : Conds) {
658 Constant *Cond = C.first;
Frits van Bommel3d180342010-12-15 09:51:20 +0000659
660 // Figure out what value to use for the condition.
661 bool KnownCond;
662 if (ConstantInt *CI = dyn_cast<ConstantInt>(Cond)) {
663 // A known boolean.
664 KnownCond = CI->isOne();
665 } else {
666 assert(isa<UndefValue>(Cond) && "Unexpected condition value");
667 // Either operand will do, so be sure to pick the one that's a known
668 // constant.
669 // FIXME: Do this more cleverly if both values are known constants?
Craig Topperf40110f2014-04-25 05:29:35 +0000670 KnownCond = (TrueVal != nullptr);
Frits van Bommel3d180342010-12-15 09:51:20 +0000671 }
672
673 // See if the select has a known constant value for this predecessor.
674 if (Constant *Val = KnownCond ? TrueVal : FalseVal)
Benjamin Kramer543762d2016-01-09 18:43:01 +0000675 Result.push_back(std::make_pair(Val, C.second));
Frits van Bommel3d180342010-12-15 09:51:20 +0000676 }
677
678 return !Result.empty();
679 }
680 }
681
Owen Andersond361aac2010-09-14 20:57:41 +0000682 // If all else fails, see if LVI can figure out a constant value for us.
Hal Finkel7e184492014-09-07 20:29:59 +0000683 Constant *CI = LVI->getConstant(V, BB, CxtI);
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000684 if (Constant *KC = getKnownConstant(CI, Preference)) {
Benjamin Kramer543762d2016-01-09 18:43:01 +0000685 for (BasicBlock *Pred : predecessors(BB))
686 Result.push_back(std::make_pair(KC, Pred));
Owen Andersonbd2ecc72010-08-26 17:40:24 +0000687 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000688
Owen Andersond361aac2010-09-14 20:57:41 +0000689 return !Result.empty();
Chris Lattner5ff7f562009-11-07 08:05:03 +0000690}
691
692
Chris Lattner3df4c152008-04-22 07:05:46 +0000693
Chris Lattnerf99a74e2009-10-11 04:18:15 +0000694/// GetBestDestForBranchOnUndef - If we determine that the specified block ends
695/// in an undefined jump, decide which block is best to revector to.
696///
697/// Since we can pick an arbitrary destination, we pick the successor with the
698/// fewest predecessors. This should reduce the in-degree of the others.
699///
700static unsigned GetBestDestForJumpOnUndef(BasicBlock *BB) {
701 TerminatorInst *BBTerm = BB->getTerminator();
702 unsigned MinSucc = 0;
703 BasicBlock *TestBB = BBTerm->getSuccessor(MinSucc);
704 // Compute the successor with the minimum number of predecessors.
705 unsigned MinNumPreds = std::distance(pred_begin(TestBB), pred_end(TestBB));
706 for (unsigned i = 1, e = BBTerm->getNumSuccessors(); i != e; ++i) {
707 TestBB = BBTerm->getSuccessor(i);
708 unsigned NumPreds = std::distance(pred_begin(TestBB), pred_end(TestBB));
Jakub Staszak423651e2011-06-27 21:51:12 +0000709 if (NumPreds < MinNumPreds) {
Chris Lattnerf99a74e2009-10-11 04:18:15 +0000710 MinSucc = i;
Jakub Staszak423651e2011-06-27 21:51:12 +0000711 MinNumPreds = NumPreds;
712 }
Chris Lattnerf99a74e2009-10-11 04:18:15 +0000713 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000714
Chris Lattnerf99a74e2009-10-11 04:18:15 +0000715 return MinSucc;
716}
717
Chris Lattner1a924e72011-02-18 04:43:06 +0000718static bool hasAddressTakenAndUsed(BasicBlock *BB) {
719 if (!BB->hasAddressTaken()) return false;
Owen Anderson92651ec2011-04-14 21:35:50 +0000720
Chris Lattner1a924e72011-02-18 04:43:06 +0000721 // If the block has its address taken, it may be a tree of dead constants
722 // hanging off of it. These shouldn't keep the block alive.
723 BlockAddress *BA = BlockAddress::get(BB);
724 BA->removeDeadConstantUsers();
725 return !BA->use_empty();
726}
727
Chris Lattner240051a2008-11-27 07:20:04 +0000728/// ProcessBlock - If there are any predecessors whose control can be threaded
Chris Lattner21157222008-04-20 21:13:06 +0000729/// through to a successor, transform them now.
Chris Lattner240051a2008-11-27 07:20:04 +0000730bool JumpThreading::ProcessBlock(BasicBlock *BB) {
Chris Lattnerde5ab4862010-01-23 18:56:07 +0000731 // If the block is trivially dead, just return and let the caller nuke it.
732 // This simplifies other transformations.
Ramkumar Ramachandra40c3e032015-01-13 03:46:47 +0000733 if (pred_empty(BB) &&
Chris Lattnerde5ab4862010-01-23 18:56:07 +0000734 BB != &BB->getParent()->getEntryBlock())
735 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000736
Chris Lattner98d89d12008-11-27 05:07:53 +0000737 // If this block has a single predecessor, and if that pred has a single
738 // successor, merge the blocks. This encourages recursive jump threading
739 // because now the condition in this block can be threaded through
740 // predecessors of our predecessor block.
Chris Lattner5ff7f562009-11-07 08:05:03 +0000741 if (BasicBlock *SinglePred = BB->getSinglePredecessor()) {
David Majnemer654e1302015-07-31 17:58:14 +0000742 const TerminatorInst *TI = SinglePred->getTerminator();
743 if (!TI->isExceptional() && TI->getNumSuccessors() == 1 &&
Chris Lattner1a924e72011-02-18 04:43:06 +0000744 SinglePred != BB && !hasAddressTakenAndUsed(BB)) {
Chris Lattnerd579cb12009-05-04 02:28:08 +0000745 // If SinglePred was a loop header, BB becomes one.
746 if (LoopHeaders.erase(SinglePred))
747 LoopHeaders.insert(BB);
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000748
Owen Andersond361aac2010-09-14 20:57:41 +0000749 LVI->eraseBlock(SinglePred);
Chris Lattner98d89d12008-11-27 05:07:53 +0000750 MergeBasicBlockIntoOnlyPred(BB);
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000751
Chris Lattner98d89d12008-11-27 05:07:53 +0000752 return true;
753 }
Chris Lattner5ff7f562009-11-07 08:05:03 +0000754 }
755
Haicheng Wua6a32792016-01-08 19:39:39 +0000756 if (TryToUnfoldSelectInCurrBB(BB))
757 return true;
758
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000759 // What kind of constant we're looking for.
760 ConstantPreference Preference = WantInteger;
761
762 // Look to see if the terminator is a conditional branch, switch or indirect
763 // branch, if not we can't thread it.
Chris Lattner21157222008-04-20 21:13:06 +0000764 Value *Condition;
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000765 Instruction *Terminator = BB->getTerminator();
766 if (BranchInst *BI = dyn_cast<BranchInst>(Terminator)) {
Chris Lattnerff1c6e32008-04-20 22:39:42 +0000767 // Can't thread an unconditional jump.
768 if (BI->isUnconditional()) return false;
Chris Lattner21157222008-04-20 21:13:06 +0000769 Condition = BI->getCondition();
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000770 } else if (SwitchInst *SI = dyn_cast<SwitchInst>(Terminator)) {
Chris Lattner21157222008-04-20 21:13:06 +0000771 Condition = SI->getCondition();
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000772 } else if (IndirectBrInst *IB = dyn_cast<IndirectBrInst>(Terminator)) {
Richard Osborne0ab2b0d2012-07-20 10:36:17 +0000773 // Can't thread indirect branch with no successors.
774 if (IB->getNumSuccessors() == 0) return false;
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000775 Condition = IB->getAddress()->stripPointerCasts();
776 Preference = WantBlockAddress;
777 } else {
Chris Lattner21157222008-04-20 21:13:06 +0000778 return false; // Must be an invoke.
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000779 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000780
Owen Anderson92651ec2011-04-14 21:35:50 +0000781 // Run constant folding to see if we can reduce the condition to a simple
782 // constant.
783 if (Instruction *I = dyn_cast<Instruction>(Condition)) {
Mehdi Aminia28d91d2015-03-10 02:37:25 +0000784 Value *SimpleVal =
785 ConstantFoldInstruction(I, BB->getModule()->getDataLayout(), TLI);
Owen Anderson92651ec2011-04-14 21:35:50 +0000786 if (SimpleVal) {
787 I->replaceAllUsesWith(SimpleVal);
788 I->eraseFromParent();
789 Condition = SimpleVal;
790 }
791 }
792
Chris Lattner595c7272008-12-03 07:48:08 +0000793 // If the terminator is branching on an undef, we can pick any of the
Chris Lattnerf99a74e2009-10-11 04:18:15 +0000794 // successors to branch to. Let GetBestDestForJumpOnUndef decide.
Chris Lattner595c7272008-12-03 07:48:08 +0000795 if (isa<UndefValue>(Condition)) {
Chris Lattnerf99a74e2009-10-11 04:18:15 +0000796 unsigned BestSucc = GetBestDestForJumpOnUndef(BB);
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000797
Chris Lattner595c7272008-12-03 07:48:08 +0000798 // Fold the branch/switch.
Chris Lattnerf99a74e2009-10-11 04:18:15 +0000799 TerminatorInst *BBTerm = BB->getTerminator();
Chris Lattner595c7272008-12-03 07:48:08 +0000800 for (unsigned i = 0, e = BBTerm->getNumSuccessors(); i != e; ++i) {
Chris Lattnerf99a74e2009-10-11 04:18:15 +0000801 if (i == BestSucc) continue;
Owen Anderson99c985c2010-09-29 20:34:41 +0000802 BBTerm->getSuccessor(i)->removePredecessor(BB, true);
Chris Lattner595c7272008-12-03 07:48:08 +0000803 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000804
David Greene1efdb452010-01-05 01:27:19 +0000805 DEBUG(dbgs() << " In block '" << BB->getName()
Chris Lattner97b14052009-10-11 07:24:57 +0000806 << "' folding undef terminator: " << *BBTerm << '\n');
Chris Lattnerf99a74e2009-10-11 04:18:15 +0000807 BranchInst::Create(BBTerm->getSuccessor(BestSucc), BBTerm);
Chris Lattner595c7272008-12-03 07:48:08 +0000808 BBTerm->eraseFromParent();
809 return true;
810 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000811
Frits van Bommel76244862010-12-05 19:06:41 +0000812 // If the terminator of this block is branching on a constant, simplify the
813 // terminator to an unconditional branch. This can occur due to threading in
814 // other blocks.
Frits van Bommeld9df6ea2010-12-06 23:36:56 +0000815 if (getKnownConstant(Condition, Preference)) {
Frits van Bommel76244862010-12-05 19:06:41 +0000816 DEBUG(dbgs() << " In block '" << BB->getName()
817 << "' folding terminator: " << *BB->getTerminator() << '\n');
818 ++NumFolds;
Frits van Bommelad964552011-05-22 16:24:18 +0000819 ConstantFoldTerminator(BB, true);
Frits van Bommel76244862010-12-05 19:06:41 +0000820 return true;
821 }
822
Chris Lattner595c7272008-12-03 07:48:08 +0000823 Instruction *CondInst = dyn_cast<Instruction>(Condition);
824
Chris Lattner595c7272008-12-03 07:48:08 +0000825 // All the rest of our checks depend on the condition being an instruction.
Craig Topperf40110f2014-04-25 05:29:35 +0000826 if (!CondInst) {
Chris Lattnerba456162009-11-12 01:41:34 +0000827 // FIXME: Unify this with code below.
Hal Finkel7e184492014-09-07 20:29:59 +0000828 if (ProcessThreadableEdges(Condition, BB, Preference, Terminator))
Chris Lattnerba456162009-11-12 01:41:34 +0000829 return true;
Chris Lattner595c7272008-12-03 07:48:08 +0000830 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000831 }
832
833
Nick Lewycky77585a22009-06-19 04:56:29 +0000834 if (CmpInst *CondCmp = dyn_cast<CmpInst>(CondInst)) {
Philip Reames66ab0f02015-06-16 00:49:59 +0000835 // If we're branching on a conditional, LVI might be able to determine
Eric Christopher572e03a2015-06-19 01:53:21 +0000836 // it's value at the branch instruction. We only handle comparisons
Philip Reames66ab0f02015-06-16 00:49:59 +0000837 // against a constant at this time.
Larisse Voufo532bf712015-09-18 19:14:35 +0000838 // TODO: This should be extended to handle switches as well.
Owen Anderson6ebbd922010-08-27 17:12:29 +0000839 BranchInst *CondBr = dyn_cast<BranchInst>(BB->getTerminator());
840 Constant *CondConst = dyn_cast<Constant>(CondCmp->getOperand(1));
Philip Reames66ab0f02015-06-16 00:49:59 +0000841 if (CondBr && CondConst && CondBr->isConditional()) {
Hal Finkel7e184492014-09-07 20:29:59 +0000842 LazyValueInfo::Tristate Ret =
843 LVI->getPredicateAt(CondCmp->getPredicate(), CondCmp->getOperand(0),
Philip Reames66ab0f02015-06-16 00:49:59 +0000844 CondConst, CondBr);
Hal Finkel7e184492014-09-07 20:29:59 +0000845 if (Ret != LazyValueInfo::Unknown) {
846 unsigned ToRemove = Ret == LazyValueInfo::True ? 1 : 0;
847 unsigned ToKeep = Ret == LazyValueInfo::True ? 0 : 1;
848 CondBr->getSuccessor(ToRemove)->removePredecessor(BB, true);
849 BranchInst::Create(CondBr->getSuccessor(ToKeep), CondBr);
850 CondBr->eraseFromParent();
Philip Reames66ab0f02015-06-16 00:49:59 +0000851 if (CondCmp->use_empty())
852 CondCmp->eraseFromParent();
853 else if (CondCmp->getParent() == BB) {
854 // If the fact we just learned is true for all uses of the
855 // condition, replace it with a constant value
856 auto *CI = Ret == LazyValueInfo::True ?
857 ConstantInt::getTrue(CondCmp->getType()) :
858 ConstantInt::getFalse(CondCmp->getType());
859 CondCmp->replaceAllUsesWith(CI);
860 CondCmp->eraseFromParent();
861 }
Hal Finkel7e184492014-09-07 20:29:59 +0000862 return true;
863 }
Owen Anderson6ebbd922010-08-27 17:12:29 +0000864 }
Benjamin Kramer6a4976d2013-08-07 10:29:38 +0000865
866 if (CondBr && CondConst && TryToUnfoldSelect(CondCmp, BB))
867 return true;
Nick Lewycky77585a22009-06-19 04:56:29 +0000868 }
Chris Lattner98d89d12008-11-27 05:07:53 +0000869
870 // Check for some cases that are worth simplifying. Right now we want to look
871 // for loads that are used by a switch or by the condition for the branch. If
872 // we see one, check to see if it's partially redundant. If so, insert a PHI
873 // which can then be used to thread the values.
874 //
Chris Lattner595c7272008-12-03 07:48:08 +0000875 Value *SimplifyValue = CondInst;
Chris Lattner98d89d12008-11-27 05:07:53 +0000876 if (CmpInst *CondCmp = dyn_cast<CmpInst>(SimplifyValue))
877 if (isa<Constant>(CondCmp->getOperand(1)))
878 SimplifyValue = CondCmp->getOperand(0);
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000879
Chris Lattner9d9812a2009-11-15 19:58:31 +0000880 // TODO: There are other places where load PRE would be profitable, such as
881 // more complex comparisons.
Chris Lattner98d89d12008-11-27 05:07:53 +0000882 if (LoadInst *LI = dyn_cast<LoadInst>(SimplifyValue))
883 if (SimplifyPartiallyRedundantLoad(LI))
884 return true;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000885
886
Chris Lattner5ff7f562009-11-07 08:05:03 +0000887 // Handle a variety of cases where we are branching on something derived from
888 // a PHI node in the current block. If we can prove that any predecessors
889 // compute a predictable value based on a PHI node, thread those predecessors.
890 //
Hal Finkel7e184492014-09-07 20:29:59 +0000891 if (ProcessThreadableEdges(CondInst, BB, Preference, Terminator))
Chris Lattnerfde1f8d2009-11-11 02:08:33 +0000892 return true;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000893
Chris Lattner6a19ed02010-01-11 23:41:09 +0000894 // If this is an otherwise-unfoldable branch on a phi node in the current
895 // block, see if we can simplify.
896 if (PHINode *PN = dyn_cast<PHINode>(CondInst))
897 if (PN->getParent() == BB && isa<BranchInst>(BB->getTerminator()))
898 return ProcessBranchOnPHI(PN);
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000899
900
Chris Lattnereb73bdb2010-01-12 02:07:17 +0000901 // If this is an otherwise-unfoldable branch on a XOR, see if we can simplify.
902 if (CondInst->getOpcode() == Instruction::Xor &&
903 CondInst->getParent() == BB && isa<BranchInst>(BB->getTerminator()))
904 return ProcessBranchOnXOR(cast<BinaryOperator>(CondInst));
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000905
Sanjoy Das13e63a22015-10-28 21:27:08 +0000906 // Search for a stronger dominating condition that can be used to simplify a
907 // conditional branch leaving BB.
908 if (ProcessImpliedCondition(BB))
909 return true;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000910
Sanjoy Das13e63a22015-10-28 21:27:08 +0000911 return false;
912}
913
914bool JumpThreading::ProcessImpliedCondition(BasicBlock *BB) {
915 auto *BI = dyn_cast<BranchInst>(BB->getTerminator());
916 if (!BI || !BI->isConditional())
917 return false;
918
919 Value *Cond = BI->getCondition();
920 BasicBlock *CurrentBB = BB;
921 BasicBlock *CurrentPred = BB->getSinglePredecessor();
922 unsigned Iter = 0;
923
Sanjoy Das55ea67c2015-11-06 19:01:08 +0000924 auto &DL = BB->getModule()->getDataLayout();
925
Sanjoy Das13e63a22015-10-28 21:27:08 +0000926 while (CurrentPred && Iter++ < ImplicationSearchThreshold) {
927 auto *PBI = dyn_cast<BranchInst>(CurrentPred->getTerminator());
928 if (!PBI || !PBI->isConditional() || PBI->getSuccessor(0) != CurrentBB)
929 return false;
930
Sanjoy Das55ea67c2015-11-06 19:01:08 +0000931 if (isImpliedCondition(PBI->getCondition(), Cond, DL)) {
Sanjoy Das13e63a22015-10-28 21:27:08 +0000932 BI->getSuccessor(1)->removePredecessor(BB);
933 BranchInst::Create(BI->getSuccessor(0), BI);
934 BI->eraseFromParent();
935 return true;
936 }
937 CurrentBB = CurrentPred;
938 CurrentPred = CurrentBB->getSinglePredecessor();
939 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000940
Chris Lattnere369c352008-04-22 06:36:15 +0000941 return false;
942}
943
Chris Lattner98d89d12008-11-27 05:07:53 +0000944/// SimplifyPartiallyRedundantLoad - If LI is an obviously partially redundant
945/// load instruction, eliminate it by replacing it with a PHI node. This is an
946/// important optimization that encourages jump threading, and needs to be run
947/// interlaced with other jump threading tasks.
948bool JumpThreading::SimplifyPartiallyRedundantLoad(LoadInst *LI) {
Eli Friedman7c5dc122011-09-12 20:23:13 +0000949 // Don't hack volatile/atomic loads.
950 if (!LI->isSimple()) return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000951
Chris Lattner98d89d12008-11-27 05:07:53 +0000952 // If the load is defined in a block with exactly one predecessor, it can't be
953 // partially redundant.
954 BasicBlock *LoadBB = LI->getParent();
955 if (LoadBB->getSinglePredecessor())
956 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000957
David Majnemereb518bd2015-08-04 08:21:40 +0000958 // If the load is defined in an EH pad, it can't be partially redundant,
959 // because the edges between the invoke and the EH pad cannot have other
Bill Wendling90dd90a2013-10-21 04:09:17 +0000960 // instructions between them.
David Majnemereb518bd2015-08-04 08:21:40 +0000961 if (LoadBB->isEHPad())
Bill Wendling90dd90a2013-10-21 04:09:17 +0000962 return false;
963
Chris Lattner98d89d12008-11-27 05:07:53 +0000964 Value *LoadedPtr = LI->getOperand(0);
965
966 // If the loaded operand is defined in the LoadBB, it can't be available.
Chris Lattner9d9812a2009-11-15 19:58:31 +0000967 // TODO: Could do simple PHI translation, that would be fun :)
Chris Lattner98d89d12008-11-27 05:07:53 +0000968 if (Instruction *PtrOp = dyn_cast<Instruction>(LoadedPtr))
969 if (PtrOp->getParent() == LoadBB)
970 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000971
Chris Lattner98d89d12008-11-27 05:07:53 +0000972 // Scan a few instructions up from the load, to see if it is obviously live at
973 // the entry to its block.
Duncan P. N. Exon Smith3a9c9e32015-10-13 18:26:00 +0000974 BasicBlock::iterator BBIt(LI);
Chris Lattner98d89d12008-11-27 05:07:53 +0000975
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000976 if (Value *AvailableVal =
Eduard Burtescue2a69172016-01-22 01:51:51 +0000977 FindAvailableLoadedValue(LI, LoadBB, BBIt, DefMaxInstsToScan)) {
David L Kreitzerda700ce2015-09-16 13:27:30 +0000978 // If the value of the load is locally available within the block, just use
Chris Lattner98d89d12008-11-27 05:07:53 +0000979 // it. This frequently occurs for reg2mem'd allocas.
980 //cerr << "LOAD ELIMINATED:\n" << *BBIt << *LI << "\n";
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000981
Chris Lattner482eb702009-01-09 06:08:12 +0000982 // If the returned value is the load itself, replace with an undef. This can
983 // only happen in dead loops.
Owen Andersonb292b8c2009-07-30 23:03:37 +0000984 if (AvailableVal == LI) AvailableVal = UndefValue::get(LI->getType());
Chandler Carrutheeec35a2014-10-20 00:24:14 +0000985 if (AvailableVal->getType() != LI->getType())
Chandler Carruth1a3c2c42014-11-25 08:20:27 +0000986 AvailableVal =
987 CastInst::CreateBitOrPointerCast(AvailableVal, LI->getType(), "", LI);
Chris Lattner98d89d12008-11-27 05:07:53 +0000988 LI->replaceAllUsesWith(AvailableVal);
989 LI->eraseFromParent();
990 return true;
991 }
992
993 // Otherwise, if we scanned the whole block and got to the top of the block,
994 // we know the block is locally transparent to the load. If not, something
995 // might clobber its value.
996 if (BBIt != LoadBB->begin())
997 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000998
Hal Finkelcc39b672014-07-24 12:16:19 +0000999 // If all of the loads and stores that feed the value have the same AA tags,
1000 // then we can propagate them onto any newly inserted loads.
1001 AAMDNodes AATags;
1002 LI->getAAMetadata(AATags);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001003
Chris Lattner98d89d12008-11-27 05:07:53 +00001004 SmallPtrSet<BasicBlock*, 8> PredsScanned;
1005 typedef SmallVector<std::pair<BasicBlock*, Value*>, 8> AvailablePredsTy;
1006 AvailablePredsTy AvailablePreds;
Craig Topperf40110f2014-04-25 05:29:35 +00001007 BasicBlock *OneUnavailablePred = nullptr;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001008
Chris Lattner98d89d12008-11-27 05:07:53 +00001009 // If we got here, the loaded value is transparent through to the start of the
1010 // block. Check to see if it is available in any of the predecessor blocks.
Benjamin Kramer543762d2016-01-09 18:43:01 +00001011 for (BasicBlock *PredBB : predecessors(LoadBB)) {
Chris Lattner98d89d12008-11-27 05:07:53 +00001012 // If we already scanned this predecessor, skip it.
David Blaikie70573dc2014-11-19 07:49:26 +00001013 if (!PredsScanned.insert(PredBB).second)
Chris Lattner98d89d12008-11-27 05:07:53 +00001014 continue;
1015
1016 // Scan the predecessor to see if the value is available in the pred.
1017 BBIt = PredBB->end();
Hal Finkelcc39b672014-07-24 12:16:19 +00001018 AAMDNodes ThisAATags;
Eduard Burtescue2a69172016-01-22 01:51:51 +00001019 Value *PredAvailable = FindAvailableLoadedValue(LI, PredBB, BBIt,
Larisse Voufo532bf712015-09-18 19:14:35 +00001020 DefMaxInstsToScan,
Hal Finkelcc39b672014-07-24 12:16:19 +00001021 nullptr, &ThisAATags);
Chris Lattner98d89d12008-11-27 05:07:53 +00001022 if (!PredAvailable) {
1023 OneUnavailablePred = PredBB;
1024 continue;
1025 }
Nadav Rotem465834c2012-07-24 10:51:42 +00001026
Hal Finkelcc39b672014-07-24 12:16:19 +00001027 // If AA tags disagree or are not present, forget about them.
1028 if (AATags != ThisAATags) AATags = AAMDNodes();
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001029
Chris Lattner98d89d12008-11-27 05:07:53 +00001030 // If so, this load is partially redundant. Remember this info so that we
1031 // can create a PHI node.
1032 AvailablePreds.push_back(std::make_pair(PredBB, PredAvailable));
1033 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001034
Chris Lattner98d89d12008-11-27 05:07:53 +00001035 // If the loaded value isn't available in any predecessor, it isn't partially
1036 // redundant.
1037 if (AvailablePreds.empty()) return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001038
Chris Lattner98d89d12008-11-27 05:07:53 +00001039 // Okay, the loaded value is available in at least one (and maybe all!)
1040 // predecessors. If the value is unavailable in more than one unique
1041 // predecessor, we want to insert a merge block for those common predecessors.
1042 // This ensures that we only have to insert one reload, thus not increasing
1043 // code size.
Craig Topperf40110f2014-04-25 05:29:35 +00001044 BasicBlock *UnavailablePred = nullptr;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001045
Chris Lattner98d89d12008-11-27 05:07:53 +00001046 // If there is exactly one predecessor where the value is unavailable, the
1047 // already computed 'OneUnavailablePred' block is it. If it ends in an
1048 // unconditional branch, we know that it isn't a critical edge.
1049 if (PredsScanned.size() == AvailablePreds.size()+1 &&
1050 OneUnavailablePred->getTerminator()->getNumSuccessors() == 1) {
1051 UnavailablePred = OneUnavailablePred;
1052 } else if (PredsScanned.size() != AvailablePreds.size()) {
1053 // Otherwise, we had multiple unavailable predecessors or we had a critical
1054 // edge from the one.
1055 SmallVector<BasicBlock*, 8> PredsToSplit;
1056 SmallPtrSet<BasicBlock*, 8> AvailablePredSet;
1057
Benjamin Kramer543762d2016-01-09 18:43:01 +00001058 for (const auto &AvailablePred : AvailablePreds)
1059 AvailablePredSet.insert(AvailablePred.first);
Chris Lattner98d89d12008-11-27 05:07:53 +00001060
1061 // Add all the unavailable predecessors to the PredsToSplit list.
Benjamin Kramer543762d2016-01-09 18:43:01 +00001062 for (BasicBlock *P : predecessors(LoadBB)) {
Chris Lattner329ea062010-06-14 19:45:43 +00001063 // If the predecessor is an indirect goto, we can't split the edge.
Gabor Greifa5fa8852010-07-12 14:10:24 +00001064 if (isa<IndirectBrInst>(P->getTerminator()))
Chris Lattner329ea062010-06-14 19:45:43 +00001065 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001066
Gabor Greifa5fa8852010-07-12 14:10:24 +00001067 if (!AvailablePredSet.count(P))
1068 PredsToSplit.push_back(P);
Chris Lattner329ea062010-06-14 19:45:43 +00001069 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001070
Chris Lattner98d89d12008-11-27 05:07:53 +00001071 // Split them out to their own block.
Manman Ren72d44b12015-10-15 14:59:40 +00001072 UnavailablePred = SplitBlockPreds(LoadBB, PredsToSplit, "thread-pre-split");
Chris Lattner98d89d12008-11-27 05:07:53 +00001073 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001074
Chris Lattner98d89d12008-11-27 05:07:53 +00001075 // If the value isn't available in all predecessors, then there will be
1076 // exactly one where it isn't available. Insert a load on that edge and add
1077 // it to the AvailablePreds list.
1078 if (UnavailablePred) {
1079 assert(UnavailablePred->getTerminator()->getNumSuccessors() == 1 &&
1080 "Can't handle critical edge here!");
Devang Patel306f8db2011-05-04 22:48:19 +00001081 LoadInst *NewVal = new LoadInst(LoadedPtr, LI->getName()+".pr", false,
Chris Lattner9d9812a2009-11-15 19:58:31 +00001082 LI->getAlignment(),
Chris Lattner98d89d12008-11-27 05:07:53 +00001083 UnavailablePred->getTerminator());
Devang Patel306f8db2011-05-04 22:48:19 +00001084 NewVal->setDebugLoc(LI->getDebugLoc());
Hal Finkelcc39b672014-07-24 12:16:19 +00001085 if (AATags)
1086 NewVal->setAAMetadata(AATags);
Nadav Rotem465834c2012-07-24 10:51:42 +00001087
Chris Lattner98d89d12008-11-27 05:07:53 +00001088 AvailablePreds.push_back(std::make_pair(UnavailablePred, NewVal));
1089 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001090
Chris Lattner98d89d12008-11-27 05:07:53 +00001091 // Now we know that each predecessor of this block has a value in
1092 // AvailablePreds, sort them for efficient access as we're walking the preds.
Chris Lattner2b07d3c2008-12-01 06:52:57 +00001093 array_pod_sort(AvailablePreds.begin(), AvailablePreds.end());
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001094
Chris Lattner98d89d12008-11-27 05:07:53 +00001095 // Create a PHI node at the start of the block for the PRE'd load value.
Jay Foade0938d82011-03-30 11:19:20 +00001096 pred_iterator PB = pred_begin(LoadBB), PE = pred_end(LoadBB);
Jay Foad52131342011-03-30 11:28:46 +00001097 PHINode *PN = PHINode::Create(LI->getType(), std::distance(PB, PE), "",
Duncan P. N. Exon Smith3a9c9e32015-10-13 18:26:00 +00001098 &LoadBB->front());
Chris Lattner98d89d12008-11-27 05:07:53 +00001099 PN->takeName(LI);
Devang Patel306f8db2011-05-04 22:48:19 +00001100 PN->setDebugLoc(LI->getDebugLoc());
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001101
Chris Lattner98d89d12008-11-27 05:07:53 +00001102 // Insert new entries into the PHI for each predecessor. A single block may
1103 // have multiple entries here.
Jay Foade0938d82011-03-30 11:19:20 +00001104 for (pred_iterator PI = PB; PI != PE; ++PI) {
Gabor Greifa5fa8852010-07-12 14:10:24 +00001105 BasicBlock *P = *PI;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001106 AvailablePredsTy::iterator I =
Chris Lattner98d89d12008-11-27 05:07:53 +00001107 std::lower_bound(AvailablePreds.begin(), AvailablePreds.end(),
Craig Topperf40110f2014-04-25 05:29:35 +00001108 std::make_pair(P, (Value*)nullptr));
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001109
Gabor Greifa5fa8852010-07-12 14:10:24 +00001110 assert(I != AvailablePreds.end() && I->first == P &&
Chris Lattner98d89d12008-11-27 05:07:53 +00001111 "Didn't find entry for predecessor!");
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001112
Chandler Carrutheeec35a2014-10-20 00:24:14 +00001113 // If we have an available predecessor but it requires casting, insert the
Chandler Carruth6665d622014-10-20 05:34:36 +00001114 // cast in the predecessor and use the cast. Note that we have to update the
1115 // AvailablePreds vector as we go so that all of the PHI entries for this
1116 // predecessor use the same bitcast.
1117 Value *&PredV = I->second;
Chandler Carrutheeec35a2014-10-20 00:24:14 +00001118 if (PredV->getType() != LI->getType())
Chandler Carruth1a3c2c42014-11-25 08:20:27 +00001119 PredV = CastInst::CreateBitOrPointerCast(PredV, LI->getType(), "",
1120 P->getTerminator());
Chandler Carrutheeec35a2014-10-20 00:24:14 +00001121
1122 PN->addIncoming(PredV, I->first);
Chris Lattner98d89d12008-11-27 05:07:53 +00001123 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001124
Chris Lattner98d89d12008-11-27 05:07:53 +00001125 //cerr << "PRE: " << *LI << *PN << "\n";
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001126
Chris Lattner98d89d12008-11-27 05:07:53 +00001127 LI->replaceAllUsesWith(PN);
1128 LI->eraseFromParent();
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001129
Chris Lattner98d89d12008-11-27 05:07:53 +00001130 return true;
1131}
1132
Chris Lattner5ff7f562009-11-07 08:05:03 +00001133/// FindMostPopularDest - The specified list contains multiple possible
1134/// threadable destinations. Pick the one that occurs the most frequently in
1135/// the list.
1136static BasicBlock *
1137FindMostPopularDest(BasicBlock *BB,
1138 const SmallVectorImpl<std::pair<BasicBlock*,
1139 BasicBlock*> > &PredToDestList) {
1140 assert(!PredToDestList.empty());
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001141
Chris Lattner5ff7f562009-11-07 08:05:03 +00001142 // Determine popularity. If there are multiple possible destinations, we
1143 // explicitly choose to ignore 'undef' destinations. We prefer to thread
1144 // blocks with known and real destinations to threading undef. We'll handle
1145 // them later if interesting.
1146 DenseMap<BasicBlock*, unsigned> DestPopularity;
Benjamin Kramer543762d2016-01-09 18:43:01 +00001147 for (const auto &PredToDest : PredToDestList)
1148 if (PredToDest.second)
1149 DestPopularity[PredToDest.second]++;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001150
Chris Lattner5ff7f562009-11-07 08:05:03 +00001151 // Find the most popular dest.
1152 DenseMap<BasicBlock*, unsigned>::iterator DPI = DestPopularity.begin();
1153 BasicBlock *MostPopularDest = DPI->first;
1154 unsigned Popularity = DPI->second;
1155 SmallVector<BasicBlock*, 4> SamePopularity;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001156
Chris Lattner5ff7f562009-11-07 08:05:03 +00001157 for (++DPI; DPI != DestPopularity.end(); ++DPI) {
1158 // If the popularity of this entry isn't higher than the popularity we've
1159 // seen so far, ignore it.
1160 if (DPI->second < Popularity)
1161 ; // ignore.
1162 else if (DPI->second == Popularity) {
1163 // If it is the same as what we've seen so far, keep track of it.
1164 SamePopularity.push_back(DPI->first);
1165 } else {
1166 // If it is more popular, remember it.
1167 SamePopularity.clear();
1168 MostPopularDest = DPI->first;
1169 Popularity = DPI->second;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001170 }
Chris Lattner5ff7f562009-11-07 08:05:03 +00001171 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001172
Frits van Bommel9bbe8492010-12-16 12:16:00 +00001173 // Okay, now we know the most popular destination. If there is more than one
Chris Lattner5ff7f562009-11-07 08:05:03 +00001174 // destination, we need to determine one. This is arbitrary, but we need
1175 // to make a deterministic decision. Pick the first one that appears in the
1176 // successor list.
1177 if (!SamePopularity.empty()) {
1178 SamePopularity.push_back(MostPopularDest);
1179 TerminatorInst *TI = BB->getTerminator();
1180 for (unsigned i = 0; ; ++i) {
1181 assert(i != TI->getNumSuccessors() && "Didn't find any successor!");
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001182
Chris Lattner5ff7f562009-11-07 08:05:03 +00001183 if (std::find(SamePopularity.begin(), SamePopularity.end(),
1184 TI->getSuccessor(i)) == SamePopularity.end())
1185 continue;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001186
Chris Lattner5ff7f562009-11-07 08:05:03 +00001187 MostPopularDest = TI->getSuccessor(i);
1188 break;
1189 }
1190 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001191
Chris Lattner5ff7f562009-11-07 08:05:03 +00001192 // Okay, we have finally picked the most popular destination.
1193 return MostPopularDest;
1194}
1195
Frits van Bommeld9df6ea2010-12-06 23:36:56 +00001196bool JumpThreading::ProcessThreadableEdges(Value *Cond, BasicBlock *BB,
Hal Finkel7e184492014-09-07 20:29:59 +00001197 ConstantPreference Preference,
1198 Instruction *CxtI) {
Chris Lattner5ff7f562009-11-07 08:05:03 +00001199 // If threading this would thread across a loop header, don't even try to
1200 // thread the edge.
1201 if (LoopHeaders.count(BB))
1202 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001203
Frits van Bommel76244862010-12-05 19:06:41 +00001204 PredValueInfoTy PredValues;
Haicheng Wu64d9d7c2016-03-15 23:38:47 +00001205 if (!ComputeValueKnownInPredecessors(Cond, BB, PredValues, Preference, CxtI))
1206 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001207
Chris Lattner5ff7f562009-11-07 08:05:03 +00001208 assert(!PredValues.empty() &&
1209 "ComputeValueKnownInPredecessors returned true with no values");
1210
David Greene1efdb452010-01-05 01:27:19 +00001211 DEBUG(dbgs() << "IN BB: " << *BB;
Benjamin Kramer543762d2016-01-09 18:43:01 +00001212 for (const auto &PredValue : PredValues) {
Frits van Bommel76244862010-12-05 19:06:41 +00001213 dbgs() << " BB '" << BB->getName() << "': FOUND condition = "
Benjamin Kramer543762d2016-01-09 18:43:01 +00001214 << *PredValue.first
1215 << " for pred '" << PredValue.second->getName() << "'.\n";
Chris Lattner5ff7f562009-11-07 08:05:03 +00001216 });
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001217
Chris Lattner5ff7f562009-11-07 08:05:03 +00001218 // Decide what we want to thread through. Convert our list of known values to
1219 // a list of known destinations for each pred. This also discards duplicate
1220 // predecessors and keeps track of the undefined inputs (which are represented
Chris Lattnerea465e22009-11-09 00:41:49 +00001221 // as a null dest in the PredToDestList).
Chris Lattner5ff7f562009-11-07 08:05:03 +00001222 SmallPtrSet<BasicBlock*, 16> SeenPreds;
1223 SmallVector<std::pair<BasicBlock*, BasicBlock*>, 16> PredToDestList;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001224
Craig Topperf40110f2014-04-25 05:29:35 +00001225 BasicBlock *OnlyDest = nullptr;
Chris Lattner5ff7f562009-11-07 08:05:03 +00001226 BasicBlock *MultipleDestSentinel = (BasicBlock*)(intptr_t)~0ULL;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001227
Benjamin Kramer543762d2016-01-09 18:43:01 +00001228 for (const auto &PredValue : PredValues) {
1229 BasicBlock *Pred = PredValue.second;
David Blaikie70573dc2014-11-19 07:49:26 +00001230 if (!SeenPreds.insert(Pred).second)
Chris Lattner5ff7f562009-11-07 08:05:03 +00001231 continue; // Duplicate predecessor entry.
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001232
Chris Lattner5ff7f562009-11-07 08:05:03 +00001233 // If the predecessor ends with an indirect goto, we can't change its
1234 // destination.
1235 if (isa<IndirectBrInst>(Pred->getTerminator()))
1236 continue;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001237
Benjamin Kramer543762d2016-01-09 18:43:01 +00001238 Constant *Val = PredValue.first;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001239
Chris Lattner5ff7f562009-11-07 08:05:03 +00001240 BasicBlock *DestBB;
Frits van Bommel76244862010-12-05 19:06:41 +00001241 if (isa<UndefValue>(Val))
Craig Topperf40110f2014-04-25 05:29:35 +00001242 DestBB = nullptr;
Chris Lattner5ff7f562009-11-07 08:05:03 +00001243 else if (BranchInst *BI = dyn_cast<BranchInst>(BB->getTerminator()))
Frits van Bommel76244862010-12-05 19:06:41 +00001244 DestBB = BI->getSuccessor(cast<ConstantInt>(Val)->isZero());
Stepan Dyatkovskiy513aaa52012-02-01 07:49:51 +00001245 else if (SwitchInst *SI = dyn_cast<SwitchInst>(BB->getTerminator())) {
Stepan Dyatkovskiy5b648af2012-03-08 07:06:20 +00001246 DestBB = SI->findCaseValue(cast<ConstantInt>(Val)).getCaseSuccessor();
Stepan Dyatkovskiy513aaa52012-02-01 07:49:51 +00001247 } else {
Frits van Bommeld9df6ea2010-12-06 23:36:56 +00001248 assert(isa<IndirectBrInst>(BB->getTerminator())
1249 && "Unexpected terminator");
1250 DestBB = cast<BlockAddress>(Val)->getBasicBlock();
Chris Lattner5ff7f562009-11-07 08:05:03 +00001251 }
1252
1253 // If we have exactly one destination, remember it for efficiency below.
Frits van Bommel9bbe8492010-12-16 12:16:00 +00001254 if (PredToDestList.empty())
Chris Lattner5ff7f562009-11-07 08:05:03 +00001255 OnlyDest = DestBB;
1256 else if (OnlyDest != DestBB)
1257 OnlyDest = MultipleDestSentinel;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001258
Chris Lattner5ff7f562009-11-07 08:05:03 +00001259 PredToDestList.push_back(std::make_pair(Pred, DestBB));
1260 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001261
Chris Lattner5ff7f562009-11-07 08:05:03 +00001262 // If all edges were unthreadable, we fail.
1263 if (PredToDestList.empty())
Haicheng Wu64d9d7c2016-03-15 23:38:47 +00001264 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001265
Chris Lattner5ff7f562009-11-07 08:05:03 +00001266 // Determine which is the most common successor. If we have many inputs and
1267 // this block is a switch, we want to start by threading the batch that goes
1268 // to the most popular destination first. If we only know about one
1269 // threadable destination (the common case) we can avoid this.
1270 BasicBlock *MostPopularDest = OnlyDest;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001271
Chris Lattner5ff7f562009-11-07 08:05:03 +00001272 if (MostPopularDest == MultipleDestSentinel)
1273 MostPopularDest = FindMostPopularDest(BB, PredToDestList);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001274
Chris Lattner5ff7f562009-11-07 08:05:03 +00001275 // Now that we know what the most popular destination is, factor all
1276 // predecessors that will jump to it into a single predecessor.
1277 SmallVector<BasicBlock*, 16> PredsToFactor;
Benjamin Kramer543762d2016-01-09 18:43:01 +00001278 for (const auto &PredToDest : PredToDestList)
1279 if (PredToDest.second == MostPopularDest) {
1280 BasicBlock *Pred = PredToDest.first;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001281
Chris Lattner5ff7f562009-11-07 08:05:03 +00001282 // This predecessor may be a switch or something else that has multiple
1283 // edges to the block. Factor each of these edges by listing them
1284 // according to # occurrences in PredsToFactor.
Benjamin Kramer543762d2016-01-09 18:43:01 +00001285 for (BasicBlock *Succ : successors(Pred))
1286 if (Succ == BB)
Chris Lattner5ff7f562009-11-07 08:05:03 +00001287 PredsToFactor.push_back(Pred);
1288 }
1289
1290 // If the threadable edges are branching on an undefined value, we get to pick
1291 // the destination that these predecessors should get to.
Craig Topperf40110f2014-04-25 05:29:35 +00001292 if (!MostPopularDest)
Chris Lattner5ff7f562009-11-07 08:05:03 +00001293 MostPopularDest = BB->getTerminator()->
1294 getSuccessor(GetBestDestForJumpOnUndef(BB));
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001295
Chris Lattner5ff7f562009-11-07 08:05:03 +00001296 // Ok, try to thread it!
Haicheng Wu64d9d7c2016-03-15 23:38:47 +00001297 return ThreadEdge(BB, PredsToFactor, MostPopularDest);
Chris Lattner5ff7f562009-11-07 08:05:03 +00001298}
Chris Lattner98d89d12008-11-27 05:07:53 +00001299
Chris Lattner6a19ed02010-01-11 23:41:09 +00001300/// ProcessBranchOnPHI - We have an otherwise unthreadable conditional branch on
1301/// a PHI node in the current block. See if there are any simplifications we
1302/// can do based on inputs to the phi node.
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001303///
Chris Lattner6a19ed02010-01-11 23:41:09 +00001304bool JumpThreading::ProcessBranchOnPHI(PHINode *PN) {
Chris Lattner6ce85e82009-10-11 04:40:21 +00001305 BasicBlock *BB = PN->getParent();
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001306
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001307 // TODO: We could make use of this to do it once for blocks with common PHI
1308 // values.
1309 SmallVector<BasicBlock*, 1> PredBBs;
1310 PredBBs.resize(1);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001311
Chris Lattner5ff7f562009-11-07 08:05:03 +00001312 // If any of the predecessor blocks end in an unconditional branch, we can
Chris Lattner6a19ed02010-01-11 23:41:09 +00001313 // *duplicate* the conditional branch into that block in order to further
1314 // encourage jump threading and to eliminate cases where we have branch on a
1315 // phi of an icmp (branch on icmp is much better).
Chris Lattner97b14052009-10-11 07:24:57 +00001316 for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) {
1317 BasicBlock *PredBB = PN->getIncomingBlock(i);
1318 if (BranchInst *PredBr = dyn_cast<BranchInst>(PredBB->getTerminator()))
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001319 if (PredBr->isUnconditional()) {
1320 PredBBs[0] = PredBB;
1321 // Try to duplicate BB into PredBB.
1322 if (DuplicateCondBranchOnPHIIntoPred(BB, PredBBs))
1323 return true;
1324 }
Chris Lattner97b14052009-10-11 07:24:57 +00001325 }
1326
Chris Lattner6ce85e82009-10-11 04:40:21 +00001327 return false;
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001328}
1329
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001330/// ProcessBranchOnXOR - We have an otherwise unthreadable conditional branch on
1331/// a xor instruction in the current block. See if there are any
1332/// simplifications we can do based on inputs to the xor.
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001333///
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001334bool JumpThreading::ProcessBranchOnXOR(BinaryOperator *BO) {
1335 BasicBlock *BB = BO->getParent();
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001336
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001337 // If either the LHS or RHS of the xor is a constant, don't do this
1338 // optimization.
1339 if (isa<ConstantInt>(BO->getOperand(0)) ||
1340 isa<ConstantInt>(BO->getOperand(1)))
1341 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001342
Chris Lattnerba2d0b82010-01-23 19:16:25 +00001343 // If the first instruction in BB isn't a phi, we won't be able to infer
1344 // anything special about any particular predecessor.
1345 if (!isa<PHINode>(BB->front()))
1346 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001347
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001348 // If we have a xor as the branch input to this block, and we know that the
1349 // LHS or RHS of the xor in any predecessor is true/false, then we can clone
1350 // the condition into the predecessor and fix that value to true, saving some
1351 // logical ops on that path and encouraging other paths to simplify.
1352 //
1353 // This copies something like this:
1354 //
1355 // BB:
1356 // %X = phi i1 [1], [%X']
1357 // %Y = icmp eq i32 %A, %B
1358 // %Z = xor i1 %X, %Y
1359 // br i1 %Z, ...
1360 //
1361 // Into:
1362 // BB':
1363 // %Y = icmp ne i32 %A, %B
David L Kreitzerda700ce2015-09-16 13:27:30 +00001364 // br i1 %Y, ...
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001365
Frits van Bommel76244862010-12-05 19:06:41 +00001366 PredValueInfoTy XorOpValues;
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001367 bool isLHS = true;
Frits van Bommeld9df6ea2010-12-06 23:36:56 +00001368 if (!ComputeValueKnownInPredecessors(BO->getOperand(0), BB, XorOpValues,
Haicheng Wu64d9d7c2016-03-15 23:38:47 +00001369 WantInteger, BO)) {
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001370 assert(XorOpValues.empty());
Frits van Bommeld9df6ea2010-12-06 23:36:56 +00001371 if (!ComputeValueKnownInPredecessors(BO->getOperand(1), BB, XorOpValues,
Haicheng Wu64d9d7c2016-03-15 23:38:47 +00001372 WantInteger, BO))
1373 return false;
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001374 isLHS = false;
1375 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001376
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001377 assert(!XorOpValues.empty() &&
1378 "ComputeValueKnownInPredecessors returned true with no values");
1379
1380 // Scan the information to see which is most popular: true or false. The
1381 // predecessors can be of the set true, false, or undef.
1382 unsigned NumTrue = 0, NumFalse = 0;
Benjamin Kramer543762d2016-01-09 18:43:01 +00001383 for (const auto &XorOpValue : XorOpValues) {
1384 if (isa<UndefValue>(XorOpValue.first))
Frits van Bommel76244862010-12-05 19:06:41 +00001385 // Ignore undefs for the count.
1386 continue;
Benjamin Kramer543762d2016-01-09 18:43:01 +00001387 if (cast<ConstantInt>(XorOpValue.first)->isZero())
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001388 ++NumFalse;
1389 else
1390 ++NumTrue;
1391 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001392
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001393 // Determine which value to split on, true, false, or undef if neither.
Craig Topperf40110f2014-04-25 05:29:35 +00001394 ConstantInt *SplitVal = nullptr;
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001395 if (NumTrue > NumFalse)
1396 SplitVal = ConstantInt::getTrue(BB->getContext());
1397 else if (NumTrue != 0 || NumFalse != 0)
1398 SplitVal = ConstantInt::getFalse(BB->getContext());
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001399
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001400 // Collect all of the blocks that this can be folded into so that we can
1401 // factor this once and clone it once.
1402 SmallVector<BasicBlock*, 8> BlocksToFoldInto;
Benjamin Kramer543762d2016-01-09 18:43:01 +00001403 for (const auto &XorOpValue : XorOpValues) {
1404 if (XorOpValue.first != SplitVal && !isa<UndefValue>(XorOpValue.first))
Frits van Bommel76244862010-12-05 19:06:41 +00001405 continue;
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001406
Benjamin Kramer543762d2016-01-09 18:43:01 +00001407 BlocksToFoldInto.push_back(XorOpValue.second);
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001408 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001409
Chris Lattnerba2d0b82010-01-23 19:16:25 +00001410 // If we inferred a value for all of the predecessors, then duplication won't
1411 // help us. However, we can just replace the LHS or RHS with the constant.
1412 if (BlocksToFoldInto.size() ==
1413 cast<PHINode>(BB->front()).getNumIncomingValues()) {
Craig Topperf40110f2014-04-25 05:29:35 +00001414 if (!SplitVal) {
Chris Lattnerba2d0b82010-01-23 19:16:25 +00001415 // If all preds provide undef, just nuke the xor, because it is undef too.
1416 BO->replaceAllUsesWith(UndefValue::get(BO->getType()));
1417 BO->eraseFromParent();
1418 } else if (SplitVal->isZero()) {
1419 // If all preds provide 0, replace the xor with the other input.
1420 BO->replaceAllUsesWith(BO->getOperand(isLHS));
1421 BO->eraseFromParent();
1422 } else {
1423 // If all preds provide 1, set the computed value to 1.
1424 BO->setOperand(!isLHS, SplitVal);
1425 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001426
Chris Lattnerba2d0b82010-01-23 19:16:25 +00001427 return true;
1428 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001429
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001430 // Try to duplicate BB into PredBB.
Haicheng Wu64d9d7c2016-03-15 23:38:47 +00001431 return DuplicateCondBranchOnPHIIntoPred(BB, BlocksToFoldInto);
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001432}
1433
1434
Chris Lattner97b14052009-10-11 07:24:57 +00001435/// AddPHINodeEntriesForMappedBlock - We're adding 'NewPred' as a new
1436/// predecessor to the PHIBB block. If it has PHI nodes, add entries for
1437/// NewPred using the entries from OldPred (suitably mapped).
1438static void AddPHINodeEntriesForMappedBlock(BasicBlock *PHIBB,
1439 BasicBlock *OldPred,
1440 BasicBlock *NewPred,
1441 DenseMap<Instruction*, Value*> &ValueMap) {
1442 for (BasicBlock::iterator PNI = PHIBB->begin();
1443 PHINode *PN = dyn_cast<PHINode>(PNI); ++PNI) {
1444 // Ok, we have a PHI node. Figure out what the incoming value was for the
1445 // DestBlock.
1446 Value *IV = PN->getIncomingValueForBlock(OldPred);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001447
Chris Lattner97b14052009-10-11 07:24:57 +00001448 // Remap the value if necessary.
1449 if (Instruction *Inst = dyn_cast<Instruction>(IV)) {
1450 DenseMap<Instruction*, Value*>::iterator I = ValueMap.find(Inst);
1451 if (I != ValueMap.end())
1452 IV = I->second;
1453 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001454
Chris Lattner97b14052009-10-11 07:24:57 +00001455 PN->addIncoming(IV, NewPred);
1456 }
1457}
Chris Lattner3df4c152008-04-22 07:05:46 +00001458
Chris Lattner5ff7f562009-11-07 08:05:03 +00001459/// ThreadEdge - We have decided that it is safe and profitable to factor the
1460/// blocks in PredBBs to one predecessor, then thread an edge from it to SuccBB
1461/// across BB. Transform the IR to reflect this change.
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001462bool JumpThreading::ThreadEdge(BasicBlock *BB,
1463 const SmallVectorImpl<BasicBlock*> &PredBBs,
Chris Lattnerf466bc82009-10-11 04:33:43 +00001464 BasicBlock *SuccBB) {
Chris Lattnerd579cb12009-05-04 02:28:08 +00001465 // If threading to the same block as we come from, we would infinite loop.
1466 if (SuccBB == BB) {
David Greene1efdb452010-01-05 01:27:19 +00001467 DEBUG(dbgs() << " Not threading across BB '" << BB->getName()
Daniel Dunbar9813b0b2009-07-26 07:49:05 +00001468 << "' - would thread to self!\n");
Chris Lattnerd579cb12009-05-04 02:28:08 +00001469 return false;
1470 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001471
Chris Lattnerd579cb12009-05-04 02:28:08 +00001472 // If threading this would thread across a loop header, don't thread the edge.
1473 // See the comments above FindLoopHeaders for justifications and caveats.
1474 if (LoopHeaders.count(BB)) {
David Greene1efdb452010-01-05 01:27:19 +00001475 DEBUG(dbgs() << " Not threading across loop header BB '" << BB->getName()
Daniel Dunbar9813b0b2009-07-26 07:49:05 +00001476 << "' to dest BB '" << SuccBB->getName()
1477 << "' - it might create an irreducible loop!\n");
Chris Lattnerd579cb12009-05-04 02:28:08 +00001478 return false;
1479 }
1480
Michael Liaod1209162014-09-24 04:59:06 +00001481 unsigned JumpThreadCost = getJumpThreadDuplicationCost(BB, BBDupThreshold);
1482 if (JumpThreadCost > BBDupThreshold) {
David Greene1efdb452010-01-05 01:27:19 +00001483 DEBUG(dbgs() << " Not threading BB '" << BB->getName()
Chris Lattner97b14052009-10-11 07:24:57 +00001484 << "' - Cost is too high: " << JumpThreadCost << "\n");
1485 return false;
1486 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001487
David L Kreitzerda700ce2015-09-16 13:27:30 +00001488 // And finally, do it! Start by factoring the predecessors if needed.
Chris Lattner5ff7f562009-11-07 08:05:03 +00001489 BasicBlock *PredBB;
1490 if (PredBBs.size() == 1)
1491 PredBB = PredBBs[0];
1492 else {
David Greene1efdb452010-01-05 01:27:19 +00001493 DEBUG(dbgs() << " Factoring out " << PredBBs.size()
Chris Lattner5ff7f562009-11-07 08:05:03 +00001494 << " common predecessors.\n");
Manman Ren72d44b12015-10-15 14:59:40 +00001495 PredBB = SplitBlockPreds(BB, PredBBs, ".thr_comm");
Chris Lattner5ff7f562009-11-07 08:05:03 +00001496 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001497
Chris Lattnerd579cb12009-05-04 02:28:08 +00001498 // And finally, do it!
David Greene1efdb452010-01-05 01:27:19 +00001499 DEBUG(dbgs() << " Threading edge from '" << PredBB->getName() << "' to '"
Daniel Dunbar6115b392009-07-26 09:48:23 +00001500 << SuccBB->getName() << "' with cost: " << JumpThreadCost
Daniel Dunbar9813b0b2009-07-26 07:49:05 +00001501 << ", across block:\n "
1502 << *BB << "\n");
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001503
Owen Andersond361aac2010-09-14 20:57:41 +00001504 LVI->threadEdge(PredBB, BB, SuccBB);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001505
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001506 // We are going to have to map operands from the original BB block to the new
1507 // copy of the block 'NewBB'. If there are PHI nodes in BB, evaluate them to
1508 // account for entry from PredBB.
1509 DenseMap<Instruction*, Value*> ValueMapping;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001510
1511 BasicBlock *NewBB = BasicBlock::Create(BB->getContext(),
1512 BB->getName()+".thread",
Owen Anderson55f1c092009-08-13 21:58:54 +00001513 BB->getParent(), BB);
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001514 NewBB->moveAfter(PredBB);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001515
Manman Ren72d44b12015-10-15 14:59:40 +00001516 // Set the block frequency of NewBB.
1517 if (HasProfileData) {
1518 auto NewBBFreq =
1519 BFI->getBlockFreq(PredBB) * BPI->getEdgeProbability(PredBB, BB);
1520 BFI->setBlockFreq(NewBB, NewBBFreq.getFrequency());
1521 }
1522
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001523 BasicBlock::iterator BI = BB->begin();
1524 for (; PHINode *PN = dyn_cast<PHINode>(BI); ++BI)
1525 ValueMapping[PN] = PN->getIncomingValueForBlock(PredBB);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001526
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001527 // Clone the non-phi instructions of BB into NewBB, keeping track of the
1528 // mapping and using it to remap operands in the cloned instructions.
1529 for (; !isa<TerminatorInst>(BI); ++BI) {
Nick Lewycky42fb7452009-09-27 07:38:41 +00001530 Instruction *New = BI->clone();
Daniel Dunbar6115b392009-07-26 09:48:23 +00001531 New->setName(BI->getName());
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001532 NewBB->getInstList().push_back(New);
Duncan P. N. Exon Smith3a9c9e32015-10-13 18:26:00 +00001533 ValueMapping[&*BI] = New;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001534
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001535 // Remap operands to patch up intra-block references.
1536 for (unsigned i = 0, e = New->getNumOperands(); i != e; ++i)
Dan Gohman43f33dd2009-07-02 00:17:47 +00001537 if (Instruction *Inst = dyn_cast<Instruction>(New->getOperand(i))) {
1538 DenseMap<Instruction*, Value*>::iterator I = ValueMapping.find(Inst);
1539 if (I != ValueMapping.end())
1540 New->setOperand(i, I->second);
1541 }
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001542 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001543
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001544 // We didn't copy the terminator from BB over to NewBB, because there is now
1545 // an unconditional jump to SuccBB. Insert the unconditional jump.
Manman Ren72d44b12015-10-15 14:59:40 +00001546 BranchInst *NewBI = BranchInst::Create(SuccBB, NewBB);
Devang Patel306f8db2011-05-04 22:48:19 +00001547 NewBI->setDebugLoc(BB->getTerminator()->getDebugLoc());
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001548
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001549 // Check to see if SuccBB has PHI nodes. If so, we need to add entries to the
1550 // PHI nodes for NewBB now.
Chris Lattner97b14052009-10-11 07:24:57 +00001551 AddPHINodeEntriesForMappedBlock(SuccBB, BB, NewBB, ValueMapping);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001552
Chris Lattner84095072009-10-10 09:05:58 +00001553 // If there were values defined in BB that are used outside the block, then we
1554 // now have to update all uses of the value to use either the original value,
1555 // the cloned value, or some PHI derived value. This can require arbitrary
1556 // PHI insertion, of which we are prepared to do, clean these up now.
1557 SSAUpdater SSAUpdate;
1558 SmallVector<Use*, 16> UsesToRename;
Benjamin Kramer543762d2016-01-09 18:43:01 +00001559 for (Instruction &I : *BB) {
Chris Lattner84095072009-10-10 09:05:58 +00001560 // Scan all uses of this instruction to see if it is used outside of its
1561 // block, and if so, record them in UsesToRename.
Benjamin Kramer543762d2016-01-09 18:43:01 +00001562 for (Use &U : I.uses()) {
Chandler Carruthcdf47882014-03-09 03:16:01 +00001563 Instruction *User = cast<Instruction>(U.getUser());
Chris Lattner84095072009-10-10 09:05:58 +00001564 if (PHINode *UserPN = dyn_cast<PHINode>(User)) {
Chandler Carruthcdf47882014-03-09 03:16:01 +00001565 if (UserPN->getIncomingBlock(U) == BB)
Chris Lattner84095072009-10-10 09:05:58 +00001566 continue;
1567 } else if (User->getParent() == BB)
1568 continue;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001569
Chandler Carruthcdf47882014-03-09 03:16:01 +00001570 UsesToRename.push_back(&U);
Chris Lattner84095072009-10-10 09:05:58 +00001571 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001572
Chris Lattner84095072009-10-10 09:05:58 +00001573 // If there are no uses outside the block, we're done with this instruction.
1574 if (UsesToRename.empty())
1575 continue;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001576
Benjamin Kramer543762d2016-01-09 18:43:01 +00001577 DEBUG(dbgs() << "JT: Renaming non-local uses of: " << I << "\n");
Chris Lattner84095072009-10-10 09:05:58 +00001578
1579 // We found a use of I outside of BB. Rename all uses of I that are outside
1580 // its block to be uses of the appropriate PHI node etc. See ValuesInBlocks
1581 // with the two values we know.
Benjamin Kramer543762d2016-01-09 18:43:01 +00001582 SSAUpdate.Initialize(I.getType(), I.getName());
1583 SSAUpdate.AddAvailableValue(BB, &I);
1584 SSAUpdate.AddAvailableValue(NewBB, ValueMapping[&I]);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001585
Chris Lattner84095072009-10-10 09:05:58 +00001586 while (!UsesToRename.empty())
1587 SSAUpdate.RewriteUse(*UsesToRename.pop_back_val());
David Greene1efdb452010-01-05 01:27:19 +00001588 DEBUG(dbgs() << "\n");
Chris Lattner84095072009-10-10 09:05:58 +00001589 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001590
1591
Chris Lattner88a1f022008-12-01 04:48:07 +00001592 // Ok, NewBB is good to go. Update the terminator of PredBB to jump to
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001593 // NewBB instead of BB. This eliminates predecessors from BB, which requires
1594 // us to simplify any PHI nodes in BB.
1595 TerminatorInst *PredTerm = PredBB->getTerminator();
1596 for (unsigned i = 0, e = PredTerm->getNumSuccessors(); i != e; ++i)
1597 if (PredTerm->getSuccessor(i) == BB) {
Owen Anderson99c985c2010-09-29 20:34:41 +00001598 BB->removePredecessor(PredBB, true);
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001599 PredTerm->setSuccessor(i, NewBB);
1600 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001601
Chris Lattner88a1f022008-12-01 04:48:07 +00001602 // At this point, the IR is fully up to date and consistent. Do a quick scan
1603 // over the new instructions and zap any that are constants or dead. This
1604 // frequently happens because of phi translation.
Mehdi Aminia28d91d2015-03-10 02:37:25 +00001605 SimplifyInstructionsInBlock(NewBB, TLI);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001606
Manman Ren72d44b12015-10-15 14:59:40 +00001607 // Update the edge weight from BB to SuccBB, which should be less than before.
1608 UpdateBlockFreqAndEdgeWeight(PredBB, BB, NewBB, SuccBB);
1609
Chris Lattnerd579cb12009-05-04 02:28:08 +00001610 // Threaded an edge!
1611 ++NumThreads;
1612 return true;
Chris Lattner21157222008-04-20 21:13:06 +00001613}
Chris Lattner97b14052009-10-11 07:24:57 +00001614
Manman Ren72d44b12015-10-15 14:59:40 +00001615/// Create a new basic block that will be the predecessor of BB and successor of
1616/// all blocks in Preds. When profile data is availble, update the frequency of
1617/// this new block.
1618BasicBlock *JumpThreading::SplitBlockPreds(BasicBlock *BB,
1619 ArrayRef<BasicBlock *> Preds,
1620 const char *Suffix) {
1621 // Collect the frequencies of all predecessors of BB, which will be used to
1622 // update the edge weight on BB->SuccBB.
1623 BlockFrequency PredBBFreq(0);
1624 if (HasProfileData)
1625 for (auto Pred : Preds)
1626 PredBBFreq += BFI->getBlockFreq(Pred) * BPI->getEdgeProbability(Pred, BB);
1627
1628 BasicBlock *PredBB = SplitBlockPredecessors(BB, Preds, Suffix);
1629
1630 // Set the block frequency of the newly created PredBB, which is the sum of
1631 // frequencies of Preds.
1632 if (HasProfileData)
1633 BFI->setBlockFreq(PredBB, PredBBFreq.getFrequency());
1634 return PredBB;
1635}
1636
1637/// Update the block frequency of BB and branch weight and the metadata on the
1638/// edge BB->SuccBB. This is done by scaling the weight of BB->SuccBB by 1 -
1639/// Freq(PredBB->BB) / Freq(BB->SuccBB).
1640void JumpThreading::UpdateBlockFreqAndEdgeWeight(BasicBlock *PredBB,
1641 BasicBlock *BB,
1642 BasicBlock *NewBB,
1643 BasicBlock *SuccBB) {
1644 if (!HasProfileData)
1645 return;
1646
1647 assert(BFI && BPI && "BFI & BPI should have been created here");
1648
1649 // As the edge from PredBB to BB is deleted, we have to update the block
1650 // frequency of BB.
1651 auto BBOrigFreq = BFI->getBlockFreq(BB);
1652 auto NewBBFreq = BFI->getBlockFreq(NewBB);
1653 auto BB2SuccBBFreq = BBOrigFreq * BPI->getEdgeProbability(BB, SuccBB);
1654 auto BBNewFreq = BBOrigFreq - NewBBFreq;
1655 BFI->setBlockFreq(BB, BBNewFreq.getFrequency());
1656
1657 // Collect updated outgoing edges' frequencies from BB and use them to update
Cong Houe93b8e12015-12-22 18:56:14 +00001658 // edge probabilities.
Manman Ren72d44b12015-10-15 14:59:40 +00001659 SmallVector<uint64_t, 4> BBSuccFreq;
Benjamin Kramer543762d2016-01-09 18:43:01 +00001660 for (BasicBlock *Succ : successors(BB)) {
1661 auto SuccFreq = (Succ == SuccBB)
Manman Ren72d44b12015-10-15 14:59:40 +00001662 ? BB2SuccBBFreq - NewBBFreq
Benjamin Kramer543762d2016-01-09 18:43:01 +00001663 : BBOrigFreq * BPI->getEdgeProbability(BB, Succ);
Manman Ren72d44b12015-10-15 14:59:40 +00001664 BBSuccFreq.push_back(SuccFreq.getFrequency());
1665 }
1666
Cong Houe93b8e12015-12-22 18:56:14 +00001667 uint64_t MaxBBSuccFreq =
1668 *std::max_element(BBSuccFreq.begin(), BBSuccFreq.end());
Manman Ren72d44b12015-10-15 14:59:40 +00001669
Cong Hou6a2c71a2015-12-22 23:45:55 +00001670 SmallVector<BranchProbability, 4> BBSuccProbs;
1671 if (MaxBBSuccFreq == 0)
1672 BBSuccProbs.assign(BBSuccFreq.size(),
1673 {1, static_cast<unsigned>(BBSuccFreq.size())});
1674 else {
1675 for (uint64_t Freq : BBSuccFreq)
1676 BBSuccProbs.push_back(
1677 BranchProbability::getBranchProbability(Freq, MaxBBSuccFreq));
1678 // Normalize edge probabilities so that they sum up to one.
1679 BranchProbability::normalizeProbabilities(BBSuccProbs.begin(),
1680 BBSuccProbs.end());
1681 }
Manman Ren72d44b12015-10-15 14:59:40 +00001682
Cong Houe93b8e12015-12-22 18:56:14 +00001683 // Update edge probabilities in BPI.
1684 for (int I = 0, E = BBSuccProbs.size(); I < E; I++)
1685 BPI->setEdgeProbability(BB, I, BBSuccProbs[I]);
Manman Ren72d44b12015-10-15 14:59:40 +00001686
Cong Houe93b8e12015-12-22 18:56:14 +00001687 if (BBSuccProbs.size() >= 2) {
1688 SmallVector<uint32_t, 4> Weights;
1689 for (auto Prob : BBSuccProbs)
1690 Weights.push_back(Prob.getNumerator());
1691
Manman Ren72d44b12015-10-15 14:59:40 +00001692 auto TI = BB->getTerminator();
1693 TI->setMetadata(
1694 LLVMContext::MD_prof,
1695 MDBuilder(TI->getParent()->getContext()).createBranchWeights(Weights));
1696 }
1697}
1698
Chris Lattner97b14052009-10-11 07:24:57 +00001699/// DuplicateCondBranchOnPHIIntoPred - PredBB contains an unconditional branch
1700/// to BB which contains an i1 PHI node and a conditional branch on that PHI.
1701/// If we can duplicate the contents of BB up into PredBB do so now, this
1702/// improves the odds that the branch will be on an analyzable instruction like
1703/// a compare.
1704bool JumpThreading::DuplicateCondBranchOnPHIIntoPred(BasicBlock *BB,
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001705 const SmallVectorImpl<BasicBlock *> &PredBBs) {
1706 assert(!PredBBs.empty() && "Can't handle an empty set");
1707
Chris Lattner97b14052009-10-11 07:24:57 +00001708 // If BB is a loop header, then duplicating this block outside the loop would
1709 // cause us to transform this into an irreducible loop, don't do this.
1710 // See the comments above FindLoopHeaders for justifications and caveats.
1711 if (LoopHeaders.count(BB)) {
David Greene1efdb452010-01-05 01:27:19 +00001712 DEBUG(dbgs() << " Not duplicating loop header '" << BB->getName()
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001713 << "' into predecessor block '" << PredBBs[0]->getName()
Chris Lattner97b14052009-10-11 07:24:57 +00001714 << "' - it might create an irreducible loop!\n");
1715 return false;
1716 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001717
Michael Liaod1209162014-09-24 04:59:06 +00001718 unsigned DuplicationCost = getJumpThreadDuplicationCost(BB, BBDupThreshold);
1719 if (DuplicationCost > BBDupThreshold) {
David Greene1efdb452010-01-05 01:27:19 +00001720 DEBUG(dbgs() << " Not duplicating BB '" << BB->getName()
Chris Lattner97b14052009-10-11 07:24:57 +00001721 << "' - Cost is too high: " << DuplicationCost << "\n");
1722 return false;
1723 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001724
David L Kreitzerda700ce2015-09-16 13:27:30 +00001725 // And finally, do it! Start by factoring the predecessors if needed.
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001726 BasicBlock *PredBB;
1727 if (PredBBs.size() == 1)
1728 PredBB = PredBBs[0];
1729 else {
1730 DEBUG(dbgs() << " Factoring out " << PredBBs.size()
1731 << " common predecessors.\n");
Manman Ren72d44b12015-10-15 14:59:40 +00001732 PredBB = SplitBlockPreds(BB, PredBBs, ".thr_comm");
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001733 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001734
Chris Lattner97b14052009-10-11 07:24:57 +00001735 // Okay, we decided to do this! Clone all the instructions in BB onto the end
1736 // of PredBB.
David Greene1efdb452010-01-05 01:27:19 +00001737 DEBUG(dbgs() << " Duplicating block '" << BB->getName() << "' into end of '"
Chris Lattner97b14052009-10-11 07:24:57 +00001738 << PredBB->getName() << "' to eliminate branch on phi. Cost: "
1739 << DuplicationCost << " block is:" << *BB << "\n");
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001740
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001741 // Unless PredBB ends with an unconditional branch, split the edge so that we
1742 // can just clone the bits from BB into the end of the new PredBB.
Chris Lattner29b15c52010-01-23 19:21:31 +00001743 BranchInst *OldPredBranch = dyn_cast<BranchInst>(PredBB->getTerminator());
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001744
Craig Topperf40110f2014-04-25 05:29:35 +00001745 if (!OldPredBranch || !OldPredBranch->isUnconditional()) {
Chandler Carruthd4500562015-01-19 12:36:53 +00001746 PredBB = SplitEdge(PredBB, BB);
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001747 OldPredBranch = cast<BranchInst>(PredBB->getTerminator());
1748 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001749
Chris Lattner97b14052009-10-11 07:24:57 +00001750 // We are going to have to map operands from the original BB block into the
1751 // PredBB block. Evaluate PHI nodes in BB.
1752 DenseMap<Instruction*, Value*> ValueMapping;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001753
Chris Lattner97b14052009-10-11 07:24:57 +00001754 BasicBlock::iterator BI = BB->begin();
1755 for (; PHINode *PN = dyn_cast<PHINode>(BI); ++BI)
1756 ValueMapping[PN] = PN->getIncomingValueForBlock(PredBB);
Chris Lattner97b14052009-10-11 07:24:57 +00001757 // Clone the non-phi instructions of BB into PredBB, keeping track of the
1758 // mapping and using it to remap operands in the cloned instructions.
1759 for (; BI != BB->end(); ++BI) {
1760 Instruction *New = BI->clone();
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001761
Chris Lattner97b14052009-10-11 07:24:57 +00001762 // Remap operands to patch up intra-block references.
1763 for (unsigned i = 0, e = New->getNumOperands(); i != e; ++i)
1764 if (Instruction *Inst = dyn_cast<Instruction>(New->getOperand(i))) {
1765 DenseMap<Instruction*, Value*>::iterator I = ValueMapping.find(Inst);
1766 if (I != ValueMapping.end())
1767 New->setOperand(i, I->second);
1768 }
Chris Lattner573da8a2010-01-12 20:41:47 +00001769
1770 // If this instruction can be simplified after the operands are updated,
1771 // just use the simplified value instead. This frequently happens due to
1772 // phi translation.
Mehdi Aminia28d91d2015-03-10 02:37:25 +00001773 if (Value *IV =
1774 SimplifyInstruction(New, BB->getModule()->getDataLayout())) {
Chris Lattner573da8a2010-01-12 20:41:47 +00001775 delete New;
Duncan P. N. Exon Smith3a9c9e32015-10-13 18:26:00 +00001776 ValueMapping[&*BI] = IV;
Chris Lattner573da8a2010-01-12 20:41:47 +00001777 } else {
1778 // Otherwise, insert the new instruction into the block.
1779 New->setName(BI->getName());
Duncan P. N. Exon Smith3a9c9e32015-10-13 18:26:00 +00001780 PredBB->getInstList().insert(OldPredBranch->getIterator(), New);
1781 ValueMapping[&*BI] = New;
Chris Lattner573da8a2010-01-12 20:41:47 +00001782 }
Chris Lattner97b14052009-10-11 07:24:57 +00001783 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001784
Chris Lattner97b14052009-10-11 07:24:57 +00001785 // Check to see if the targets of the branch had PHI nodes. If so, we need to
1786 // add entries to the PHI nodes for branch from PredBB now.
1787 BranchInst *BBBranch = cast<BranchInst>(BB->getTerminator());
1788 AddPHINodeEntriesForMappedBlock(BBBranch->getSuccessor(0), BB, PredBB,
1789 ValueMapping);
1790 AddPHINodeEntriesForMappedBlock(BBBranch->getSuccessor(1), BB, PredBB,
1791 ValueMapping);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001792
Chris Lattner97b14052009-10-11 07:24:57 +00001793 // If there were values defined in BB that are used outside the block, then we
1794 // now have to update all uses of the value to use either the original value,
1795 // the cloned value, or some PHI derived value. This can require arbitrary
1796 // PHI insertion, of which we are prepared to do, clean these up now.
1797 SSAUpdater SSAUpdate;
1798 SmallVector<Use*, 16> UsesToRename;
Benjamin Kramer543762d2016-01-09 18:43:01 +00001799 for (Instruction &I : *BB) {
Chris Lattner97b14052009-10-11 07:24:57 +00001800 // Scan all uses of this instruction to see if it is used outside of its
1801 // block, and if so, record them in UsesToRename.
Benjamin Kramer543762d2016-01-09 18:43:01 +00001802 for (Use &U : I.uses()) {
Chandler Carruthcdf47882014-03-09 03:16:01 +00001803 Instruction *User = cast<Instruction>(U.getUser());
Chris Lattner97b14052009-10-11 07:24:57 +00001804 if (PHINode *UserPN = dyn_cast<PHINode>(User)) {
Chandler Carruthcdf47882014-03-09 03:16:01 +00001805 if (UserPN->getIncomingBlock(U) == BB)
Chris Lattner97b14052009-10-11 07:24:57 +00001806 continue;
1807 } else if (User->getParent() == BB)
1808 continue;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001809
Chandler Carruthcdf47882014-03-09 03:16:01 +00001810 UsesToRename.push_back(&U);
Chris Lattner97b14052009-10-11 07:24:57 +00001811 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001812
Chris Lattner97b14052009-10-11 07:24:57 +00001813 // If there are no uses outside the block, we're done with this instruction.
1814 if (UsesToRename.empty())
1815 continue;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001816
Benjamin Kramer543762d2016-01-09 18:43:01 +00001817 DEBUG(dbgs() << "JT: Renaming non-local uses of: " << I << "\n");
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001818
Chris Lattner97b14052009-10-11 07:24:57 +00001819 // We found a use of I outside of BB. Rename all uses of I that are outside
1820 // its block to be uses of the appropriate PHI node etc. See ValuesInBlocks
1821 // with the two values we know.
Benjamin Kramer543762d2016-01-09 18:43:01 +00001822 SSAUpdate.Initialize(I.getType(), I.getName());
1823 SSAUpdate.AddAvailableValue(BB, &I);
1824 SSAUpdate.AddAvailableValue(PredBB, ValueMapping[&I]);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001825
Chris Lattner97b14052009-10-11 07:24:57 +00001826 while (!UsesToRename.empty())
1827 SSAUpdate.RewriteUse(*UsesToRename.pop_back_val());
David Greene1efdb452010-01-05 01:27:19 +00001828 DEBUG(dbgs() << "\n");
Chris Lattner97b14052009-10-11 07:24:57 +00001829 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001830
Chris Lattner97b14052009-10-11 07:24:57 +00001831 // PredBB no longer jumps to BB, remove entries in the PHI node for the edge
1832 // that we nuked.
Owen Anderson99c985c2010-09-29 20:34:41 +00001833 BB->removePredecessor(PredBB, true);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001834
Chris Lattner97b14052009-10-11 07:24:57 +00001835 // Remove the unconditional branch at the end of the PredBB block.
1836 OldPredBranch->eraseFromParent();
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001837
Chris Lattner97b14052009-10-11 07:24:57 +00001838 ++NumDupes;
1839 return true;
1840}
Benjamin Kramer6a4976d2013-08-07 10:29:38 +00001841
1842/// TryToUnfoldSelect - Look for blocks of the form
1843/// bb1:
1844/// %a = select
1845/// br bb
1846///
1847/// bb2:
1848/// %p = phi [%a, %bb] ...
1849/// %c = icmp %p
1850/// br i1 %c
1851///
1852/// And expand the select into a branch structure if one of its arms allows %c
1853/// to be folded. This later enables threading from bb1 over bb2.
1854bool JumpThreading::TryToUnfoldSelect(CmpInst *CondCmp, BasicBlock *BB) {
1855 BranchInst *CondBr = dyn_cast<BranchInst>(BB->getTerminator());
1856 PHINode *CondLHS = dyn_cast<PHINode>(CondCmp->getOperand(0));
1857 Constant *CondRHS = cast<Constant>(CondCmp->getOperand(1));
1858
1859 if (!CondBr || !CondBr->isConditional() || !CondLHS ||
1860 CondLHS->getParent() != BB)
1861 return false;
1862
1863 for (unsigned I = 0, E = CondLHS->getNumIncomingValues(); I != E; ++I) {
1864 BasicBlock *Pred = CondLHS->getIncomingBlock(I);
1865 SelectInst *SI = dyn_cast<SelectInst>(CondLHS->getIncomingValue(I));
1866
1867 // Look if one of the incoming values is a select in the corresponding
1868 // predecessor.
1869 if (!SI || SI->getParent() != Pred || !SI->hasOneUse())
1870 continue;
1871
1872 BranchInst *PredTerm = dyn_cast<BranchInst>(Pred->getTerminator());
1873 if (!PredTerm || !PredTerm->isUnconditional())
1874 continue;
1875
1876 // Now check if one of the select values would allow us to constant fold the
1877 // terminator in BB. We don't do the transform if both sides fold, those
1878 // cases will be threaded in any case.
1879 LazyValueInfo::Tristate LHSFolds =
1880 LVI->getPredicateOnEdge(CondCmp->getPredicate(), SI->getOperand(1),
Hal Finkel7e184492014-09-07 20:29:59 +00001881 CondRHS, Pred, BB, CondCmp);
Benjamin Kramer6a4976d2013-08-07 10:29:38 +00001882 LazyValueInfo::Tristate RHSFolds =
1883 LVI->getPredicateOnEdge(CondCmp->getPredicate(), SI->getOperand(2),
Hal Finkel7e184492014-09-07 20:29:59 +00001884 CondRHS, Pred, BB, CondCmp);
Benjamin Kramer6a4976d2013-08-07 10:29:38 +00001885 if ((LHSFolds != LazyValueInfo::Unknown ||
1886 RHSFolds != LazyValueInfo::Unknown) &&
1887 LHSFolds != RHSFolds) {
1888 // Expand the select.
1889 //
1890 // Pred --
1891 // | v
1892 // | NewBB
1893 // | |
1894 // |-----
1895 // v
1896 // BB
1897 BasicBlock *NewBB = BasicBlock::Create(BB->getContext(), "select.unfold",
1898 BB->getParent(), BB);
1899 // Move the unconditional branch to NewBB.
1900 PredTerm->removeFromParent();
1901 NewBB->getInstList().insert(NewBB->end(), PredTerm);
1902 // Create a conditional branch and update PHI nodes.
1903 BranchInst::Create(NewBB, BB, SI->getCondition(), Pred);
1904 CondLHS->setIncomingValue(I, SI->getFalseValue());
1905 CondLHS->addIncoming(SI->getTrueValue(), NewBB);
1906 // The select is now dead.
1907 SI->eraseFromParent();
1908
1909 // Update any other PHI nodes in BB.
1910 for (BasicBlock::iterator BI = BB->begin();
1911 PHINode *Phi = dyn_cast<PHINode>(BI); ++BI)
1912 if (Phi != CondLHS)
1913 Phi->addIncoming(Phi->getIncomingValueForBlock(Pred), NewBB);
1914 return true;
1915 }
1916 }
1917 return false;
1918}
Haicheng Wua6a32792016-01-08 19:39:39 +00001919
1920/// TryToUnfoldSelectInCurrBB - Look for PHI/Select in the same BB of the form
1921/// bb:
1922/// %p = phi [false, %bb1], [true, %bb2], [false, %bb3], [true, %bb4], ...
1923/// %s = select p, trueval, falseval
1924///
1925/// And expand the select into a branch structure. This later enables
1926/// jump-threading over bb in this pass.
1927///
1928/// Using the similar approach of SimplifyCFG::FoldCondBranchOnPHI(), unfold
1929/// select if the associated PHI has at least one constant. If the unfolded
1930/// select is not jump-threaded, it will be folded again in the later
1931/// optimizations.
1932bool JumpThreading::TryToUnfoldSelectInCurrBB(BasicBlock *BB) {
1933 // If threading this would thread across a loop header, don't thread the edge.
1934 // See the comments above FindLoopHeaders for justifications and caveats.
1935 if (LoopHeaders.count(BB))
1936 return false;
1937
1938 // Look for a Phi/Select pair in the same basic block. The Phi feeds the
1939 // condition of the Select and at least one of the incoming values is a
1940 // constant.
1941 for (BasicBlock::iterator BI = BB->begin();
1942 PHINode *PN = dyn_cast<PHINode>(BI); ++BI) {
1943 unsigned NumPHIValues = PN->getNumIncomingValues();
1944 if (NumPHIValues == 0 || !PN->hasOneUse())
1945 continue;
1946
1947 SelectInst *SI = dyn_cast<SelectInst>(PN->user_back());
1948 if (!SI || SI->getParent() != BB)
1949 continue;
1950
1951 Value *Cond = SI->getCondition();
1952 if (!Cond || Cond != PN || !Cond->getType()->isIntegerTy(1))
1953 continue;
1954
1955 bool HasConst = false;
1956 for (unsigned i = 0; i != NumPHIValues; ++i) {
1957 if (PN->getIncomingBlock(i) == BB)
1958 return false;
1959 if (isa<ConstantInt>(PN->getIncomingValue(i)))
1960 HasConst = true;
1961 }
1962
1963 if (HasConst) {
1964 // Expand the select.
1965 TerminatorInst *Term =
1966 SplitBlockAndInsertIfThen(SI->getCondition(), SI, false);
1967 PHINode *NewPN = PHINode::Create(SI->getType(), 2, "", SI);
1968 NewPN->addIncoming(SI->getTrueValue(), Term->getParent());
1969 NewPN->addIncoming(SI->getFalseValue(), BB);
1970 SI->replaceAllUsesWith(NewPN);
1971 SI->eraseFromParent();
1972 return true;
1973 }
1974 }
1975
1976 return false;
1977}