RPCS3/llvm-mirror
1
0
Fork 0
mirror of https://github.com/RPCS3/llvm-mirror.git synced 2025-03-03 01:48:15 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Do NOT inline self recursive calls into other functions. This is causing the

Browse Source 
pool allocator no end of trouble, and doesn't make a lot of sense anyway.  This
does not solve the problem with mutually recursive functions, but they are much less common.

llvm-svn: 9828
This commit is contained in:
Chris Lattner 2003-11-09 05:05:36 +00:00
parent a1040ce57e
commit 953f23925f
2 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
lib/Transforms/IPO/Inliner.cpp
View File

@ -64,10 +64,13 @@ bool Inliner::runOnSCC(const std::vector<CallGraphNode*> &SCC) {
if (isa<CallInst>(I) || isa<InvokeInst>(I)) {
CallSite CS = CallSite::get(I);
if (Function *Callee = CS.getCalledFunction())
if (!Callee->isExternal()) {
if (!Callee->isExternal() && !IsRecursiveFunction.count(Callee)) {
// Determine whether this is a function IN the SCC...
bool inSCC = SCCFunctions.count(Callee);
// Keep track of whether this is a directly recursive function.
if (Callee == F) IsRecursiveFunction.insert(F);
// If the policy determines that we should inline this function,
// try to do so...
int InlineCost = inSCC ? getRecursiveInlineCost(CS) :

6
lib/Transforms/IPO/Inliner.h
View File

@ -51,7 +51,13 @@ struct Inliner : public CallGraphSCCPass {
virtual int getRecursiveInlineCost(CallSite CS);
private:
// InlineThreshold - Cache the value here for easy access.
unsigned InlineThreshold;
// IsRecursiveFunction - This contains all functions which are directly
// recursive, which we do NOT want to inline into other functions.
std::set<Function*> IsRecursiveFunction;
bool performInlining(CallSite CS, std::set<Function*> &SCC);
};