blob: 517a9c082a4cf6f4fe46c79a7be73ca77d4a0a75 [file] [log] [blame]
Eugene Zelenkof27d1612017-10-19 21:21:30 +00001//===- ArgumentPromotion.cpp - Promote by-reference arguments -------------===//
Misha Brukmanb1c93172005-04-21 23:48:37 +00002//
Chris Lattner483ae012004-03-07 21:29:54 +00003// The LLVM Compiler Infrastructure
4//
Chris Lattnerf3ebc3f2007-12-29 20:36:04 +00005// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
Misha Brukmanb1c93172005-04-21 23:48:37 +00007//
Chris Lattner483ae012004-03-07 21:29:54 +00008//===----------------------------------------------------------------------===//
9//
10// This pass promotes "by reference" arguments to be "by value" arguments. In
11// practice, this means looking for internal functions that have pointer
Gordon Henriksen78c63ac2007-10-26 03:03:51 +000012// arguments. If it can prove, through the use of alias analysis, that an
13// argument is *only* loaded, then it can pass the value into the function
Chris Lattner483ae012004-03-07 21:29:54 +000014// instead of the address of the value. This can cause recursive simplification
Chris Lattner254f8f82004-05-23 21:21:17 +000015// of code and lead to the elimination of allocas (especially in C++ template
16// code like the STL).
Chris Lattner483ae012004-03-07 21:29:54 +000017//
Chris Lattnerfe6f2e32004-03-08 01:04:36 +000018// This pass also handles aggregate arguments that are passed into a function,
19// scalarizing them if the elements of the aggregate are only loaded. Note that
Matthijs Kooijmanfd307042008-07-29 10:00:13 +000020// by default it refuses to scalarize aggregates which would require passing in
21// more than three operands to the function, because passing thousands of
Duncan Sands1ea0d2e2008-09-07 09:54:09 +000022// operands for a large array or structure is unprofitable! This limit can be
Matthijs Kooijmanfd307042008-07-29 10:00:13 +000023// configured or disabled, however.
Chris Lattnerfe6f2e32004-03-08 01:04:36 +000024//
Chris Lattner483ae012004-03-07 21:29:54 +000025// Note that this transformation could also be done for arguments that are only
Gordon Henriksen78c63ac2007-10-26 03:03:51 +000026// stored to (returning the value instead), but does not currently. This case
27// would be best handled when and if LLVM begins supporting multiple return
28// values from functions.
Chris Lattner483ae012004-03-07 21:29:54 +000029//
30//===----------------------------------------------------------------------===//
31
Chandler Carruthaddcda42017-02-09 23:46:27 +000032#include "llvm/Transforms/IPO/ArgumentPromotion.h"
Reid Spencer7c16caa2004-09-01 22:55:40 +000033#include "llvm/ADT/DepthFirstIterator.h"
Eugene Zelenkof27d1612017-10-19 21:21:30 +000034#include "llvm/ADT/None.h"
Chandler Carruthaddcda42017-02-09 23:46:27 +000035#include "llvm/ADT/Optional.h"
Eugene Zelenkof27d1612017-10-19 21:21:30 +000036#include "llvm/ADT/STLExtras.h"
37#include "llvm/ADT/SmallPtrSet.h"
38#include "llvm/ADT/SmallVector.h"
Reid Spencer7c16caa2004-09-01 22:55:40 +000039#include "llvm/ADT/Statistic.h"
Chris Lattner06fa1762009-08-24 03:52:50 +000040#include "llvm/ADT/StringExtras.h"
Eugene Zelenkof27d1612017-10-19 21:21:30 +000041#include "llvm/ADT/Twine.h"
Chandler Carruthed0881b2012-12-03 16:50:05 +000042#include "llvm/Analysis/AliasAnalysis.h"
Daniel Jasperaec2fa32016-12-19 08:22:17 +000043#include "llvm/Analysis/AssumptionCache.h"
Chandler Carruth7b560d42015-09-09 17:55:00 +000044#include "llvm/Analysis/BasicAliasAnalysis.h"
Eugene Zelenkof27d1612017-10-19 21:21:30 +000045#include "llvm/Analysis/CGSCCPassManager.h"
Chandler Carruthed0881b2012-12-03 16:50:05 +000046#include "llvm/Analysis/CallGraph.h"
Chandler Carruth839a98e2013-01-07 15:26:48 +000047#include "llvm/Analysis/CallGraphSCCPass.h"
Chandler Carruthaddcda42017-02-09 23:46:27 +000048#include "llvm/Analysis/LazyCallGraph.h"
Artur Pilipenko31bcca42016-02-24 12:49:04 +000049#include "llvm/Analysis/Loads.h"
Eugene Zelenkof27d1612017-10-19 21:21:30 +000050#include "llvm/Analysis/MemoryLocation.h"
Chandler Carruth7b560d42015-09-09 17:55:00 +000051#include "llvm/Analysis/TargetLibraryInfo.h"
Eugene Zelenkof27d1612017-10-19 21:21:30 +000052#include "llvm/IR/Argument.h"
53#include "llvm/IR/Attributes.h"
54#include "llvm/IR/BasicBlock.h"
Chandler Carruth1305dc32014-03-04 11:45:46 +000055#include "llvm/IR/CFG.h"
Chandler Carruth219b89b2014-03-04 11:01:28 +000056#include "llvm/IR/CallSite.h"
Chandler Carruth9fb823b2013-01-02 11:36:10 +000057#include "llvm/IR/Constants.h"
Hal Finkel2e42c342014-07-10 05:27:53 +000058#include "llvm/IR/DataLayout.h"
Chandler Carruth9fb823b2013-01-02 11:36:10 +000059#include "llvm/IR/DerivedTypes.h"
Eugene Zelenkof27d1612017-10-19 21:21:30 +000060#include "llvm/IR/Function.h"
61#include "llvm/IR/InstrTypes.h"
62#include "llvm/IR/Instruction.h"
Chandler Carruth9fb823b2013-01-02 11:36:10 +000063#include "llvm/IR/Instructions.h"
Eugene Zelenkof27d1612017-10-19 21:21:30 +000064#include "llvm/IR/Metadata.h"
Chandler Carruth9fb823b2013-01-02 11:36:10 +000065#include "llvm/IR/Module.h"
Eugene Zelenkof27d1612017-10-19 21:21:30 +000066#include "llvm/IR/PassManager.h"
67#include "llvm/IR/Type.h"
68#include "llvm/IR/Use.h"
69#include "llvm/IR/User.h"
70#include "llvm/IR/Value.h"
71#include "llvm/Pass.h"
72#include "llvm/Support/Casting.h"
Chandler Carruthed0881b2012-12-03 16:50:05 +000073#include "llvm/Support/Debug.h"
74#include "llvm/Support/raw_ostream.h"
Chandler Carruthae9ce3d2017-01-29 08:03:19 +000075#include "llvm/Transforms/IPO.h"
Eugene Zelenkof27d1612017-10-19 21:21:30 +000076#include <algorithm>
77#include <cassert>
78#include <cstdint>
79#include <functional>
80#include <iterator>
81#include <map>
Chris Lattner483ae012004-03-07 21:29:54 +000082#include <set>
Eugene Zelenkof27d1612017-10-19 21:21:30 +000083#include <string>
84#include <utility>
85#include <vector>
86
Chris Lattner483ae012004-03-07 21:29:54 +000087using namespace llvm;
88
Chandler Carruth964daaa2014-04-22 02:55:47 +000089#define DEBUG_TYPE "argpromotion"
90
Chandler Carruthae9ce3d2017-01-29 08:03:19 +000091STATISTIC(NumArgumentsPromoted, "Number of pointer arguments promoted");
Chris Lattner1631bcb2006-12-19 22:09:18 +000092STATISTIC(NumAggregatesPromoted, "Number of aggregate arguments promoted");
Chandler Carruthae9ce3d2017-01-29 08:03:19 +000093STATISTIC(NumByValArgsPromoted, "Number of byval arguments promoted");
94STATISTIC(NumArgumentsDead, "Number of dead pointer args eliminated");
Chris Lattner483ae012004-03-07 21:29:54 +000095
Sean Silvae2133e72016-07-02 18:59:51 +000096/// A vector used to hold the indices of a single GEP instruction
Eugene Zelenkof27d1612017-10-19 21:21:30 +000097using IndicesVector = std::vector<uint64_t>;
Sean Silvae2133e72016-07-02 18:59:51 +000098
Chris Lattner254f8f82004-05-23 21:21:17 +000099/// DoPromotion - This method actually performs the promotion of the specified
Chris Lattner37b6c4f2004-09-18 00:34:13 +0000100/// arguments, and returns the new function. At this point, we know that it's
101/// safe to do so.
Chandler Carruthaddcda42017-02-09 23:46:27 +0000102static Function *
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000103doPromotion(Function *F, SmallPtrSetImpl<Argument *> &ArgsToPromote,
Chandler Carruthaddcda42017-02-09 23:46:27 +0000104 SmallPtrSetImpl<Argument *> &ByValArgsToTransform,
105 Optional<function_ref<void(CallSite OldCS, CallSite NewCS)>>
106 ReplaceCallSite) {
Chris Lattner483ae012004-03-07 21:29:54 +0000107 // Start by computing a new prototype for the function, which is the same as
108 // the old function, but has modified arguments.
Chris Lattner229907c2011-07-18 04:54:35 +0000109 FunctionType *FTy = F->getFunctionType();
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000110 std::vector<Type *> Params;
Chris Lattner483ae012004-03-07 21:29:54 +0000111
Eugene Zelenkof27d1612017-10-19 21:21:30 +0000112 using ScalarizeTable = std::set<std::pair<Type *, IndicesVector>>;
Chris Lattner1c676f72004-06-21 00:07:58 +0000113
Chris Lattnerfe6f2e32004-03-08 01:04:36 +0000114 // ScalarizedElements - If we are promoting a pointer that has elements
115 // accessed out of it, keep track of which elements are accessed so that we
116 // can add one argument for each.
117 //
118 // Arguments that are directly loaded will have a zero element value here, to
119 // handle cases where there are both a direct load and GEP accesses.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000120 std::map<Argument *, ScalarizeTable> ScalarizedElements;
Chris Lattnerfe6f2e32004-03-08 01:04:36 +0000121
Chris Lattner254f8f82004-05-23 21:21:17 +0000122 // OriginalLoads - Keep track of a representative load instruction from the
123 // original function so that we can tell the alias analysis implementation
124 // what the new GEP/Load instructions we are inserting look like.
Manman Renbc376582013-11-15 20:41:15 +0000125 // We need to keep the original loads for each argument and the elements
126 // of the argument that are accessed.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000127 std::map<std::pair<Argument *, IndicesVector>, LoadInst *> OriginalLoads;
Chris Lattner254f8f82004-05-23 21:21:17 +0000128
Bill Wendling3d7b0b82012-12-19 07:18:57 +0000129 // Attribute - Keep track of the parameter attributes for the arguments
Duncan Sandsad0ea2d2007-11-27 13:23:08 +0000130 // that we are *not* promoting. For the ones that we do promote, the parameter
131 // attributes are lost
Reid Kleckner7f720332017-04-13 00:58:09 +0000132 SmallVector<AttributeSet, 8> ArgAttrVec;
133 AttributeList PAL = F->getAttributes();
Duncan Sands1ea0d2e2008-09-07 09:54:09 +0000134
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000135 // First, determine the new argument list
Reid Kleckner6652a522017-04-28 18:37:16 +0000136 unsigned ArgNo = 0;
Duncan Sandsad0ea2d2007-11-27 13:23:08 +0000137 for (Function::arg_iterator I = F->arg_begin(), E = F->arg_end(); I != E;
Reid Kleckner6652a522017-04-28 18:37:16 +0000138 ++I, ++ArgNo) {
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000139 if (ByValArgsToTransform.count(&*I)) {
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000140 // Simple byval argument? Just add all the struct element types.
Chris Lattner229907c2011-07-18 04:54:35 +0000141 Type *AgTy = cast<PointerType>(I->getType())->getElementType();
142 StructType *STy = cast<StructType>(AgTy);
Benjamin Kramer5fbfe2f2015-02-28 13:20:15 +0000143 Params.insert(Params.end(), STy->element_begin(), STy->element_end());
Reid Kleckner7f720332017-04-13 00:58:09 +0000144 ArgAttrVec.insert(ArgAttrVec.end(), STy->getNumElements(),
145 AttributeSet());
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000146 ++NumByValArgsPromoted;
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000147 } else if (!ArgsToPromote.count(&*I)) {
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000148 // Unchanged argument
Chris Lattner483ae012004-03-07 21:29:54 +0000149 Params.push_back(I->getType());
Reid Kleckner6652a522017-04-28 18:37:16 +0000150 ArgAttrVec.push_back(PAL.getParamAttributes(ArgNo));
Chris Lattner254f8f82004-05-23 21:21:17 +0000151 } else if (I->use_empty()) {
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000152 // Dead argument (which are always marked as promotable)
Chris Lattner254f8f82004-05-23 21:21:17 +0000153 ++NumArgumentsDead;
Mikael Holmene0ced142017-07-10 06:07:24 +0000154
155 // There may be remaining metadata uses of the argument for things like
156 // llvm.dbg.value. Replace them with undef.
157 I->replaceAllUsesWith(UndefValue::get(I->getType()));
Chris Lattner254f8f82004-05-23 21:21:17 +0000158 } else {
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000159 // Okay, this is being promoted. This means that the only uses are loads
160 // or GEPs which are only used by loads
161
162 // In this table, we will track which indices are loaded from the argument
163 // (where direct loads are tracked as no indices).
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000164 ScalarizeTable &ArgIndices = ScalarizedElements[&*I];
Chandler Carruthcdf47882014-03-09 03:16:01 +0000165 for (User *U : I->users()) {
166 Instruction *UI = cast<Instruction>(U);
David Blaikie76826632015-03-14 21:11:26 +0000167 Type *SrcTy;
168 if (LoadInst *L = dyn_cast<LoadInst>(UI))
169 SrcTy = L->getType();
170 else
171 SrcTy = cast<GetElementPtrInst>(UI)->getSourceElementType();
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000172 IndicesVector Indices;
Chandler Carruthcdf47882014-03-09 03:16:01 +0000173 Indices.reserve(UI->getNumOperands() - 1);
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000174 // Since loads will only have a single operand, and GEPs only a single
175 // non-index operand, this will record direct loads without any indices,
176 // and gep+loads with the GEP indices.
Chandler Carruthcdf47882014-03-09 03:16:01 +0000177 for (User::op_iterator II = UI->op_begin() + 1, IE = UI->op_end();
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000178 II != IE; ++II)
179 Indices.push_back(cast<ConstantInt>(*II)->getSExtValue());
180 // GEPs with a single 0 index can be merged with direct loads
181 if (Indices.size() == 1 && Indices.front() == 0)
182 Indices.clear();
David Blaikie76826632015-03-14 21:11:26 +0000183 ArgIndices.insert(std::make_pair(SrcTy, Indices));
Owen Andersonedadd3f2006-09-15 05:22:51 +0000184 LoadInst *OrigLoad;
Chandler Carruthcdf47882014-03-09 03:16:01 +0000185 if (LoadInst *L = dyn_cast<LoadInst>(UI))
Owen Andersonedadd3f2006-09-15 05:22:51 +0000186 OrigLoad = L;
187 else
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000188 // Take any load, we will use it only to update Alias Analysis
Chandler Carruthcdf47882014-03-09 03:16:01 +0000189 OrigLoad = cast<LoadInst>(UI->user_back());
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000190 OriginalLoads[std::make_pair(&*I, Indices)] = OrigLoad;
Chris Lattnerfe6f2e32004-03-08 01:04:36 +0000191 }
192
193 // Add a parameter to the function for each element passed in.
Benjamin Kramer135f7352016-06-26 12:28:59 +0000194 for (const auto &ArgIndex : ArgIndices) {
Torok Edwin026259f2008-11-16 17:21:25 +0000195 // not allowed to dereference ->begin() if size() is 0
David Blaikied288fb82015-03-30 21:41:43 +0000196 Params.push_back(GetElementPtrInst::getIndexedType(
197 cast<PointerType>(I->getType()->getScalarType())->getElementType(),
Benjamin Kramer135f7352016-06-26 12:28:59 +0000198 ArgIndex.second));
Reid Kleckner7f720332017-04-13 00:58:09 +0000199 ArgAttrVec.push_back(AttributeSet());
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000200 assert(Params.back());
201 }
Chris Lattnerfe6f2e32004-03-08 01:04:36 +0000202
David Blaikie76826632015-03-14 21:11:26 +0000203 if (ArgIndices.size() == 1 && ArgIndices.begin()->second.empty())
Chris Lattnerfe6f2e32004-03-08 01:04:36 +0000204 ++NumArgumentsPromoted;
205 else
206 ++NumAggregatesPromoted;
Chris Lattner483ae012004-03-07 21:29:54 +0000207 }
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000208 }
Chris Lattner483ae012004-03-07 21:29:54 +0000209
Chris Lattner229907c2011-07-18 04:54:35 +0000210 Type *RetTy = FTy->getReturnType();
Chris Lattner483ae012004-03-07 21:29:54 +0000211
Duncan Sandsad0ea2d2007-11-27 13:23:08 +0000212 // Construct the new function type using the new arguments.
Owen Anderson4056ca92009-07-29 22:17:13 +0000213 FunctionType *NFTy = FunctionType::get(RetTy, Params, FTy->isVarArg());
Misha Brukmanb1c93172005-04-21 23:48:37 +0000214
Chris Lattnere0987212009-09-15 05:40:35 +0000215 // Create the new function body and insert it into the module.
Dylan McKayf920da02018-12-18 09:52:52 +0000216 Function *NF = Function::Create(NFTy, F->getLinkage(), F->getAddressSpace(),
217 F->getName());
Duncan Sandsdd7daee2008-05-26 19:58:59 +0000218 NF->copyAttributesFrom(F);
Chris Lattner5630c4f2008-01-17 01:17:03 +0000219
David Blaikiee844cd52014-07-01 21:13:37 +0000220 // Patch the pointer to LLVM function in debug info descriptor.
Peter Collingbourned4bff302015-11-05 22:03:56 +0000221 NF->setSubprogram(F->getSubprogram());
222 F->setSubprogram(nullptr);
David Blaikie8e9cfa52014-07-23 22:09:29 +0000223
Nicola Zaghend34e60c2018-05-14 12:53:11 +0000224 LLVM_DEBUG(dbgs() << "ARG PROMOTION: Promoting to:" << *NF << "\n"
225 << "From: " << *F);
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000226
Chris Lattner5630c4f2008-01-17 01:17:03 +0000227 // Recompute the parameter attributes list based on the new arguments for
228 // the function.
Reid Kleckner7f720332017-04-13 00:58:09 +0000229 NF->setAttributes(AttributeList::get(F->getContext(), PAL.getFnAttributes(),
230 PAL.getRetAttributes(), ArgAttrVec));
231 ArgAttrVec.clear();
Duncan Sandsdd7daee2008-05-26 19:58:59 +0000232
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000233 F->getParent()->getFunctionList().insert(F->getIterator(), NF);
Zhou Shenga30cdb92008-03-20 08:05:05 +0000234 NF->takeName(F);
Chris Lattner254f8f82004-05-23 21:21:17 +0000235
Chris Lattner483ae012004-03-07 21:29:54 +0000236 // Loop over all of the callers of the function, transforming the call sites
237 // to pass in the loaded pointers.
238 //
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000239 SmallVector<Value *, 16> Args;
Chris Lattner483ae012004-03-07 21:29:54 +0000240 while (!F->use_empty()) {
Chandler Carruthcdf47882014-03-09 03:16:01 +0000241 CallSite CS(F->user_back());
Gabor Greif161cb042010-03-23 14:40:20 +0000242 assert(CS.getCalledFunction() == F);
Chris Lattner483ae012004-03-07 21:29:54 +0000243 Instruction *Call = CS.getInstruction();
Reid Klecknerb5180542017-03-21 16:57:19 +0000244 const AttributeList &CallPAL = CS.getAttributes();
Duncan Sands1ea0d2e2008-09-07 09:54:09 +0000245
Chris Lattner254f8f82004-05-23 21:21:17 +0000246 // Loop over the operands, inserting GEP and loads in the caller as
247 // appropriate.
Chris Lattner483ae012004-03-07 21:29:54 +0000248 CallSite::arg_iterator AI = CS.arg_begin();
Reid Kleckner6652a522017-04-28 18:37:16 +0000249 ArgNo = 0;
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000250 for (Function::arg_iterator I = F->arg_begin(), E = F->arg_end(); I != E;
Reid Kleckner6652a522017-04-28 18:37:16 +0000251 ++I, ++AI, ++ArgNo)
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000252 if (!ArgsToPromote.count(&*I) && !ByValArgsToTransform.count(&*I)) {
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000253 Args.push_back(*AI); // Unmodified argument
Reid Kleckner6652a522017-04-28 18:37:16 +0000254 ArgAttrVec.push_back(CallPAL.getParamAttributes(ArgNo));
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000255 } else if (ByValArgsToTransform.count(&*I)) {
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000256 // Emit a GEP and load for each element of the struct.
Chris Lattner229907c2011-07-18 04:54:35 +0000257 Type *AgTy = cast<PointerType>(I->getType())->getElementType();
258 StructType *STy = cast<StructType>(AgTy);
Owen Anderson55f1c092009-08-13 21:58:54 +0000259 Value *Idxs[2] = {
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000260 ConstantInt::get(Type::getInt32Ty(F->getContext()), 0), nullptr};
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000261 for (unsigned i = 0, e = STy->getNumElements(); i != e; ++i) {
Owen Anderson55f1c092009-08-13 21:58:54 +0000262 Idxs[1] = ConstantInt::get(Type::getInt32Ty(F->getContext()), i);
David Blaikie76826632015-03-14 21:11:26 +0000263 Value *Idx = GetElementPtrInst::Create(
Benjamin Kramerdba7ee92015-05-28 11:24:24 +0000264 STy, *AI, Idxs, (*AI)->getName() + "." + Twine(i), Call);
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000265 // TODO: Tell AA about the new values?
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000266 Args.push_back(new LoadInst(Idx, Idx->getName() + ".val", Call));
Reid Kleckner7f720332017-04-13 00:58:09 +0000267 ArgAttrVec.push_back(AttributeSet());
Duncan Sands1ea0d2e2008-09-07 09:54:09 +0000268 }
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000269 } else if (!I->use_empty()) {
Chris Lattner254f8f82004-05-23 21:21:17 +0000270 // Non-dead argument: insert GEPs and loads as appropriate.
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000271 ScalarizeTable &ArgIndices = ScalarizedElements[&*I];
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000272 // Store the Value* version of the indices in here, but declare it now
Gabor Greif161cb042010-03-23 14:40:20 +0000273 // for reuse.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000274 std::vector<Value *> Ops;
Benjamin Kramer135f7352016-06-26 12:28:59 +0000275 for (const auto &ArgIndex : ArgIndices) {
Chris Lattnerfe6f2e32004-03-08 01:04:36 +0000276 Value *V = *AI;
Benjamin Kramer135f7352016-06-26 12:28:59 +0000277 LoadInst *OrigLoad =
278 OriginalLoads[std::make_pair(&*I, ArgIndex.second)];
279 if (!ArgIndex.second.empty()) {
280 Ops.reserve(ArgIndex.second.size());
Chris Lattner229907c2011-07-18 04:54:35 +0000281 Type *ElTy = V->getType();
Martin Storsjoe81233d2017-05-04 10:54:35 +0000282 for (auto II : ArgIndex.second) {
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000283 // Use i32 to index structs, and i64 for others (pointers/arrays).
284 // This satisfies GEP constraints.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000285 Type *IdxTy =
286 (ElTy->isStructTy() ? Type::getInt32Ty(F->getContext())
287 : Type::getInt64Ty(F->getContext()));
Benjamin Kramer135f7352016-06-26 12:28:59 +0000288 Ops.push_back(ConstantInt::get(IdxTy, II));
Gabor Greif161cb042010-03-23 14:40:20 +0000289 // Keep track of the type we're currently indexing.
Peter Collingbourne45681582016-12-02 03:05:41 +0000290 if (auto *ElPTy = dyn_cast<PointerType>(ElTy))
291 ElTy = ElPTy->getElementType();
292 else
293 ElTy = cast<CompositeType>(ElTy)->getTypeAtIndex(II);
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000294 }
Gabor Greif161cb042010-03-23 14:40:20 +0000295 // And create a GEP to extract those indices.
Benjamin Kramer135f7352016-06-26 12:28:59 +0000296 V = GetElementPtrInst::Create(ArgIndex.first, V, Ops,
David Blaikie72edd882015-03-14 21:40:12 +0000297 V->getName() + ".idx", Call);
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000298 Ops.clear();
Chris Lattner254f8f82004-05-23 21:21:17 +0000299 }
Eric Christopher81c03442010-03-27 01:54:00 +0000300 // Since we're replacing a load make sure we take the alignment
301 // of the previous load.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000302 LoadInst *newLoad = new LoadInst(V, V->getName() + ".val", Call);
Eric Christopher81c03442010-03-27 01:54:00 +0000303 newLoad->setAlignment(OrigLoad->getAlignment());
Hal Finkelcc39b672014-07-24 12:16:19 +0000304 // Transfer the AA info too.
305 AAMDNodes AAInfo;
306 OrigLoad->getAAMetadata(AAInfo);
307 newLoad->setAAMetadata(AAInfo);
308
Eric Christopher81c03442010-03-27 01:54:00 +0000309 Args.push_back(newLoad);
Reid Kleckner7f720332017-04-13 00:58:09 +0000310 ArgAttrVec.push_back(AttributeSet());
Chris Lattnerfe6f2e32004-03-08 01:04:36 +0000311 }
Chris Lattner483ae012004-03-07 21:29:54 +0000312 }
313
Gabor Greif161cb042010-03-23 14:40:20 +0000314 // Push any varargs arguments on the list.
Reid Kleckner6652a522017-04-28 18:37:16 +0000315 for (; AI != CS.arg_end(); ++AI, ++ArgNo) {
Chris Lattner483ae012004-03-07 21:29:54 +0000316 Args.push_back(*AI);
Reid Kleckner6652a522017-04-28 18:37:16 +0000317 ArgAttrVec.push_back(CallPAL.getParamAttributes(ArgNo));
Chris Lattner5630c4f2008-01-17 01:17:03 +0000318 }
Chris Lattner483ae012004-03-07 21:29:54 +0000319
David Majnemercd24bb12016-04-29 04:56:12 +0000320 SmallVector<OperandBundleDef, 1> OpBundles;
321 CS.getOperandBundlesAsDefs(OpBundles);
322
Reid Kleckner7f720332017-04-13 00:58:09 +0000323 CallSite NewCS;
Chris Lattner483ae012004-03-07 21:29:54 +0000324 if (InvokeInst *II = dyn_cast<InvokeInst>(Call)) {
Reid Kleckner7f720332017-04-13 00:58:09 +0000325 NewCS = InvokeInst::Create(NF, II->getNormalDest(), II->getUnwindDest(),
Reid Kleckner3a115032017-04-13 18:10:30 +0000326 Args, OpBundles, "", Call);
Chris Lattner483ae012004-03-07 21:29:54 +0000327 } else {
Reid Kleckner7f720332017-04-13 00:58:09 +0000328 auto *NewCall = CallInst::Create(NF, Args, OpBundles, "", Call);
329 NewCall->setTailCallKind(cast<CallInst>(Call)->getTailCallKind());
330 NewCS = NewCall;
Chris Lattner483ae012004-03-07 21:29:54 +0000331 }
Reid Kleckner7f720332017-04-13 00:58:09 +0000332 NewCS.setCallingConv(CS.getCallingConv());
333 NewCS.setAttributes(
334 AttributeList::get(F->getContext(), CallPAL.getFnAttributes(),
335 CallPAL.getRetAttributes(), ArgAttrVec));
336 NewCS->setDebugLoc(Call->getDebugLoc());
Reid Kleckner3a115032017-04-13 18:10:30 +0000337 uint64_t W;
338 if (Call->extractProfTotalWeight(W))
339 NewCS->setProfWeight(W);
Chris Lattner483ae012004-03-07 21:29:54 +0000340 Args.clear();
Reid Kleckner7f720332017-04-13 00:58:09 +0000341 ArgAttrVec.clear();
Chris Lattner483ae012004-03-07 21:29:54 +0000342
Duncan Sands3cf7d862008-09-08 11:07:35 +0000343 // Update the callgraph to know that the callsite has been transformed.
Chandler Carruthaddcda42017-02-09 23:46:27 +0000344 if (ReplaceCallSite)
Reid Kleckner7f720332017-04-13 00:58:09 +0000345 (*ReplaceCallSite)(CS, NewCS);
Duncan Sands3cf7d862008-09-08 11:07:35 +0000346
Chris Lattner483ae012004-03-07 21:29:54 +0000347 if (!Call->use_empty()) {
Reid Kleckner7f720332017-04-13 00:58:09 +0000348 Call->replaceAllUsesWith(NewCS.getInstruction());
349 NewCS->takeName(Call);
Chris Lattner483ae012004-03-07 21:29:54 +0000350 }
Misha Brukmanb1c93172005-04-21 23:48:37 +0000351
Chris Lattner483ae012004-03-07 21:29:54 +0000352 // Finally, remove the old call from the program, reducing the use-count of
353 // F.
Chris Lattner4062a622008-01-11 19:20:39 +0000354 Call->eraseFromParent();
Chris Lattner483ae012004-03-07 21:29:54 +0000355 }
356
Matt Arsenault3c1fc762017-04-10 22:27:50 +0000357 const DataLayout &DL = F->getParent()->getDataLayout();
358
Chris Lattner483ae012004-03-07 21:29:54 +0000359 // Since we have now created the new function, splice the body of the old
360 // function right into the new function, leaving the old rotting hulk of the
361 // function empty.
362 NF->getBasicBlockList().splice(NF->begin(), F->getBasicBlockList());
363
Chris Lattner0ab5e2c2011-04-15 05:18:47 +0000364 // Loop over the argument list, transferring uses of the old arguments over to
365 // the new arguments, also transferring over the names as well.
Chris Lattnera06a8fd2007-02-13 02:10:56 +0000366 for (Function::arg_iterator I = F->arg_begin(), E = F->arg_end(),
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000367 I2 = NF->arg_begin();
368 I != E; ++I) {
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000369 if (!ArgsToPromote.count(&*I) && !ByValArgsToTransform.count(&*I)) {
Chris Lattner483ae012004-03-07 21:29:54 +0000370 // If this is an unmodified argument, move the name and users over to the
371 // new version.
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000372 I->replaceAllUsesWith(&*I2);
373 I2->takeName(&*I);
Chris Lattner483ae012004-03-07 21:29:54 +0000374 ++I2;
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000375 continue;
Chris Lattner483ae012004-03-07 21:29:54 +0000376 }
Duncan Sands1ea0d2e2008-09-07 09:54:09 +0000377
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000378 if (ByValArgsToTransform.count(&*I)) {
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000379 // In the callee, we create an alloca, and store each of the new incoming
380 // arguments into the alloca.
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000381 Instruction *InsertPt = &NF->begin()->front();
Duncan Sands1ea0d2e2008-09-07 09:54:09 +0000382
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000383 // Just add all the struct element types.
Chris Lattner229907c2011-07-18 04:54:35 +0000384 Type *AgTy = cast<PointerType>(I->getType())->getElementType();
Matt Arsenault3c1fc762017-04-10 22:27:50 +0000385 Value *TheAlloca = new AllocaInst(AgTy, DL.getAllocaAddrSpace(), nullptr,
Reid Klecknerda748f12017-08-04 17:09:11 +0000386 I->getParamAlignment(), "", InsertPt);
Chris Lattner229907c2011-07-18 04:54:35 +0000387 StructType *STy = cast<StructType>(AgTy);
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000388 Value *Idxs[2] = {ConstantInt::get(Type::getInt32Ty(F->getContext()), 0),
389 nullptr};
Duncan Sands1ea0d2e2008-09-07 09:54:09 +0000390
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000391 for (unsigned i = 0, e = STy->getNumElements(); i != e; ++i) {
Owen Anderson55f1c092009-08-13 21:58:54 +0000392 Idxs[1] = ConstantInt::get(Type::getInt32Ty(F->getContext()), i);
David Blaikie096b1da2015-03-14 19:53:33 +0000393 Value *Idx = GetElementPtrInst::Create(
394 AgTy, TheAlloca, Idxs, TheAlloca->getName() + "." + Twine(i),
395 InsertPt);
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000396 I2->setName(I->getName() + "." + Twine(i));
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000397 new StoreInst(&*I2++, Idx, InsertPt);
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000398 }
Duncan Sands1ea0d2e2008-09-07 09:54:09 +0000399
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000400 // Anything that used the arg should now use the alloca.
401 I->replaceAllUsesWith(TheAlloca);
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000402 TheAlloca->takeName(&*I);
Rafael Espindola2a05ea52014-01-23 17:19:42 +0000403
404 // If the alloca is used in a call, we must clear the tail flag since
405 // the callee now uses an alloca from the caller.
Chandler Carruthcdf47882014-03-09 03:16:01 +0000406 for (User *U : TheAlloca->users()) {
407 CallInst *Call = dyn_cast<CallInst>(U);
Rafael Espindola2a05ea52014-01-23 17:19:42 +0000408 if (!Call)
409 continue;
410 Call->setTailCall(false);
411 }
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000412 continue;
Duncan Sands1ea0d2e2008-09-07 09:54:09 +0000413 }
414
Chandler Carrutha1032a02015-07-22 09:49:59 +0000415 if (I->use_empty())
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000416 continue;
Duncan Sands1ea0d2e2008-09-07 09:54:09 +0000417
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000418 // Otherwise, if we promoted this argument, then all users are load
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000419 // instructions (or GEPs with only load users), and all loads should be
420 // using the new argument that we added.
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000421 ScalarizeTable &ArgIndices = ScalarizedElements[&*I];
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000422
423 while (!I->use_empty()) {
Chandler Carruthcdf47882014-03-09 03:16:01 +0000424 if (LoadInst *LI = dyn_cast<LoadInst>(I->user_back())) {
David Blaikie76826632015-03-14 21:11:26 +0000425 assert(ArgIndices.begin()->second.empty() &&
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000426 "Load element should sort to front!");
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000427 I2->setName(I->getName() + ".val");
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000428 LI->replaceAllUsesWith(&*I2);
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000429 LI->eraseFromParent();
Nicola Zaghend34e60c2018-05-14 12:53:11 +0000430 LLVM_DEBUG(dbgs() << "*** Promoted load of argument '" << I->getName()
431 << "' in function '" << F->getName() << "'\n");
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000432 } else {
Chandler Carruthcdf47882014-03-09 03:16:01 +0000433 GetElementPtrInst *GEP = cast<GetElementPtrInst>(I->user_back());
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000434 IndicesVector Operands;
435 Operands.reserve(GEP->getNumIndices());
436 for (User::op_iterator II = GEP->idx_begin(), IE = GEP->idx_end();
437 II != IE; ++II)
438 Operands.push_back(cast<ConstantInt>(*II)->getSExtValue());
439
440 // GEPs with a single 0 index can be merged with direct loads
441 if (Operands.size() == 1 && Operands.front() == 0)
442 Operands.clear();
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000443
444 Function::arg_iterator TheArg = I2;
445 for (ScalarizeTable::iterator It = ArgIndices.begin();
David Blaikie76826632015-03-14 21:11:26 +0000446 It->second != Operands; ++It, ++TheArg) {
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000447 assert(It != ArgIndices.end() && "GEP not handled??");
448 }
449
450 std::string NewName = I->getName();
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000451 for (unsigned i = 0, e = Operands.size(); i != e; ++i) {
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000452 NewName += "." + utostr(Operands[i]);
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000453 }
454 NewName += ".val";
455 TheArg->setName(NewName);
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000456
Nicola Zaghend34e60c2018-05-14 12:53:11 +0000457 LLVM_DEBUG(dbgs() << "*** Promoted agg argument '" << TheArg->getName()
458 << "' of function '" << NF->getName() << "'\n");
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000459
460 // All of the uses must be load instructions. Replace them all with
461 // the argument specified by ArgNo.
462 while (!GEP->use_empty()) {
Chandler Carruthcdf47882014-03-09 03:16:01 +0000463 LoadInst *L = cast<LoadInst>(GEP->user_back());
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000464 L->replaceAllUsesWith(&*TheArg);
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000465 L->eraseFromParent();
466 }
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000467 GEP->eraseFromParent();
468 }
469 }
470
471 // Increment I2 past all of the arguments added for this promoted pointer.
Benjamin Kramer84036252012-09-30 17:31:56 +0000472 std::advance(I2, ArgIndices.size());
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000473 }
Chris Lattner483ae012004-03-07 21:29:54 +0000474
Chandler Carruthaddcda42017-02-09 23:46:27 +0000475 return NF;
Chris Lattner483ae012004-03-07 21:29:54 +0000476}
David Blaikiee844cd52014-07-01 21:13:37 +0000477
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000478/// AllCallersPassInValidPointerForArgument - Return true if we can prove that
479/// all callees pass in a valid pointer for the specified function argument.
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000480static bool allCallersPassInValidPointerForArgument(Argument *Arg) {
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000481 Function *Callee = Arg->getParent();
482 const DataLayout &DL = Callee->getParent()->getDataLayout();
483
484 unsigned ArgNo = Arg->getArgNo();
485
486 // Look at all call sites of the function. At this point we know we only have
487 // direct callees.
488 for (User *U : Callee->users()) {
489 CallSite CS(U);
490 assert(CS && "Should only have direct calls!");
491
492 if (!isDereferenceablePointer(CS.getArgument(ArgNo), DL))
493 return false;
494 }
495 return true;
496}
497
498/// Returns true if Prefix is a prefix of longer. That means, Longer has a size
499/// that is greater than or equal to the size of prefix, and each of the
500/// elements in Prefix is the same as the corresponding elements in Longer.
501///
502/// This means it also returns true when Prefix and Longer are equal!
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000503static bool isPrefix(const IndicesVector &Prefix, const IndicesVector &Longer) {
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000504 if (Prefix.size() > Longer.size())
505 return false;
506 return std::equal(Prefix.begin(), Prefix.end(), Longer.begin());
507}
508
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000509/// Checks if Indices, or a prefix of Indices, is in Set.
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000510static bool prefixIn(const IndicesVector &Indices,
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000511 std::set<IndicesVector> &Set) {
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000512 std::set<IndicesVector>::iterator Low;
513 Low = Set.upper_bound(Indices);
514 if (Low != Set.begin())
515 Low--;
516 // Low is now the last element smaller than or equal to Indices. This means
517 // it points to a prefix of Indices (possibly Indices itself), if such
518 // prefix exists.
519 //
520 // This load is safe if any prefix of its operands is safe to load.
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000521 return Low != Set.end() && isPrefix(*Low, Indices);
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000522}
523
524/// Mark the given indices (ToMark) as safe in the given set of indices
525/// (Safe). Marking safe usually means adding ToMark to Safe. However, if there
526/// is already a prefix of Indices in Safe, Indices are implicitely marked safe
527/// already. Furthermore, any indices that Indices is itself a prefix of, are
528/// removed from Safe (since they are implicitely safe because of Indices now).
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000529static void markIndicesSafe(const IndicesVector &ToMark,
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000530 std::set<IndicesVector> &Safe) {
531 std::set<IndicesVector>::iterator Low;
532 Low = Safe.upper_bound(ToMark);
533 // Guard against the case where Safe is empty
534 if (Low != Safe.begin())
535 Low--;
536 // Low is now the last element smaller than or equal to Indices. This
537 // means it points to a prefix of Indices (possibly Indices itself), if
538 // such prefix exists.
539 if (Low != Safe.end()) {
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000540 if (isPrefix(*Low, ToMark))
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000541 // If there is already a prefix of these indices (or exactly these
542 // indices) marked a safe, don't bother adding these indices
543 return;
544
545 // Increment Low, so we can use it as a "insert before" hint
546 ++Low;
547 }
548 // Insert
549 Low = Safe.insert(Low, ToMark);
550 ++Low;
551 // If there we're a prefix of longer index list(s), remove those
552 std::set<IndicesVector>::iterator End = Safe.end();
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000553 while (Low != End && isPrefix(ToMark, *Low)) {
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000554 std::set<IndicesVector>::iterator Remove = Low;
555 ++Low;
556 Safe.erase(Remove);
557 }
558}
559
560/// isSafeToPromoteArgument - As you might guess from the name of this method,
561/// it checks to see if it is both safe and useful to promote the argument.
562/// This method limits promotion of aggregates to only promote up to three
563/// elements of the aggregate in order to avoid exploding the number of
564/// arguments passed in.
565static bool isSafeToPromoteArgument(Argument *Arg, bool isByValOrInAlloca,
566 AAResults &AAR, unsigned MaxElements) {
Eugene Zelenkof27d1612017-10-19 21:21:30 +0000567 using GEPIndicesSet = std::set<IndicesVector>;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000568
569 // Quick exit for unused arguments
570 if (Arg->use_empty())
571 return true;
572
573 // We can only promote this argument if all of the uses are loads, or are GEP
574 // instructions (with constant indices) that are subsequently loaded.
575 //
576 // Promoting the argument causes it to be loaded in the caller
577 // unconditionally. This is only safe if we can prove that either the load
578 // would have happened in the callee anyway (ie, there is a load in the entry
579 // block) or the pointer passed in at every call site is guaranteed to be
580 // valid.
581 // In the former case, invalid loads can happen, but would have happened
582 // anyway, in the latter case, invalid loads won't happen. This prevents us
583 // from introducing an invalid load that wouldn't have happened in the
584 // original code.
585 //
586 // This set will contain all sets of indices that are loaded in the entry
587 // block, and thus are safe to unconditionally load in the caller.
588 //
589 // This optimization is also safe for InAlloca parameters, because it verifies
590 // that the address isn't captured.
591 GEPIndicesSet SafeToUnconditionallyLoad;
592
593 // This set contains all the sets of indices that we are planning to promote.
594 // This makes it possible to limit the number of arguments added.
595 GEPIndicesSet ToPromote;
596
597 // If the pointer is always valid, any load with first index 0 is valid.
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000598 if (isByValOrInAlloca || allCallersPassInValidPointerForArgument(Arg))
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000599 SafeToUnconditionallyLoad.insert(IndicesVector(1, 0));
600
601 // First, iterate the entry block and mark loads of (geps of) arguments as
602 // safe.
603 BasicBlock &EntryBlock = Arg->getParent()->front();
604 // Declare this here so we can reuse it
605 IndicesVector Indices;
606 for (Instruction &I : EntryBlock)
607 if (LoadInst *LI = dyn_cast<LoadInst>(&I)) {
608 Value *V = LI->getPointerOperand();
609 if (GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(V)) {
610 V = GEP->getPointerOperand();
611 if (V == Arg) {
612 // This load actually loads (part of) Arg? Check the indices then.
613 Indices.reserve(GEP->getNumIndices());
614 for (User::op_iterator II = GEP->idx_begin(), IE = GEP->idx_end();
615 II != IE; ++II)
616 if (ConstantInt *CI = dyn_cast<ConstantInt>(*II))
617 Indices.push_back(CI->getSExtValue());
618 else
619 // We found a non-constant GEP index for this argument? Bail out
620 // right away, can't promote this argument at all.
621 return false;
622
623 // Indices checked out, mark them as safe
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000624 markIndicesSafe(Indices, SafeToUnconditionallyLoad);
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000625 Indices.clear();
626 }
627 } else if (V == Arg) {
628 // Direct loads are equivalent to a GEP with a single 0 index.
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000629 markIndicesSafe(IndicesVector(1, 0), SafeToUnconditionallyLoad);
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000630 }
631 }
632
633 // Now, iterate all uses of the argument to see if there are any uses that are
634 // not (GEP+)loads, or any (GEP+)loads that are not safe to promote.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000635 SmallVector<LoadInst *, 16> Loads;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000636 IndicesVector Operands;
637 for (Use &U : Arg->uses()) {
638 User *UR = U.getUser();
639 Operands.clear();
640 if (LoadInst *LI = dyn_cast<LoadInst>(UR)) {
641 // Don't hack volatile/atomic loads
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000642 if (!LI->isSimple())
643 return false;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000644 Loads.push_back(LI);
645 // Direct loads are equivalent to a GEP with a zero index and then a load.
646 Operands.push_back(0);
647 } else if (GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(UR)) {
648 if (GEP->use_empty()) {
649 // Dead GEP's cause trouble later. Just remove them if we run into
650 // them.
651 GEP->eraseFromParent();
652 // TODO: This runs the above loop over and over again for dead GEPs
653 // Couldn't we just do increment the UI iterator earlier and erase the
654 // use?
655 return isSafeToPromoteArgument(Arg, isByValOrInAlloca, AAR,
656 MaxElements);
657 }
658
659 // Ensure that all of the indices are constants.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000660 for (User::op_iterator i = GEP->idx_begin(), e = GEP->idx_end(); i != e;
661 ++i)
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000662 if (ConstantInt *C = dyn_cast<ConstantInt>(*i))
663 Operands.push_back(C->getSExtValue());
664 else
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000665 return false; // Not a constant operand GEP!
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000666
667 // Ensure that the only users of the GEP are load instructions.
668 for (User *GEPU : GEP->users())
669 if (LoadInst *LI = dyn_cast<LoadInst>(GEPU)) {
670 // Don't hack volatile/atomic loads
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000671 if (!LI->isSimple())
672 return false;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000673 Loads.push_back(LI);
674 } else {
675 // Other uses than load?
676 return false;
677 }
678 } else {
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000679 return false; // Not a load or a GEP.
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000680 }
681
682 // Now, see if it is safe to promote this load / loads of this GEP. Loading
683 // is safe if Operands, or a prefix of Operands, is marked as safe.
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000684 if (!prefixIn(Operands, SafeToUnconditionallyLoad))
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000685 return false;
686
687 // See if we are already promoting a load with these indices. If not, check
688 // to make sure that we aren't promoting too many elements. If so, nothing
689 // to do.
690 if (ToPromote.find(Operands) == ToPromote.end()) {
691 if (MaxElements > 0 && ToPromote.size() == MaxElements) {
Nicola Zaghend34e60c2018-05-14 12:53:11 +0000692 LLVM_DEBUG(dbgs() << "argpromotion not promoting argument '"
693 << Arg->getName()
694 << "' because it would require adding more "
695 << "than " << MaxElements
696 << " arguments to the function.\n");
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000697 // We limit aggregate promotion to only promoting up to a fixed number
698 // of elements of the aggregate.
699 return false;
700 }
701 ToPromote.insert(std::move(Operands));
702 }
703 }
704
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000705 if (Loads.empty())
706 return true; // No users, this is a dead argument.
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000707
708 // Okay, now we know that the argument is only used by load instructions and
709 // it is safe to unconditionally perform all of them. Use alias analysis to
710 // check to see if the pointer is guaranteed to not be modified from entry of
711 // the function to each of the load instructions.
712
713 // Because there could be several/many load instructions, remember which
714 // blocks we know to be transparent to the load.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000715 df_iterator_default_set<BasicBlock *, 16> TranspBlocks;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000716
717 for (LoadInst *Load : Loads) {
718 // Check to see if the load is invalidated from the start of the block to
719 // the load itself.
720 BasicBlock *BB = Load->getParent();
721
722 MemoryLocation Loc = MemoryLocation::get(Load);
Alina Sbirlea193429f2017-12-07 22:41:34 +0000723 if (AAR.canInstructionRangeModRef(BB->front(), *Load, Loc, ModRefInfo::Mod))
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000724 return false; // Pointer is invalidated!
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000725
726 // Now check every path from the entry block to the load for transparency.
727 // To do this, we perform a depth first search on the inverse CFG from the
728 // loading block.
729 for (BasicBlock *P : predecessors(BB)) {
730 for (BasicBlock *TranspBB : inverse_depth_first_ext(P, TranspBlocks))
731 if (AAR.canBasicBlockModify(*TranspBB, Loc))
732 return false;
733 }
734 }
735
736 // If the path from the entry of the function to each load is free of
737 // instructions that potentially invalidate the load, we can make the
738 // transformation!
739 return true;
740}
741
Adrian Prantl5f8f34e42018-05-01 15:54:18 +0000742/// Checks if a type could have padding bytes.
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000743static bool isDenselyPacked(Type *type, const DataLayout &DL) {
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000744 // There is no size information, so be conservative.
745 if (!type->isSized())
746 return false;
747
748 // If the alloc size is not equal to the storage size, then there are padding
749 // bytes. For x86_fp80 on x86-64, size: 80 alloc size: 128.
750 if (DL.getTypeSizeInBits(type) != DL.getTypeAllocSizeInBits(type))
751 return false;
752
753 if (!isa<CompositeType>(type))
754 return true;
755
756 // For homogenous sequential types, check for padding within members.
757 if (SequentialType *seqTy = dyn_cast<SequentialType>(type))
758 return isDenselyPacked(seqTy->getElementType(), DL);
759
760 // Check for padding within and between elements of a struct.
761 StructType *StructTy = cast<StructType>(type);
762 const StructLayout *Layout = DL.getStructLayout(StructTy);
763 uint64_t StartPos = 0;
764 for (unsigned i = 0, E = StructTy->getNumElements(); i < E; ++i) {
765 Type *ElTy = StructTy->getElementType(i);
766 if (!isDenselyPacked(ElTy, DL))
767 return false;
768 if (StartPos != Layout->getElementOffsetInBits(i))
769 return false;
770 StartPos += DL.getTypeAllocSizeInBits(ElTy);
771 }
772
773 return true;
774}
775
Adrian Prantl5f8f34e42018-05-01 15:54:18 +0000776/// Checks if the padding bytes of an argument could be accessed.
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000777static bool canPaddingBeAccessed(Argument *arg) {
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000778 assert(arg->hasByValAttr());
779
780 // Track all the pointers to the argument to make sure they are not captured.
781 SmallPtrSet<Value *, 16> PtrValues;
782 PtrValues.insert(arg);
783
784 // Track all of the stores.
785 SmallVector<StoreInst *, 16> Stores;
786
787 // Scan through the uses recursively to make sure the pointer is always used
788 // sanely.
789 SmallVector<Value *, 16> WorkList;
790 WorkList.insert(WorkList.end(), arg->user_begin(), arg->user_end());
791 while (!WorkList.empty()) {
792 Value *V = WorkList.back();
793 WorkList.pop_back();
794 if (isa<GetElementPtrInst>(V) || isa<PHINode>(V)) {
795 if (PtrValues.insert(V).second)
796 WorkList.insert(WorkList.end(), V->user_begin(), V->user_end());
797 } else if (StoreInst *Store = dyn_cast<StoreInst>(V)) {
798 Stores.push_back(Store);
799 } else if (!isa<LoadInst>(V)) {
800 return true;
801 }
802 }
803
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000804 // Check to make sure the pointers aren't captured
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000805 for (StoreInst *Store : Stores)
806 if (PtrValues.count(Store->getValueOperand()))
807 return true;
808
809 return false;
810}
811
812/// PromoteArguments - This method checks the specified function to see if there
813/// are any promotable arguments and if it is safe to promote the function (for
814/// example, all callers are direct). If safe to promote some arguments, it
815/// calls the DoPromotion method.
Chandler Carruthaddcda42017-02-09 23:46:27 +0000816static Function *
817promoteArguments(Function *F, function_ref<AAResults &(Function &F)> AARGetter,
818 unsigned MaxElements,
819 Optional<function_ref<void(CallSite OldCS, CallSite NewCS)>>
820 ReplaceCallSite) {
Luke Cheeseman6c1e6bb2018-02-22 14:42:08 +0000821 // Don't perform argument promotion for naked functions; otherwise we can end
822 // up removing parameters that are seemingly 'not used' as they are referred
823 // to in the assembly.
824 if(F->hasFnAttribute(Attribute::Naked))
825 return nullptr;
826
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000827 // Make sure that it is local to this module.
Chandler Carruthaddcda42017-02-09 23:46:27 +0000828 if (!F->hasLocalLinkage())
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000829 return nullptr;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000830
831 // Don't promote arguments for variadic functions. Adding, removing, or
832 // changing non-pack parameters can change the classification of pack
833 // parameters. Frontends encode that classification at the call site in the
834 // IR, while in the callee the classification is determined dynamically based
835 // on the number of registers consumed so far.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000836 if (F->isVarArg())
837 return nullptr;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000838
839 // First check: see if there are any pointer arguments! If not, quick exit.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000840 SmallVector<Argument *, 16> PointerArgs;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000841 for (Argument &I : F->args())
842 if (I.getType()->isPointerTy())
843 PointerArgs.push_back(&I);
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000844 if (PointerArgs.empty())
845 return nullptr;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000846
847 // Second check: make sure that all callers are direct callers. We can't
848 // transform functions that have indirect callers. Also see if the function
849 // is self-recursive.
850 bool isSelfRecursive = false;
851 for (Use &U : F->uses()) {
852 CallSite CS(U.getUser());
853 // Must be a direct call.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000854 if (CS.getInstruction() == nullptr || !CS.isCallee(&U))
855 return nullptr;
856
Fedor Indutny1571b122018-03-02 00:59:27 +0000857 // Can't change signature of musttail callee
858 if (CS.isMustTailCall())
859 return nullptr;
860
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000861 if (CS.getInstruction()->getParent()->getParent() == F)
862 isSelfRecursive = true;
863 }
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000864
Fedor Indutny1571b122018-03-02 00:59:27 +0000865 // Can't change signature of musttail caller
866 // FIXME: Support promoting whole chain of musttail functions
867 for (BasicBlock &BB : *F)
868 if (BB.getTerminatingMustTailCall())
869 return nullptr;
870
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000871 const DataLayout &DL = F->getParent()->getDataLayout();
872
873 AAResults &AAR = AARGetter(*F);
874
875 // Check to see which arguments are promotable. If an argument is promotable,
876 // add it to ArgsToPromote.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000877 SmallPtrSet<Argument *, 8> ArgsToPromote;
878 SmallPtrSet<Argument *, 8> ByValArgsToTransform;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000879 for (Argument *PtrArg : PointerArgs) {
880 Type *AgTy = cast<PointerType>(PtrArg->getType())->getElementType();
881
882 // Replace sret attribute with noalias. This reduces register pressure by
883 // avoiding a register copy.
884 if (PtrArg->hasStructRetAttr()) {
885 unsigned ArgNo = PtrArg->getArgNo();
Reid Klecknera0b45f42017-05-03 18:17:31 +0000886 F->removeParamAttr(ArgNo, Attribute::StructRet);
887 F->addParamAttr(ArgNo, Attribute::NoAlias);
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000888 for (Use &U : F->uses()) {
889 CallSite CS(U.getUser());
Reid Klecknera0b45f42017-05-03 18:17:31 +0000890 CS.removeParamAttr(ArgNo, Attribute::StructRet);
891 CS.addParamAttr(ArgNo, Attribute::NoAlias);
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000892 }
893 }
894
895 // If this is a byval argument, and if the aggregate type is small, just
896 // pass the elements, which is always safe, if the passed value is densely
897 // packed or if we can prove the padding bytes are never accessed. This does
898 // not apply to inalloca.
899 bool isSafeToPromote =
900 PtrArg->hasByValAttr() &&
901 (isDenselyPacked(AgTy, DL) || !canPaddingBeAccessed(PtrArg));
902 if (isSafeToPromote) {
903 if (StructType *STy = dyn_cast<StructType>(AgTy)) {
904 if (MaxElements > 0 && STy->getNumElements() > MaxElements) {
Nicola Zaghend34e60c2018-05-14 12:53:11 +0000905 LLVM_DEBUG(dbgs() << "argpromotion disable promoting argument '"
906 << PtrArg->getName()
907 << "' because it would require adding more"
908 << " than " << MaxElements
909 << " arguments to the function.\n");
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000910 continue;
911 }
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000912
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000913 // If all the elements are single-value types, we can promote it.
914 bool AllSimple = true;
915 for (const auto *EltTy : STy->elements()) {
916 if (!EltTy->isSingleValueType()) {
917 AllSimple = false;
918 break;
919 }
920 }
921
922 // Safe to transform, don't even bother trying to "promote" it.
923 // Passing the elements as a scalar will allow sroa to hack on
924 // the new alloca we introduce.
925 if (AllSimple) {
926 ByValArgsToTransform.insert(PtrArg);
927 continue;
928 }
929 }
930 }
931
932 // If the argument is a recursive type and we're in a recursive
933 // function, we could end up infinitely peeling the function argument.
934 if (isSelfRecursive) {
935 if (StructType *STy = dyn_cast<StructType>(AgTy)) {
936 bool RecursiveType = false;
937 for (const auto *EltTy : STy->elements()) {
938 if (EltTy == PtrArg->getType()) {
939 RecursiveType = true;
940 break;
941 }
942 }
943 if (RecursiveType)
944 continue;
945 }
946 }
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000947
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000948 // Otherwise, see if we can promote the pointer to its value.
949 if (isSafeToPromoteArgument(PtrArg, PtrArg->hasByValOrInAllocaAttr(), AAR,
950 MaxElements))
951 ArgsToPromote.insert(PtrArg);
952 }
953
954 // No promotable pointer arguments.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000955 if (ArgsToPromote.empty() && ByValArgsToTransform.empty())
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000956 return nullptr;
957
Chandler Carruthaddcda42017-02-09 23:46:27 +0000958 return doPromotion(F, ArgsToPromote, ByValArgsToTransform, ReplaceCallSite);
959}
960
961PreservedAnalyses ArgumentPromotionPass::run(LazyCallGraph::SCC &C,
962 CGSCCAnalysisManager &AM,
963 LazyCallGraph &CG,
964 CGSCCUpdateResult &UR) {
965 bool Changed = false, LocalChange;
966
967 // Iterate until we stop promoting from this SCC.
968 do {
969 LocalChange = false;
970
971 for (LazyCallGraph::Node &N : C) {
972 Function &OldF = N.getFunction();
973
974 FunctionAnalysisManager &FAM =
975 AM.getResult<FunctionAnalysisManagerCGSCCProxy>(C, CG).getManager();
976 // FIXME: This lambda must only be used with this function. We should
977 // skip the lambda and just get the AA results directly.
978 auto AARGetter = [&](Function &F) -> AAResults & {
979 assert(&F == &OldF && "Called with an unexpected function!");
980 return FAM.getResult<AAManager>(F);
981 };
982
Justin Bogner92fe5632018-01-08 21:13:35 +0000983 Function *NewF = promoteArguments(&OldF, AARGetter, MaxElements, None);
Chandler Carruthaddcda42017-02-09 23:46:27 +0000984 if (!NewF)
985 continue;
986 LocalChange = true;
987
988 // Directly substitute the functions in the call graph. Note that this
989 // requires the old function to be completely dead and completely
990 // replaced by the new function. It does no call graph updates, it merely
991 // swaps out the particular function mapped to a particular node in the
992 // graph.
993 C.getOuterRefSCC().replaceNodeFunction(N, *NewF);
994 OldF.eraseFromParent();
995 }
996
997 Changed |= LocalChange;
998 } while (LocalChange);
999
1000 if (!Changed)
1001 return PreservedAnalyses::all();
1002
1003 return PreservedAnalyses::none();
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001004}
1005
1006namespace {
Eugene Zelenkof27d1612017-10-19 21:21:30 +00001007
Chandler Carruthae9ce3d2017-01-29 08:03:19 +00001008/// ArgPromotion - The 'by reference' to 'by value' argument promotion pass.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +00001009struct ArgPromotion : public CallGraphSCCPass {
Eugene Zelenkof27d1612017-10-19 21:21:30 +00001010 // Pass identification, replacement for typeid
1011 static char ID;
1012
1013 explicit ArgPromotion(unsigned MaxElements = 3)
1014 : CallGraphSCCPass(ID), MaxElements(MaxElements) {
1015 initializeArgPromotionPass(*PassRegistry::getPassRegistry());
1016 }
1017
Chandler Carruthae9ce3d2017-01-29 08:03:19 +00001018 void getAnalysisUsage(AnalysisUsage &AU) const override {
1019 AU.addRequired<AssumptionCacheTracker>();
1020 AU.addRequired<TargetLibraryInfoWrapperPass>();
1021 getAAResultsAnalysisUsage(AU);
1022 CallGraphSCCPass::getAnalysisUsage(AU);
1023 }
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001024
Chandler Carruthae9ce3d2017-01-29 08:03:19 +00001025 bool runOnSCC(CallGraphSCC &SCC) override;
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001026
Chandler Carruthae9ce3d2017-01-29 08:03:19 +00001027private:
1028 using llvm::Pass::doInitialization;
Eugene Zelenkof27d1612017-10-19 21:21:30 +00001029
Chandler Carruthae9ce3d2017-01-29 08:03:19 +00001030 bool doInitialization(CallGraph &CG) override;
Eugene Zelenkof27d1612017-10-19 21:21:30 +00001031
Chandler Carruthae9ce3d2017-01-29 08:03:19 +00001032 /// The maximum number of elements to expand, or 0 for unlimited.
1033 unsigned MaxElements;
1034};
Eugene Zelenkof27d1612017-10-19 21:21:30 +00001035
1036} // end anonymous namespace
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001037
1038char ArgPromotion::ID = 0;
Eugene Zelenkof27d1612017-10-19 21:21:30 +00001039
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001040INITIALIZE_PASS_BEGIN(ArgPromotion, "argpromotion",
Chandler Carruthae9ce3d2017-01-29 08:03:19 +00001041 "Promote 'by reference' arguments to scalars", false,
1042 false)
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001043INITIALIZE_PASS_DEPENDENCY(AssumptionCacheTracker)
1044INITIALIZE_PASS_DEPENDENCY(CallGraphWrapperPass)
1045INITIALIZE_PASS_DEPENDENCY(TargetLibraryInfoWrapperPass)
1046INITIALIZE_PASS_END(ArgPromotion, "argpromotion",
Chandler Carruthae9ce3d2017-01-29 08:03:19 +00001047 "Promote 'by reference' arguments to scalars", false, false)
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001048
1049Pass *llvm::createArgumentPromotionPass(unsigned MaxElements) {
1050 return new ArgPromotion(MaxElements);
1051}
1052
1053bool ArgPromotion::runOnSCC(CallGraphSCC &SCC) {
1054 if (skipSCC(SCC))
1055 return false;
1056
1057 // Get the callgraph information that we need to update to reflect our
1058 // changes.
1059 CallGraph &CG = getAnalysis<CallGraphWrapperPass>().getCallGraph();
1060
Peter Collingbournecea1e4e2017-02-09 23:11:52 +00001061 LegacyAARGetter AARGetter(*this);
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001062
1063 bool Changed = false, LocalChange;
1064
1065 // Iterate until we stop promoting from this SCC.
1066 do {
1067 LocalChange = false;
1068 // Attempt to promote arguments from all functions in this SCC.
1069 for (CallGraphNode *OldNode : SCC) {
Chandler Carruthaddcda42017-02-09 23:46:27 +00001070 Function *OldF = OldNode->getFunction();
1071 if (!OldF)
1072 continue;
1073
1074 auto ReplaceCallSite = [&](CallSite OldCS, CallSite NewCS) {
1075 Function *Caller = OldCS.getInstruction()->getParent()->getParent();
1076 CallGraphNode *NewCalleeNode =
1077 CG.getOrInsertFunction(NewCS.getCalledFunction());
1078 CallGraphNode *CallerNode = CG[Caller];
1079 CallerNode->replaceCallEdge(OldCS, NewCS, NewCalleeNode);
1080 };
1081
1082 if (Function *NewF = promoteArguments(OldF, AARGetter, MaxElements,
1083 {ReplaceCallSite})) {
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001084 LocalChange = true;
Chandler Carruthaddcda42017-02-09 23:46:27 +00001085
1086 // Update the call graph for the newly promoted function.
1087 CallGraphNode *NewNode = CG.getOrInsertFunction(NewF);
1088 NewNode->stealCalledFunctionsFrom(OldNode);
1089 if (OldNode->getNumReferences() == 0)
1090 delete CG.removeFunctionFromModule(OldNode);
1091 else
1092 OldF->setLinkage(Function::ExternalLinkage);
1093
1094 // And updat ethe SCC we're iterating as well.
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001095 SCC.ReplaceNode(OldNode, NewNode);
1096 }
1097 }
1098 // Remember that we changed something.
1099 Changed |= LocalChange;
1100 } while (LocalChange);
1101
1102 return Changed;
1103}
1104
David Blaikiee844cd52014-07-01 21:13:37 +00001105bool ArgPromotion::doInitialization(CallGraph &CG) {
David Blaikiee844cd52014-07-01 21:13:37 +00001106 return CallGraphSCCPass::doInitialization(CG);
1107}