diff --git a/clang/lib/CodeGen/CodeGenFunction.cpp b/clang/lib/CodeGen/CodeGenFunction.cpp index 108c14778198..cffbca37bddc 100644 --- a/clang/lib/CodeGen/CodeGenFunction.cpp +++ b/clang/lib/CodeGen/CodeGenFunction.cpp @@ -509,15 +509,20 @@ void CodeGenFunction::StartFunction(GlobalDecl GD, } // Pass inline keyword to optimizer if it appears explicitly on any - // declaration. - if (!CGM.getCodeGenOpts().NoInline) - if (const FunctionDecl *FD = dyn_cast_or_null(D)) + // declaration. Also, in the case of -fno-inline attach NoInline + // attribute to all function that are not marked AlwaysInline or ForceInline. + if (const FunctionDecl *FD = dyn_cast_or_null(D)) { + if (!CGM.getCodeGenOpts().NoInline) { for (FunctionDecl::redecl_iterator RI = FD->redecls_begin(), RE = FD->redecls_end(); RI != RE; ++RI) if (RI->isInlineSpecified()) { Fn->addFnAttr(llvm::Attribute::InlineHint); break; } + } else if (!FD->hasAttr() && + !FD->hasAttr()) + Fn->addFnAttr(llvm::Attribute::NoInline); + } if (getLangOpts().OpenCL) { // Add metadata for a kernel function. diff --git a/clang/test/CodeGen/noinline.c b/clang/test/CodeGen/noinline.c index e64a1a539f4b..bd5a9d8c35ff 100644 --- a/clang/test/CodeGen/noinline.c +++ b/clang/test/CodeGen/noinline.c @@ -5,10 +5,17 @@ inline int dont_inline_me(int a, int b) { return(a+b); } +inline __attribute__ ((__always_inline__)) int inline_me(int a, int b) { return(a*b); } + volatile int *pa = (int*) 0x1000; void foo() { // NOINLINE: @foo // NOINLINE: dont_inline_me // NOINLINE-NOT: inlinehint pa[0] = dont_inline_me(pa[1],pa[2]); +// NOINLINE-NOT: inline_me + pa[3] = inline_me(pa[4],pa[5]); } + +// NOINLINE: Function Attrs: noinline +// NOINLINE: @dont_inline_me