From d49e1529ed855b9b788a9d438f4536404e0c8f5b Mon Sep 17 00:00:00 2001 From: David Nadlinger Date: Sat, 9 Feb 2013 10:39:43 +0100 Subject: [PATCH] Revert "Revert "Fix overly conservative inlining prediction".", The previous failures were caused by the issue in callWithStackShell(), which is fixed now. This reverts commit 4a6444c3207c01238484785dbbca44257fde6370. --- gen/llvmhelpers.cpp | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/gen/llvmhelpers.cpp b/gen/llvmhelpers.cpp index ef570268..673c5ad0 100644 --- a/gen/llvmhelpers.cpp +++ b/gen/llvmhelpers.cpp @@ -1654,10 +1654,7 @@ bool mustDefineSymbol(Dsymbol* s) if ( !fd->isStaticCtorDeclaration() && !fd->isStaticDtorDeclaration() && !fd->isUnitTestDeclaration() - // FIXME: Should be canInline(true, false, false), but this - // causes a misoptimization in druntime on x86, likely due to - // an LLVM bug. - && fd->canInline(true)) + && fd->canInline(true, false, false)) { return true; }