diff options
Diffstat (limited to 'llvm/lib/Transforms/Scalar/SCCP.cpp')
| -rw-r--r-- | llvm/lib/Transforms/Scalar/SCCP.cpp | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/Scalar/SCCP.cpp b/llvm/lib/Transforms/Scalar/SCCP.cpp index dd1123fd4fa..054ad712d38 100644 --- a/llvm/lib/Transforms/Scalar/SCCP.cpp +++ b/llvm/lib/Transforms/Scalar/SCCP.cpp @@ -1715,7 +1715,12 @@ static bool runIPSCCP(Module &M, const DataLayout &DL, // Don't touch naked functions. They may contain asm returning a // value we don't see, so we may end up interprocedurally propagating // the return value incorrectly. - if (F.hasExactDefinition() && !F.hasFnAttribute(Attribute::Naked)) + // Also, don't touch functions marked as noinline. Trivial functions may + // essentially be inlined because of return value propagation. + // (e.g. int tinkywinky(void) { return 666; }) + if (F.hasExactDefinition() && + !(F.hasFnAttribute(Attribute::Naked) || + F.hasFnAttribute(Attribute::NoInline))) Solver.AddTrackedFunction(&F); // If this function only has direct calls that we can see, we can track its |
