xref: /llvm-project/llvm/lib/Transforms/Scalar/CallSiteSplitting.cpp (revision 9bc0bc4b9b77c3b1fdfcf07477ec560ed0951090)
1 //===- CallSiteSplitting.cpp ----------------------------------------------===//
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 //
10 // This file implements a transformation that tries to split a call-site to pass
11 // more constrained arguments if its argument is predicated in the control flow
12 // so that we can expose better context to the later passes (e.g, inliner, jump
13 // threading, or IPA-CP based function cloning, etc.).
14 // As of now we support two cases :
15 //
16 // 1) Try to a split call-site with constrained arguments, if any constraints
17 // on any argument can be found by following the single predecessors of the
18 // all site's predecessors. Currently this pass only handles call-sites with 2
19 // predecessors. For example, in the code below, we try to split the call-site
20 // since we can predicate the argument(ptr) based on the OR condition.
21 //
22 // Split from :
23 //   if (!ptr || c)
24 //     callee(ptr);
25 // to :
26 //   if (!ptr)
27 //     callee(null)         // set the known constant value
28 //   else if (c)
29 //     callee(nonnull ptr)  // set non-null attribute in the argument
30 //
31 // 2) We can also split a call-site based on constant incoming values of a PHI
32 // For example,
33 // from :
34 //   Header:
35 //    %c = icmp eq i32 %i1, %i2
36 //    br i1 %c, label %Tail, label %TBB
37 //   TBB:
38 //    br label Tail%
39 //   Tail:
40 //    %p = phi i32 [ 0, %Header], [ 1, %TBB]
41 //    call void @bar(i32 %p)
42 // to
43 //   Header:
44 //    %c = icmp eq i32 %i1, %i2
45 //    br i1 %c, label %Tail-split0, label %TBB
46 //   TBB:
47 //    br label %Tail-split1
48 //   Tail-split0:
49 //    call void @bar(i32 0)
50 //    br label %Tail
51 //   Tail-split1:
52 //    call void @bar(i32 1)
53 //    br label %Tail
54 //   Tail:
55 //    %p = phi i32 [ 0, %Tail-split0 ], [ 1, %Tail-split1 ]
56 //
57 //===----------------------------------------------------------------------===//
58 
59 #include "llvm/Transforms/Scalar/CallSiteSplitting.h"
60 #include "llvm/ADT/Statistic.h"
61 #include "llvm/Analysis/TargetLibraryInfo.h"
62 #include "llvm/Analysis/TargetTransformInfo.h"
63 #include "llvm/Analysis/Utils/Local.h"
64 #include "llvm/IR/IntrinsicInst.h"
65 #include "llvm/IR/PatternMatch.h"
66 #include "llvm/Support/Debug.h"
67 #include "llvm/Transforms/Scalar.h"
68 #include "llvm/Transforms/Utils/BasicBlockUtils.h"
69 #include "llvm/Transforms/Utils/Cloning.h"
70 
71 using namespace llvm;
72 using namespace PatternMatch;
73 
74 #define DEBUG_TYPE "callsite-splitting"
75 
76 STATISTIC(NumCallSiteSplit, "Number of call-site split");
77 
78 /// Only allow instructions before a call, if their CodeSize cost is below
79 /// DuplicationThreshold. Those instructions need to be duplicated in all
80 /// split blocks.
81 static cl::opt<unsigned>
82     DuplicationThreshold("callsite-splitting-duplication-threshold", cl::Hidden,
83                          cl::desc("Only allow instructions before a call, if "
84                                   "their cost is below DuplicationThreshold"),
85                          cl::init(5));
86 
87 static void addNonNullAttribute(CallSite CS, Value *Op) {
88   unsigned ArgNo = 0;
89   for (auto &I : CS.args()) {
90     if (&*I == Op)
91       CS.addParamAttr(ArgNo, Attribute::NonNull);
92     ++ArgNo;
93   }
94 }
95 
96 static void setConstantInArgument(CallSite CS, Value *Op,
97                                   Constant *ConstValue) {
98   unsigned ArgNo = 0;
99   for (auto &I : CS.args()) {
100     if (&*I == Op)
101       CS.setArgument(ArgNo, ConstValue);
102     ++ArgNo;
103   }
104 }
105 
106 static bool isCondRelevantToAnyCallArgument(ICmpInst *Cmp, CallSite CS) {
107   assert(isa<Constant>(Cmp->getOperand(1)) && "Expected a constant operand.");
108   Value *Op0 = Cmp->getOperand(0);
109   unsigned ArgNo = 0;
110   for (CallSite::arg_iterator I = CS.arg_begin(), E = CS.arg_end(); I != E;
111        ++I, ++ArgNo) {
112     // Don't consider constant or arguments that are already known non-null.
113     if (isa<Constant>(*I) || CS.paramHasAttr(ArgNo, Attribute::NonNull))
114       continue;
115 
116     if (*I == Op0)
117       return true;
118   }
119   return false;
120 }
121 
122 typedef std::pair<ICmpInst *, unsigned> ConditionTy;
123 typedef SmallVector<ConditionTy, 2> ConditionsTy;
124 
125 /// If From has a conditional jump to To, add the condition to Conditions,
126 /// if it is relevant to any argument at CS.
127 static void recordCondition(CallSite CS, BasicBlock *From, BasicBlock *To,
128                             ConditionsTy &Conditions) {
129   auto *BI = dyn_cast<BranchInst>(From->getTerminator());
130   if (!BI || !BI->isConditional())
131     return;
132 
133   CmpInst::Predicate Pred;
134   Value *Cond = BI->getCondition();
135   if (!match(Cond, m_ICmp(Pred, m_Value(), m_Constant())))
136     return;
137 
138   ICmpInst *Cmp = cast<ICmpInst>(Cond);
139   if (Pred == ICmpInst::ICMP_EQ || Pred == ICmpInst::ICMP_NE)
140     if (isCondRelevantToAnyCallArgument(Cmp, CS))
141       Conditions.push_back({Cmp, From->getTerminator()->getSuccessor(0) == To
142                                      ? Pred
143                                      : Cmp->getInversePredicate()});
144 }
145 
146 /// Record ICmp conditions relevant to any argument in CS following Pred's
147 /// single successors. If there are conflicting conditions along a path, like
148 /// x == 1 and x == 0, the first condition will be used.
149 static void recordConditions(CallSite CS, BasicBlock *Pred,
150                              ConditionsTy &Conditions) {
151   recordCondition(CS, Pred, CS.getInstruction()->getParent(), Conditions);
152   BasicBlock *From = Pred;
153   BasicBlock *To = Pred;
154   SmallPtrSet<BasicBlock *, 4> Visited;
155   while (!Visited.count(From->getSinglePredecessor()) &&
156          (From = From->getSinglePredecessor())) {
157     recordCondition(CS, From, To, Conditions);
158     Visited.insert(From);
159     To = From;
160   }
161 }
162 
163 static void addConditions(CallSite CS, const ConditionsTy &Conditions) {
164   for (auto &Cond : Conditions) {
165     Value *Arg = Cond.first->getOperand(0);
166     Constant *ConstVal = cast<Constant>(Cond.first->getOperand(1));
167     if (Cond.second == ICmpInst::ICMP_EQ)
168       setConstantInArgument(CS, Arg, ConstVal);
169     else if (ConstVal->getType()->isPointerTy() && ConstVal->isNullValue()) {
170       assert(Cond.second == ICmpInst::ICMP_NE);
171       addNonNullAttribute(CS, Arg);
172     }
173   }
174 }
175 
176 static SmallVector<BasicBlock *, 2> getTwoPredecessors(BasicBlock *BB) {
177   SmallVector<BasicBlock *, 2> Preds(predecessors((BB)));
178   assert(Preds.size() == 2 && "Expected exactly 2 predecessors!");
179   return Preds;
180 }
181 
182 static bool canSplitCallSite(CallSite CS, TargetTransformInfo &TTI) {
183   // FIXME: As of now we handle only CallInst. InvokeInst could be handled
184   // without too much effort.
185   Instruction *Instr = CS.getInstruction();
186   if (!isa<CallInst>(Instr))
187     return false;
188 
189   BasicBlock *CallSiteBB = Instr->getParent();
190   // Allow splitting a call-site only when the CodeSize cost of the
191   // instructions before the call is less then DuplicationThreshold. The
192   // instructions before the call will be duplicated in the split blocks and
193   // corresponding uses will be updated.
194   unsigned Cost = 0;
195   for (auto &InstBeforeCall :
196        llvm::make_range(CallSiteBB->begin(), Instr->getIterator())) {
197     Cost += TTI.getInstructionCost(&InstBeforeCall,
198                                    TargetTransformInfo::TCK_CodeSize);
199     if (Cost >= DuplicationThreshold)
200       return false;
201   }
202 
203   // Need 2 predecessors and cannot split an edge from an IndirectBrInst.
204   SmallVector<BasicBlock *, 2> Preds(predecessors(CallSiteBB));
205   if (Preds.size() != 2 || isa<IndirectBrInst>(Preds[0]->getTerminator()) ||
206       isa<IndirectBrInst>(Preds[1]->getTerminator()))
207     return false;
208 
209   return CallSiteBB->canSplitPredecessors();
210 }
211 
212 static Instruction *cloneInstForMustTail(Instruction *I, Instruction *Before,
213                                          Value *V) {
214   Instruction *Copy = I->clone();
215   Copy->setName(I->getName());
216   Copy->insertBefore(Before);
217   if (V)
218     Copy->setOperand(0, V);
219   return Copy;
220 }
221 
222 /// Copy mandatory `musttail` return sequence that follows original `CI`, and
223 /// link it up to `NewCI` value instead:
224 ///
225 ///   * (optional) `bitcast NewCI to ...`
226 ///   * `ret bitcast or NewCI`
227 ///
228 /// Insert this sequence right before `SplitBB`'s terminator, which will be
229 /// cleaned up later in `splitCallSite` below.
230 static void copyMustTailReturn(BasicBlock *SplitBB, Instruction *CI,
231                                Instruction *NewCI) {
232   bool IsVoid = SplitBB->getParent()->getReturnType()->isVoidTy();
233   auto II = std::next(CI->getIterator());
234 
235   BitCastInst* BCI = dyn_cast<BitCastInst>(&*II);
236   if (BCI)
237     ++II;
238 
239   ReturnInst* RI = dyn_cast<ReturnInst>(&*II);
240   assert(RI && "`musttail` call must be followed by `ret` instruction");
241 
242   TerminatorInst *TI = SplitBB->getTerminator();
243   Value *V = NewCI;
244   if (BCI)
245     V = cloneInstForMustTail(BCI, TI, V);
246   cloneInstForMustTail(RI, TI, IsVoid ? nullptr : V);
247 
248   // FIXME: remove TI here, `DuplicateInstructionsInSplitBetween` has a bug
249   // that prevents doing this now.
250 }
251 
252 /// For each (predecessor, conditions from predecessors) pair, it will split the
253 /// basic block containing the call site, hook it up to the predecessor and
254 /// replace the call instruction with new call instructions, which contain
255 /// constraints based on the conditions from their predecessors.
256 /// For example, in the IR below with an OR condition, the call-site can
257 /// be split. In this case, Preds for Tail is [(Header, a == null),
258 /// (TBB, a != null, b == null)]. Tail is replaced by 2 split blocks, containing
259 /// CallInst1, which has constraints based on the conditions from Head and
260 /// CallInst2, which has constraints based on the conditions coming from TBB.
261 ///
262 /// From :
263 ///
264 ///   Header:
265 ///     %c = icmp eq i32* %a, null
266 ///     br i1 %c %Tail, %TBB
267 ///   TBB:
268 ///     %c2 = icmp eq i32* %b, null
269 ///     br i1 %c %Tail, %End
270 ///   Tail:
271 ///     %ca = call i1  @callee (i32* %a, i32* %b)
272 ///
273 ///  to :
274 ///
275 ///   Header:                          // PredBB1 is Header
276 ///     %c = icmp eq i32* %a, null
277 ///     br i1 %c %Tail-split1, %TBB
278 ///   TBB:                             // PredBB2 is TBB
279 ///     %c2 = icmp eq i32* %b, null
280 ///     br i1 %c %Tail-split2, %End
281 ///   Tail-split1:
282 ///     %ca1 = call @callee (i32* null, i32* %b)         // CallInst1
283 ///    br %Tail
284 ///   Tail-split2:
285 ///     %ca2 = call @callee (i32* nonnull %a, i32* null) // CallInst2
286 ///    br %Tail
287 ///   Tail:
288 ///    %p = phi i1 [%ca1, %Tail-split1],[%ca2, %Tail-split2]
289 ///
290 /// Note that in case any arguments at the call-site are constrained by its
291 /// predecessors, new call-sites with more constrained arguments will be
292 /// created in createCallSitesOnPredicatedArgument().
293 static void splitCallSite(
294     CallSite CS,
295     const SmallVectorImpl<std::pair<BasicBlock *, ConditionsTy>> &Preds,
296     DominatorTree *DT) {
297   Instruction *Instr = CS.getInstruction();
298   BasicBlock *TailBB = Instr->getParent();
299   bool IsMustTailCall = CS.isMustTailCall();
300 
301   PHINode *CallPN = nullptr;
302 
303   // `musttail` calls must be followed by optional `bitcast`, and `ret`. The
304   // split blocks will be terminated right after that so there're no users for
305   // this phi in a `TailBB`.
306   if (!IsMustTailCall && !Instr->use_empty())
307     CallPN = PHINode::Create(Instr->getType(), Preds.size(), "phi.call");
308 
309   DEBUG(dbgs() << "split call-site : " << *Instr << " into \n");
310 
311   assert(Preds.size() == 2 && "The ValueToValueMaps array has size 2.");
312   // ValueToValueMapTy is neither copy nor moveable, so we use a simple array
313   // here.
314   ValueToValueMapTy ValueToValueMaps[2];
315   for (unsigned i = 0; i < Preds.size(); i++) {
316     BasicBlock *PredBB = Preds[i].first;
317     BasicBlock *SplitBlock = DuplicateInstructionsInSplitBetween(
318         TailBB, PredBB, &*std::next(Instr->getIterator()), ValueToValueMaps[i],
319         DT);
320     assert(SplitBlock && "Unexpected new basic block split.");
321 
322     Instruction *NewCI =
323         &*std::prev(SplitBlock->getTerminator()->getIterator());
324     CallSite NewCS(NewCI);
325     addConditions(NewCS, Preds[i].second);
326 
327     // Handle PHIs used as arguments in the call-site.
328     for (PHINode &PN : TailBB->phis()) {
329       unsigned ArgNo = 0;
330       for (auto &CI : CS.args()) {
331         if (&*CI == &PN) {
332           NewCS.setArgument(ArgNo, PN.getIncomingValueForBlock(SplitBlock));
333         }
334         ++ArgNo;
335       }
336     }
337     DEBUG(dbgs() << "    " << *NewCI << " in " << SplitBlock->getName()
338                  << "\n");
339     if (CallPN)
340       CallPN->addIncoming(NewCI, SplitBlock);
341 
342     // Clone and place bitcast and return instructions before `TI`
343     if (IsMustTailCall)
344       copyMustTailReturn(SplitBlock, Instr, NewCI);
345   }
346 
347   NumCallSiteSplit++;
348 
349   // FIXME: remove TI in `copyMustTailReturn`
350   if (IsMustTailCall) {
351     // Remove superfluous `br` terminators from the end of the Split blocks
352     // NOTE: Removing terminator removes the SplitBlock from the TailBB's
353     // predecessors. Therefore we must get complete list of Splits before
354     // attempting removal.
355     SmallVector<BasicBlock *, 2> Splits(predecessors((TailBB)));
356     assert(Splits.size() == 2 && "Expected exactly 2 splits!");
357     for (unsigned i = 0; i < Splits.size(); i++)
358       Splits[i]->getTerminator()->eraseFromParent();
359 
360     // Erase the tail block once done with musttail patching
361     TailBB->eraseFromParent();
362     return;
363   }
364 
365   auto *OriginalBegin = &*TailBB->begin();
366   // Replace users of the original call with a PHI mering call-sites split.
367   if (CallPN) {
368     CallPN->insertBefore(OriginalBegin);
369     Instr->replaceAllUsesWith(CallPN);
370   }
371 
372   // Remove instructions moved to split blocks from TailBB, from the duplicated
373   // call instruction to the beginning of the basic block. If an instruction
374   // has any uses, add a new PHI node to combine the values coming from the
375   // split blocks. The new PHI nodes are placed before the first original
376   // instruction, so we do not end up deleting them. By using reverse-order, we
377   // do not introduce unnecessary PHI nodes for def-use chains from the call
378   // instruction to the beginning of the block.
379   auto I = Instr->getReverseIterator();
380   while (I != TailBB->rend()) {
381     Instruction *CurrentI = &*I++;
382     if (!CurrentI->use_empty()) {
383       // If an existing PHI has users after the call, there is no need to create
384       // a new one.
385       if (isa<PHINode>(CurrentI))
386         continue;
387       PHINode *NewPN = PHINode::Create(CurrentI->getType(), Preds.size());
388       for (auto &Mapping : ValueToValueMaps)
389         NewPN->addIncoming(Mapping[CurrentI],
390                            cast<Instruction>(Mapping[CurrentI])->getParent());
391       NewPN->insertBefore(&*TailBB->begin());
392       CurrentI->replaceAllUsesWith(NewPN);
393     }
394     CurrentI->eraseFromParent();
395     // We are done once we handled the first original instruction in TailBB.
396     if (CurrentI == OriginalBegin)
397       break;
398   }
399 }
400 
401 // Return true if the call-site has an argument which is a PHI with only
402 // constant incoming values.
403 static bool isPredicatedOnPHI(CallSite CS) {
404   Instruction *Instr = CS.getInstruction();
405   BasicBlock *Parent = Instr->getParent();
406   if (Instr != Parent->getFirstNonPHIOrDbg())
407     return false;
408 
409   for (auto &BI : *Parent) {
410     if (PHINode *PN = dyn_cast<PHINode>(&BI)) {
411       for (auto &I : CS.args())
412         if (&*I == PN) {
413           assert(PN->getNumIncomingValues() == 2 &&
414                  "Unexpected number of incoming values");
415           if (PN->getIncomingBlock(0) == PN->getIncomingBlock(1))
416             return false;
417           if (PN->getIncomingValue(0) == PN->getIncomingValue(1))
418             continue;
419           if (isa<Constant>(PN->getIncomingValue(0)) &&
420               isa<Constant>(PN->getIncomingValue(1)))
421             return true;
422         }
423     }
424     break;
425   }
426   return false;
427 }
428 
429 static bool tryToSplitOnPHIPredicatedArgument(CallSite CS, DominatorTree *DT) {
430   if (!isPredicatedOnPHI(CS))
431     return false;
432 
433   auto Preds = getTwoPredecessors(CS.getInstruction()->getParent());
434   SmallVector<std::pair<BasicBlock *, ConditionsTy>, 2> PredsCS = {
435       {Preds[0], {}}, {Preds[1], {}}};
436   splitCallSite(CS, PredsCS, DT);
437   return true;
438 }
439 
440 static bool tryToSplitOnPredicatedArgument(CallSite CS, DominatorTree *DT) {
441   auto Preds = getTwoPredecessors(CS.getInstruction()->getParent());
442   if (Preds[0] == Preds[1])
443     return false;
444 
445   SmallVector<std::pair<BasicBlock *, ConditionsTy>, 2> PredsCS;
446   for (auto *Pred : make_range(Preds.rbegin(), Preds.rend())) {
447     ConditionsTy Conditions;
448     recordConditions(CS, Pred, Conditions);
449     PredsCS.push_back({Pred, Conditions});
450   }
451 
452   if (std::all_of(PredsCS.begin(), PredsCS.end(),
453                   [](const std::pair<BasicBlock *, ConditionsTy> &P) {
454                     return P.second.empty();
455                   }))
456     return false;
457 
458   splitCallSite(CS, PredsCS, DT);
459   return true;
460 }
461 
462 static bool tryToSplitCallSite(CallSite CS, TargetTransformInfo &TTI,
463                                DominatorTree *DT) {
464   if (!CS.arg_size() || !canSplitCallSite(CS, TTI))
465     return false;
466   return tryToSplitOnPredicatedArgument(CS, DT) ||
467          tryToSplitOnPHIPredicatedArgument(CS, DT);
468 }
469 
470 static bool doCallSiteSplitting(Function &F, TargetLibraryInfo &TLI,
471                                 TargetTransformInfo &TTI, DominatorTree *DT) {
472   bool Changed = false;
473   for (Function::iterator BI = F.begin(), BE = F.end(); BI != BE;) {
474     BasicBlock &BB = *BI++;
475     auto II = BB.getFirstNonPHIOrDbg()->getIterator();
476     auto IE = BB.getTerminator()->getIterator();
477     // Iterate until we reach the terminator instruction. tryToSplitCallSite
478     // can replace BB's terminator in case BB is a successor of itself. In that
479     // case, IE will be invalidated and we also have to check the current
480     // terminator.
481     while (II != IE && &*II != BB.getTerminator()) {
482       Instruction *I = &*II++;
483       CallSite CS(cast<Value>(I));
484       if (!CS || isa<IntrinsicInst>(I) || isInstructionTriviallyDead(I, &TLI))
485         continue;
486 
487       Function *Callee = CS.getCalledFunction();
488       if (!Callee || Callee->isDeclaration())
489         continue;
490 
491       // Successful musttail call-site splits result in erased CI and erased BB.
492       // Check if such path is possible before attempting the splitting.
493       bool IsMustTail = CS.isMustTailCall();
494 
495       Changed |= tryToSplitCallSite(CS, TTI, DT);
496 
497       // There're no interesting instructions after this. The call site
498       // itself might have been erased on splitting.
499       if (IsMustTail)
500         break;
501     }
502   }
503   return Changed;
504 }
505 
506 namespace {
507 struct CallSiteSplittingLegacyPass : public FunctionPass {
508   static char ID;
509   CallSiteSplittingLegacyPass() : FunctionPass(ID) {
510     initializeCallSiteSplittingLegacyPassPass(*PassRegistry::getPassRegistry());
511   }
512 
513   void getAnalysisUsage(AnalysisUsage &AU) const override {
514     AU.addRequired<TargetLibraryInfoWrapperPass>();
515     AU.addRequired<TargetTransformInfoWrapperPass>();
516     AU.addPreserved<DominatorTreeWrapperPass>();
517     FunctionPass::getAnalysisUsage(AU);
518   }
519 
520   bool runOnFunction(Function &F) override {
521     if (skipFunction(F))
522       return false;
523 
524     auto &TLI = getAnalysis<TargetLibraryInfoWrapperPass>().getTLI();
525     auto &TTI = getAnalysis<TargetTransformInfoWrapperPass>().getTTI(F);
526     auto *DTWP = getAnalysisIfAvailable<DominatorTreeWrapperPass>();
527     return doCallSiteSplitting(F, TLI, TTI,
528                                DTWP ? &DTWP->getDomTree() : nullptr);
529   }
530 };
531 } // namespace
532 
533 char CallSiteSplittingLegacyPass::ID = 0;
534 INITIALIZE_PASS_BEGIN(CallSiteSplittingLegacyPass, "callsite-splitting",
535                       "Call-site splitting", false, false)
536 INITIALIZE_PASS_DEPENDENCY(TargetLibraryInfoWrapperPass)
537 INITIALIZE_PASS_DEPENDENCY(TargetTransformInfoWrapperPass)
538 INITIALIZE_PASS_END(CallSiteSplittingLegacyPass, "callsite-splitting",
539                     "Call-site splitting", false, false)
540 FunctionPass *llvm::createCallSiteSplittingPass() {
541   return new CallSiteSplittingLegacyPass();
542 }
543 
544 PreservedAnalyses CallSiteSplittingPass::run(Function &F,
545                                              FunctionAnalysisManager &AM) {
546   auto &TLI = AM.getResult<TargetLibraryAnalysis>(F);
547   auto &TTI = AM.getResult<TargetIRAnalysis>(F);
548   auto *DT = AM.getCachedResult<DominatorTreeAnalysis>(F);
549 
550   if (!doCallSiteSplitting(F, TLI, TTI, DT))
551     return PreservedAnalyses::all();
552   PreservedAnalyses PA;
553   PA.preserve<DominatorTreeAnalysis>();
554   return PA;
555 }
556