blob: e159920116ca8de5620443c7e24f52009d63636d [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.
Gabor Greife9ecc682008-04-06 20:25:17 +0000216 Function *NF = Function::Create(NFTy, F->getLinkage(), F->getName());
Duncan Sandsdd7daee2008-05-26 19:58:59 +0000217 NF->copyAttributesFrom(F);
Chris Lattner5630c4f2008-01-17 01:17:03 +0000218
David Blaikiee844cd52014-07-01 21:13:37 +0000219 // Patch the pointer to LLVM function in debug info descriptor.
Peter Collingbourned4bff302015-11-05 22:03:56 +0000220 NF->setSubprogram(F->getSubprogram());
221 F->setSubprogram(nullptr);
David Blaikie8e9cfa52014-07-23 22:09:29 +0000222
David Greenecf0addf2010-01-05 01:28:37 +0000223 DEBUG(dbgs() << "ARG PROMOTION: Promoting to:" << *NF << "\n"
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000224 << "From: " << *F);
225
Chris Lattner5630c4f2008-01-17 01:17:03 +0000226 // Recompute the parameter attributes list based on the new arguments for
227 // the function.
Reid Kleckner7f720332017-04-13 00:58:09 +0000228 NF->setAttributes(AttributeList::get(F->getContext(), PAL.getFnAttributes(),
229 PAL.getRetAttributes(), ArgAttrVec));
230 ArgAttrVec.clear();
Duncan Sandsdd7daee2008-05-26 19:58:59 +0000231
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000232 F->getParent()->getFunctionList().insert(F->getIterator(), NF);
Zhou Shenga30cdb92008-03-20 08:05:05 +0000233 NF->takeName(F);
Chris Lattner254f8f82004-05-23 21:21:17 +0000234
Chris Lattner483ae012004-03-07 21:29:54 +0000235 // Loop over all of the callers of the function, transforming the call sites
236 // to pass in the loaded pointers.
237 //
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000238 SmallVector<Value *, 16> Args;
Chris Lattner483ae012004-03-07 21:29:54 +0000239 while (!F->use_empty()) {
Chandler Carruthcdf47882014-03-09 03:16:01 +0000240 CallSite CS(F->user_back());
Gabor Greif161cb042010-03-23 14:40:20 +0000241 assert(CS.getCalledFunction() == F);
Chris Lattner483ae012004-03-07 21:29:54 +0000242 Instruction *Call = CS.getInstruction();
Reid Klecknerb5180542017-03-21 16:57:19 +0000243 const AttributeList &CallPAL = CS.getAttributes();
Duncan Sands1ea0d2e2008-09-07 09:54:09 +0000244
Chris Lattner254f8f82004-05-23 21:21:17 +0000245 // Loop over the operands, inserting GEP and loads in the caller as
246 // appropriate.
Chris Lattner483ae012004-03-07 21:29:54 +0000247 CallSite::arg_iterator AI = CS.arg_begin();
Reid Kleckner6652a522017-04-28 18:37:16 +0000248 ArgNo = 0;
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000249 for (Function::arg_iterator I = F->arg_begin(), E = F->arg_end(); I != E;
Reid Kleckner6652a522017-04-28 18:37:16 +0000250 ++I, ++AI, ++ArgNo)
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000251 if (!ArgsToPromote.count(&*I) && !ByValArgsToTransform.count(&*I)) {
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000252 Args.push_back(*AI); // Unmodified argument
Reid Kleckner6652a522017-04-28 18:37:16 +0000253 ArgAttrVec.push_back(CallPAL.getParamAttributes(ArgNo));
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000254 } else if (ByValArgsToTransform.count(&*I)) {
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000255 // Emit a GEP and load for each element of the struct.
Chris Lattner229907c2011-07-18 04:54:35 +0000256 Type *AgTy = cast<PointerType>(I->getType())->getElementType();
257 StructType *STy = cast<StructType>(AgTy);
Owen Anderson55f1c092009-08-13 21:58:54 +0000258 Value *Idxs[2] = {
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000259 ConstantInt::get(Type::getInt32Ty(F->getContext()), 0), nullptr};
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000260 for (unsigned i = 0, e = STy->getNumElements(); i != e; ++i) {
Owen Anderson55f1c092009-08-13 21:58:54 +0000261 Idxs[1] = ConstantInt::get(Type::getInt32Ty(F->getContext()), i);
David Blaikie76826632015-03-14 21:11:26 +0000262 Value *Idx = GetElementPtrInst::Create(
Benjamin Kramerdba7ee92015-05-28 11:24:24 +0000263 STy, *AI, Idxs, (*AI)->getName() + "." + Twine(i), Call);
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000264 // TODO: Tell AA about the new values?
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000265 Args.push_back(new LoadInst(Idx, Idx->getName() + ".val", Call));
Reid Kleckner7f720332017-04-13 00:58:09 +0000266 ArgAttrVec.push_back(AttributeSet());
Duncan Sands1ea0d2e2008-09-07 09:54:09 +0000267 }
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000268 } else if (!I->use_empty()) {
Chris Lattner254f8f82004-05-23 21:21:17 +0000269 // Non-dead argument: insert GEPs and loads as appropriate.
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000270 ScalarizeTable &ArgIndices = ScalarizedElements[&*I];
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000271 // Store the Value* version of the indices in here, but declare it now
Gabor Greif161cb042010-03-23 14:40:20 +0000272 // for reuse.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000273 std::vector<Value *> Ops;
Benjamin Kramer135f7352016-06-26 12:28:59 +0000274 for (const auto &ArgIndex : ArgIndices) {
Chris Lattnerfe6f2e32004-03-08 01:04:36 +0000275 Value *V = *AI;
Benjamin Kramer135f7352016-06-26 12:28:59 +0000276 LoadInst *OrigLoad =
277 OriginalLoads[std::make_pair(&*I, ArgIndex.second)];
278 if (!ArgIndex.second.empty()) {
279 Ops.reserve(ArgIndex.second.size());
Chris Lattner229907c2011-07-18 04:54:35 +0000280 Type *ElTy = V->getType();
Martin Storsjoe81233d2017-05-04 10:54:35 +0000281 for (auto II : ArgIndex.second) {
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000282 // Use i32 to index structs, and i64 for others (pointers/arrays).
283 // This satisfies GEP constraints.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000284 Type *IdxTy =
285 (ElTy->isStructTy() ? Type::getInt32Ty(F->getContext())
286 : Type::getInt64Ty(F->getContext()));
Benjamin Kramer135f7352016-06-26 12:28:59 +0000287 Ops.push_back(ConstantInt::get(IdxTy, II));
Gabor Greif161cb042010-03-23 14:40:20 +0000288 // Keep track of the type we're currently indexing.
Peter Collingbourne45681582016-12-02 03:05:41 +0000289 if (auto *ElPTy = dyn_cast<PointerType>(ElTy))
290 ElTy = ElPTy->getElementType();
291 else
292 ElTy = cast<CompositeType>(ElTy)->getTypeAtIndex(II);
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000293 }
Gabor Greif161cb042010-03-23 14:40:20 +0000294 // And create a GEP to extract those indices.
Benjamin Kramer135f7352016-06-26 12:28:59 +0000295 V = GetElementPtrInst::Create(ArgIndex.first, V, Ops,
David Blaikie72edd882015-03-14 21:40:12 +0000296 V->getName() + ".idx", Call);
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000297 Ops.clear();
Chris Lattner254f8f82004-05-23 21:21:17 +0000298 }
Eric Christopher81c03442010-03-27 01:54:00 +0000299 // Since we're replacing a load make sure we take the alignment
300 // of the previous load.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000301 LoadInst *newLoad = new LoadInst(V, V->getName() + ".val", Call);
Eric Christopher81c03442010-03-27 01:54:00 +0000302 newLoad->setAlignment(OrigLoad->getAlignment());
Hal Finkelcc39b672014-07-24 12:16:19 +0000303 // Transfer the AA info too.
304 AAMDNodes AAInfo;
305 OrigLoad->getAAMetadata(AAInfo);
306 newLoad->setAAMetadata(AAInfo);
307
Eric Christopher81c03442010-03-27 01:54:00 +0000308 Args.push_back(newLoad);
Reid Kleckner7f720332017-04-13 00:58:09 +0000309 ArgAttrVec.push_back(AttributeSet());
Chris Lattnerfe6f2e32004-03-08 01:04:36 +0000310 }
Chris Lattner483ae012004-03-07 21:29:54 +0000311 }
312
Gabor Greif161cb042010-03-23 14:40:20 +0000313 // Push any varargs arguments on the list.
Reid Kleckner6652a522017-04-28 18:37:16 +0000314 for (; AI != CS.arg_end(); ++AI, ++ArgNo) {
Chris Lattner483ae012004-03-07 21:29:54 +0000315 Args.push_back(*AI);
Reid Kleckner6652a522017-04-28 18:37:16 +0000316 ArgAttrVec.push_back(CallPAL.getParamAttributes(ArgNo));
Chris Lattner5630c4f2008-01-17 01:17:03 +0000317 }
Chris Lattner483ae012004-03-07 21:29:54 +0000318
David Majnemercd24bb12016-04-29 04:56:12 +0000319 SmallVector<OperandBundleDef, 1> OpBundles;
320 CS.getOperandBundlesAsDefs(OpBundles);
321
Reid Kleckner7f720332017-04-13 00:58:09 +0000322 CallSite NewCS;
Chris Lattner483ae012004-03-07 21:29:54 +0000323 if (InvokeInst *II = dyn_cast<InvokeInst>(Call)) {
Reid Kleckner7f720332017-04-13 00:58:09 +0000324 NewCS = InvokeInst::Create(NF, II->getNormalDest(), II->getUnwindDest(),
Reid Kleckner3a115032017-04-13 18:10:30 +0000325 Args, OpBundles, "", Call);
Chris Lattner483ae012004-03-07 21:29:54 +0000326 } else {
Reid Kleckner7f720332017-04-13 00:58:09 +0000327 auto *NewCall = CallInst::Create(NF, Args, OpBundles, "", Call);
328 NewCall->setTailCallKind(cast<CallInst>(Call)->getTailCallKind());
329 NewCS = NewCall;
Chris Lattner483ae012004-03-07 21:29:54 +0000330 }
Reid Kleckner7f720332017-04-13 00:58:09 +0000331 NewCS.setCallingConv(CS.getCallingConv());
332 NewCS.setAttributes(
333 AttributeList::get(F->getContext(), CallPAL.getFnAttributes(),
334 CallPAL.getRetAttributes(), ArgAttrVec));
335 NewCS->setDebugLoc(Call->getDebugLoc());
Reid Kleckner3a115032017-04-13 18:10:30 +0000336 uint64_t W;
337 if (Call->extractProfTotalWeight(W))
338 NewCS->setProfWeight(W);
Chris Lattner483ae012004-03-07 21:29:54 +0000339 Args.clear();
Reid Kleckner7f720332017-04-13 00:58:09 +0000340 ArgAttrVec.clear();
Chris Lattner483ae012004-03-07 21:29:54 +0000341
Duncan Sands3cf7d862008-09-08 11:07:35 +0000342 // Update the callgraph to know that the callsite has been transformed.
Chandler Carruthaddcda42017-02-09 23:46:27 +0000343 if (ReplaceCallSite)
Reid Kleckner7f720332017-04-13 00:58:09 +0000344 (*ReplaceCallSite)(CS, NewCS);
Duncan Sands3cf7d862008-09-08 11:07:35 +0000345
Chris Lattner483ae012004-03-07 21:29:54 +0000346 if (!Call->use_empty()) {
Reid Kleckner7f720332017-04-13 00:58:09 +0000347 Call->replaceAllUsesWith(NewCS.getInstruction());
348 NewCS->takeName(Call);
Chris Lattner483ae012004-03-07 21:29:54 +0000349 }
Misha Brukmanb1c93172005-04-21 23:48:37 +0000350
Chris Lattner483ae012004-03-07 21:29:54 +0000351 // Finally, remove the old call from the program, reducing the use-count of
352 // F.
Chris Lattner4062a622008-01-11 19:20:39 +0000353 Call->eraseFromParent();
Chris Lattner483ae012004-03-07 21:29:54 +0000354 }
355
Matt Arsenault3c1fc762017-04-10 22:27:50 +0000356 const DataLayout &DL = F->getParent()->getDataLayout();
357
Chris Lattner483ae012004-03-07 21:29:54 +0000358 // Since we have now created the new function, splice the body of the old
359 // function right into the new function, leaving the old rotting hulk of the
360 // function empty.
361 NF->getBasicBlockList().splice(NF->begin(), F->getBasicBlockList());
362
Chris Lattner0ab5e2c2011-04-15 05:18:47 +0000363 // Loop over the argument list, transferring uses of the old arguments over to
364 // the new arguments, also transferring over the names as well.
Chris Lattnera06a8fd2007-02-13 02:10:56 +0000365 for (Function::arg_iterator I = F->arg_begin(), E = F->arg_end(),
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000366 I2 = NF->arg_begin();
367 I != E; ++I) {
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000368 if (!ArgsToPromote.count(&*I) && !ByValArgsToTransform.count(&*I)) {
Chris Lattner483ae012004-03-07 21:29:54 +0000369 // If this is an unmodified argument, move the name and users over to the
370 // new version.
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000371 I->replaceAllUsesWith(&*I2);
372 I2->takeName(&*I);
Chris Lattner483ae012004-03-07 21:29:54 +0000373 ++I2;
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000374 continue;
Chris Lattner483ae012004-03-07 21:29:54 +0000375 }
Duncan Sands1ea0d2e2008-09-07 09:54:09 +0000376
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000377 if (ByValArgsToTransform.count(&*I)) {
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000378 // In the callee, we create an alloca, and store each of the new incoming
379 // arguments into the alloca.
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000380 Instruction *InsertPt = &NF->begin()->front();
Duncan Sands1ea0d2e2008-09-07 09:54:09 +0000381
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000382 // Just add all the struct element types.
Chris Lattner229907c2011-07-18 04:54:35 +0000383 Type *AgTy = cast<PointerType>(I->getType())->getElementType();
Matt Arsenault3c1fc762017-04-10 22:27:50 +0000384 Value *TheAlloca = new AllocaInst(AgTy, DL.getAllocaAddrSpace(), nullptr,
Reid Klecknerda748f12017-08-04 17:09:11 +0000385 I->getParamAlignment(), "", InsertPt);
Chris Lattner229907c2011-07-18 04:54:35 +0000386 StructType *STy = cast<StructType>(AgTy);
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000387 Value *Idxs[2] = {ConstantInt::get(Type::getInt32Ty(F->getContext()), 0),
388 nullptr};
Duncan Sands1ea0d2e2008-09-07 09:54:09 +0000389
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000390 for (unsigned i = 0, e = STy->getNumElements(); i != e; ++i) {
Owen Anderson55f1c092009-08-13 21:58:54 +0000391 Idxs[1] = ConstantInt::get(Type::getInt32Ty(F->getContext()), i);
David Blaikie096b1da2015-03-14 19:53:33 +0000392 Value *Idx = GetElementPtrInst::Create(
393 AgTy, TheAlloca, Idxs, TheAlloca->getName() + "." + Twine(i),
394 InsertPt);
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000395 I2->setName(I->getName() + "." + Twine(i));
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000396 new StoreInst(&*I2++, Idx, InsertPt);
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000397 }
Duncan Sands1ea0d2e2008-09-07 09:54:09 +0000398
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000399 // Anything that used the arg should now use the alloca.
400 I->replaceAllUsesWith(TheAlloca);
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000401 TheAlloca->takeName(&*I);
Rafael Espindola2a05ea52014-01-23 17:19:42 +0000402
403 // If the alloca is used in a call, we must clear the tail flag since
404 // the callee now uses an alloca from the caller.
Chandler Carruthcdf47882014-03-09 03:16:01 +0000405 for (User *U : TheAlloca->users()) {
406 CallInst *Call = dyn_cast<CallInst>(U);
Rafael Espindola2a05ea52014-01-23 17:19:42 +0000407 if (!Call)
408 continue;
409 Call->setTailCall(false);
410 }
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000411 continue;
Duncan Sands1ea0d2e2008-09-07 09:54:09 +0000412 }
413
Chandler Carrutha1032a02015-07-22 09:49:59 +0000414 if (I->use_empty())
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000415 continue;
Duncan Sands1ea0d2e2008-09-07 09:54:09 +0000416
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000417 // Otherwise, if we promoted this argument, then all users are load
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000418 // instructions (or GEPs with only load users), and all loads should be
419 // using the new argument that we added.
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000420 ScalarizeTable &ArgIndices = ScalarizedElements[&*I];
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000421
422 while (!I->use_empty()) {
Chandler Carruthcdf47882014-03-09 03:16:01 +0000423 if (LoadInst *LI = dyn_cast<LoadInst>(I->user_back())) {
David Blaikie76826632015-03-14 21:11:26 +0000424 assert(ArgIndices.begin()->second.empty() &&
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000425 "Load element should sort to front!");
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000426 I2->setName(I->getName() + ".val");
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000427 LI->replaceAllUsesWith(&*I2);
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000428 LI->eraseFromParent();
David Greenecf0addf2010-01-05 01:28:37 +0000429 DEBUG(dbgs() << "*** Promoted load of argument '" << I->getName()
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000430 << "' in function '" << F->getName() << "'\n");
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000431 } else {
Chandler Carruthcdf47882014-03-09 03:16:01 +0000432 GetElementPtrInst *GEP = cast<GetElementPtrInst>(I->user_back());
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000433 IndicesVector Operands;
434 Operands.reserve(GEP->getNumIndices());
435 for (User::op_iterator II = GEP->idx_begin(), IE = GEP->idx_end();
436 II != IE; ++II)
437 Operands.push_back(cast<ConstantInt>(*II)->getSExtValue());
438
439 // GEPs with a single 0 index can be merged with direct loads
440 if (Operands.size() == 1 && Operands.front() == 0)
441 Operands.clear();
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000442
443 Function::arg_iterator TheArg = I2;
444 for (ScalarizeTable::iterator It = ArgIndices.begin();
David Blaikie76826632015-03-14 21:11:26 +0000445 It->second != Operands; ++It, ++TheArg) {
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000446 assert(It != ArgIndices.end() && "GEP not handled??");
447 }
448
449 std::string NewName = I->getName();
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000450 for (unsigned i = 0, e = Operands.size(); i != e; ++i) {
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000451 NewName += "." + utostr(Operands[i]);
Matthijs Kooijmanfd307042008-07-29 10:00:13 +0000452 }
453 NewName += ".val";
454 TheArg->setName(NewName);
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000455
David Greenecf0addf2010-01-05 01:28:37 +0000456 DEBUG(dbgs() << "*** Promoted agg argument '" << TheArg->getName()
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000457 << "' of function '" << NF->getName() << "'\n");
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000458
459 // All of the uses must be load instructions. Replace them all with
460 // the argument specified by ArgNo.
461 while (!GEP->use_empty()) {
Chandler Carruthcdf47882014-03-09 03:16:01 +0000462 LoadInst *L = cast<LoadInst>(GEP->user_back());
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000463 L->replaceAllUsesWith(&*TheArg);
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000464 L->eraseFromParent();
465 }
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000466 GEP->eraseFromParent();
467 }
468 }
469
470 // Increment I2 past all of the arguments added for this promoted pointer.
Benjamin Kramer84036252012-09-30 17:31:56 +0000471 std::advance(I2, ArgIndices.size());
Chris Lattnerb5bd9242008-01-11 22:31:41 +0000472 }
Chris Lattner483ae012004-03-07 21:29:54 +0000473
Chandler Carruthaddcda42017-02-09 23:46:27 +0000474 return NF;
Chris Lattner483ae012004-03-07 21:29:54 +0000475}
David Blaikiee844cd52014-07-01 21:13:37 +0000476
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000477/// AllCallersPassInValidPointerForArgument - Return true if we can prove that
478/// all callees pass in a valid pointer for the specified function argument.
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000479static bool allCallersPassInValidPointerForArgument(Argument *Arg) {
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000480 Function *Callee = Arg->getParent();
481 const DataLayout &DL = Callee->getParent()->getDataLayout();
482
483 unsigned ArgNo = Arg->getArgNo();
484
485 // Look at all call sites of the function. At this point we know we only have
486 // direct callees.
487 for (User *U : Callee->users()) {
488 CallSite CS(U);
489 assert(CS && "Should only have direct calls!");
490
491 if (!isDereferenceablePointer(CS.getArgument(ArgNo), DL))
492 return false;
493 }
494 return true;
495}
496
497/// Returns true if Prefix is a prefix of longer. That means, Longer has a size
498/// that is greater than or equal to the size of prefix, and each of the
499/// elements in Prefix is the same as the corresponding elements in Longer.
500///
501/// This means it also returns true when Prefix and Longer are equal!
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000502static bool isPrefix(const IndicesVector &Prefix, const IndicesVector &Longer) {
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000503 if (Prefix.size() > Longer.size())
504 return false;
505 return std::equal(Prefix.begin(), Prefix.end(), Longer.begin());
506}
507
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000508/// Checks if Indices, or a prefix of Indices, is in Set.
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000509static bool prefixIn(const IndicesVector &Indices,
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000510 std::set<IndicesVector> &Set) {
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000511 std::set<IndicesVector>::iterator Low;
512 Low = Set.upper_bound(Indices);
513 if (Low != Set.begin())
514 Low--;
515 // Low is now the last element smaller than or equal to Indices. This means
516 // it points to a prefix of Indices (possibly Indices itself), if such
517 // prefix exists.
518 //
519 // This load is safe if any prefix of its operands is safe to load.
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000520 return Low != Set.end() && isPrefix(*Low, Indices);
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000521}
522
523/// Mark the given indices (ToMark) as safe in the given set of indices
524/// (Safe). Marking safe usually means adding ToMark to Safe. However, if there
525/// is already a prefix of Indices in Safe, Indices are implicitely marked safe
526/// already. Furthermore, any indices that Indices is itself a prefix of, are
527/// removed from Safe (since they are implicitely safe because of Indices now).
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000528static void markIndicesSafe(const IndicesVector &ToMark,
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000529 std::set<IndicesVector> &Safe) {
530 std::set<IndicesVector>::iterator Low;
531 Low = Safe.upper_bound(ToMark);
532 // Guard against the case where Safe is empty
533 if (Low != Safe.begin())
534 Low--;
535 // Low is now the last element smaller than or equal to Indices. This
536 // means it points to a prefix of Indices (possibly Indices itself), if
537 // such prefix exists.
538 if (Low != Safe.end()) {
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000539 if (isPrefix(*Low, ToMark))
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000540 // If there is already a prefix of these indices (or exactly these
541 // indices) marked a safe, don't bother adding these indices
542 return;
543
544 // Increment Low, so we can use it as a "insert before" hint
545 ++Low;
546 }
547 // Insert
548 Low = Safe.insert(Low, ToMark);
549 ++Low;
550 // If there we're a prefix of longer index list(s), remove those
551 std::set<IndicesVector>::iterator End = Safe.end();
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000552 while (Low != End && isPrefix(ToMark, *Low)) {
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000553 std::set<IndicesVector>::iterator Remove = Low;
554 ++Low;
555 Safe.erase(Remove);
556 }
557}
558
559/// isSafeToPromoteArgument - As you might guess from the name of this method,
560/// it checks to see if it is both safe and useful to promote the argument.
561/// This method limits promotion of aggregates to only promote up to three
562/// elements of the aggregate in order to avoid exploding the number of
563/// arguments passed in.
564static bool isSafeToPromoteArgument(Argument *Arg, bool isByValOrInAlloca,
565 AAResults &AAR, unsigned MaxElements) {
Eugene Zelenkof27d1612017-10-19 21:21:30 +0000566 using GEPIndicesSet = std::set<IndicesVector>;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000567
568 // Quick exit for unused arguments
569 if (Arg->use_empty())
570 return true;
571
572 // We can only promote this argument if all of the uses are loads, or are GEP
573 // instructions (with constant indices) that are subsequently loaded.
574 //
575 // Promoting the argument causes it to be loaded in the caller
576 // unconditionally. This is only safe if we can prove that either the load
577 // would have happened in the callee anyway (ie, there is a load in the entry
578 // block) or the pointer passed in at every call site is guaranteed to be
579 // valid.
580 // In the former case, invalid loads can happen, but would have happened
581 // anyway, in the latter case, invalid loads won't happen. This prevents us
582 // from introducing an invalid load that wouldn't have happened in the
583 // original code.
584 //
585 // This set will contain all sets of indices that are loaded in the entry
586 // block, and thus are safe to unconditionally load in the caller.
587 //
588 // This optimization is also safe for InAlloca parameters, because it verifies
589 // that the address isn't captured.
590 GEPIndicesSet SafeToUnconditionallyLoad;
591
592 // This set contains all the sets of indices that we are planning to promote.
593 // This makes it possible to limit the number of arguments added.
594 GEPIndicesSet ToPromote;
595
596 // If the pointer is always valid, any load with first index 0 is valid.
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000597 if (isByValOrInAlloca || allCallersPassInValidPointerForArgument(Arg))
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000598 SafeToUnconditionallyLoad.insert(IndicesVector(1, 0));
599
600 // First, iterate the entry block and mark loads of (geps of) arguments as
601 // safe.
602 BasicBlock &EntryBlock = Arg->getParent()->front();
603 // Declare this here so we can reuse it
604 IndicesVector Indices;
605 for (Instruction &I : EntryBlock)
606 if (LoadInst *LI = dyn_cast<LoadInst>(&I)) {
607 Value *V = LI->getPointerOperand();
608 if (GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(V)) {
609 V = GEP->getPointerOperand();
610 if (V == Arg) {
611 // This load actually loads (part of) Arg? Check the indices then.
612 Indices.reserve(GEP->getNumIndices());
613 for (User::op_iterator II = GEP->idx_begin(), IE = GEP->idx_end();
614 II != IE; ++II)
615 if (ConstantInt *CI = dyn_cast<ConstantInt>(*II))
616 Indices.push_back(CI->getSExtValue());
617 else
618 // We found a non-constant GEP index for this argument? Bail out
619 // right away, can't promote this argument at all.
620 return false;
621
622 // Indices checked out, mark them as safe
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000623 markIndicesSafe(Indices, SafeToUnconditionallyLoad);
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000624 Indices.clear();
625 }
626 } else if (V == Arg) {
627 // Direct loads are equivalent to a GEP with a single 0 index.
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000628 markIndicesSafe(IndicesVector(1, 0), SafeToUnconditionallyLoad);
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000629 }
630 }
631
632 // Now, iterate all uses of the argument to see if there are any uses that are
633 // not (GEP+)loads, or any (GEP+)loads that are not safe to promote.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000634 SmallVector<LoadInst *, 16> Loads;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000635 IndicesVector Operands;
636 for (Use &U : Arg->uses()) {
637 User *UR = U.getUser();
638 Operands.clear();
639 if (LoadInst *LI = dyn_cast<LoadInst>(UR)) {
640 // Don't hack volatile/atomic loads
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000641 if (!LI->isSimple())
642 return false;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000643 Loads.push_back(LI);
644 // Direct loads are equivalent to a GEP with a zero index and then a load.
645 Operands.push_back(0);
646 } else if (GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(UR)) {
647 if (GEP->use_empty()) {
648 // Dead GEP's cause trouble later. Just remove them if we run into
649 // them.
650 GEP->eraseFromParent();
651 // TODO: This runs the above loop over and over again for dead GEPs
652 // Couldn't we just do increment the UI iterator earlier and erase the
653 // use?
654 return isSafeToPromoteArgument(Arg, isByValOrInAlloca, AAR,
655 MaxElements);
656 }
657
658 // Ensure that all of the indices are constants.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000659 for (User::op_iterator i = GEP->idx_begin(), e = GEP->idx_end(); i != e;
660 ++i)
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000661 if (ConstantInt *C = dyn_cast<ConstantInt>(*i))
662 Operands.push_back(C->getSExtValue());
663 else
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000664 return false; // Not a constant operand GEP!
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000665
666 // Ensure that the only users of the GEP are load instructions.
667 for (User *GEPU : GEP->users())
668 if (LoadInst *LI = dyn_cast<LoadInst>(GEPU)) {
669 // Don't hack volatile/atomic loads
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000670 if (!LI->isSimple())
671 return false;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000672 Loads.push_back(LI);
673 } else {
674 // Other uses than load?
675 return false;
676 }
677 } else {
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000678 return false; // Not a load or a GEP.
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000679 }
680
681 // Now, see if it is safe to promote this load / loads of this GEP. Loading
682 // is safe if Operands, or a prefix of Operands, is marked as safe.
Chandler Carruth8e9c0a82017-01-29 08:03:21 +0000683 if (!prefixIn(Operands, SafeToUnconditionallyLoad))
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000684 return false;
685
686 // See if we are already promoting a load with these indices. If not, check
687 // to make sure that we aren't promoting too many elements. If so, nothing
688 // to do.
689 if (ToPromote.find(Operands) == ToPromote.end()) {
690 if (MaxElements > 0 && ToPromote.size() == MaxElements) {
691 DEBUG(dbgs() << "argpromotion not promoting argument '"
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000692 << Arg->getName()
693 << "' because it would require adding more "
694 << "than " << MaxElements
695 << " arguments to the function.\n");
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000696 // We limit aggregate promotion to only promoting up to a fixed number
697 // of elements of the aggregate.
698 return false;
699 }
700 ToPromote.insert(std::move(Operands));
701 }
702 }
703
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000704 if (Loads.empty())
705 return true; // No users, this is a dead argument.
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000706
707 // Okay, now we know that the argument is only used by load instructions and
708 // it is safe to unconditionally perform all of them. Use alias analysis to
709 // check to see if the pointer is guaranteed to not be modified from entry of
710 // the function to each of the load instructions.
711
712 // Because there could be several/many load instructions, remember which
713 // blocks we know to be transparent to the load.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000714 df_iterator_default_set<BasicBlock *, 16> TranspBlocks;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000715
716 for (LoadInst *Load : Loads) {
717 // Check to see if the load is invalidated from the start of the block to
718 // the load itself.
719 BasicBlock *BB = Load->getParent();
720
721 MemoryLocation Loc = MemoryLocation::get(Load);
Alina Sbirlea193429f2017-12-07 22:41:34 +0000722 if (AAR.canInstructionRangeModRef(BB->front(), *Load, Loc, ModRefInfo::Mod))
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000723 return false; // Pointer is invalidated!
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000724
725 // Now check every path from the entry block to the load for transparency.
726 // To do this, we perform a depth first search on the inverse CFG from the
727 // loading block.
728 for (BasicBlock *P : predecessors(BB)) {
729 for (BasicBlock *TranspBB : inverse_depth_first_ext(P, TranspBlocks))
730 if (AAR.canBasicBlockModify(*TranspBB, Loc))
731 return false;
732 }
733 }
734
735 // If the path from the entry of the function to each load is free of
736 // instructions that potentially invalidate the load, we can make the
737 // transformation!
738 return true;
739}
740
Adrian Prantl5f8f34e42018-05-01 15:54:18 +0000741/// Checks if a type could have padding bytes.
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000742static bool isDenselyPacked(Type *type, const DataLayout &DL) {
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000743 // There is no size information, so be conservative.
744 if (!type->isSized())
745 return false;
746
747 // If the alloc size is not equal to the storage size, then there are padding
748 // bytes. For x86_fp80 on x86-64, size: 80 alloc size: 128.
749 if (DL.getTypeSizeInBits(type) != DL.getTypeAllocSizeInBits(type))
750 return false;
751
752 if (!isa<CompositeType>(type))
753 return true;
754
755 // For homogenous sequential types, check for padding within members.
756 if (SequentialType *seqTy = dyn_cast<SequentialType>(type))
757 return isDenselyPacked(seqTy->getElementType(), DL);
758
759 // Check for padding within and between elements of a struct.
760 StructType *StructTy = cast<StructType>(type);
761 const StructLayout *Layout = DL.getStructLayout(StructTy);
762 uint64_t StartPos = 0;
763 for (unsigned i = 0, E = StructTy->getNumElements(); i < E; ++i) {
764 Type *ElTy = StructTy->getElementType(i);
765 if (!isDenselyPacked(ElTy, DL))
766 return false;
767 if (StartPos != Layout->getElementOffsetInBits(i))
768 return false;
769 StartPos += DL.getTypeAllocSizeInBits(ElTy);
770 }
771
772 return true;
773}
774
Adrian Prantl5f8f34e42018-05-01 15:54:18 +0000775/// Checks if the padding bytes of an argument could be accessed.
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000776static bool canPaddingBeAccessed(Argument *arg) {
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000777 assert(arg->hasByValAttr());
778
779 // Track all the pointers to the argument to make sure they are not captured.
780 SmallPtrSet<Value *, 16> PtrValues;
781 PtrValues.insert(arg);
782
783 // Track all of the stores.
784 SmallVector<StoreInst *, 16> Stores;
785
786 // Scan through the uses recursively to make sure the pointer is always used
787 // sanely.
788 SmallVector<Value *, 16> WorkList;
789 WorkList.insert(WorkList.end(), arg->user_begin(), arg->user_end());
790 while (!WorkList.empty()) {
791 Value *V = WorkList.back();
792 WorkList.pop_back();
793 if (isa<GetElementPtrInst>(V) || isa<PHINode>(V)) {
794 if (PtrValues.insert(V).second)
795 WorkList.insert(WorkList.end(), V->user_begin(), V->user_end());
796 } else if (StoreInst *Store = dyn_cast<StoreInst>(V)) {
797 Stores.push_back(Store);
798 } else if (!isa<LoadInst>(V)) {
799 return true;
800 }
801 }
802
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000803 // Check to make sure the pointers aren't captured
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000804 for (StoreInst *Store : Stores)
805 if (PtrValues.count(Store->getValueOperand()))
806 return true;
807
808 return false;
809}
810
811/// PromoteArguments - This method checks the specified function to see if there
812/// are any promotable arguments and if it is safe to promote the function (for
813/// example, all callers are direct). If safe to promote some arguments, it
814/// calls the DoPromotion method.
Chandler Carruthaddcda42017-02-09 23:46:27 +0000815static Function *
816promoteArguments(Function *F, function_ref<AAResults &(Function &F)> AARGetter,
817 unsigned MaxElements,
818 Optional<function_ref<void(CallSite OldCS, CallSite NewCS)>>
819 ReplaceCallSite) {
Luke Cheeseman6c1e6bb2018-02-22 14:42:08 +0000820 // Don't perform argument promotion for naked functions; otherwise we can end
821 // up removing parameters that are seemingly 'not used' as they are referred
822 // to in the assembly.
823 if(F->hasFnAttribute(Attribute::Naked))
824 return nullptr;
825
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000826 // Make sure that it is local to this module.
Chandler Carruthaddcda42017-02-09 23:46:27 +0000827 if (!F->hasLocalLinkage())
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000828 return nullptr;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000829
830 // Don't promote arguments for variadic functions. Adding, removing, or
831 // changing non-pack parameters can change the classification of pack
832 // parameters. Frontends encode that classification at the call site in the
833 // IR, while in the callee the classification is determined dynamically based
834 // on the number of registers consumed so far.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000835 if (F->isVarArg())
836 return nullptr;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000837
838 // First check: see if there are any pointer arguments! If not, quick exit.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000839 SmallVector<Argument *, 16> PointerArgs;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000840 for (Argument &I : F->args())
841 if (I.getType()->isPointerTy())
842 PointerArgs.push_back(&I);
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000843 if (PointerArgs.empty())
844 return nullptr;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000845
846 // Second check: make sure that all callers are direct callers. We can't
847 // transform functions that have indirect callers. Also see if the function
848 // is self-recursive.
849 bool isSelfRecursive = false;
850 for (Use &U : F->uses()) {
851 CallSite CS(U.getUser());
852 // Must be a direct call.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000853 if (CS.getInstruction() == nullptr || !CS.isCallee(&U))
854 return nullptr;
855
Fedor Indutny1571b122018-03-02 00:59:27 +0000856 // Can't change signature of musttail callee
857 if (CS.isMustTailCall())
858 return nullptr;
859
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000860 if (CS.getInstruction()->getParent()->getParent() == F)
861 isSelfRecursive = true;
862 }
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000863
Fedor Indutny1571b122018-03-02 00:59:27 +0000864 // Can't change signature of musttail caller
865 // FIXME: Support promoting whole chain of musttail functions
866 for (BasicBlock &BB : *F)
867 if (BB.getTerminatingMustTailCall())
868 return nullptr;
869
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000870 const DataLayout &DL = F->getParent()->getDataLayout();
871
872 AAResults &AAR = AARGetter(*F);
873
874 // Check to see which arguments are promotable. If an argument is promotable,
875 // add it to ArgsToPromote.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000876 SmallPtrSet<Argument *, 8> ArgsToPromote;
877 SmallPtrSet<Argument *, 8> ByValArgsToTransform;
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000878 for (Argument *PtrArg : PointerArgs) {
879 Type *AgTy = cast<PointerType>(PtrArg->getType())->getElementType();
880
881 // Replace sret attribute with noalias. This reduces register pressure by
882 // avoiding a register copy.
883 if (PtrArg->hasStructRetAttr()) {
884 unsigned ArgNo = PtrArg->getArgNo();
Reid Klecknera0b45f42017-05-03 18:17:31 +0000885 F->removeParamAttr(ArgNo, Attribute::StructRet);
886 F->addParamAttr(ArgNo, Attribute::NoAlias);
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000887 for (Use &U : F->uses()) {
888 CallSite CS(U.getUser());
Reid Klecknera0b45f42017-05-03 18:17:31 +0000889 CS.removeParamAttr(ArgNo, Attribute::StructRet);
890 CS.addParamAttr(ArgNo, Attribute::NoAlias);
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000891 }
892 }
893
894 // If this is a byval argument, and if the aggregate type is small, just
895 // pass the elements, which is always safe, if the passed value is densely
896 // packed or if we can prove the padding bytes are never accessed. This does
897 // not apply to inalloca.
898 bool isSafeToPromote =
899 PtrArg->hasByValAttr() &&
900 (isDenselyPacked(AgTy, DL) || !canPaddingBeAccessed(PtrArg));
901 if (isSafeToPromote) {
902 if (StructType *STy = dyn_cast<StructType>(AgTy)) {
903 if (MaxElements > 0 && STy->getNumElements() > MaxElements) {
904 DEBUG(dbgs() << "argpromotion disable promoting argument '"
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000905 << PtrArg->getName()
906 << "' because it would require adding more"
907 << " than " << MaxElements
908 << " arguments to the function.\n");
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000909 continue;
910 }
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000911
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000912 // If all the elements are single-value types, we can promote it.
913 bool AllSimple = true;
914 for (const auto *EltTy : STy->elements()) {
915 if (!EltTy->isSingleValueType()) {
916 AllSimple = false;
917 break;
918 }
919 }
920
921 // Safe to transform, don't even bother trying to "promote" it.
922 // Passing the elements as a scalar will allow sroa to hack on
923 // the new alloca we introduce.
924 if (AllSimple) {
925 ByValArgsToTransform.insert(PtrArg);
926 continue;
927 }
928 }
929 }
930
931 // If the argument is a recursive type and we're in a recursive
932 // function, we could end up infinitely peeling the function argument.
933 if (isSelfRecursive) {
934 if (StructType *STy = dyn_cast<StructType>(AgTy)) {
935 bool RecursiveType = false;
936 for (const auto *EltTy : STy->elements()) {
937 if (EltTy == PtrArg->getType()) {
938 RecursiveType = true;
939 break;
940 }
941 }
942 if (RecursiveType)
943 continue;
944 }
945 }
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000946
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000947 // Otherwise, see if we can promote the pointer to its value.
948 if (isSafeToPromoteArgument(PtrArg, PtrArg->hasByValOrInAllocaAttr(), AAR,
949 MaxElements))
950 ArgsToPromote.insert(PtrArg);
951 }
952
953 // No promotable pointer arguments.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +0000954 if (ArgsToPromote.empty() && ByValArgsToTransform.empty())
Chandler Carruthcd836cd2017-01-29 08:03:16 +0000955 return nullptr;
956
Chandler Carruthaddcda42017-02-09 23:46:27 +0000957 return doPromotion(F, ArgsToPromote, ByValArgsToTransform, ReplaceCallSite);
958}
959
960PreservedAnalyses ArgumentPromotionPass::run(LazyCallGraph::SCC &C,
961 CGSCCAnalysisManager &AM,
962 LazyCallGraph &CG,
963 CGSCCUpdateResult &UR) {
964 bool Changed = false, LocalChange;
965
966 // Iterate until we stop promoting from this SCC.
967 do {
968 LocalChange = false;
969
970 for (LazyCallGraph::Node &N : C) {
971 Function &OldF = N.getFunction();
972
973 FunctionAnalysisManager &FAM =
974 AM.getResult<FunctionAnalysisManagerCGSCCProxy>(C, CG).getManager();
975 // FIXME: This lambda must only be used with this function. We should
976 // skip the lambda and just get the AA results directly.
977 auto AARGetter = [&](Function &F) -> AAResults & {
978 assert(&F == &OldF && "Called with an unexpected function!");
979 return FAM.getResult<AAManager>(F);
980 };
981
Justin Bogner92fe5632018-01-08 21:13:35 +0000982 Function *NewF = promoteArguments(&OldF, AARGetter, MaxElements, None);
Chandler Carruthaddcda42017-02-09 23:46:27 +0000983 if (!NewF)
984 continue;
985 LocalChange = true;
986
987 // Directly substitute the functions in the call graph. Note that this
988 // requires the old function to be completely dead and completely
989 // replaced by the new function. It does no call graph updates, it merely
990 // swaps out the particular function mapped to a particular node in the
991 // graph.
992 C.getOuterRefSCC().replaceNodeFunction(N, *NewF);
993 OldF.eraseFromParent();
994 }
995
996 Changed |= LocalChange;
997 } while (LocalChange);
998
999 if (!Changed)
1000 return PreservedAnalyses::all();
1001
1002 return PreservedAnalyses::none();
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001003}
1004
1005namespace {
Eugene Zelenkof27d1612017-10-19 21:21:30 +00001006
Chandler Carruthae9ce3d2017-01-29 08:03:19 +00001007/// ArgPromotion - The 'by reference' to 'by value' argument promotion pass.
Chandler Carruthae9ce3d2017-01-29 08:03:19 +00001008struct ArgPromotion : public CallGraphSCCPass {
Eugene Zelenkof27d1612017-10-19 21:21:30 +00001009 // Pass identification, replacement for typeid
1010 static char ID;
1011
1012 explicit ArgPromotion(unsigned MaxElements = 3)
1013 : CallGraphSCCPass(ID), MaxElements(MaxElements) {
1014 initializeArgPromotionPass(*PassRegistry::getPassRegistry());
1015 }
1016
Chandler Carruthae9ce3d2017-01-29 08:03:19 +00001017 void getAnalysisUsage(AnalysisUsage &AU) const override {
1018 AU.addRequired<AssumptionCacheTracker>();
1019 AU.addRequired<TargetLibraryInfoWrapperPass>();
1020 getAAResultsAnalysisUsage(AU);
1021 CallGraphSCCPass::getAnalysisUsage(AU);
1022 }
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001023
Chandler Carruthae9ce3d2017-01-29 08:03:19 +00001024 bool runOnSCC(CallGraphSCC &SCC) override;
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001025
Chandler Carruthae9ce3d2017-01-29 08:03:19 +00001026private:
1027 using llvm::Pass::doInitialization;
Eugene Zelenkof27d1612017-10-19 21:21:30 +00001028
Chandler Carruthae9ce3d2017-01-29 08:03:19 +00001029 bool doInitialization(CallGraph &CG) override;
Eugene Zelenkof27d1612017-10-19 21:21:30 +00001030
Chandler Carruthae9ce3d2017-01-29 08:03:19 +00001031 /// The maximum number of elements to expand, or 0 for unlimited.
1032 unsigned MaxElements;
1033};
Eugene Zelenkof27d1612017-10-19 21:21:30 +00001034
1035} // end anonymous namespace
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001036
1037char ArgPromotion::ID = 0;
Eugene Zelenkof27d1612017-10-19 21:21:30 +00001038
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001039INITIALIZE_PASS_BEGIN(ArgPromotion, "argpromotion",
Chandler Carruthae9ce3d2017-01-29 08:03:19 +00001040 "Promote 'by reference' arguments to scalars", false,
1041 false)
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001042INITIALIZE_PASS_DEPENDENCY(AssumptionCacheTracker)
1043INITIALIZE_PASS_DEPENDENCY(CallGraphWrapperPass)
1044INITIALIZE_PASS_DEPENDENCY(TargetLibraryInfoWrapperPass)
1045INITIALIZE_PASS_END(ArgPromotion, "argpromotion",
Chandler Carruthae9ce3d2017-01-29 08:03:19 +00001046 "Promote 'by reference' arguments to scalars", false, false)
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001047
1048Pass *llvm::createArgumentPromotionPass(unsigned MaxElements) {
1049 return new ArgPromotion(MaxElements);
1050}
1051
1052bool ArgPromotion::runOnSCC(CallGraphSCC &SCC) {
1053 if (skipSCC(SCC))
1054 return false;
1055
1056 // Get the callgraph information that we need to update to reflect our
1057 // changes.
1058 CallGraph &CG = getAnalysis<CallGraphWrapperPass>().getCallGraph();
1059
Peter Collingbournecea1e4e2017-02-09 23:11:52 +00001060 LegacyAARGetter AARGetter(*this);
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001061
1062 bool Changed = false, LocalChange;
1063
1064 // Iterate until we stop promoting from this SCC.
1065 do {
1066 LocalChange = false;
1067 // Attempt to promote arguments from all functions in this SCC.
1068 for (CallGraphNode *OldNode : SCC) {
Chandler Carruthaddcda42017-02-09 23:46:27 +00001069 Function *OldF = OldNode->getFunction();
1070 if (!OldF)
1071 continue;
1072
1073 auto ReplaceCallSite = [&](CallSite OldCS, CallSite NewCS) {
1074 Function *Caller = OldCS.getInstruction()->getParent()->getParent();
1075 CallGraphNode *NewCalleeNode =
1076 CG.getOrInsertFunction(NewCS.getCalledFunction());
1077 CallGraphNode *CallerNode = CG[Caller];
1078 CallerNode->replaceCallEdge(OldCS, NewCS, NewCalleeNode);
1079 };
1080
1081 if (Function *NewF = promoteArguments(OldF, AARGetter, MaxElements,
1082 {ReplaceCallSite})) {
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001083 LocalChange = true;
Chandler Carruthaddcda42017-02-09 23:46:27 +00001084
1085 // Update the call graph for the newly promoted function.
1086 CallGraphNode *NewNode = CG.getOrInsertFunction(NewF);
1087 NewNode->stealCalledFunctionsFrom(OldNode);
1088 if (OldNode->getNumReferences() == 0)
1089 delete CG.removeFunctionFromModule(OldNode);
1090 else
1091 OldF->setLinkage(Function::ExternalLinkage);
1092
1093 // And updat ethe SCC we're iterating as well.
Chandler Carruthcd836cd2017-01-29 08:03:16 +00001094 SCC.ReplaceNode(OldNode, NewNode);
1095 }
1096 }
1097 // Remember that we changed something.
1098 Changed |= LocalChange;
1099 } while (LocalChange);
1100
1101 return Changed;
1102}
1103
David Blaikiee844cd52014-07-01 21:13:37 +00001104bool ArgPromotion::doInitialization(CallGraph &CG) {
David Blaikiee844cd52014-07-01 21:13:37 +00001105 return CallGraphSCCPass::doInitialization(CG);
1106}