blob: ce648e2ea518f67fd55a22ac0a4cfbc7c7706480 [file] [log] [blame]
Meador Inge6b6a1612013-03-21 00:55:59 +00001//===- FunctionAttrs.cpp - Pass which marks functions attributes ----------===//
Duncan Sands44c8cd92008-12-31 16:14:43 +00002//
3// The LLVM Compiler Infrastructure
4//
5// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
Chandler Carruth1926b702016-01-08 10:55:52 +00009///
10/// \file
11/// This file implements interprocedural passes which walk the
12/// call-graph deducing and/or propagating function attributes.
13///
Duncan Sands44c8cd92008-12-31 16:14:43 +000014//===----------------------------------------------------------------------===//
15
Chandler Carruth9c4ed172016-02-18 11:03:11 +000016#include "llvm/Transforms/IPO/FunctionAttrs.h"
Duncan Sands44c8cd92008-12-31 16:14:43 +000017#include "llvm/Transforms/IPO.h"
Nick Lewycky4c378a42011-12-28 23:24:21 +000018#include "llvm/ADT/SCCIterator.h"
Benjamin Kramer15591272012-10-31 13:45:49 +000019#include "llvm/ADT/SetVector.h"
Duncan Sandsb193a372009-01-02 11:54:37 +000020#include "llvm/ADT/SmallSet.h"
Duncan Sands44c8cd92008-12-31 16:14:43 +000021#include "llvm/ADT/Statistic.h"
James Molloy0ecdbe72015-11-19 08:49:57 +000022#include "llvm/ADT/StringSwitch.h"
Chandler Carruthed0881b2012-12-03 16:50:05 +000023#include "llvm/Analysis/AliasAnalysis.h"
Chandler Carruth7b560d42015-09-09 17:55:00 +000024#include "llvm/Analysis/AssumptionCache.h"
25#include "llvm/Analysis/BasicAliasAnalysis.h"
Chandler Carruthed0881b2012-12-03 16:50:05 +000026#include "llvm/Analysis/CallGraph.h"
Chandler Carruth839a98e2013-01-07 15:26:48 +000027#include "llvm/Analysis/CallGraphSCCPass.h"
Chandler Carruthed0881b2012-12-03 16:50:05 +000028#include "llvm/Analysis/CaptureTracking.h"
Chandler Carruth7b560d42015-09-09 17:55:00 +000029#include "llvm/Analysis/TargetLibraryInfo.h"
Philip Reamesa88caea2015-08-31 19:44:38 +000030#include "llvm/Analysis/ValueTracking.h"
Chandler Carruth9fb823b2013-01-02 11:36:10 +000031#include "llvm/IR/GlobalVariable.h"
Chandler Carruth83948572014-03-04 10:30:26 +000032#include "llvm/IR/InstIterator.h"
Chandler Carruth9fb823b2013-01-02 11:36:10 +000033#include "llvm/IR/IntrinsicInst.h"
34#include "llvm/IR/LLVMContext.h"
Philip Reamesa88caea2015-08-31 19:44:38 +000035#include "llvm/Support/Debug.h"
Hans Wennborg043bf5b2015-08-31 21:19:18 +000036#include "llvm/Support/raw_ostream.h"
Chandler Carruth62d42152015-01-15 02:16:27 +000037#include "llvm/Analysis/TargetLibraryInfo.h"
Duncan Sands44c8cd92008-12-31 16:14:43 +000038using namespace llvm;
39
Chandler Carruth964daaa2014-04-22 02:55:47 +000040#define DEBUG_TYPE "functionattrs"
41
Duncan Sands44c8cd92008-12-31 16:14:43 +000042STATISTIC(NumReadNone, "Number of functions marked readnone");
43STATISTIC(NumReadOnly, "Number of functions marked readonly");
44STATISTIC(NumNoCapture, "Number of arguments marked nocapture");
David Majnemer5246e0b2016-07-19 18:50:26 +000045STATISTIC(NumReturned, "Number of arguments marked returned");
Nick Lewyckyc2ec0722013-07-06 00:29:58 +000046STATISTIC(NumReadNoneArg, "Number of arguments marked readnone");
47STATISTIC(NumReadOnlyArg, "Number of arguments marked readonly");
Nick Lewyckyfbed86a2009-03-08 06:20:47 +000048STATISTIC(NumNoAlias, "Number of function returns marked noalias");
Philip Reamesa88caea2015-08-31 19:44:38 +000049STATISTIC(NumNonNullReturn, "Number of function returns marked nonnull");
James Molloy7e9bdd52015-11-12 10:55:20 +000050STATISTIC(NumNoRecurse, "Number of functions marked as norecurse");
Duncan Sands44c8cd92008-12-31 16:14:43 +000051
52namespace {
Chandler Carruthc518ebd2015-10-29 18:29:15 +000053typedef SmallSetVector<Function *, 8> SCCNodeSet;
54}
55
56namespace {
Chandler Carruth7542d372015-09-21 17:39:41 +000057/// The three kinds of memory access relevant to 'readonly' and
58/// 'readnone' attributes.
59enum MemoryAccessKind {
60 MAK_ReadNone = 0,
61 MAK_ReadOnly = 1,
62 MAK_MayWrite = 2
63};
64}
65
Chandler Carruthc518ebd2015-10-29 18:29:15 +000066static MemoryAccessKind checkFunctionMemoryAccess(Function &F, AAResults &AAR,
67 const SCCNodeSet &SCCNodes) {
Chandler Carruth7542d372015-09-21 17:39:41 +000068 FunctionModRefBehavior MRB = AAR.getModRefBehavior(&F);
69 if (MRB == FMRB_DoesNotAccessMemory)
70 // Already perfect!
71 return MAK_ReadNone;
72
Sanjoy Das5ce32722016-04-08 00:48:30 +000073 // Non-exact function definitions may not be selected at link time, and an
74 // alternative version that writes to memory may be selected. See the comment
75 // on GlobalValue::isDefinitionExact for more details.
76 if (!F.hasExactDefinition()) {
Chandler Carruth7542d372015-09-21 17:39:41 +000077 if (AliasAnalysis::onlyReadsMemory(MRB))
78 return MAK_ReadOnly;
79
80 // Conservatively assume it writes to memory.
81 return MAK_MayWrite;
82 }
83
84 // Scan the function body for instructions that may read or write memory.
85 bool ReadsMemory = false;
86 for (inst_iterator II = inst_begin(F), E = inst_end(F); II != E; ++II) {
87 Instruction *I = &*II;
88
89 // Some instructions can be ignored even if they read or write memory.
90 // Detect these now, skipping to the next instruction if one is found.
91 CallSite CS(cast<Value>(I));
92 if (CS) {
Sanjoy Das10c8a042016-02-09 18:40:40 +000093 // Ignore calls to functions in the same SCC, as long as the call sites
94 // don't have operand bundles. Calls with operand bundles are allowed to
95 // have memory effects not described by the memory effects of the call
96 // target.
97 if (!CS.hasOperandBundles() && CS.getCalledFunction() &&
98 SCCNodes.count(CS.getCalledFunction()))
Chandler Carruth7542d372015-09-21 17:39:41 +000099 continue;
100 FunctionModRefBehavior MRB = AAR.getModRefBehavior(CS);
Chandler Carruth7542d372015-09-21 17:39:41 +0000101
Chandler Carruth69798fb2015-10-27 01:41:43 +0000102 // If the call doesn't access memory, we're done.
103 if (!(MRB & MRI_ModRef))
104 continue;
105
106 if (!AliasAnalysis::onlyAccessesArgPointees(MRB)) {
107 // The call could access any memory. If that includes writes, give up.
108 if (MRB & MRI_Mod)
109 return MAK_MayWrite;
110 // If it reads, note it.
111 if (MRB & MRI_Ref)
112 ReadsMemory = true;
Chandler Carruth7542d372015-09-21 17:39:41 +0000113 continue;
114 }
Chandler Carruth69798fb2015-10-27 01:41:43 +0000115
116 // Check whether all pointer arguments point to local memory, and
117 // ignore calls that only access local memory.
118 for (CallSite::arg_iterator CI = CS.arg_begin(), CE = CS.arg_end();
119 CI != CE; ++CI) {
120 Value *Arg = *CI;
Elena Demikhovsky3ec9e152015-11-17 19:30:51 +0000121 if (!Arg->getType()->isPtrOrPtrVectorTy())
Chandler Carruth69798fb2015-10-27 01:41:43 +0000122 continue;
123
124 AAMDNodes AAInfo;
125 I->getAAMetadata(AAInfo);
126 MemoryLocation Loc(Arg, MemoryLocation::UnknownSize, AAInfo);
127
128 // Skip accesses to local or constant memory as they don't impact the
129 // externally visible mod/ref behavior.
130 if (AAR.pointsToConstantMemory(Loc, /*OrLocal=*/true))
131 continue;
132
133 if (MRB & MRI_Mod)
134 // Writes non-local memory. Give up.
135 return MAK_MayWrite;
136 if (MRB & MRI_Ref)
137 // Ok, it reads non-local memory.
138 ReadsMemory = true;
139 }
Chandler Carruth7542d372015-09-21 17:39:41 +0000140 continue;
141 } else if (LoadInst *LI = dyn_cast<LoadInst>(I)) {
142 // Ignore non-volatile loads from local memory. (Atomic is okay here.)
143 if (!LI->isVolatile()) {
144 MemoryLocation Loc = MemoryLocation::get(LI);
145 if (AAR.pointsToConstantMemory(Loc, /*OrLocal=*/true))
146 continue;
147 }
148 } else if (StoreInst *SI = dyn_cast<StoreInst>(I)) {
149 // Ignore non-volatile stores to local memory. (Atomic is okay here.)
150 if (!SI->isVolatile()) {
151 MemoryLocation Loc = MemoryLocation::get(SI);
152 if (AAR.pointsToConstantMemory(Loc, /*OrLocal=*/true))
153 continue;
154 }
155 } else if (VAArgInst *VI = dyn_cast<VAArgInst>(I)) {
156 // Ignore vaargs on local memory.
157 MemoryLocation Loc = MemoryLocation::get(VI);
158 if (AAR.pointsToConstantMemory(Loc, /*OrLocal=*/true))
159 continue;
160 }
161
162 // Any remaining instructions need to be taken seriously! Check if they
163 // read or write memory.
164 if (I->mayWriteToMemory())
165 // Writes memory. Just give up.
166 return MAK_MayWrite;
167
168 // If this instruction may read memory, remember that.
169 ReadsMemory |= I->mayReadFromMemory();
170 }
171
172 return ReadsMemory ? MAK_ReadOnly : MAK_ReadNone;
173}
174
Chandler Carrutha632fb92015-09-13 06:57:25 +0000175/// Deduce readonly/readnone attributes for the SCC.
Chandler Carrutha8125352015-10-30 16:48:08 +0000176template <typename AARGetterT>
177static bool addReadAttrs(const SCCNodeSet &SCCNodes, AARGetterT AARGetter) {
Duncan Sands44c8cd92008-12-31 16:14:43 +0000178 // Check if any of the functions in the SCC read or write memory. If they
179 // write memory then they can't be marked readnone or readonly.
180 bool ReadsMemory = false;
Chandler Carruthc518ebd2015-10-29 18:29:15 +0000181 for (Function *F : SCCNodes) {
Chandler Carrutha8125352015-10-30 16:48:08 +0000182 // Call the callable parameter to look up AA results for this function.
183 AAResults &AAR = AARGetter(*F);
Chandler Carruth7b560d42015-09-09 17:55:00 +0000184
Chandler Carruth7542d372015-09-21 17:39:41 +0000185 switch (checkFunctionMemoryAccess(*F, AAR, SCCNodes)) {
186 case MAK_MayWrite:
187 return false;
188 case MAK_ReadOnly:
Duncan Sands44c8cd92008-12-31 16:14:43 +0000189 ReadsMemory = true;
Chandler Carruth7542d372015-09-21 17:39:41 +0000190 break;
191 case MAK_ReadNone:
192 // Nothing to do!
193 break;
Duncan Sands44c8cd92008-12-31 16:14:43 +0000194 }
195 }
196
197 // Success! Functions in this SCC do not access memory, or only read memory.
198 // Give them the appropriate attribute.
199 bool MadeChange = false;
Chandler Carruthc518ebd2015-10-29 18:29:15 +0000200 for (Function *F : SCCNodes) {
Duncan Sands44c8cd92008-12-31 16:14:43 +0000201 if (F->doesNotAccessMemory())
202 // Already perfect!
203 continue;
204
205 if (F->onlyReadsMemory() && ReadsMemory)
206 // No change.
207 continue;
208
209 MadeChange = true;
210
211 // Clear out any existing attributes.
Bill Wendling50d27842012-10-15 20:35:56 +0000212 AttrBuilder B;
Chandler Carruth63559d72015-09-13 06:47:20 +0000213 B.addAttribute(Attribute::ReadOnly).addAttribute(Attribute::ReadNone);
214 F->removeAttributes(
215 AttributeSet::FunctionIndex,
216 AttributeSet::get(F->getContext(), AttributeSet::FunctionIndex, B));
Duncan Sands44c8cd92008-12-31 16:14:43 +0000217
218 // Add in the new attribute.
Bill Wendlinge94d8432012-12-07 23:16:57 +0000219 F->addAttribute(AttributeSet::FunctionIndex,
Bill Wendlingc0e2a1f2013-01-23 00:20:53 +0000220 ReadsMemory ? Attribute::ReadOnly : Attribute::ReadNone);
Duncan Sands44c8cd92008-12-31 16:14:43 +0000221
222 if (ReadsMemory)
Duncan Sandscefc8602009-01-02 11:46:24 +0000223 ++NumReadOnly;
Duncan Sands44c8cd92008-12-31 16:14:43 +0000224 else
Duncan Sandscefc8602009-01-02 11:46:24 +0000225 ++NumReadNone;
Duncan Sands44c8cd92008-12-31 16:14:43 +0000226 }
227
228 return MadeChange;
229}
230
Nick Lewycky4c378a42011-12-28 23:24:21 +0000231namespace {
Chandler Carrutha632fb92015-09-13 06:57:25 +0000232/// For a given pointer Argument, this retains a list of Arguments of functions
233/// in the same SCC that the pointer data flows into. We use this to build an
234/// SCC of the arguments.
Chandler Carruth63559d72015-09-13 06:47:20 +0000235struct ArgumentGraphNode {
236 Argument *Definition;
237 SmallVector<ArgumentGraphNode *, 4> Uses;
238};
Nick Lewycky4c378a42011-12-28 23:24:21 +0000239
Chandler Carruth63559d72015-09-13 06:47:20 +0000240class ArgumentGraph {
241 // We store pointers to ArgumentGraphNode objects, so it's important that
242 // that they not move around upon insert.
243 typedef std::map<Argument *, ArgumentGraphNode> ArgumentMapTy;
Nick Lewycky4c378a42011-12-28 23:24:21 +0000244
Chandler Carruth63559d72015-09-13 06:47:20 +0000245 ArgumentMapTy ArgumentMap;
Nick Lewycky4c378a42011-12-28 23:24:21 +0000246
Chandler Carruth63559d72015-09-13 06:47:20 +0000247 // There is no root node for the argument graph, in fact:
248 // void f(int *x, int *y) { if (...) f(x, y); }
249 // is an example where the graph is disconnected. The SCCIterator requires a
250 // single entry point, so we maintain a fake ("synthetic") root node that
251 // uses every node. Because the graph is directed and nothing points into
252 // the root, it will not participate in any SCCs (except for its own).
253 ArgumentGraphNode SyntheticRoot;
Nick Lewycky4c378a42011-12-28 23:24:21 +0000254
Chandler Carruth63559d72015-09-13 06:47:20 +0000255public:
256 ArgumentGraph() { SyntheticRoot.Definition = nullptr; }
Nick Lewycky4c378a42011-12-28 23:24:21 +0000257
Chandler Carruth63559d72015-09-13 06:47:20 +0000258 typedef SmallVectorImpl<ArgumentGraphNode *>::iterator iterator;
Nick Lewycky4c378a42011-12-28 23:24:21 +0000259
Chandler Carruth63559d72015-09-13 06:47:20 +0000260 iterator begin() { return SyntheticRoot.Uses.begin(); }
261 iterator end() { return SyntheticRoot.Uses.end(); }
262 ArgumentGraphNode *getEntryNode() { return &SyntheticRoot; }
Nick Lewycky4c378a42011-12-28 23:24:21 +0000263
Chandler Carruth63559d72015-09-13 06:47:20 +0000264 ArgumentGraphNode *operator[](Argument *A) {
265 ArgumentGraphNode &Node = ArgumentMap[A];
266 Node.Definition = A;
267 SyntheticRoot.Uses.push_back(&Node);
268 return &Node;
269 }
270};
Nick Lewycky4c378a42011-12-28 23:24:21 +0000271
Chandler Carrutha632fb92015-09-13 06:57:25 +0000272/// This tracker checks whether callees are in the SCC, and if so it does not
273/// consider that a capture, instead adding it to the "Uses" list and
274/// continuing with the analysis.
Chandler Carruth63559d72015-09-13 06:47:20 +0000275struct ArgumentUsesTracker : public CaptureTracker {
Chandler Carruthc518ebd2015-10-29 18:29:15 +0000276 ArgumentUsesTracker(const SCCNodeSet &SCCNodes)
Nick Lewycky4c378a42011-12-28 23:24:21 +0000277 : Captured(false), SCCNodes(SCCNodes) {}
278
Chandler Carruth63559d72015-09-13 06:47:20 +0000279 void tooManyUses() override { Captured = true; }
Nick Lewycky4c378a42011-12-28 23:24:21 +0000280
Chandler Carruth63559d72015-09-13 06:47:20 +0000281 bool captured(const Use *U) override {
282 CallSite CS(U->getUser());
283 if (!CS.getInstruction()) {
284 Captured = true;
285 return true;
Nick Lewycky4c378a42011-12-28 23:24:21 +0000286 }
287
Chandler Carruth63559d72015-09-13 06:47:20 +0000288 Function *F = CS.getCalledFunction();
Sanjoy Das5ce32722016-04-08 00:48:30 +0000289 if (!F || !F->hasExactDefinition() || !SCCNodes.count(F)) {
Chandler Carruth63559d72015-09-13 06:47:20 +0000290 Captured = true;
291 return true;
292 }
Nick Lewycky4c378a42011-12-28 23:24:21 +0000293
Sanjoy Das98bfe262015-11-05 03:04:40 +0000294 // Note: the callee and the two successor blocks *follow* the argument
295 // operands. This means there is no need to adjust UseIndex to account for
296 // these.
297
298 unsigned UseIndex =
299 std::distance(const_cast<const Use *>(CS.arg_begin()), U);
300
Sanjoy Das71fe81f2015-11-07 01:56:00 +0000301 assert(UseIndex < CS.data_operands_size() &&
302 "Indirect function calls should have been filtered above!");
303
304 if (UseIndex >= CS.getNumArgOperands()) {
305 // Data operand, but not a argument operand -- must be a bundle operand
306 assert(CS.hasOperandBundles() && "Must be!");
307
308 // CaptureTracking told us that we're being captured by an operand bundle
309 // use. In this case it does not matter if the callee is within our SCC
310 // or not -- we've been captured in some unknown way, and we have to be
311 // conservative.
312 Captured = true;
313 return true;
314 }
315
Sanjoy Das98bfe262015-11-05 03:04:40 +0000316 if (UseIndex >= F->arg_size()) {
317 assert(F->isVarArg() && "More params than args in non-varargs call");
318 Captured = true;
319 return true;
Chandler Carruth63559d72015-09-13 06:47:20 +0000320 }
Sanjoy Das98bfe262015-11-05 03:04:40 +0000321
Duncan P. N. Exon Smith83c4b682015-11-07 00:01:16 +0000322 Uses.push_back(&*std::next(F->arg_begin(), UseIndex));
Chandler Carruth63559d72015-09-13 06:47:20 +0000323 return false;
324 }
325
326 bool Captured; // True only if certainly captured (used outside our SCC).
327 SmallVector<Argument *, 4> Uses; // Uses within our SCC.
328
Chandler Carruthc518ebd2015-10-29 18:29:15 +0000329 const SCCNodeSet &SCCNodes;
Chandler Carruth63559d72015-09-13 06:47:20 +0000330};
Alexander Kornienkof00654e2015-06-23 09:49:53 +0000331}
Nick Lewycky4c378a42011-12-28 23:24:21 +0000332
333namespace llvm {
Chandler Carruth63559d72015-09-13 06:47:20 +0000334template <> struct GraphTraits<ArgumentGraphNode *> {
335 typedef ArgumentGraphNode NodeType;
Tim Shenb44909e2016-08-01 22:32:20 +0000336 typedef ArgumentGraphNode *NodeRef;
Chandler Carruth63559d72015-09-13 06:47:20 +0000337 typedef SmallVectorImpl<ArgumentGraphNode *>::iterator ChildIteratorType;
Nick Lewycky4c378a42011-12-28 23:24:21 +0000338
Chandler Carruth63559d72015-09-13 06:47:20 +0000339 static inline NodeType *getEntryNode(NodeType *A) { return A; }
340 static inline ChildIteratorType child_begin(NodeType *N) {
341 return N->Uses.begin();
342 }
343 static inline ChildIteratorType child_end(NodeType *N) {
344 return N->Uses.end();
345 }
346};
347template <>
348struct GraphTraits<ArgumentGraph *> : public GraphTraits<ArgumentGraphNode *> {
349 static NodeType *getEntryNode(ArgumentGraph *AG) {
350 return AG->getEntryNode();
351 }
352 static ChildIteratorType nodes_begin(ArgumentGraph *AG) {
353 return AG->begin();
354 }
355 static ChildIteratorType nodes_end(ArgumentGraph *AG) { return AG->end(); }
356};
Alexander Kornienkof00654e2015-06-23 09:49:53 +0000357}
Nick Lewycky4c378a42011-12-28 23:24:21 +0000358
Chandler Carrutha632fb92015-09-13 06:57:25 +0000359/// Returns Attribute::None, Attribute::ReadOnly or Attribute::ReadNone.
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000360static Attribute::AttrKind
361determinePointerReadAttrs(Argument *A,
Chandler Carruth63559d72015-09-13 06:47:20 +0000362 const SmallPtrSet<Argument *, 8> &SCCNodes) {
363
364 SmallVector<Use *, 32> Worklist;
365 SmallSet<Use *, 32> Visited;
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000366
Reid Kleckner26af2ca2014-01-28 02:38:36 +0000367 // inalloca arguments are always clobbered by the call.
368 if (A->hasInAllocaAttr())
369 return Attribute::None;
370
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000371 bool IsRead = false;
372 // We don't need to track IsWritten. If A is written to, return immediately.
373
Chandler Carruthcdf47882014-03-09 03:16:01 +0000374 for (Use &U : A->uses()) {
Chandler Carruthcdf47882014-03-09 03:16:01 +0000375 Visited.insert(&U);
376 Worklist.push_back(&U);
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000377 }
378
379 while (!Worklist.empty()) {
380 Use *U = Worklist.pop_back_val();
381 Instruction *I = cast<Instruction>(U->getUser());
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000382
383 switch (I->getOpcode()) {
384 case Instruction::BitCast:
385 case Instruction::GetElementPtr:
386 case Instruction::PHI:
387 case Instruction::Select:
Matt Arsenaulte55a2c22014-01-14 19:11:52 +0000388 case Instruction::AddrSpaceCast:
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000389 // The original value is not read/written via this if the new value isn't.
Chandler Carruthcdf47882014-03-09 03:16:01 +0000390 for (Use &UU : I->uses())
David Blaikie70573dc2014-11-19 07:49:26 +0000391 if (Visited.insert(&UU).second)
Chandler Carruthcdf47882014-03-09 03:16:01 +0000392 Worklist.push_back(&UU);
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000393 break;
394
395 case Instruction::Call:
396 case Instruction::Invoke: {
Nick Lewycky59633cb2014-05-30 02:31:27 +0000397 bool Captures = true;
398
399 if (I->getType()->isVoidTy())
400 Captures = false;
401
402 auto AddUsersToWorklistIfCapturing = [&] {
403 if (Captures)
404 for (Use &UU : I->uses())
David Blaikie70573dc2014-11-19 07:49:26 +0000405 if (Visited.insert(&UU).second)
Nick Lewycky59633cb2014-05-30 02:31:27 +0000406 Worklist.push_back(&UU);
407 };
408
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000409 CallSite CS(I);
Nick Lewycky59633cb2014-05-30 02:31:27 +0000410 if (CS.doesNotAccessMemory()) {
411 AddUsersToWorklistIfCapturing();
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000412 continue;
Nick Lewycky59633cb2014-05-30 02:31:27 +0000413 }
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000414
415 Function *F = CS.getCalledFunction();
416 if (!F) {
417 if (CS.onlyReadsMemory()) {
418 IsRead = true;
Nick Lewycky59633cb2014-05-30 02:31:27 +0000419 AddUsersToWorklistIfCapturing();
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000420 continue;
421 }
422 return Attribute::None;
423 }
424
Sanjoy Das436e2392015-11-07 01:55:53 +0000425 // Note: the callee and the two successor blocks *follow* the argument
426 // operands. This means there is no need to adjust UseIndex to account
427 // for these.
428
429 unsigned UseIndex = std::distance(CS.arg_begin(), U);
430
Sanjoy Dasea1df7f2015-11-07 01:56:07 +0000431 // U cannot be the callee operand use: since we're exploring the
432 // transitive uses of an Argument, having such a use be a callee would
433 // imply the CallSite is an indirect call or invoke; and we'd take the
434 // early exit above.
435 assert(UseIndex < CS.data_operands_size() &&
436 "Data operand use expected!");
Sanjoy Das71fe81f2015-11-07 01:56:00 +0000437
438 bool IsOperandBundleUse = UseIndex >= CS.getNumArgOperands();
439
440 if (UseIndex >= F->arg_size() && !IsOperandBundleUse) {
Sanjoy Das436e2392015-11-07 01:55:53 +0000441 assert(F->isVarArg() && "More params than args in non-varargs call");
442 return Attribute::None;
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000443 }
Sanjoy Das436e2392015-11-07 01:55:53 +0000444
Tilmann Scheller925b1932015-11-20 19:17:10 +0000445 Captures &= !CS.doesNotCapture(UseIndex);
446
Sanjoy Das71fe81f2015-11-07 01:56:00 +0000447 // Since the optimizer (by design) cannot see the data flow corresponding
448 // to a operand bundle use, these cannot participate in the optimistic SCC
449 // analysis. Instead, we model the operand bundle uses as arguments in
450 // call to a function external to the SCC.
Duncan P. N. Exon Smith9e3edad2016-08-17 01:23:58 +0000451 if (IsOperandBundleUse ||
452 !SCCNodes.count(&*std::next(F->arg_begin(), UseIndex))) {
Sanjoy Das71fe81f2015-11-07 01:56:00 +0000453
454 // The accessors used on CallSite here do the right thing for calls and
455 // invokes with operand bundles.
456
Sanjoy Das436e2392015-11-07 01:55:53 +0000457 if (!CS.onlyReadsMemory() && !CS.onlyReadsMemory(UseIndex))
458 return Attribute::None;
459 if (!CS.doesNotAccessMemory(UseIndex))
460 IsRead = true;
461 }
462
Nick Lewycky59633cb2014-05-30 02:31:27 +0000463 AddUsersToWorklistIfCapturing();
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000464 break;
465 }
466
467 case Instruction::Load:
David Majnemer124bdb72016-05-25 05:53:04 +0000468 // A volatile load has side effects beyond what readonly can be relied
469 // upon.
470 if (cast<LoadInst>(I)->isVolatile())
471 return Attribute::None;
472
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000473 IsRead = true;
474 break;
475
476 case Instruction::ICmp:
477 case Instruction::Ret:
478 break;
479
480 default:
481 return Attribute::None;
482 }
483 }
484
485 return IsRead ? Attribute::ReadOnly : Attribute::ReadNone;
486}
487
David Majnemer5246e0b2016-07-19 18:50:26 +0000488/// Deduce returned attributes for the SCC.
489static bool addArgumentReturnedAttrs(const SCCNodeSet &SCCNodes) {
490 bool Changed = false;
491
492 AttrBuilder B;
493 B.addAttribute(Attribute::Returned);
494
495 // Check each function in turn, determining if an argument is always returned.
496 for (Function *F : SCCNodes) {
497 // We can infer and propagate function attributes only when we know that the
498 // definition we'll get at link time is *exactly* the definition we see now.
499 // For more details, see GlobalValue::mayBeDerefined.
500 if (!F->hasExactDefinition())
501 continue;
502
503 if (F->getReturnType()->isVoidTy())
504 continue;
505
506 auto FindRetArg = [&]() -> Value * {
507 Value *RetArg = nullptr;
508 for (BasicBlock &BB : *F)
509 if (auto *Ret = dyn_cast<ReturnInst>(BB.getTerminator())) {
510 // Note that stripPointerCasts should look through functions with
511 // returned arguments.
512 Value *RetVal = Ret->getReturnValue()->stripPointerCasts();
513 if (!isa<Argument>(RetVal) || RetVal->getType() != F->getReturnType())
514 return nullptr;
515
516 if (!RetArg)
517 RetArg = RetVal;
518 else if (RetArg != RetVal)
519 return nullptr;
520 }
521
522 return RetArg;
523 };
524
525 if (Value *RetArg = FindRetArg()) {
526 auto *A = cast<Argument>(RetArg);
527 A->addAttr(AttributeSet::get(F->getContext(), A->getArgNo() + 1, B));
528 ++NumReturned;
529 Changed = true;
530 }
531 }
532
533 return Changed;
534}
535
Chandler Carrutha632fb92015-09-13 06:57:25 +0000536/// Deduce nocapture attributes for the SCC.
Chandler Carruthc518ebd2015-10-29 18:29:15 +0000537static bool addArgumentAttrs(const SCCNodeSet &SCCNodes) {
Duncan Sands44c8cd92008-12-31 16:14:43 +0000538 bool Changed = false;
539
Nick Lewycky4c378a42011-12-28 23:24:21 +0000540 ArgumentGraph AG;
541
Benjamin Kramer40d7f352013-06-22 16:56:32 +0000542 AttrBuilder B;
543 B.addAttribute(Attribute::NoCapture);
544
Duncan Sands44c8cd92008-12-31 16:14:43 +0000545 // Check each function in turn, determining which pointer arguments are not
546 // captured.
Chandler Carruthc518ebd2015-10-29 18:29:15 +0000547 for (Function *F : SCCNodes) {
Sanjoy Das5ce32722016-04-08 00:48:30 +0000548 // We can infer and propagate function attributes only when we know that the
549 // definition we'll get at link time is *exactly* the definition we see now.
550 // For more details, see GlobalValue::mayBeDerefined.
551 if (!F->hasExactDefinition())
Duncan Sands44c8cd92008-12-31 16:14:43 +0000552 continue;
553
Nick Lewycky4c378a42011-12-28 23:24:21 +0000554 // Functions that are readonly (or readnone) and nounwind and don't return
555 // a value can't capture arguments. Don't analyze them.
556 if (F->onlyReadsMemory() && F->doesNotThrow() &&
557 F->getReturnType()->isVoidTy()) {
Chandler Carruth63559d72015-09-13 06:47:20 +0000558 for (Function::arg_iterator A = F->arg_begin(), E = F->arg_end(); A != E;
559 ++A) {
Benjamin Kramer40d7f352013-06-22 16:56:32 +0000560 if (A->getType()->isPointerTy() && !A->hasNoCaptureAttr()) {
561 A->addAttr(AttributeSet::get(F->getContext(), A->getArgNo() + 1, B));
562 ++NumNoCapture;
563 Changed = true;
Nick Lewycky4c378a42011-12-28 23:24:21 +0000564 }
Nick Lewycky4c378a42011-12-28 23:24:21 +0000565 }
Benjamin Kramer40d7f352013-06-22 16:56:32 +0000566 continue;
Benjamin Kramer76b7bd02013-06-22 15:51:19 +0000567 }
568
Chandler Carruth63559d72015-09-13 06:47:20 +0000569 for (Function::arg_iterator A = F->arg_begin(), E = F->arg_end(); A != E;
570 ++A) {
571 if (!A->getType()->isPointerTy())
572 continue;
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000573 bool HasNonLocalUses = false;
574 if (!A->hasNoCaptureAttr()) {
Benjamin Kramer40d7f352013-06-22 16:56:32 +0000575 ArgumentUsesTracker Tracker(SCCNodes);
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000576 PointerMayBeCaptured(&*A, &Tracker);
Benjamin Kramer40d7f352013-06-22 16:56:32 +0000577 if (!Tracker.Captured) {
578 if (Tracker.Uses.empty()) {
579 // If it's trivially not captured, mark it nocapture now.
Chandler Carruth63559d72015-09-13 06:47:20 +0000580 A->addAttr(
581 AttributeSet::get(F->getContext(), A->getArgNo() + 1, B));
Benjamin Kramer40d7f352013-06-22 16:56:32 +0000582 ++NumNoCapture;
583 Changed = true;
584 } else {
585 // If it's not trivially captured and not trivially not captured,
586 // then it must be calling into another function in our SCC. Save
587 // its particulars for Argument-SCC analysis later.
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000588 ArgumentGraphNode *Node = AG[&*A];
Benjamin Kramer135f7352016-06-26 12:28:59 +0000589 for (Argument *Use : Tracker.Uses) {
590 Node->Uses.push_back(AG[Use]);
591 if (Use != &*A)
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000592 HasNonLocalUses = true;
593 }
Benjamin Kramer40d7f352013-06-22 16:56:32 +0000594 }
595 }
596 // Otherwise, it's captured. Don't bother doing SCC analysis on it.
597 }
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000598 if (!HasNonLocalUses && !A->onlyReadsMemory()) {
599 // Can we determine that it's readonly/readnone without doing an SCC?
600 // Note that we don't allow any calls at all here, or else our result
601 // will be dependent on the iteration order through the functions in the
602 // SCC.
Chandler Carruth63559d72015-09-13 06:47:20 +0000603 SmallPtrSet<Argument *, 8> Self;
Duncan P. N. Exon Smith17323402015-10-13 17:51:03 +0000604 Self.insert(&*A);
605 Attribute::AttrKind R = determinePointerReadAttrs(&*A, Self);
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000606 if (R != Attribute::None) {
607 AttrBuilder B;
608 B.addAttribute(R);
609 A->addAttr(AttributeSet::get(A->getContext(), A->getArgNo() + 1, B));
610 Changed = true;
611 R == Attribute::ReadOnly ? ++NumReadOnlyArg : ++NumReadNoneArg;
612 }
613 }
614 }
Nick Lewycky4c378a42011-12-28 23:24:21 +0000615 }
616
617 // The graph we've collected is partial because we stopped scanning for
618 // argument uses once we solved the argument trivially. These partial nodes
619 // show up as ArgumentGraphNode objects with an empty Uses list, and for
620 // these nodes the final decision about whether they capture has already been
621 // made. If the definition doesn't have a 'nocapture' attribute by now, it
622 // captures.
623
Chandler Carruth63559d72015-09-13 06:47:20 +0000624 for (scc_iterator<ArgumentGraph *> I = scc_begin(&AG); !I.isAtEnd(); ++I) {
Duncan P. N. Exon Smithd2b2fac2014-04-25 18:24:50 +0000625 const std::vector<ArgumentGraphNode *> &ArgumentSCC = *I;
Nick Lewycky4c378a42011-12-28 23:24:21 +0000626 if (ArgumentSCC.size() == 1) {
Chandler Carruth63559d72015-09-13 06:47:20 +0000627 if (!ArgumentSCC[0]->Definition)
628 continue; // synthetic root node
Nick Lewycky4c378a42011-12-28 23:24:21 +0000629
630 // eg. "void f(int* x) { if (...) f(x); }"
631 if (ArgumentSCC[0]->Uses.size() == 1 &&
632 ArgumentSCC[0]->Uses[0] == ArgumentSCC[0]) {
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000633 Argument *A = ArgumentSCC[0]->Definition;
634 A->addAttr(AttributeSet::get(A->getContext(), A->getArgNo() + 1, B));
Nick Lewycky7e820552009-01-02 03:46:56 +0000635 ++NumNoCapture;
Duncan Sands44c8cd92008-12-31 16:14:43 +0000636 Changed = true;
637 }
Nick Lewycky4c378a42011-12-28 23:24:21 +0000638 continue;
639 }
640
641 bool SCCCaptured = false;
Duncan P. N. Exon Smithd2b2fac2014-04-25 18:24:50 +0000642 for (auto I = ArgumentSCC.begin(), E = ArgumentSCC.end();
643 I != E && !SCCCaptured; ++I) {
Nick Lewycky4c378a42011-12-28 23:24:21 +0000644 ArgumentGraphNode *Node = *I;
645 if (Node->Uses.empty()) {
646 if (!Node->Definition->hasNoCaptureAttr())
647 SCCCaptured = true;
648 }
649 }
Chandler Carruth63559d72015-09-13 06:47:20 +0000650 if (SCCCaptured)
651 continue;
Nick Lewycky4c378a42011-12-28 23:24:21 +0000652
Chandler Carruth63559d72015-09-13 06:47:20 +0000653 SmallPtrSet<Argument *, 8> ArgumentSCCNodes;
Nick Lewycky4c378a42011-12-28 23:24:21 +0000654 // Fill ArgumentSCCNodes with the elements of the ArgumentSCC. Used for
655 // quickly looking up whether a given Argument is in this ArgumentSCC.
Benjamin Kramer135f7352016-06-26 12:28:59 +0000656 for (ArgumentGraphNode *I : ArgumentSCC) {
657 ArgumentSCCNodes.insert(I->Definition);
Nick Lewycky4c378a42011-12-28 23:24:21 +0000658 }
659
Duncan P. N. Exon Smithd2b2fac2014-04-25 18:24:50 +0000660 for (auto I = ArgumentSCC.begin(), E = ArgumentSCC.end();
661 I != E && !SCCCaptured; ++I) {
Nick Lewycky4c378a42011-12-28 23:24:21 +0000662 ArgumentGraphNode *N = *I;
Benjamin Kramer135f7352016-06-26 12:28:59 +0000663 for (ArgumentGraphNode *Use : N->Uses) {
664 Argument *A = Use->Definition;
Nick Lewycky4c378a42011-12-28 23:24:21 +0000665 if (A->hasNoCaptureAttr() || ArgumentSCCNodes.count(A))
666 continue;
667 SCCCaptured = true;
668 break;
669 }
670 }
Chandler Carruth63559d72015-09-13 06:47:20 +0000671 if (SCCCaptured)
672 continue;
Nick Lewycky4c378a42011-12-28 23:24:21 +0000673
Nick Lewyckyf740db32012-01-05 22:21:45 +0000674 for (unsigned i = 0, e = ArgumentSCC.size(); i != e; ++i) {
Nick Lewycky4c378a42011-12-28 23:24:21 +0000675 Argument *A = ArgumentSCC[i]->Definition;
Benjamin Kramer40d7f352013-06-22 16:56:32 +0000676 A->addAttr(AttributeSet::get(A->getContext(), A->getArgNo() + 1, B));
Nick Lewycky4c378a42011-12-28 23:24:21 +0000677 ++NumNoCapture;
678 Changed = true;
679 }
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000680
681 // We also want to compute readonly/readnone. With a small number of false
682 // negatives, we can assume that any pointer which is captured isn't going
683 // to be provably readonly or readnone, since by definition we can't
684 // analyze all uses of a captured pointer.
685 //
686 // The false negatives happen when the pointer is captured by a function
687 // that promises readonly/readnone behaviour on the pointer, then the
688 // pointer's lifetime ends before anything that writes to arbitrary memory.
689 // Also, a readonly/readnone pointer may be returned, but returning a
690 // pointer is capturing it.
691
692 Attribute::AttrKind ReadAttr = Attribute::ReadNone;
693 for (unsigned i = 0, e = ArgumentSCC.size(); i != e; ++i) {
694 Argument *A = ArgumentSCC[i]->Definition;
695 Attribute::AttrKind K = determinePointerReadAttrs(A, ArgumentSCCNodes);
696 if (K == Attribute::ReadNone)
697 continue;
698 if (K == Attribute::ReadOnly) {
699 ReadAttr = Attribute::ReadOnly;
700 continue;
701 }
702 ReadAttr = K;
703 break;
704 }
705
706 if (ReadAttr != Attribute::None) {
Bjorn Steinbrink236446c2015-05-25 19:46:38 +0000707 AttrBuilder B, R;
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000708 B.addAttribute(ReadAttr);
Chandler Carruth63559d72015-09-13 06:47:20 +0000709 R.addAttribute(Attribute::ReadOnly).addAttribute(Attribute::ReadNone);
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000710 for (unsigned i = 0, e = ArgumentSCC.size(); i != e; ++i) {
711 Argument *A = ArgumentSCC[i]->Definition;
Bjorn Steinbrink236446c2015-05-25 19:46:38 +0000712 // Clear out existing readonly/readnone attributes
713 A->removeAttr(AttributeSet::get(A->getContext(), A->getArgNo() + 1, R));
Nick Lewyckyc2ec0722013-07-06 00:29:58 +0000714 A->addAttr(AttributeSet::get(A->getContext(), A->getArgNo() + 1, B));
715 ReadAttr == Attribute::ReadOnly ? ++NumReadOnlyArg : ++NumReadNoneArg;
716 Changed = true;
717 }
718 }
Duncan Sands44c8cd92008-12-31 16:14:43 +0000719 }
720
721 return Changed;
722}
723
Chandler Carrutha632fb92015-09-13 06:57:25 +0000724/// Tests whether a function is "malloc-like".
725///
726/// A function is "malloc-like" if it returns either null or a pointer that
727/// doesn't alias any other pointer visible to the caller.
Chandler Carruthc518ebd2015-10-29 18:29:15 +0000728static bool isFunctionMallocLike(Function *F, const SCCNodeSet &SCCNodes) {
Benjamin Kramer15591272012-10-31 13:45:49 +0000729 SmallSetVector<Value *, 8> FlowsToReturn;
Benjamin Kramer135f7352016-06-26 12:28:59 +0000730 for (BasicBlock &BB : *F)
731 if (ReturnInst *Ret = dyn_cast<ReturnInst>(BB.getTerminator()))
Nick Lewyckyfbed86a2009-03-08 06:20:47 +0000732 FlowsToReturn.insert(Ret->getReturnValue());
733
734 for (unsigned i = 0; i != FlowsToReturn.size(); ++i) {
Benjamin Kramer15591272012-10-31 13:45:49 +0000735 Value *RetVal = FlowsToReturn[i];
Nick Lewyckyfbed86a2009-03-08 06:20:47 +0000736
737 if (Constant *C = dyn_cast<Constant>(RetVal)) {
738 if (!C->isNullValue() && !isa<UndefValue>(C))
739 return false;
740
741 continue;
742 }
743
744 if (isa<Argument>(RetVal))
745 return false;
746
747 if (Instruction *RVI = dyn_cast<Instruction>(RetVal))
748 switch (RVI->getOpcode()) {
Chandler Carruth63559d72015-09-13 06:47:20 +0000749 // Extend the analysis by looking upwards.
750 case Instruction::BitCast:
751 case Instruction::GetElementPtr:
752 case Instruction::AddrSpaceCast:
753 FlowsToReturn.insert(RVI->getOperand(0));
754 continue;
755 case Instruction::Select: {
756 SelectInst *SI = cast<SelectInst>(RVI);
757 FlowsToReturn.insert(SI->getTrueValue());
758 FlowsToReturn.insert(SI->getFalseValue());
759 continue;
760 }
761 case Instruction::PHI: {
762 PHINode *PN = cast<PHINode>(RVI);
763 for (Value *IncValue : PN->incoming_values())
764 FlowsToReturn.insert(IncValue);
765 continue;
766 }
Nick Lewyckyfbed86a2009-03-08 06:20:47 +0000767
Chandler Carruth63559d72015-09-13 06:47:20 +0000768 // Check whether the pointer came from an allocation.
769 case Instruction::Alloca:
770 break;
771 case Instruction::Call:
772 case Instruction::Invoke: {
773 CallSite CS(RVI);
774 if (CS.paramHasAttr(0, Attribute::NoAlias))
Nick Lewyckyfbed86a2009-03-08 06:20:47 +0000775 break;
Chandler Carruth63559d72015-09-13 06:47:20 +0000776 if (CS.getCalledFunction() && SCCNodes.count(CS.getCalledFunction()))
777 break;
Justin Bognercd1d5aa2016-08-17 20:30:52 +0000778 LLVM_FALLTHROUGH;
779 }
Chandler Carruth63559d72015-09-13 06:47:20 +0000780 default:
781 return false; // Did not come from an allocation.
Nick Lewyckyfbed86a2009-03-08 06:20:47 +0000782 }
783
Dan Gohman94e61762009-11-19 21:57:48 +0000784 if (PointerMayBeCaptured(RetVal, false, /*StoreCaptures=*/false))
Nick Lewyckyfbed86a2009-03-08 06:20:47 +0000785 return false;
786 }
787
788 return true;
789}
790
Chandler Carrutha632fb92015-09-13 06:57:25 +0000791/// Deduce noalias attributes for the SCC.
Chandler Carruthc518ebd2015-10-29 18:29:15 +0000792static bool addNoAliasAttrs(const SCCNodeSet &SCCNodes) {
Nick Lewycky9ec96d12009-03-08 17:08:09 +0000793 // Check each function in turn, determining which functions return noalias
794 // pointers.
Chandler Carruthc518ebd2015-10-29 18:29:15 +0000795 for (Function *F : SCCNodes) {
Nick Lewyckyfbed86a2009-03-08 06:20:47 +0000796 // Already noalias.
797 if (F->doesNotAlias(0))
798 continue;
799
Sanjoy Das5ce32722016-04-08 00:48:30 +0000800 // We can infer and propagate function attributes only when we know that the
801 // definition we'll get at link time is *exactly* the definition we see now.
802 // For more details, see GlobalValue::mayBeDerefined.
803 if (!F->hasExactDefinition())
Nick Lewyckyfbed86a2009-03-08 06:20:47 +0000804 return false;
805
Chandler Carruth63559d72015-09-13 06:47:20 +0000806 // We annotate noalias return values, which are only applicable to
Nick Lewyckyfbed86a2009-03-08 06:20:47 +0000807 // pointer types.
Duncan Sands19d0b472010-02-16 11:11:14 +0000808 if (!F->getReturnType()->isPointerTy())
Nick Lewyckyfbed86a2009-03-08 06:20:47 +0000809 continue;
810
Chandler Carruth3824f852015-09-13 08:23:27 +0000811 if (!isFunctionMallocLike(F, SCCNodes))
Nick Lewyckyfbed86a2009-03-08 06:20:47 +0000812 return false;
813 }
814
815 bool MadeChange = false;
Chandler Carruthc518ebd2015-10-29 18:29:15 +0000816 for (Function *F : SCCNodes) {
Duncan Sands19d0b472010-02-16 11:11:14 +0000817 if (F->doesNotAlias(0) || !F->getReturnType()->isPointerTy())
Nick Lewyckyfbed86a2009-03-08 06:20:47 +0000818 continue;
819
820 F->setDoesNotAlias(0);
821 ++NumNoAlias;
822 MadeChange = true;
823 }
824
825 return MadeChange;
826}
827
Chandler Carrutha632fb92015-09-13 06:57:25 +0000828/// Tests whether this function is known to not return null.
Chandler Carruth8874b782015-09-13 08:17:14 +0000829///
830/// Requires that the function returns a pointer.
831///
832/// Returns true if it believes the function will not return a null, and sets
833/// \p Speculative based on whether the returned conclusion is a speculative
834/// conclusion due to SCC calls.
Chandler Carruthc518ebd2015-10-29 18:29:15 +0000835static bool isReturnNonNull(Function *F, const SCCNodeSet &SCCNodes,
Sean Silva45835e72016-07-02 23:47:27 +0000836 bool &Speculative) {
Philip Reamesa88caea2015-08-31 19:44:38 +0000837 assert(F->getReturnType()->isPointerTy() &&
838 "nonnull only meaningful on pointer types");
839 Speculative = false;
Chandler Carruth63559d72015-09-13 06:47:20 +0000840
Philip Reamesa88caea2015-08-31 19:44:38 +0000841 SmallSetVector<Value *, 8> FlowsToReturn;
842 for (BasicBlock &BB : *F)
843 if (auto *Ret = dyn_cast<ReturnInst>(BB.getTerminator()))
844 FlowsToReturn.insert(Ret->getReturnValue());
845
846 for (unsigned i = 0; i != FlowsToReturn.size(); ++i) {
847 Value *RetVal = FlowsToReturn[i];
848
849 // If this value is locally known to be non-null, we're good
Sean Silva45835e72016-07-02 23:47:27 +0000850 if (isKnownNonNull(RetVal))
Philip Reamesa88caea2015-08-31 19:44:38 +0000851 continue;
852
853 // Otherwise, we need to look upwards since we can't make any local
Chandler Carruth63559d72015-09-13 06:47:20 +0000854 // conclusions.
Philip Reamesa88caea2015-08-31 19:44:38 +0000855 Instruction *RVI = dyn_cast<Instruction>(RetVal);
856 if (!RVI)
857 return false;
858 switch (RVI->getOpcode()) {
Chandler Carruth63559d72015-09-13 06:47:20 +0000859 // Extend the analysis by looking upwards.
Philip Reamesa88caea2015-08-31 19:44:38 +0000860 case Instruction::BitCast:
861 case Instruction::GetElementPtr:
862 case Instruction::AddrSpaceCast:
863 FlowsToReturn.insert(RVI->getOperand(0));
864 continue;
865 case Instruction::Select: {
866 SelectInst *SI = cast<SelectInst>(RVI);
867 FlowsToReturn.insert(SI->getTrueValue());
868 FlowsToReturn.insert(SI->getFalseValue());
869 continue;
870 }
871 case Instruction::PHI: {
872 PHINode *PN = cast<PHINode>(RVI);
873 for (int i = 0, e = PN->getNumIncomingValues(); i != e; ++i)
874 FlowsToReturn.insert(PN->getIncomingValue(i));
875 continue;
876 }
877 case Instruction::Call:
878 case Instruction::Invoke: {
879 CallSite CS(RVI);
880 Function *Callee = CS.getCalledFunction();
881 // A call to a node within the SCC is assumed to return null until
882 // proven otherwise
883 if (Callee && SCCNodes.count(Callee)) {
884 Speculative = true;
885 continue;
886 }
887 return false;
888 }
889 default:
Chandler Carruth63559d72015-09-13 06:47:20 +0000890 return false; // Unknown source, may be null
Philip Reamesa88caea2015-08-31 19:44:38 +0000891 };
892 llvm_unreachable("should have either continued or returned");
893 }
894
895 return true;
896}
897
Chandler Carrutha632fb92015-09-13 06:57:25 +0000898/// Deduce nonnull attributes for the SCC.
Sean Silva45835e72016-07-02 23:47:27 +0000899static bool addNonNullAttrs(const SCCNodeSet &SCCNodes) {
Philip Reamesa88caea2015-08-31 19:44:38 +0000900 // Speculative that all functions in the SCC return only nonnull
901 // pointers. We may refute this as we analyze functions.
902 bool SCCReturnsNonNull = true;
903
904 bool MadeChange = false;
905
906 // Check each function in turn, determining which functions return nonnull
907 // pointers.
Chandler Carruthc518ebd2015-10-29 18:29:15 +0000908 for (Function *F : SCCNodes) {
Philip Reamesa88caea2015-08-31 19:44:38 +0000909 // Already nonnull.
910 if (F->getAttributes().hasAttribute(AttributeSet::ReturnIndex,
911 Attribute::NonNull))
912 continue;
913
Sanjoy Das5ce32722016-04-08 00:48:30 +0000914 // We can infer and propagate function attributes only when we know that the
915 // definition we'll get at link time is *exactly* the definition we see now.
916 // For more details, see GlobalValue::mayBeDerefined.
917 if (!F->hasExactDefinition())
Philip Reamesa88caea2015-08-31 19:44:38 +0000918 return false;
919
Chandler Carruth63559d72015-09-13 06:47:20 +0000920 // We annotate nonnull return values, which are only applicable to
Philip Reamesa88caea2015-08-31 19:44:38 +0000921 // pointer types.
922 if (!F->getReturnType()->isPointerTy())
923 continue;
924
925 bool Speculative = false;
Sean Silva45835e72016-07-02 23:47:27 +0000926 if (isReturnNonNull(F, SCCNodes, Speculative)) {
Philip Reamesa88caea2015-08-31 19:44:38 +0000927 if (!Speculative) {
928 // Mark the function eagerly since we may discover a function
929 // which prevents us from speculating about the entire SCC
930 DEBUG(dbgs() << "Eagerly marking " << F->getName() << " as nonnull\n");
931 F->addAttribute(AttributeSet::ReturnIndex, Attribute::NonNull);
932 ++NumNonNullReturn;
933 MadeChange = true;
934 }
935 continue;
936 }
937 // At least one function returns something which could be null, can't
938 // speculate any more.
939 SCCReturnsNonNull = false;
940 }
941
942 if (SCCReturnsNonNull) {
Chandler Carruthc518ebd2015-10-29 18:29:15 +0000943 for (Function *F : SCCNodes) {
Philip Reamesa88caea2015-08-31 19:44:38 +0000944 if (F->getAttributes().hasAttribute(AttributeSet::ReturnIndex,
945 Attribute::NonNull) ||
946 !F->getReturnType()->isPointerTy())
947 continue;
948
949 DEBUG(dbgs() << "SCC marking " << F->getName() << " as nonnull\n");
950 F->addAttribute(AttributeSet::ReturnIndex, Attribute::NonNull);
951 ++NumNonNullReturn;
952 MadeChange = true;
953 }
954 }
955
956 return MadeChange;
957}
958
Justin Lebar9d943972016-03-14 20:18:54 +0000959/// Remove the convergent attribute from all functions in the SCC if every
960/// callsite within the SCC is not convergent (except for calls to functions
961/// within the SCC). Returns true if changes were made.
Chandler Carruth3937bc72016-02-12 09:47:49 +0000962static bool removeConvergentAttrs(const SCCNodeSet &SCCNodes) {
Justin Lebar9d943972016-03-14 20:18:54 +0000963 // For every function in SCC, ensure that either
964 // * it is not convergent, or
965 // * we can remove its convergent attribute.
966 bool HasConvergentFn = false;
Chandler Carruth3937bc72016-02-12 09:47:49 +0000967 for (Function *F : SCCNodes) {
Justin Lebar9d943972016-03-14 20:18:54 +0000968 if (!F->isConvergent()) continue;
969 HasConvergentFn = true;
970
971 // Can't remove convergent from function declarations.
972 if (F->isDeclaration()) return false;
973
974 // Can't remove convergent if any of our functions has a convergent call to a
975 // function not in the SCC.
976 for (Instruction &I : instructions(*F)) {
977 CallSite CS(&I);
978 // Bail if CS is a convergent call to a function not in the SCC.
979 if (CS && CS.isConvergent() &&
980 SCCNodes.count(CS.getCalledFunction()) == 0)
981 return false;
982 }
983 }
984
985 // If the SCC doesn't have any convergent functions, we have nothing to do.
986 if (!HasConvergentFn) return false;
987
988 // If we got here, all of the calls the SCC makes to functions not in the SCC
989 // are non-convergent. Therefore all of the SCC's functions can also be made
990 // non-convergent. We'll remove the attr from the callsites in
991 // InstCombineCalls.
992 for (Function *F : SCCNodes) {
993 if (!F->isConvergent()) continue;
994
995 DEBUG(dbgs() << "Removing convergent attr from fn " << F->getName()
996 << "\n");
Chandler Carruth3937bc72016-02-12 09:47:49 +0000997 F->setNotConvergent();
998 }
Justin Lebar260854b2016-02-09 23:03:22 +0000999 return true;
1000}
1001
James Molloy7e9bdd52015-11-12 10:55:20 +00001002static bool setDoesNotRecurse(Function &F) {
1003 if (F.doesNotRecurse())
1004 return false;
1005 F.setDoesNotRecurse();
1006 ++NumNoRecurse;
1007 return true;
1008}
1009
Chandler Carruth632d2082016-02-13 08:47:51 +00001010static bool addNoRecurseAttrs(const SCCNodeSet &SCCNodes) {
James Molloy7e9bdd52015-11-12 10:55:20 +00001011 // Try and identify functions that do not recurse.
1012
1013 // If the SCC contains multiple nodes we know for sure there is recursion.
Chandler Carruth632d2082016-02-13 08:47:51 +00001014 if (SCCNodes.size() != 1)
James Molloy7e9bdd52015-11-12 10:55:20 +00001015 return false;
1016
Chandler Carruth632d2082016-02-13 08:47:51 +00001017 Function *F = *SCCNodes.begin();
James Molloy7e9bdd52015-11-12 10:55:20 +00001018 if (!F || F->isDeclaration() || F->doesNotRecurse())
1019 return false;
1020
1021 // If all of the calls in F are identifiable and are to norecurse functions, F
1022 // is norecurse. This check also detects self-recursion as F is not currently
1023 // marked norecurse, so any called from F to F will not be marked norecurse.
Chandler Carruth632d2082016-02-13 08:47:51 +00001024 for (Instruction &I : instructions(*F))
1025 if (auto CS = CallSite(&I)) {
1026 Function *Callee = CS.getCalledFunction();
1027 if (!Callee || Callee == F || !Callee->doesNotRecurse())
1028 // Function calls a potentially recursive function.
1029 return false;
1030 }
James Molloy7e9bdd52015-11-12 10:55:20 +00001031
Chandler Carruth632d2082016-02-13 08:47:51 +00001032 // Every call was to a non-recursive function other than this function, and
1033 // we have no indirect recursion as the SCC size is one. This function cannot
1034 // recurse.
1035 return setDoesNotRecurse(*F);
James Molloy7e9bdd52015-11-12 10:55:20 +00001036}
1037
Chandler Carruthb47f8012016-03-11 11:05:24 +00001038PreservedAnalyses PostOrderFunctionAttrsPass::run(LazyCallGraph::SCC &C,
1039 CGSCCAnalysisManager &AM) {
Chandler Carruth9c4ed172016-02-18 11:03:11 +00001040 FunctionAnalysisManager &FAM =
Chandler Carruthb47f8012016-03-11 11:05:24 +00001041 AM.getResult<FunctionAnalysisManagerCGSCCProxy>(C).getManager();
Chandler Carruth9c4ed172016-02-18 11:03:11 +00001042
Chandler Carruth9c4ed172016-02-18 11:03:11 +00001043 // We pass a lambda into functions to wire them up to the analysis manager
1044 // for getting function analyses.
1045 auto AARGetter = [&](Function &F) -> AAResults & {
1046 return FAM.getResult<AAManager>(F);
1047 };
1048
1049 // Fill SCCNodes with the elements of the SCC. Also track whether there are
1050 // any external or opt-none nodes that will prevent us from optimizing any
1051 // part of the SCC.
1052 SCCNodeSet SCCNodes;
1053 bool HasUnknownCall = false;
1054 for (LazyCallGraph::Node &N : C) {
1055 Function &F = N.getFunction();
1056 if (F.hasFnAttribute(Attribute::OptimizeNone)) {
1057 // Treat any function we're trying not to optimize as if it were an
1058 // indirect call and omit it from the node set used below.
1059 HasUnknownCall = true;
1060 continue;
1061 }
1062 // Track whether any functions in this SCC have an unknown call edge.
1063 // Note: if this is ever a performance hit, we can common it with
1064 // subsequent routines which also do scans over the instructions of the
1065 // function.
1066 if (!HasUnknownCall)
1067 for (Instruction &I : instructions(F))
1068 if (auto CS = CallSite(&I))
1069 if (!CS.getCalledFunction()) {
1070 HasUnknownCall = true;
1071 break;
1072 }
1073
1074 SCCNodes.insert(&F);
1075 }
1076
1077 bool Changed = false;
David Majnemer5246e0b2016-07-19 18:50:26 +00001078 Changed |= addArgumentReturnedAttrs(SCCNodes);
Chandler Carruth9c4ed172016-02-18 11:03:11 +00001079 Changed |= addReadAttrs(SCCNodes, AARGetter);
1080 Changed |= addArgumentAttrs(SCCNodes);
1081
1082 // If we have no external nodes participating in the SCC, we can deduce some
1083 // more precise attributes as well.
1084 if (!HasUnknownCall) {
1085 Changed |= addNoAliasAttrs(SCCNodes);
Sean Silva45835e72016-07-02 23:47:27 +00001086 Changed |= addNonNullAttrs(SCCNodes);
Chandler Carruth9c4ed172016-02-18 11:03:11 +00001087 Changed |= removeConvergentAttrs(SCCNodes);
1088 Changed |= addNoRecurseAttrs(SCCNodes);
1089 }
1090
1091 return Changed ? PreservedAnalyses::none() : PreservedAnalyses::all();
1092}
1093
1094namespace {
1095struct PostOrderFunctionAttrsLegacyPass : public CallGraphSCCPass {
1096 static char ID; // Pass identification, replacement for typeid
1097 PostOrderFunctionAttrsLegacyPass() : CallGraphSCCPass(ID) {
1098 initializePostOrderFunctionAttrsLegacyPassPass(*PassRegistry::getPassRegistry());
1099 }
1100
1101 bool runOnSCC(CallGraphSCC &SCC) override;
1102
1103 void getAnalysisUsage(AnalysisUsage &AU) const override {
1104 AU.setPreservesCFG();
1105 AU.addRequired<AssumptionCacheTracker>();
Chandler Carruth12884f72016-03-02 15:56:53 +00001106 getAAResultsAnalysisUsage(AU);
Chandler Carruth9c4ed172016-02-18 11:03:11 +00001107 CallGraphSCCPass::getAnalysisUsage(AU);
1108 }
Chandler Carruth9c4ed172016-02-18 11:03:11 +00001109};
1110}
1111
1112char PostOrderFunctionAttrsLegacyPass::ID = 0;
1113INITIALIZE_PASS_BEGIN(PostOrderFunctionAttrsLegacyPass, "functionattrs",
1114 "Deduce function attributes", false, false)
1115INITIALIZE_PASS_DEPENDENCY(AssumptionCacheTracker)
1116INITIALIZE_PASS_DEPENDENCY(CallGraphWrapperPass)
Chandler Carruth9c4ed172016-02-18 11:03:11 +00001117INITIALIZE_PASS_END(PostOrderFunctionAttrsLegacyPass, "functionattrs",
1118 "Deduce function attributes", false, false)
1119
1120Pass *llvm::createPostOrderFunctionAttrsLegacyPass() { return new PostOrderFunctionAttrsLegacyPass(); }
1121
Sean Silva997cbea2016-07-03 03:35:03 +00001122template <typename AARGetterT>
1123static bool runImpl(CallGraphSCC &SCC, AARGetterT AARGetter) {
Chandler Carruthcada2d82015-10-31 00:28:37 +00001124 bool Changed = false;
Chandler Carruthc518ebd2015-10-29 18:29:15 +00001125
1126 // Fill SCCNodes with the elements of the SCC. Used for quickly looking up
1127 // whether a given CallGraphNode is in this SCC. Also track whether there are
1128 // any external or opt-none nodes that will prevent us from optimizing any
1129 // part of the SCC.
1130 SCCNodeSet SCCNodes;
1131 bool ExternalNode = false;
Benjamin Kramer135f7352016-06-26 12:28:59 +00001132 for (CallGraphNode *I : SCC) {
1133 Function *F = I->getFunction();
Chandler Carruthc518ebd2015-10-29 18:29:15 +00001134 if (!F || F->hasFnAttribute(Attribute::OptimizeNone)) {
1135 // External node or function we're trying not to optimize - we both avoid
1136 // transform them and avoid leveraging information they provide.
1137 ExternalNode = true;
1138 continue;
1139 }
1140
1141 SCCNodes.insert(F);
1142 }
1143
David Majnemer5246e0b2016-07-19 18:50:26 +00001144 Changed |= addArgumentReturnedAttrs(SCCNodes);
Chandler Carrutha8125352015-10-30 16:48:08 +00001145 Changed |= addReadAttrs(SCCNodes, AARGetter);
Chandler Carruthc518ebd2015-10-29 18:29:15 +00001146 Changed |= addArgumentAttrs(SCCNodes);
1147
Chandler Carruth3a040e62015-12-27 08:41:34 +00001148 // If we have no external nodes participating in the SCC, we can deduce some
Chandler Carruthc518ebd2015-10-29 18:29:15 +00001149 // more precise attributes as well.
1150 if (!ExternalNode) {
1151 Changed |= addNoAliasAttrs(SCCNodes);
Sean Silva45835e72016-07-02 23:47:27 +00001152 Changed |= addNonNullAttrs(SCCNodes);
Chandler Carruth3937bc72016-02-12 09:47:49 +00001153 Changed |= removeConvergentAttrs(SCCNodes);
Chandler Carruth632d2082016-02-13 08:47:51 +00001154 Changed |= addNoRecurseAttrs(SCCNodes);
Chandler Carruthc518ebd2015-10-29 18:29:15 +00001155 }
Chandler Carruth1926b702016-01-08 10:55:52 +00001156
James Molloy7e9bdd52015-11-12 10:55:20 +00001157 return Changed;
1158}
Chandler Carruthc518ebd2015-10-29 18:29:15 +00001159
Sean Silva997cbea2016-07-03 03:35:03 +00001160bool PostOrderFunctionAttrsLegacyPass::runOnSCC(CallGraphSCC &SCC) {
1161 if (skipSCC(SCC))
1162 return false;
1163
1164 // We compute dedicated AA results for each function in the SCC as needed. We
1165 // use a lambda referencing external objects so that they live long enough to
1166 // be queried, but we re-use them each time.
1167 Optional<BasicAAResult> BAR;
1168 Optional<AAResults> AAR;
1169 auto AARGetter = [&](Function &F) -> AAResults & {
1170 BAR.emplace(createLegacyPMBasicAAResult(*this, F));
1171 AAR.emplace(createLegacyPMAAResults(*this, F, *BAR));
1172 return *AAR;
1173 };
1174
1175 return runImpl(SCC, AARGetter);
1176}
1177
Chandler Carruth1926b702016-01-08 10:55:52 +00001178namespace {
Sean Silvaf5080192016-06-12 07:48:51 +00001179struct ReversePostOrderFunctionAttrsLegacyPass : public ModulePass {
Chandler Carruth1926b702016-01-08 10:55:52 +00001180 static char ID; // Pass identification, replacement for typeid
Sean Silvaf5080192016-06-12 07:48:51 +00001181 ReversePostOrderFunctionAttrsLegacyPass() : ModulePass(ID) {
1182 initializeReversePostOrderFunctionAttrsLegacyPassPass(*PassRegistry::getPassRegistry());
Chandler Carruth1926b702016-01-08 10:55:52 +00001183 }
1184
1185 bool runOnModule(Module &M) override;
1186
1187 void getAnalysisUsage(AnalysisUsage &AU) const override {
1188 AU.setPreservesCFG();
1189 AU.addRequired<CallGraphWrapperPass>();
Mehdi Amini0ddf4042016-05-02 18:03:33 +00001190 AU.addPreserved<CallGraphWrapperPass>();
Chandler Carruth1926b702016-01-08 10:55:52 +00001191 }
1192};
1193}
1194
Sean Silvaf5080192016-06-12 07:48:51 +00001195char ReversePostOrderFunctionAttrsLegacyPass::ID = 0;
1196INITIALIZE_PASS_BEGIN(ReversePostOrderFunctionAttrsLegacyPass, "rpo-functionattrs",
Chandler Carruth1926b702016-01-08 10:55:52 +00001197 "Deduce function attributes in RPO", false, false)
1198INITIALIZE_PASS_DEPENDENCY(CallGraphWrapperPass)
Sean Silvaf5080192016-06-12 07:48:51 +00001199INITIALIZE_PASS_END(ReversePostOrderFunctionAttrsLegacyPass, "rpo-functionattrs",
Chandler Carruth1926b702016-01-08 10:55:52 +00001200 "Deduce function attributes in RPO", false, false)
1201
1202Pass *llvm::createReversePostOrderFunctionAttrsPass() {
Sean Silvaf5080192016-06-12 07:48:51 +00001203 return new ReversePostOrderFunctionAttrsLegacyPass();
Chandler Carruth1926b702016-01-08 10:55:52 +00001204}
1205
1206static bool addNoRecurseAttrsTopDown(Function &F) {
1207 // We check the preconditions for the function prior to calling this to avoid
1208 // the cost of building up a reversible post-order list. We assert them here
1209 // to make sure none of the invariants this relies on were violated.
1210 assert(!F.isDeclaration() && "Cannot deduce norecurse without a definition!");
1211 assert(!F.doesNotRecurse() &&
1212 "This function has already been deduced as norecurs!");
1213 assert(F.hasInternalLinkage() &&
1214 "Can only do top-down deduction for internal linkage functions!");
1215
1216 // If F is internal and all of its uses are calls from a non-recursive
1217 // functions, then none of its calls could in fact recurse without going
1218 // through a function marked norecurse, and so we can mark this function too
1219 // as norecurse. Note that the uses must actually be calls -- otherwise
1220 // a pointer to this function could be returned from a norecurse function but
1221 // this function could be recursively (indirectly) called. Note that this
1222 // also detects if F is directly recursive as F is not yet marked as
1223 // a norecurse function.
1224 for (auto *U : F.users()) {
1225 auto *I = dyn_cast<Instruction>(U);
1226 if (!I)
1227 return false;
1228 CallSite CS(I);
1229 if (!CS || !CS.getParent()->getParent()->doesNotRecurse())
1230 return false;
1231 }
1232 return setDoesNotRecurse(F);
1233}
1234
Sean Silvaadc79392016-06-12 05:44:51 +00001235static bool deduceFunctionAttributeInRPO(Module &M, CallGraph &CG) {
Chandler Carruth1926b702016-01-08 10:55:52 +00001236 // We only have a post-order SCC traversal (because SCCs are inherently
1237 // discovered in post-order), so we accumulate them in a vector and then walk
1238 // it in reverse. This is simpler than using the RPO iterator infrastructure
1239 // because we need to combine SCC detection and the PO walk of the call
1240 // graph. We can also cheat egregiously because we're primarily interested in
1241 // synthesizing norecurse and so we can only save the singular SCCs as SCCs
1242 // with multiple functions in them will clearly be recursive.
Chandler Carruth1926b702016-01-08 10:55:52 +00001243 SmallVector<Function *, 16> Worklist;
1244 for (scc_iterator<CallGraph *> I = scc_begin(&CG); !I.isAtEnd(); ++I) {
1245 if (I->size() != 1)
1246 continue;
1247
1248 Function *F = I->front()->getFunction();
1249 if (F && !F->isDeclaration() && !F->doesNotRecurse() &&
1250 F->hasInternalLinkage())
1251 Worklist.push_back(F);
1252 }
1253
James Molloy7e9bdd52015-11-12 10:55:20 +00001254 bool Changed = false;
Chandler Carruth1926b702016-01-08 10:55:52 +00001255 for (auto *F : reverse(Worklist))
1256 Changed |= addNoRecurseAttrsTopDown(*F);
1257
Duncan Sands44c8cd92008-12-31 16:14:43 +00001258 return Changed;
1259}
Sean Silvaadc79392016-06-12 05:44:51 +00001260
Sean Silvaf5080192016-06-12 07:48:51 +00001261bool ReversePostOrderFunctionAttrsLegacyPass::runOnModule(Module &M) {
Sean Silvaadc79392016-06-12 05:44:51 +00001262 if (skipModule(M))
1263 return false;
1264
1265 auto &CG = getAnalysis<CallGraphWrapperPass>().getCallGraph();
1266
1267 return deduceFunctionAttributeInRPO(M, CG);
1268}
Sean Silvaf5080192016-06-12 07:48:51 +00001269
1270PreservedAnalyses
Sean Silvafd03ac62016-08-09 00:28:38 +00001271ReversePostOrderFunctionAttrsPass::run(Module &M, ModuleAnalysisManager &AM) {
Sean Silvaf5080192016-06-12 07:48:51 +00001272 auto &CG = AM.getResult<CallGraphAnalysis>(M);
1273
1274 bool Changed = deduceFunctionAttributeInRPO(M, CG);
Sean Silva744f7a82016-08-08 05:38:01 +00001275
1276 // CallGraphAnalysis holds AssertingVH and must be invalidated eagerly so
1277 // that other passes don't delete stuff from under it.
Sean Silva0873e7d2016-08-08 07:03:49 +00001278 // FIXME: We need to invalidate this to avoid PR28400. Is there a better
1279 // solution?
Sean Silva744f7a82016-08-08 05:38:01 +00001280 AM.invalidate<CallGraphAnalysis>(M);
1281
Sean Silvaf5080192016-06-12 07:48:51 +00001282 if (!Changed)
1283 return PreservedAnalyses::all();
1284 PreservedAnalyses PA;
1285 PA.preserve<CallGraphAnalysis>();
1286 return PA;
1287}