summaryrefslogtreecommitdiff
path: root/lib/Analysis/IPA/InlineCost.cpp
diff options
context:
space:
mode:
Diffstat (limited to 'lib/Analysis/IPA/InlineCost.cpp')
-rw-r--r--lib/Analysis/IPA/InlineCost.cpp10
1 files changed, 8 insertions, 2 deletions
diff --git a/lib/Analysis/IPA/InlineCost.cpp b/lib/Analysis/IPA/InlineCost.cpp
index a803f8c3bf..683637f5b7 100644
--- a/lib/Analysis/IPA/InlineCost.cpp
+++ b/lib/Analysis/IPA/InlineCost.cpp
@@ -1300,8 +1300,14 @@ bool InlineCostAnalysis::isInlineViable(Function &F) {
F.getAttributes().hasAttribute(AttributeSet::FunctionIndex,
Attribute::ReturnsTwice);
for (Function::iterator BI = F.begin(), BE = F.end(); BI != BE; ++BI) {
- // Disallow inlining of functions which contain an indirect branch.
- if (isa<IndirectBrInst>(BI->getTerminator()))
+ // Disallow inlining of functions which contain an indirect branch,
+ // unless the always_inline attribute is set.
+ // The attribute serves as a assertion that no local address
+ // like a block label can escpape the function.
+ // Revisit enabling inlining for functions with indirect branches
+ // when a more sophisticated espape/points-to analysis becomes available.
+ if (isa<IndirectBrInst>(BI->getTerminator()) &&
+ !F.hasFnAttribute(Attribute::AlwaysInline))
return false;
for (BasicBlock::iterator II = BI->begin(), IE = BI->end(); II != IE;