blob: ba406a8e8b304ae6f2e7f7bf9148265d26c4af0e [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) {
Andrew Kayloraa641a52016-04-22 22:06:11 +0000190 if (skipFunction(F))
Paul Robinsonaf4e64d2014-02-06 00:07:05 +0000191 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
Chad Rosier41dd31f2016-04-20 19:15:26 +0000931 Optional<bool> Implication =
932 isImpliedCondition(PBI->getCondition(), Cond, DL);
933 if (Implication) {
934 BI->getSuccessor(*Implication ? 1 : 0)->removePredecessor(BB);
935 BranchInst::Create(BI->getSuccessor(*Implication ? 0 : 1), BI);
Sanjoy Das13e63a22015-10-28 21:27:08 +0000936 BI->eraseFromParent();
937 return true;
938 }
939 CurrentBB = CurrentPred;
940 CurrentPred = CurrentBB->getSinglePredecessor();
941 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000942
Chris Lattnere369c352008-04-22 06:36:15 +0000943 return false;
944}
945
Chris Lattner98d89d12008-11-27 05:07:53 +0000946/// SimplifyPartiallyRedundantLoad - If LI is an obviously partially redundant
947/// load instruction, eliminate it by replacing it with a PHI node. This is an
948/// important optimization that encourages jump threading, and needs to be run
949/// interlaced with other jump threading tasks.
950bool JumpThreading::SimplifyPartiallyRedundantLoad(LoadInst *LI) {
Eli Friedman7c5dc122011-09-12 20:23:13 +0000951 // Don't hack volatile/atomic loads.
952 if (!LI->isSimple()) return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000953
Chris Lattner98d89d12008-11-27 05:07:53 +0000954 // If the load is defined in a block with exactly one predecessor, it can't be
955 // partially redundant.
956 BasicBlock *LoadBB = LI->getParent();
957 if (LoadBB->getSinglePredecessor())
958 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000959
David Majnemereb518bd2015-08-04 08:21:40 +0000960 // If the load is defined in an EH pad, it can't be partially redundant,
961 // because the edges between the invoke and the EH pad cannot have other
Bill Wendling90dd90a2013-10-21 04:09:17 +0000962 // instructions between them.
David Majnemereb518bd2015-08-04 08:21:40 +0000963 if (LoadBB->isEHPad())
Bill Wendling90dd90a2013-10-21 04:09:17 +0000964 return false;
965
Chris Lattner98d89d12008-11-27 05:07:53 +0000966 Value *LoadedPtr = LI->getOperand(0);
967
968 // If the loaded operand is defined in the LoadBB, it can't be available.
Chris Lattner9d9812a2009-11-15 19:58:31 +0000969 // TODO: Could do simple PHI translation, that would be fun :)
Chris Lattner98d89d12008-11-27 05:07:53 +0000970 if (Instruction *PtrOp = dyn_cast<Instruction>(LoadedPtr))
971 if (PtrOp->getParent() == LoadBB)
972 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000973
Chris Lattner98d89d12008-11-27 05:07:53 +0000974 // Scan a few instructions up from the load, to see if it is obviously live at
975 // the entry to its block.
Duncan P. N. Exon Smith3a9c9e32015-10-13 18:26:00 +0000976 BasicBlock::iterator BBIt(LI);
Chris Lattner98d89d12008-11-27 05:07:53 +0000977
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000978 if (Value *AvailableVal =
Eduard Burtescue2a69172016-01-22 01:51:51 +0000979 FindAvailableLoadedValue(LI, LoadBB, BBIt, DefMaxInstsToScan)) {
David L Kreitzerda700ce2015-09-16 13:27:30 +0000980 // If the value of the load is locally available within the block, just use
Chris Lattner98d89d12008-11-27 05:07:53 +0000981 // it. This frequently occurs for reg2mem'd allocas.
982 //cerr << "LOAD ELIMINATED:\n" << *BBIt << *LI << "\n";
Frits van Bommel5e75ef42010-12-05 19:02:47 +0000983
Chris Lattner482eb702009-01-09 06:08:12 +0000984 // If the returned value is the load itself, replace with an undef. This can
985 // only happen in dead loops.
Owen Andersonb292b8c2009-07-30 23:03:37 +0000986 if (AvailableVal == LI) AvailableVal = UndefValue::get(LI->getType());
Chandler Carrutheeec35a2014-10-20 00:24:14 +0000987 if (AvailableVal->getType() != LI->getType())
Chandler Carruth1a3c2c42014-11-25 08:20:27 +0000988 AvailableVal =
989 CastInst::CreateBitOrPointerCast(AvailableVal, LI->getType(), "", LI);
Chris Lattner98d89d12008-11-27 05:07:53 +0000990 LI->replaceAllUsesWith(AvailableVal);
991 LI->eraseFromParent();
992 return true;
993 }
994
995 // Otherwise, if we scanned the whole block and got to the top of the block,
996 // we know the block is locally transparent to the load. If not, something
997 // might clobber its value.
998 if (BBIt != LoadBB->begin())
999 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001000
Hal Finkelcc39b672014-07-24 12:16:19 +00001001 // If all of the loads and stores that feed the value have the same AA tags,
1002 // then we can propagate them onto any newly inserted loads.
1003 AAMDNodes AATags;
1004 LI->getAAMetadata(AATags);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001005
Chris Lattner98d89d12008-11-27 05:07:53 +00001006 SmallPtrSet<BasicBlock*, 8> PredsScanned;
1007 typedef SmallVector<std::pair<BasicBlock*, Value*>, 8> AvailablePredsTy;
1008 AvailablePredsTy AvailablePreds;
Craig Topperf40110f2014-04-25 05:29:35 +00001009 BasicBlock *OneUnavailablePred = nullptr;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001010
Chris Lattner98d89d12008-11-27 05:07:53 +00001011 // If we got here, the loaded value is transparent through to the start of the
1012 // block. Check to see if it is available in any of the predecessor blocks.
Benjamin Kramer543762d2016-01-09 18:43:01 +00001013 for (BasicBlock *PredBB : predecessors(LoadBB)) {
Chris Lattner98d89d12008-11-27 05:07:53 +00001014 // If we already scanned this predecessor, skip it.
David Blaikie70573dc2014-11-19 07:49:26 +00001015 if (!PredsScanned.insert(PredBB).second)
Chris Lattner98d89d12008-11-27 05:07:53 +00001016 continue;
1017
1018 // Scan the predecessor to see if the value is available in the pred.
1019 BBIt = PredBB->end();
Hal Finkelcc39b672014-07-24 12:16:19 +00001020 AAMDNodes ThisAATags;
Eduard Burtescue2a69172016-01-22 01:51:51 +00001021 Value *PredAvailable = FindAvailableLoadedValue(LI, PredBB, BBIt,
Larisse Voufo532bf712015-09-18 19:14:35 +00001022 DefMaxInstsToScan,
Hal Finkelcc39b672014-07-24 12:16:19 +00001023 nullptr, &ThisAATags);
Chris Lattner98d89d12008-11-27 05:07:53 +00001024 if (!PredAvailable) {
1025 OneUnavailablePred = PredBB;
1026 continue;
1027 }
Nadav Rotem465834c2012-07-24 10:51:42 +00001028
Hal Finkelcc39b672014-07-24 12:16:19 +00001029 // If AA tags disagree or are not present, forget about them.
1030 if (AATags != ThisAATags) AATags = AAMDNodes();
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001031
Chris Lattner98d89d12008-11-27 05:07:53 +00001032 // If so, this load is partially redundant. Remember this info so that we
1033 // can create a PHI node.
1034 AvailablePreds.push_back(std::make_pair(PredBB, PredAvailable));
1035 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001036
Chris Lattner98d89d12008-11-27 05:07:53 +00001037 // If the loaded value isn't available in any predecessor, it isn't partially
1038 // redundant.
1039 if (AvailablePreds.empty()) return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001040
Chris Lattner98d89d12008-11-27 05:07:53 +00001041 // Okay, the loaded value is available in at least one (and maybe all!)
1042 // predecessors. If the value is unavailable in more than one unique
1043 // predecessor, we want to insert a merge block for those common predecessors.
1044 // This ensures that we only have to insert one reload, thus not increasing
1045 // code size.
Craig Topperf40110f2014-04-25 05:29:35 +00001046 BasicBlock *UnavailablePred = nullptr;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001047
Chris Lattner98d89d12008-11-27 05:07:53 +00001048 // If there is exactly one predecessor where the value is unavailable, the
1049 // already computed 'OneUnavailablePred' block is it. If it ends in an
1050 // unconditional branch, we know that it isn't a critical edge.
1051 if (PredsScanned.size() == AvailablePreds.size()+1 &&
1052 OneUnavailablePred->getTerminator()->getNumSuccessors() == 1) {
1053 UnavailablePred = OneUnavailablePred;
1054 } else if (PredsScanned.size() != AvailablePreds.size()) {
1055 // Otherwise, we had multiple unavailable predecessors or we had a critical
1056 // edge from the one.
1057 SmallVector<BasicBlock*, 8> PredsToSplit;
1058 SmallPtrSet<BasicBlock*, 8> AvailablePredSet;
1059
Benjamin Kramer543762d2016-01-09 18:43:01 +00001060 for (const auto &AvailablePred : AvailablePreds)
1061 AvailablePredSet.insert(AvailablePred.first);
Chris Lattner98d89d12008-11-27 05:07:53 +00001062
1063 // Add all the unavailable predecessors to the PredsToSplit list.
Benjamin Kramer543762d2016-01-09 18:43:01 +00001064 for (BasicBlock *P : predecessors(LoadBB)) {
Chris Lattner329ea062010-06-14 19:45:43 +00001065 // If the predecessor is an indirect goto, we can't split the edge.
Gabor Greifa5fa8852010-07-12 14:10:24 +00001066 if (isa<IndirectBrInst>(P->getTerminator()))
Chris Lattner329ea062010-06-14 19:45:43 +00001067 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001068
Gabor Greifa5fa8852010-07-12 14:10:24 +00001069 if (!AvailablePredSet.count(P))
1070 PredsToSplit.push_back(P);
Chris Lattner329ea062010-06-14 19:45:43 +00001071 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001072
Chris Lattner98d89d12008-11-27 05:07:53 +00001073 // Split them out to their own block.
Manman Ren72d44b12015-10-15 14:59:40 +00001074 UnavailablePred = SplitBlockPreds(LoadBB, PredsToSplit, "thread-pre-split");
Chris Lattner98d89d12008-11-27 05:07:53 +00001075 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001076
Chris Lattner98d89d12008-11-27 05:07:53 +00001077 // If the value isn't available in all predecessors, then there will be
1078 // exactly one where it isn't available. Insert a load on that edge and add
1079 // it to the AvailablePreds list.
1080 if (UnavailablePred) {
1081 assert(UnavailablePred->getTerminator()->getNumSuccessors() == 1 &&
1082 "Can't handle critical edge here!");
Devang Patel306f8db2011-05-04 22:48:19 +00001083 LoadInst *NewVal = new LoadInst(LoadedPtr, LI->getName()+".pr", false,
Chris Lattner9d9812a2009-11-15 19:58:31 +00001084 LI->getAlignment(),
Chris Lattner98d89d12008-11-27 05:07:53 +00001085 UnavailablePred->getTerminator());
Devang Patel306f8db2011-05-04 22:48:19 +00001086 NewVal->setDebugLoc(LI->getDebugLoc());
Hal Finkelcc39b672014-07-24 12:16:19 +00001087 if (AATags)
1088 NewVal->setAAMetadata(AATags);
Nadav Rotem465834c2012-07-24 10:51:42 +00001089
Chris Lattner98d89d12008-11-27 05:07:53 +00001090 AvailablePreds.push_back(std::make_pair(UnavailablePred, NewVal));
1091 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001092
Chris Lattner98d89d12008-11-27 05:07:53 +00001093 // Now we know that each predecessor of this block has a value in
1094 // AvailablePreds, sort them for efficient access as we're walking the preds.
Chris Lattner2b07d3c2008-12-01 06:52:57 +00001095 array_pod_sort(AvailablePreds.begin(), AvailablePreds.end());
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001096
Chris Lattner98d89d12008-11-27 05:07:53 +00001097 // Create a PHI node at the start of the block for the PRE'd load value.
Jay Foade0938d82011-03-30 11:19:20 +00001098 pred_iterator PB = pred_begin(LoadBB), PE = pred_end(LoadBB);
Jay Foad52131342011-03-30 11:28:46 +00001099 PHINode *PN = PHINode::Create(LI->getType(), std::distance(PB, PE), "",
Duncan P. N. Exon Smith3a9c9e32015-10-13 18:26:00 +00001100 &LoadBB->front());
Chris Lattner98d89d12008-11-27 05:07:53 +00001101 PN->takeName(LI);
Devang Patel306f8db2011-05-04 22:48:19 +00001102 PN->setDebugLoc(LI->getDebugLoc());
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001103
Chris Lattner98d89d12008-11-27 05:07:53 +00001104 // Insert new entries into the PHI for each predecessor. A single block may
1105 // have multiple entries here.
Jay Foade0938d82011-03-30 11:19:20 +00001106 for (pred_iterator PI = PB; PI != PE; ++PI) {
Gabor Greifa5fa8852010-07-12 14:10:24 +00001107 BasicBlock *P = *PI;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001108 AvailablePredsTy::iterator I =
Chris Lattner98d89d12008-11-27 05:07:53 +00001109 std::lower_bound(AvailablePreds.begin(), AvailablePreds.end(),
Craig Topperf40110f2014-04-25 05:29:35 +00001110 std::make_pair(P, (Value*)nullptr));
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001111
Gabor Greifa5fa8852010-07-12 14:10:24 +00001112 assert(I != AvailablePreds.end() && I->first == P &&
Chris Lattner98d89d12008-11-27 05:07:53 +00001113 "Didn't find entry for predecessor!");
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001114
Chandler Carrutheeec35a2014-10-20 00:24:14 +00001115 // If we have an available predecessor but it requires casting, insert the
Chandler Carruth6665d622014-10-20 05:34:36 +00001116 // cast in the predecessor and use the cast. Note that we have to update the
1117 // AvailablePreds vector as we go so that all of the PHI entries for this
1118 // predecessor use the same bitcast.
1119 Value *&PredV = I->second;
Chandler Carrutheeec35a2014-10-20 00:24:14 +00001120 if (PredV->getType() != LI->getType())
Chandler Carruth1a3c2c42014-11-25 08:20:27 +00001121 PredV = CastInst::CreateBitOrPointerCast(PredV, LI->getType(), "",
1122 P->getTerminator());
Chandler Carrutheeec35a2014-10-20 00:24:14 +00001123
1124 PN->addIncoming(PredV, I->first);
Chris Lattner98d89d12008-11-27 05:07:53 +00001125 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001126
Chris Lattner98d89d12008-11-27 05:07:53 +00001127 //cerr << "PRE: " << *LI << *PN << "\n";
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001128
Chris Lattner98d89d12008-11-27 05:07:53 +00001129 LI->replaceAllUsesWith(PN);
1130 LI->eraseFromParent();
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001131
Chris Lattner98d89d12008-11-27 05:07:53 +00001132 return true;
1133}
1134
Chris Lattner5ff7f562009-11-07 08:05:03 +00001135/// FindMostPopularDest - The specified list contains multiple possible
1136/// threadable destinations. Pick the one that occurs the most frequently in
1137/// the list.
1138static BasicBlock *
1139FindMostPopularDest(BasicBlock *BB,
1140 const SmallVectorImpl<std::pair<BasicBlock*,
1141 BasicBlock*> > &PredToDestList) {
1142 assert(!PredToDestList.empty());
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001143
Chris Lattner5ff7f562009-11-07 08:05:03 +00001144 // Determine popularity. If there are multiple possible destinations, we
1145 // explicitly choose to ignore 'undef' destinations. We prefer to thread
1146 // blocks with known and real destinations to threading undef. We'll handle
1147 // them later if interesting.
1148 DenseMap<BasicBlock*, unsigned> DestPopularity;
Benjamin Kramer543762d2016-01-09 18:43:01 +00001149 for (const auto &PredToDest : PredToDestList)
1150 if (PredToDest.second)
1151 DestPopularity[PredToDest.second]++;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001152
Chris Lattner5ff7f562009-11-07 08:05:03 +00001153 // Find the most popular dest.
1154 DenseMap<BasicBlock*, unsigned>::iterator DPI = DestPopularity.begin();
1155 BasicBlock *MostPopularDest = DPI->first;
1156 unsigned Popularity = DPI->second;
1157 SmallVector<BasicBlock*, 4> SamePopularity;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001158
Chris Lattner5ff7f562009-11-07 08:05:03 +00001159 for (++DPI; DPI != DestPopularity.end(); ++DPI) {
1160 // If the popularity of this entry isn't higher than the popularity we've
1161 // seen so far, ignore it.
1162 if (DPI->second < Popularity)
1163 ; // ignore.
1164 else if (DPI->second == Popularity) {
1165 // If it is the same as what we've seen so far, keep track of it.
1166 SamePopularity.push_back(DPI->first);
1167 } else {
1168 // If it is more popular, remember it.
1169 SamePopularity.clear();
1170 MostPopularDest = DPI->first;
1171 Popularity = DPI->second;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001172 }
Chris Lattner5ff7f562009-11-07 08:05:03 +00001173 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001174
Frits van Bommel9bbe8492010-12-16 12:16:00 +00001175 // Okay, now we know the most popular destination. If there is more than one
Chris Lattner5ff7f562009-11-07 08:05:03 +00001176 // destination, we need to determine one. This is arbitrary, but we need
1177 // to make a deterministic decision. Pick the first one that appears in the
1178 // successor list.
1179 if (!SamePopularity.empty()) {
1180 SamePopularity.push_back(MostPopularDest);
1181 TerminatorInst *TI = BB->getTerminator();
1182 for (unsigned i = 0; ; ++i) {
1183 assert(i != TI->getNumSuccessors() && "Didn't find any successor!");
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001184
Chris Lattner5ff7f562009-11-07 08:05:03 +00001185 if (std::find(SamePopularity.begin(), SamePopularity.end(),
1186 TI->getSuccessor(i)) == SamePopularity.end())
1187 continue;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001188
Chris Lattner5ff7f562009-11-07 08:05:03 +00001189 MostPopularDest = TI->getSuccessor(i);
1190 break;
1191 }
1192 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001193
Chris Lattner5ff7f562009-11-07 08:05:03 +00001194 // Okay, we have finally picked the most popular destination.
1195 return MostPopularDest;
1196}
1197
Frits van Bommeld9df6ea2010-12-06 23:36:56 +00001198bool JumpThreading::ProcessThreadableEdges(Value *Cond, BasicBlock *BB,
Hal Finkel7e184492014-09-07 20:29:59 +00001199 ConstantPreference Preference,
1200 Instruction *CxtI) {
Chris Lattner5ff7f562009-11-07 08:05:03 +00001201 // If threading this would thread across a loop header, don't even try to
1202 // thread the edge.
1203 if (LoopHeaders.count(BB))
1204 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001205
Frits van Bommel76244862010-12-05 19:06:41 +00001206 PredValueInfoTy PredValues;
Haicheng Wu64d9d7c2016-03-15 23:38:47 +00001207 if (!ComputeValueKnownInPredecessors(Cond, BB, PredValues, Preference, CxtI))
1208 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001209
Chris Lattner5ff7f562009-11-07 08:05:03 +00001210 assert(!PredValues.empty() &&
1211 "ComputeValueKnownInPredecessors returned true with no values");
1212
David Greene1efdb452010-01-05 01:27:19 +00001213 DEBUG(dbgs() << "IN BB: " << *BB;
Benjamin Kramer543762d2016-01-09 18:43:01 +00001214 for (const auto &PredValue : PredValues) {
Frits van Bommel76244862010-12-05 19:06:41 +00001215 dbgs() << " BB '" << BB->getName() << "': FOUND condition = "
Benjamin Kramer543762d2016-01-09 18:43:01 +00001216 << *PredValue.first
1217 << " for pred '" << PredValue.second->getName() << "'.\n";
Chris Lattner5ff7f562009-11-07 08:05:03 +00001218 });
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001219
Chris Lattner5ff7f562009-11-07 08:05:03 +00001220 // Decide what we want to thread through. Convert our list of known values to
1221 // a list of known destinations for each pred. This also discards duplicate
1222 // predecessors and keeps track of the undefined inputs (which are represented
Chris Lattnerea465e22009-11-09 00:41:49 +00001223 // as a null dest in the PredToDestList).
Chris Lattner5ff7f562009-11-07 08:05:03 +00001224 SmallPtrSet<BasicBlock*, 16> SeenPreds;
1225 SmallVector<std::pair<BasicBlock*, BasicBlock*>, 16> PredToDestList;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001226
Craig Topperf40110f2014-04-25 05:29:35 +00001227 BasicBlock *OnlyDest = nullptr;
Chris Lattner5ff7f562009-11-07 08:05:03 +00001228 BasicBlock *MultipleDestSentinel = (BasicBlock*)(intptr_t)~0ULL;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001229
Benjamin Kramer543762d2016-01-09 18:43:01 +00001230 for (const auto &PredValue : PredValues) {
1231 BasicBlock *Pred = PredValue.second;
David Blaikie70573dc2014-11-19 07:49:26 +00001232 if (!SeenPreds.insert(Pred).second)
Chris Lattner5ff7f562009-11-07 08:05:03 +00001233 continue; // Duplicate predecessor entry.
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001234
Chris Lattner5ff7f562009-11-07 08:05:03 +00001235 // If the predecessor ends with an indirect goto, we can't change its
1236 // destination.
1237 if (isa<IndirectBrInst>(Pred->getTerminator()))
1238 continue;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001239
Benjamin Kramer543762d2016-01-09 18:43:01 +00001240 Constant *Val = PredValue.first;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001241
Chris Lattner5ff7f562009-11-07 08:05:03 +00001242 BasicBlock *DestBB;
Frits van Bommel76244862010-12-05 19:06:41 +00001243 if (isa<UndefValue>(Val))
Craig Topperf40110f2014-04-25 05:29:35 +00001244 DestBB = nullptr;
Chris Lattner5ff7f562009-11-07 08:05:03 +00001245 else if (BranchInst *BI = dyn_cast<BranchInst>(BB->getTerminator()))
Frits van Bommel76244862010-12-05 19:06:41 +00001246 DestBB = BI->getSuccessor(cast<ConstantInt>(Val)->isZero());
Stepan Dyatkovskiy513aaa52012-02-01 07:49:51 +00001247 else if (SwitchInst *SI = dyn_cast<SwitchInst>(BB->getTerminator())) {
Stepan Dyatkovskiy5b648af2012-03-08 07:06:20 +00001248 DestBB = SI->findCaseValue(cast<ConstantInt>(Val)).getCaseSuccessor();
Stepan Dyatkovskiy513aaa52012-02-01 07:49:51 +00001249 } else {
Frits van Bommeld9df6ea2010-12-06 23:36:56 +00001250 assert(isa<IndirectBrInst>(BB->getTerminator())
1251 && "Unexpected terminator");
1252 DestBB = cast<BlockAddress>(Val)->getBasicBlock();
Chris Lattner5ff7f562009-11-07 08:05:03 +00001253 }
1254
1255 // If we have exactly one destination, remember it for efficiency below.
Frits van Bommel9bbe8492010-12-16 12:16:00 +00001256 if (PredToDestList.empty())
Chris Lattner5ff7f562009-11-07 08:05:03 +00001257 OnlyDest = DestBB;
1258 else if (OnlyDest != DestBB)
1259 OnlyDest = MultipleDestSentinel;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001260
Chris Lattner5ff7f562009-11-07 08:05:03 +00001261 PredToDestList.push_back(std::make_pair(Pred, DestBB));
1262 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001263
Chris Lattner5ff7f562009-11-07 08:05:03 +00001264 // If all edges were unthreadable, we fail.
1265 if (PredToDestList.empty())
Haicheng Wu64d9d7c2016-03-15 23:38:47 +00001266 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001267
Chris Lattner5ff7f562009-11-07 08:05:03 +00001268 // Determine which is the most common successor. If we have many inputs and
1269 // this block is a switch, we want to start by threading the batch that goes
1270 // to the most popular destination first. If we only know about one
1271 // threadable destination (the common case) we can avoid this.
1272 BasicBlock *MostPopularDest = OnlyDest;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001273
Chris Lattner5ff7f562009-11-07 08:05:03 +00001274 if (MostPopularDest == MultipleDestSentinel)
1275 MostPopularDest = FindMostPopularDest(BB, PredToDestList);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001276
Chris Lattner5ff7f562009-11-07 08:05:03 +00001277 // Now that we know what the most popular destination is, factor all
1278 // predecessors that will jump to it into a single predecessor.
1279 SmallVector<BasicBlock*, 16> PredsToFactor;
Benjamin Kramer543762d2016-01-09 18:43:01 +00001280 for (const auto &PredToDest : PredToDestList)
1281 if (PredToDest.second == MostPopularDest) {
1282 BasicBlock *Pred = PredToDest.first;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001283
Chris Lattner5ff7f562009-11-07 08:05:03 +00001284 // This predecessor may be a switch or something else that has multiple
1285 // edges to the block. Factor each of these edges by listing them
1286 // according to # occurrences in PredsToFactor.
Benjamin Kramer543762d2016-01-09 18:43:01 +00001287 for (BasicBlock *Succ : successors(Pred))
1288 if (Succ == BB)
Chris Lattner5ff7f562009-11-07 08:05:03 +00001289 PredsToFactor.push_back(Pred);
1290 }
1291
1292 // If the threadable edges are branching on an undefined value, we get to pick
1293 // the destination that these predecessors should get to.
Craig Topperf40110f2014-04-25 05:29:35 +00001294 if (!MostPopularDest)
Chris Lattner5ff7f562009-11-07 08:05:03 +00001295 MostPopularDest = BB->getTerminator()->
1296 getSuccessor(GetBestDestForJumpOnUndef(BB));
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001297
Chris Lattner5ff7f562009-11-07 08:05:03 +00001298 // Ok, try to thread it!
Haicheng Wu64d9d7c2016-03-15 23:38:47 +00001299 return ThreadEdge(BB, PredsToFactor, MostPopularDest);
Chris Lattner5ff7f562009-11-07 08:05:03 +00001300}
Chris Lattner98d89d12008-11-27 05:07:53 +00001301
Chris Lattner6a19ed02010-01-11 23:41:09 +00001302/// ProcessBranchOnPHI - We have an otherwise unthreadable conditional branch on
1303/// a PHI node in the current block. See if there are any simplifications we
1304/// can do based on inputs to the phi node.
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001305///
Chris Lattner6a19ed02010-01-11 23:41:09 +00001306bool JumpThreading::ProcessBranchOnPHI(PHINode *PN) {
Chris Lattner6ce85e82009-10-11 04:40:21 +00001307 BasicBlock *BB = PN->getParent();
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001308
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001309 // TODO: We could make use of this to do it once for blocks with common PHI
1310 // values.
1311 SmallVector<BasicBlock*, 1> PredBBs;
1312 PredBBs.resize(1);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001313
Chris Lattner5ff7f562009-11-07 08:05:03 +00001314 // If any of the predecessor blocks end in an unconditional branch, we can
Chris Lattner6a19ed02010-01-11 23:41:09 +00001315 // *duplicate* the conditional branch into that block in order to further
1316 // encourage jump threading and to eliminate cases where we have branch on a
1317 // phi of an icmp (branch on icmp is much better).
Chris Lattner97b14052009-10-11 07:24:57 +00001318 for (unsigned i = 0, e = PN->getNumIncomingValues(); i != e; ++i) {
1319 BasicBlock *PredBB = PN->getIncomingBlock(i);
1320 if (BranchInst *PredBr = dyn_cast<BranchInst>(PredBB->getTerminator()))
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001321 if (PredBr->isUnconditional()) {
1322 PredBBs[0] = PredBB;
1323 // Try to duplicate BB into PredBB.
1324 if (DuplicateCondBranchOnPHIIntoPred(BB, PredBBs))
1325 return true;
1326 }
Chris Lattner97b14052009-10-11 07:24:57 +00001327 }
1328
Chris Lattner6ce85e82009-10-11 04:40:21 +00001329 return false;
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001330}
1331
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001332/// ProcessBranchOnXOR - We have an otherwise unthreadable conditional branch on
1333/// a xor instruction in the current block. See if there are any
1334/// simplifications we can do based on inputs to the xor.
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001335///
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001336bool JumpThreading::ProcessBranchOnXOR(BinaryOperator *BO) {
1337 BasicBlock *BB = BO->getParent();
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001338
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001339 // If either the LHS or RHS of the xor is a constant, don't do this
1340 // optimization.
1341 if (isa<ConstantInt>(BO->getOperand(0)) ||
1342 isa<ConstantInt>(BO->getOperand(1)))
1343 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001344
Chris Lattnerba2d0b82010-01-23 19:16:25 +00001345 // If the first instruction in BB isn't a phi, we won't be able to infer
1346 // anything special about any particular predecessor.
1347 if (!isa<PHINode>(BB->front()))
1348 return false;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001349
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001350 // If we have a xor as the branch input to this block, and we know that the
1351 // LHS or RHS of the xor in any predecessor is true/false, then we can clone
1352 // the condition into the predecessor and fix that value to true, saving some
1353 // logical ops on that path and encouraging other paths to simplify.
1354 //
1355 // This copies something like this:
1356 //
1357 // BB:
1358 // %X = phi i1 [1], [%X']
1359 // %Y = icmp eq i32 %A, %B
1360 // %Z = xor i1 %X, %Y
1361 // br i1 %Z, ...
1362 //
1363 // Into:
1364 // BB':
1365 // %Y = icmp ne i32 %A, %B
David L Kreitzerda700ce2015-09-16 13:27:30 +00001366 // br i1 %Y, ...
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001367
Frits van Bommel76244862010-12-05 19:06:41 +00001368 PredValueInfoTy XorOpValues;
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001369 bool isLHS = true;
Frits van Bommeld9df6ea2010-12-06 23:36:56 +00001370 if (!ComputeValueKnownInPredecessors(BO->getOperand(0), BB, XorOpValues,
Haicheng Wu64d9d7c2016-03-15 23:38:47 +00001371 WantInteger, BO)) {
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001372 assert(XorOpValues.empty());
Frits van Bommeld9df6ea2010-12-06 23:36:56 +00001373 if (!ComputeValueKnownInPredecessors(BO->getOperand(1), BB, XorOpValues,
Haicheng Wu64d9d7c2016-03-15 23:38:47 +00001374 WantInteger, BO))
1375 return false;
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001376 isLHS = false;
1377 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001378
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001379 assert(!XorOpValues.empty() &&
1380 "ComputeValueKnownInPredecessors returned true with no values");
1381
1382 // Scan the information to see which is most popular: true or false. The
1383 // predecessors can be of the set true, false, or undef.
1384 unsigned NumTrue = 0, NumFalse = 0;
Benjamin Kramer543762d2016-01-09 18:43:01 +00001385 for (const auto &XorOpValue : XorOpValues) {
1386 if (isa<UndefValue>(XorOpValue.first))
Frits van Bommel76244862010-12-05 19:06:41 +00001387 // Ignore undefs for the count.
1388 continue;
Benjamin Kramer543762d2016-01-09 18:43:01 +00001389 if (cast<ConstantInt>(XorOpValue.first)->isZero())
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001390 ++NumFalse;
1391 else
1392 ++NumTrue;
1393 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001394
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001395 // Determine which value to split on, true, false, or undef if neither.
Craig Topperf40110f2014-04-25 05:29:35 +00001396 ConstantInt *SplitVal = nullptr;
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001397 if (NumTrue > NumFalse)
1398 SplitVal = ConstantInt::getTrue(BB->getContext());
1399 else if (NumTrue != 0 || NumFalse != 0)
1400 SplitVal = ConstantInt::getFalse(BB->getContext());
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001401
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001402 // Collect all of the blocks that this can be folded into so that we can
1403 // factor this once and clone it once.
1404 SmallVector<BasicBlock*, 8> BlocksToFoldInto;
Benjamin Kramer543762d2016-01-09 18:43:01 +00001405 for (const auto &XorOpValue : XorOpValues) {
1406 if (XorOpValue.first != SplitVal && !isa<UndefValue>(XorOpValue.first))
Frits van Bommel76244862010-12-05 19:06:41 +00001407 continue;
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001408
Benjamin Kramer543762d2016-01-09 18:43:01 +00001409 BlocksToFoldInto.push_back(XorOpValue.second);
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001410 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001411
Chris Lattnerba2d0b82010-01-23 19:16:25 +00001412 // If we inferred a value for all of the predecessors, then duplication won't
1413 // help us. However, we can just replace the LHS or RHS with the constant.
1414 if (BlocksToFoldInto.size() ==
1415 cast<PHINode>(BB->front()).getNumIncomingValues()) {
Craig Topperf40110f2014-04-25 05:29:35 +00001416 if (!SplitVal) {
Chris Lattnerba2d0b82010-01-23 19:16:25 +00001417 // If all preds provide undef, just nuke the xor, because it is undef too.
1418 BO->replaceAllUsesWith(UndefValue::get(BO->getType()));
1419 BO->eraseFromParent();
1420 } else if (SplitVal->isZero()) {
1421 // If all preds provide 0, replace the xor with the other input.
1422 BO->replaceAllUsesWith(BO->getOperand(isLHS));
1423 BO->eraseFromParent();
1424 } else {
1425 // If all preds provide 1, set the computed value to 1.
1426 BO->setOperand(!isLHS, SplitVal);
1427 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001428
Chris Lattnerba2d0b82010-01-23 19:16:25 +00001429 return true;
1430 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001431
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001432 // Try to duplicate BB into PredBB.
Haicheng Wu64d9d7c2016-03-15 23:38:47 +00001433 return DuplicateCondBranchOnPHIIntoPred(BB, BlocksToFoldInto);
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001434}
1435
1436
Chris Lattner97b14052009-10-11 07:24:57 +00001437/// AddPHINodeEntriesForMappedBlock - We're adding 'NewPred' as a new
1438/// predecessor to the PHIBB block. If it has PHI nodes, add entries for
1439/// NewPred using the entries from OldPred (suitably mapped).
1440static void AddPHINodeEntriesForMappedBlock(BasicBlock *PHIBB,
1441 BasicBlock *OldPred,
1442 BasicBlock *NewPred,
1443 DenseMap<Instruction*, Value*> &ValueMap) {
1444 for (BasicBlock::iterator PNI = PHIBB->begin();
1445 PHINode *PN = dyn_cast<PHINode>(PNI); ++PNI) {
1446 // Ok, we have a PHI node. Figure out what the incoming value was for the
1447 // DestBlock.
1448 Value *IV = PN->getIncomingValueForBlock(OldPred);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001449
Chris Lattner97b14052009-10-11 07:24:57 +00001450 // Remap the value if necessary.
1451 if (Instruction *Inst = dyn_cast<Instruction>(IV)) {
1452 DenseMap<Instruction*, Value*>::iterator I = ValueMap.find(Inst);
1453 if (I != ValueMap.end())
1454 IV = I->second;
1455 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001456
Chris Lattner97b14052009-10-11 07:24:57 +00001457 PN->addIncoming(IV, NewPred);
1458 }
1459}
Chris Lattner3df4c152008-04-22 07:05:46 +00001460
Chris Lattner5ff7f562009-11-07 08:05:03 +00001461/// ThreadEdge - We have decided that it is safe and profitable to factor the
1462/// blocks in PredBBs to one predecessor, then thread an edge from it to SuccBB
1463/// across BB. Transform the IR to reflect this change.
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001464bool JumpThreading::ThreadEdge(BasicBlock *BB,
1465 const SmallVectorImpl<BasicBlock*> &PredBBs,
Chris Lattnerf466bc82009-10-11 04:33:43 +00001466 BasicBlock *SuccBB) {
Chris Lattnerd579cb12009-05-04 02:28:08 +00001467 // If threading to the same block as we come from, we would infinite loop.
1468 if (SuccBB == BB) {
David Greene1efdb452010-01-05 01:27:19 +00001469 DEBUG(dbgs() << " Not threading across BB '" << BB->getName()
Daniel Dunbar9813b0b2009-07-26 07:49:05 +00001470 << "' - would thread to self!\n");
Chris Lattnerd579cb12009-05-04 02:28:08 +00001471 return false;
1472 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001473
Chris Lattnerd579cb12009-05-04 02:28:08 +00001474 // If threading this would thread across a loop header, don't thread the edge.
1475 // See the comments above FindLoopHeaders for justifications and caveats.
1476 if (LoopHeaders.count(BB)) {
David Greene1efdb452010-01-05 01:27:19 +00001477 DEBUG(dbgs() << " Not threading across loop header BB '" << BB->getName()
Daniel Dunbar9813b0b2009-07-26 07:49:05 +00001478 << "' to dest BB '" << SuccBB->getName()
1479 << "' - it might create an irreducible loop!\n");
Chris Lattnerd579cb12009-05-04 02:28:08 +00001480 return false;
1481 }
1482
Michael Liaod1209162014-09-24 04:59:06 +00001483 unsigned JumpThreadCost = getJumpThreadDuplicationCost(BB, BBDupThreshold);
1484 if (JumpThreadCost > BBDupThreshold) {
David Greene1efdb452010-01-05 01:27:19 +00001485 DEBUG(dbgs() << " Not threading BB '" << BB->getName()
Chris Lattner97b14052009-10-11 07:24:57 +00001486 << "' - Cost is too high: " << JumpThreadCost << "\n");
1487 return false;
1488 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001489
David L Kreitzerda700ce2015-09-16 13:27:30 +00001490 // And finally, do it! Start by factoring the predecessors if needed.
Chris Lattner5ff7f562009-11-07 08:05:03 +00001491 BasicBlock *PredBB;
1492 if (PredBBs.size() == 1)
1493 PredBB = PredBBs[0];
1494 else {
David Greene1efdb452010-01-05 01:27:19 +00001495 DEBUG(dbgs() << " Factoring out " << PredBBs.size()
Chris Lattner5ff7f562009-11-07 08:05:03 +00001496 << " common predecessors.\n");
Manman Ren72d44b12015-10-15 14:59:40 +00001497 PredBB = SplitBlockPreds(BB, PredBBs, ".thr_comm");
Chris Lattner5ff7f562009-11-07 08:05:03 +00001498 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001499
Chris Lattnerd579cb12009-05-04 02:28:08 +00001500 // And finally, do it!
David Greene1efdb452010-01-05 01:27:19 +00001501 DEBUG(dbgs() << " Threading edge from '" << PredBB->getName() << "' to '"
Daniel Dunbar6115b392009-07-26 09:48:23 +00001502 << SuccBB->getName() << "' with cost: " << JumpThreadCost
Daniel Dunbar9813b0b2009-07-26 07:49:05 +00001503 << ", across block:\n "
1504 << *BB << "\n");
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001505
Owen Andersond361aac2010-09-14 20:57:41 +00001506 LVI->threadEdge(PredBB, BB, SuccBB);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001507
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001508 // We are going to have to map operands from the original BB block to the new
1509 // copy of the block 'NewBB'. If there are PHI nodes in BB, evaluate them to
1510 // account for entry from PredBB.
1511 DenseMap<Instruction*, Value*> ValueMapping;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001512
1513 BasicBlock *NewBB = BasicBlock::Create(BB->getContext(),
1514 BB->getName()+".thread",
Owen Anderson55f1c092009-08-13 21:58:54 +00001515 BB->getParent(), BB);
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001516 NewBB->moveAfter(PredBB);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001517
Manman Ren72d44b12015-10-15 14:59:40 +00001518 // Set the block frequency of NewBB.
1519 if (HasProfileData) {
1520 auto NewBBFreq =
1521 BFI->getBlockFreq(PredBB) * BPI->getEdgeProbability(PredBB, BB);
1522 BFI->setBlockFreq(NewBB, NewBBFreq.getFrequency());
1523 }
1524
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001525 BasicBlock::iterator BI = BB->begin();
1526 for (; PHINode *PN = dyn_cast<PHINode>(BI); ++BI)
1527 ValueMapping[PN] = PN->getIncomingValueForBlock(PredBB);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001528
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001529 // Clone the non-phi instructions of BB into NewBB, keeping track of the
1530 // mapping and using it to remap operands in the cloned instructions.
1531 for (; !isa<TerminatorInst>(BI); ++BI) {
Nick Lewycky42fb7452009-09-27 07:38:41 +00001532 Instruction *New = BI->clone();
Daniel Dunbar6115b392009-07-26 09:48:23 +00001533 New->setName(BI->getName());
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001534 NewBB->getInstList().push_back(New);
Duncan P. N. Exon Smith3a9c9e32015-10-13 18:26:00 +00001535 ValueMapping[&*BI] = New;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001536
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001537 // Remap operands to patch up intra-block references.
1538 for (unsigned i = 0, e = New->getNumOperands(); i != e; ++i)
Dan Gohman43f33dd2009-07-02 00:17:47 +00001539 if (Instruction *Inst = dyn_cast<Instruction>(New->getOperand(i))) {
1540 DenseMap<Instruction*, Value*>::iterator I = ValueMapping.find(Inst);
1541 if (I != ValueMapping.end())
1542 New->setOperand(i, I->second);
1543 }
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001544 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001545
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001546 // We didn't copy the terminator from BB over to NewBB, because there is now
1547 // an unconditional jump to SuccBB. Insert the unconditional jump.
Manman Ren72d44b12015-10-15 14:59:40 +00001548 BranchInst *NewBI = BranchInst::Create(SuccBB, NewBB);
Devang Patel306f8db2011-05-04 22:48:19 +00001549 NewBI->setDebugLoc(BB->getTerminator()->getDebugLoc());
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001550
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001551 // Check to see if SuccBB has PHI nodes. If so, we need to add entries to the
1552 // PHI nodes for NewBB now.
Chris Lattner97b14052009-10-11 07:24:57 +00001553 AddPHINodeEntriesForMappedBlock(SuccBB, BB, NewBB, ValueMapping);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001554
Chris Lattner84095072009-10-10 09:05:58 +00001555 // If there were values defined in BB that are used outside the block, then we
1556 // now have to update all uses of the value to use either the original value,
1557 // the cloned value, or some PHI derived value. This can require arbitrary
1558 // PHI insertion, of which we are prepared to do, clean these up now.
1559 SSAUpdater SSAUpdate;
1560 SmallVector<Use*, 16> UsesToRename;
Benjamin Kramer543762d2016-01-09 18:43:01 +00001561 for (Instruction &I : *BB) {
Chris Lattner84095072009-10-10 09:05:58 +00001562 // Scan all uses of this instruction to see if it is used outside of its
1563 // block, and if so, record them in UsesToRename.
Benjamin Kramer543762d2016-01-09 18:43:01 +00001564 for (Use &U : I.uses()) {
Chandler Carruthcdf47882014-03-09 03:16:01 +00001565 Instruction *User = cast<Instruction>(U.getUser());
Chris Lattner84095072009-10-10 09:05:58 +00001566 if (PHINode *UserPN = dyn_cast<PHINode>(User)) {
Chandler Carruthcdf47882014-03-09 03:16:01 +00001567 if (UserPN->getIncomingBlock(U) == BB)
Chris Lattner84095072009-10-10 09:05:58 +00001568 continue;
1569 } else if (User->getParent() == BB)
1570 continue;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001571
Chandler Carruthcdf47882014-03-09 03:16:01 +00001572 UsesToRename.push_back(&U);
Chris Lattner84095072009-10-10 09:05:58 +00001573 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001574
Chris Lattner84095072009-10-10 09:05:58 +00001575 // If there are no uses outside the block, we're done with this instruction.
1576 if (UsesToRename.empty())
1577 continue;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001578
Benjamin Kramer543762d2016-01-09 18:43:01 +00001579 DEBUG(dbgs() << "JT: Renaming non-local uses of: " << I << "\n");
Chris Lattner84095072009-10-10 09:05:58 +00001580
1581 // We found a use of I outside of BB. Rename all uses of I that are outside
1582 // its block to be uses of the appropriate PHI node etc. See ValuesInBlocks
1583 // with the two values we know.
Benjamin Kramer543762d2016-01-09 18:43:01 +00001584 SSAUpdate.Initialize(I.getType(), I.getName());
1585 SSAUpdate.AddAvailableValue(BB, &I);
1586 SSAUpdate.AddAvailableValue(NewBB, ValueMapping[&I]);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001587
Chris Lattner84095072009-10-10 09:05:58 +00001588 while (!UsesToRename.empty())
1589 SSAUpdate.RewriteUse(*UsesToRename.pop_back_val());
David Greene1efdb452010-01-05 01:27:19 +00001590 DEBUG(dbgs() << "\n");
Chris Lattner84095072009-10-10 09:05:58 +00001591 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001592
1593
Chris Lattner88a1f022008-12-01 04:48:07 +00001594 // Ok, NewBB is good to go. Update the terminator of PredBB to jump to
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001595 // NewBB instead of BB. This eliminates predecessors from BB, which requires
1596 // us to simplify any PHI nodes in BB.
1597 TerminatorInst *PredTerm = PredBB->getTerminator();
1598 for (unsigned i = 0, e = PredTerm->getNumSuccessors(); i != e; ++i)
1599 if (PredTerm->getSuccessor(i) == BB) {
Owen Anderson99c985c2010-09-29 20:34:41 +00001600 BB->removePredecessor(PredBB, true);
Chris Lattnerff1c6e32008-04-20 22:39:42 +00001601 PredTerm->setSuccessor(i, NewBB);
1602 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001603
Chris Lattner88a1f022008-12-01 04:48:07 +00001604 // At this point, the IR is fully up to date and consistent. Do a quick scan
1605 // over the new instructions and zap any that are constants or dead. This
1606 // frequently happens because of phi translation.
Mehdi Aminia28d91d2015-03-10 02:37:25 +00001607 SimplifyInstructionsInBlock(NewBB, TLI);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001608
Manman Ren72d44b12015-10-15 14:59:40 +00001609 // Update the edge weight from BB to SuccBB, which should be less than before.
1610 UpdateBlockFreqAndEdgeWeight(PredBB, BB, NewBB, SuccBB);
1611
Chris Lattnerd579cb12009-05-04 02:28:08 +00001612 // Threaded an edge!
1613 ++NumThreads;
1614 return true;
Chris Lattner21157222008-04-20 21:13:06 +00001615}
Chris Lattner97b14052009-10-11 07:24:57 +00001616
Manman Ren72d44b12015-10-15 14:59:40 +00001617/// Create a new basic block that will be the predecessor of BB and successor of
1618/// all blocks in Preds. When profile data is availble, update the frequency of
1619/// this new block.
1620BasicBlock *JumpThreading::SplitBlockPreds(BasicBlock *BB,
1621 ArrayRef<BasicBlock *> Preds,
1622 const char *Suffix) {
1623 // Collect the frequencies of all predecessors of BB, which will be used to
1624 // update the edge weight on BB->SuccBB.
1625 BlockFrequency PredBBFreq(0);
1626 if (HasProfileData)
1627 for (auto Pred : Preds)
1628 PredBBFreq += BFI->getBlockFreq(Pred) * BPI->getEdgeProbability(Pred, BB);
1629
1630 BasicBlock *PredBB = SplitBlockPredecessors(BB, Preds, Suffix);
1631
1632 // Set the block frequency of the newly created PredBB, which is the sum of
1633 // frequencies of Preds.
1634 if (HasProfileData)
1635 BFI->setBlockFreq(PredBB, PredBBFreq.getFrequency());
1636 return PredBB;
1637}
1638
1639/// Update the block frequency of BB and branch weight and the metadata on the
1640/// edge BB->SuccBB. This is done by scaling the weight of BB->SuccBB by 1 -
1641/// Freq(PredBB->BB) / Freq(BB->SuccBB).
1642void JumpThreading::UpdateBlockFreqAndEdgeWeight(BasicBlock *PredBB,
1643 BasicBlock *BB,
1644 BasicBlock *NewBB,
1645 BasicBlock *SuccBB) {
1646 if (!HasProfileData)
1647 return;
1648
1649 assert(BFI && BPI && "BFI & BPI should have been created here");
1650
1651 // As the edge from PredBB to BB is deleted, we have to update the block
1652 // frequency of BB.
1653 auto BBOrigFreq = BFI->getBlockFreq(BB);
1654 auto NewBBFreq = BFI->getBlockFreq(NewBB);
1655 auto BB2SuccBBFreq = BBOrigFreq * BPI->getEdgeProbability(BB, SuccBB);
1656 auto BBNewFreq = BBOrigFreq - NewBBFreq;
1657 BFI->setBlockFreq(BB, BBNewFreq.getFrequency());
1658
1659 // Collect updated outgoing edges' frequencies from BB and use them to update
Cong Houe93b8e12015-12-22 18:56:14 +00001660 // edge probabilities.
Manman Ren72d44b12015-10-15 14:59:40 +00001661 SmallVector<uint64_t, 4> BBSuccFreq;
Benjamin Kramer543762d2016-01-09 18:43:01 +00001662 for (BasicBlock *Succ : successors(BB)) {
1663 auto SuccFreq = (Succ == SuccBB)
Manman Ren72d44b12015-10-15 14:59:40 +00001664 ? BB2SuccBBFreq - NewBBFreq
Benjamin Kramer543762d2016-01-09 18:43:01 +00001665 : BBOrigFreq * BPI->getEdgeProbability(BB, Succ);
Manman Ren72d44b12015-10-15 14:59:40 +00001666 BBSuccFreq.push_back(SuccFreq.getFrequency());
1667 }
1668
Cong Houe93b8e12015-12-22 18:56:14 +00001669 uint64_t MaxBBSuccFreq =
1670 *std::max_element(BBSuccFreq.begin(), BBSuccFreq.end());
Manman Ren72d44b12015-10-15 14:59:40 +00001671
Cong Hou6a2c71a2015-12-22 23:45:55 +00001672 SmallVector<BranchProbability, 4> BBSuccProbs;
1673 if (MaxBBSuccFreq == 0)
1674 BBSuccProbs.assign(BBSuccFreq.size(),
1675 {1, static_cast<unsigned>(BBSuccFreq.size())});
1676 else {
1677 for (uint64_t Freq : BBSuccFreq)
1678 BBSuccProbs.push_back(
1679 BranchProbability::getBranchProbability(Freq, MaxBBSuccFreq));
1680 // Normalize edge probabilities so that they sum up to one.
1681 BranchProbability::normalizeProbabilities(BBSuccProbs.begin(),
1682 BBSuccProbs.end());
1683 }
Manman Ren72d44b12015-10-15 14:59:40 +00001684
Cong Houe93b8e12015-12-22 18:56:14 +00001685 // Update edge probabilities in BPI.
1686 for (int I = 0, E = BBSuccProbs.size(); I < E; I++)
1687 BPI->setEdgeProbability(BB, I, BBSuccProbs[I]);
Manman Ren72d44b12015-10-15 14:59:40 +00001688
Cong Houe93b8e12015-12-22 18:56:14 +00001689 if (BBSuccProbs.size() >= 2) {
1690 SmallVector<uint32_t, 4> Weights;
1691 for (auto Prob : BBSuccProbs)
1692 Weights.push_back(Prob.getNumerator());
1693
Manman Ren72d44b12015-10-15 14:59:40 +00001694 auto TI = BB->getTerminator();
1695 TI->setMetadata(
1696 LLVMContext::MD_prof,
1697 MDBuilder(TI->getParent()->getContext()).createBranchWeights(Weights));
1698 }
1699}
1700
Chris Lattner97b14052009-10-11 07:24:57 +00001701/// DuplicateCondBranchOnPHIIntoPred - PredBB contains an unconditional branch
1702/// to BB which contains an i1 PHI node and a conditional branch on that PHI.
1703/// If we can duplicate the contents of BB up into PredBB do so now, this
1704/// improves the odds that the branch will be on an analyzable instruction like
1705/// a compare.
1706bool JumpThreading::DuplicateCondBranchOnPHIIntoPred(BasicBlock *BB,
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001707 const SmallVectorImpl<BasicBlock *> &PredBBs) {
1708 assert(!PredBBs.empty() && "Can't handle an empty set");
1709
Chris Lattner97b14052009-10-11 07:24:57 +00001710 // If BB is a loop header, then duplicating this block outside the loop would
1711 // cause us to transform this into an irreducible loop, don't do this.
1712 // See the comments above FindLoopHeaders for justifications and caveats.
1713 if (LoopHeaders.count(BB)) {
David Greene1efdb452010-01-05 01:27:19 +00001714 DEBUG(dbgs() << " Not duplicating loop header '" << BB->getName()
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001715 << "' into predecessor block '" << PredBBs[0]->getName()
Chris Lattner97b14052009-10-11 07:24:57 +00001716 << "' - it might create an irreducible loop!\n");
1717 return false;
1718 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001719
Michael Liaod1209162014-09-24 04:59:06 +00001720 unsigned DuplicationCost = getJumpThreadDuplicationCost(BB, BBDupThreshold);
1721 if (DuplicationCost > BBDupThreshold) {
David Greene1efdb452010-01-05 01:27:19 +00001722 DEBUG(dbgs() << " Not duplicating BB '" << BB->getName()
Chris Lattner97b14052009-10-11 07:24:57 +00001723 << "' - Cost is too high: " << DuplicationCost << "\n");
1724 return false;
1725 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001726
David L Kreitzerda700ce2015-09-16 13:27:30 +00001727 // And finally, do it! Start by factoring the predecessors if needed.
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001728 BasicBlock *PredBB;
1729 if (PredBBs.size() == 1)
1730 PredBB = PredBBs[0];
1731 else {
1732 DEBUG(dbgs() << " Factoring out " << PredBBs.size()
1733 << " common predecessors.\n");
Manman Ren72d44b12015-10-15 14:59:40 +00001734 PredBB = SplitBlockPreds(BB, PredBBs, ".thr_comm");
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001735 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001736
Chris Lattner97b14052009-10-11 07:24:57 +00001737 // Okay, we decided to do this! Clone all the instructions in BB onto the end
1738 // of PredBB.
David Greene1efdb452010-01-05 01:27:19 +00001739 DEBUG(dbgs() << " Duplicating block '" << BB->getName() << "' into end of '"
Chris Lattner97b14052009-10-11 07:24:57 +00001740 << PredBB->getName() << "' to eliminate branch on phi. Cost: "
1741 << DuplicationCost << " block is:" << *BB << "\n");
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001742
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001743 // Unless PredBB ends with an unconditional branch, split the edge so that we
1744 // can just clone the bits from BB into the end of the new PredBB.
Chris Lattner29b15c52010-01-23 19:21:31 +00001745 BranchInst *OldPredBranch = dyn_cast<BranchInst>(PredBB->getTerminator());
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001746
Craig Topperf40110f2014-04-25 05:29:35 +00001747 if (!OldPredBranch || !OldPredBranch->isUnconditional()) {
Chandler Carruthd4500562015-01-19 12:36:53 +00001748 PredBB = SplitEdge(PredBB, BB);
Chris Lattnereb73bdb2010-01-12 02:07:17 +00001749 OldPredBranch = cast<BranchInst>(PredBB->getTerminator());
1750 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001751
Chris Lattner97b14052009-10-11 07:24:57 +00001752 // We are going to have to map operands from the original BB block into the
1753 // PredBB block. Evaluate PHI nodes in BB.
1754 DenseMap<Instruction*, Value*> ValueMapping;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001755
Chris Lattner97b14052009-10-11 07:24:57 +00001756 BasicBlock::iterator BI = BB->begin();
1757 for (; PHINode *PN = dyn_cast<PHINode>(BI); ++BI)
1758 ValueMapping[PN] = PN->getIncomingValueForBlock(PredBB);
Chris Lattner97b14052009-10-11 07:24:57 +00001759 // Clone the non-phi instructions of BB into PredBB, keeping track of the
1760 // mapping and using it to remap operands in the cloned instructions.
1761 for (; BI != BB->end(); ++BI) {
1762 Instruction *New = BI->clone();
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001763
Chris Lattner97b14052009-10-11 07:24:57 +00001764 // Remap operands to patch up intra-block references.
1765 for (unsigned i = 0, e = New->getNumOperands(); i != e; ++i)
1766 if (Instruction *Inst = dyn_cast<Instruction>(New->getOperand(i))) {
1767 DenseMap<Instruction*, Value*>::iterator I = ValueMapping.find(Inst);
1768 if (I != ValueMapping.end())
1769 New->setOperand(i, I->second);
1770 }
Chris Lattner573da8a2010-01-12 20:41:47 +00001771
1772 // If this instruction can be simplified after the operands are updated,
1773 // just use the simplified value instead. This frequently happens due to
1774 // phi translation.
Mehdi Aminia28d91d2015-03-10 02:37:25 +00001775 if (Value *IV =
1776 SimplifyInstruction(New, BB->getModule()->getDataLayout())) {
Chris Lattner573da8a2010-01-12 20:41:47 +00001777 delete New;
Duncan P. N. Exon Smith3a9c9e32015-10-13 18:26:00 +00001778 ValueMapping[&*BI] = IV;
Chris Lattner573da8a2010-01-12 20:41:47 +00001779 } else {
1780 // Otherwise, insert the new instruction into the block.
1781 New->setName(BI->getName());
Duncan P. N. Exon Smith3a9c9e32015-10-13 18:26:00 +00001782 PredBB->getInstList().insert(OldPredBranch->getIterator(), New);
1783 ValueMapping[&*BI] = New;
Chris Lattner573da8a2010-01-12 20:41:47 +00001784 }
Chris Lattner97b14052009-10-11 07:24:57 +00001785 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001786
Chris Lattner97b14052009-10-11 07:24:57 +00001787 // Check to see if the targets of the branch had PHI nodes. If so, we need to
1788 // add entries to the PHI nodes for branch from PredBB now.
1789 BranchInst *BBBranch = cast<BranchInst>(BB->getTerminator());
1790 AddPHINodeEntriesForMappedBlock(BBBranch->getSuccessor(0), BB, PredBB,
1791 ValueMapping);
1792 AddPHINodeEntriesForMappedBlock(BBBranch->getSuccessor(1), BB, PredBB,
1793 ValueMapping);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001794
Chris Lattner97b14052009-10-11 07:24:57 +00001795 // If there were values defined in BB that are used outside the block, then we
1796 // now have to update all uses of the value to use either the original value,
1797 // the cloned value, or some PHI derived value. This can require arbitrary
1798 // PHI insertion, of which we are prepared to do, clean these up now.
1799 SSAUpdater SSAUpdate;
1800 SmallVector<Use*, 16> UsesToRename;
Benjamin Kramer543762d2016-01-09 18:43:01 +00001801 for (Instruction &I : *BB) {
Chris Lattner97b14052009-10-11 07:24:57 +00001802 // Scan all uses of this instruction to see if it is used outside of its
1803 // block, and if so, record them in UsesToRename.
Benjamin Kramer543762d2016-01-09 18:43:01 +00001804 for (Use &U : I.uses()) {
Chandler Carruthcdf47882014-03-09 03:16:01 +00001805 Instruction *User = cast<Instruction>(U.getUser());
Chris Lattner97b14052009-10-11 07:24:57 +00001806 if (PHINode *UserPN = dyn_cast<PHINode>(User)) {
Chandler Carruthcdf47882014-03-09 03:16:01 +00001807 if (UserPN->getIncomingBlock(U) == BB)
Chris Lattner97b14052009-10-11 07:24:57 +00001808 continue;
1809 } else if (User->getParent() == BB)
1810 continue;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001811
Chandler Carruthcdf47882014-03-09 03:16:01 +00001812 UsesToRename.push_back(&U);
Chris Lattner97b14052009-10-11 07:24:57 +00001813 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001814
Chris Lattner97b14052009-10-11 07:24:57 +00001815 // If there are no uses outside the block, we're done with this instruction.
1816 if (UsesToRename.empty())
1817 continue;
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001818
Benjamin Kramer543762d2016-01-09 18:43:01 +00001819 DEBUG(dbgs() << "JT: Renaming non-local uses of: " << I << "\n");
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001820
Chris Lattner97b14052009-10-11 07:24:57 +00001821 // We found a use of I outside of BB. Rename all uses of I that are outside
1822 // its block to be uses of the appropriate PHI node etc. See ValuesInBlocks
1823 // with the two values we know.
Benjamin Kramer543762d2016-01-09 18:43:01 +00001824 SSAUpdate.Initialize(I.getType(), I.getName());
1825 SSAUpdate.AddAvailableValue(BB, &I);
1826 SSAUpdate.AddAvailableValue(PredBB, ValueMapping[&I]);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001827
Chris Lattner97b14052009-10-11 07:24:57 +00001828 while (!UsesToRename.empty())
1829 SSAUpdate.RewriteUse(*UsesToRename.pop_back_val());
David Greene1efdb452010-01-05 01:27:19 +00001830 DEBUG(dbgs() << "\n");
Chris Lattner97b14052009-10-11 07:24:57 +00001831 }
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001832
Chris Lattner97b14052009-10-11 07:24:57 +00001833 // PredBB no longer jumps to BB, remove entries in the PHI node for the edge
1834 // that we nuked.
Owen Anderson99c985c2010-09-29 20:34:41 +00001835 BB->removePredecessor(PredBB, true);
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001836
Chris Lattner97b14052009-10-11 07:24:57 +00001837 // Remove the unconditional branch at the end of the PredBB block.
1838 OldPredBranch->eraseFromParent();
Frits van Bommel5e75ef42010-12-05 19:02:47 +00001839
Chris Lattner97b14052009-10-11 07:24:57 +00001840 ++NumDupes;
1841 return true;
1842}
Benjamin Kramer6a4976d2013-08-07 10:29:38 +00001843
1844/// TryToUnfoldSelect - Look for blocks of the form
1845/// bb1:
1846/// %a = select
1847/// br bb
1848///
1849/// bb2:
1850/// %p = phi [%a, %bb] ...
1851/// %c = icmp %p
1852/// br i1 %c
1853///
1854/// And expand the select into a branch structure if one of its arms allows %c
1855/// to be folded. This later enables threading from bb1 over bb2.
1856bool JumpThreading::TryToUnfoldSelect(CmpInst *CondCmp, BasicBlock *BB) {
1857 BranchInst *CondBr = dyn_cast<BranchInst>(BB->getTerminator());
1858 PHINode *CondLHS = dyn_cast<PHINode>(CondCmp->getOperand(0));
1859 Constant *CondRHS = cast<Constant>(CondCmp->getOperand(1));
1860
1861 if (!CondBr || !CondBr->isConditional() || !CondLHS ||
1862 CondLHS->getParent() != BB)
1863 return false;
1864
1865 for (unsigned I = 0, E = CondLHS->getNumIncomingValues(); I != E; ++I) {
1866 BasicBlock *Pred = CondLHS->getIncomingBlock(I);
1867 SelectInst *SI = dyn_cast<SelectInst>(CondLHS->getIncomingValue(I));
1868
1869 // Look if one of the incoming values is a select in the corresponding
1870 // predecessor.
1871 if (!SI || SI->getParent() != Pred || !SI->hasOneUse())
1872 continue;
1873
1874 BranchInst *PredTerm = dyn_cast<BranchInst>(Pred->getTerminator());
1875 if (!PredTerm || !PredTerm->isUnconditional())
1876 continue;
1877
1878 // Now check if one of the select values would allow us to constant fold the
1879 // terminator in BB. We don't do the transform if both sides fold, those
1880 // cases will be threaded in any case.
1881 LazyValueInfo::Tristate LHSFolds =
1882 LVI->getPredicateOnEdge(CondCmp->getPredicate(), SI->getOperand(1),
Hal Finkel7e184492014-09-07 20:29:59 +00001883 CondRHS, Pred, BB, CondCmp);
Benjamin Kramer6a4976d2013-08-07 10:29:38 +00001884 LazyValueInfo::Tristate RHSFolds =
1885 LVI->getPredicateOnEdge(CondCmp->getPredicate(), SI->getOperand(2),
Hal Finkel7e184492014-09-07 20:29:59 +00001886 CondRHS, Pred, BB, CondCmp);
Benjamin Kramer6a4976d2013-08-07 10:29:38 +00001887 if ((LHSFolds != LazyValueInfo::Unknown ||
1888 RHSFolds != LazyValueInfo::Unknown) &&
1889 LHSFolds != RHSFolds) {
1890 // Expand the select.
1891 //
1892 // Pred --
1893 // | v
1894 // | NewBB
1895 // | |
1896 // |-----
1897 // v
1898 // BB
1899 BasicBlock *NewBB = BasicBlock::Create(BB->getContext(), "select.unfold",
1900 BB->getParent(), BB);
1901 // Move the unconditional branch to NewBB.
1902 PredTerm->removeFromParent();
1903 NewBB->getInstList().insert(NewBB->end(), PredTerm);
1904 // Create a conditional branch and update PHI nodes.
1905 BranchInst::Create(NewBB, BB, SI->getCondition(), Pred);
1906 CondLHS->setIncomingValue(I, SI->getFalseValue());
1907 CondLHS->addIncoming(SI->getTrueValue(), NewBB);
1908 // The select is now dead.
1909 SI->eraseFromParent();
1910
1911 // Update any other PHI nodes in BB.
1912 for (BasicBlock::iterator BI = BB->begin();
1913 PHINode *Phi = dyn_cast<PHINode>(BI); ++BI)
1914 if (Phi != CondLHS)
1915 Phi->addIncoming(Phi->getIncomingValueForBlock(Pred), NewBB);
1916 return true;
1917 }
1918 }
1919 return false;
1920}
Haicheng Wua6a32792016-01-08 19:39:39 +00001921
1922/// TryToUnfoldSelectInCurrBB - Look for PHI/Select in the same BB of the form
1923/// bb:
1924/// %p = phi [false, %bb1], [true, %bb2], [false, %bb3], [true, %bb4], ...
1925/// %s = select p, trueval, falseval
1926///
1927/// And expand the select into a branch structure. This later enables
1928/// jump-threading over bb in this pass.
1929///
1930/// Using the similar approach of SimplifyCFG::FoldCondBranchOnPHI(), unfold
1931/// select if the associated PHI has at least one constant. If the unfolded
1932/// select is not jump-threaded, it will be folded again in the later
1933/// optimizations.
1934bool JumpThreading::TryToUnfoldSelectInCurrBB(BasicBlock *BB) {
1935 // If threading this would thread across a loop header, don't thread the edge.
1936 // See the comments above FindLoopHeaders for justifications and caveats.
1937 if (LoopHeaders.count(BB))
1938 return false;
1939
1940 // Look for a Phi/Select pair in the same basic block. The Phi feeds the
1941 // condition of the Select and at least one of the incoming values is a
1942 // constant.
1943 for (BasicBlock::iterator BI = BB->begin();
1944 PHINode *PN = dyn_cast<PHINode>(BI); ++BI) {
1945 unsigned NumPHIValues = PN->getNumIncomingValues();
1946 if (NumPHIValues == 0 || !PN->hasOneUse())
1947 continue;
1948
1949 SelectInst *SI = dyn_cast<SelectInst>(PN->user_back());
1950 if (!SI || SI->getParent() != BB)
1951 continue;
1952
1953 Value *Cond = SI->getCondition();
1954 if (!Cond || Cond != PN || !Cond->getType()->isIntegerTy(1))
1955 continue;
1956
1957 bool HasConst = false;
1958 for (unsigned i = 0; i != NumPHIValues; ++i) {
1959 if (PN->getIncomingBlock(i) == BB)
1960 return false;
1961 if (isa<ConstantInt>(PN->getIncomingValue(i)))
1962 HasConst = true;
1963 }
1964
1965 if (HasConst) {
1966 // Expand the select.
1967 TerminatorInst *Term =
1968 SplitBlockAndInsertIfThen(SI->getCondition(), SI, false);
1969 PHINode *NewPN = PHINode::Create(SI->getType(), 2, "", SI);
1970 NewPN->addIncoming(SI->getTrueValue(), Term->getParent());
1971 NewPN->addIncoming(SI->getFalseValue(), BB);
1972 SI->replaceAllUsesWith(NewPN);
1973 SI->eraseFromParent();
1974 return true;
1975 }
1976 }
1977
1978 return false;
1979}