diff options
author | Richard Trieu <rtrieu@google.com> | 2018-10-23 01:26:28 +0000 |
---|---|---|
committer | Richard Trieu <rtrieu@google.com> | 2018-10-23 01:26:28 +0000 |
commit | 9b36a9c8da1e22769ad9a46f7af9ff495b77bcc6 (patch) | |
tree | cab39ac9f0ff7eb08b8d9756a069823806234ece /clang/lib/CodeGen/CodeGenModule.cpp | |
parent | 3ac97e2ed4347bf787a18f7526a605865f71ebfe (diff) | |
download | bcm5719-llvm-9b36a9c8da1e22769ad9a46f7af9ff495b77bcc6.tar.gz bcm5719-llvm-9b36a9c8da1e22769ad9a46f7af9ff495b77bcc6.zip |
[CodeGen] Attach InlineHint to more functions
For instantiated functions, search the template pattern to see if it marked
inline to determine if InlineHint attribute should be added to the function.
llvm-svn: 344987
Diffstat (limited to 'clang/lib/CodeGen/CodeGenModule.cpp')
-rw-r--r-- | clang/lib/CodeGen/CodeGenModule.cpp | 16 |
1 files changed, 13 insertions, 3 deletions
diff --git a/clang/lib/CodeGen/CodeGenModule.cpp b/clang/lib/CodeGen/CodeGenModule.cpp index c8e672a5672..8faaed1ec90 100644 --- a/clang/lib/CodeGen/CodeGenModule.cpp +++ b/clang/lib/CodeGen/CodeGenModule.cpp @@ -1299,9 +1299,19 @@ void CodeGenModule::SetLLVMFunctionAttributesForDefinition(const Decl *D, // Otherwise, propagate the inline hint attribute and potentially use its // absence to mark things as noinline. if (auto *FD = dyn_cast<FunctionDecl>(D)) { - if (any_of(FD->redecls(), [&](const FunctionDecl *Redecl) { - return Redecl->isInlineSpecified(); - })) { + // Search function and template pattern redeclarations for inline. + auto CheckForInline = [](const FunctionDecl *FD) { + auto CheckRedeclForInline = [](const FunctionDecl *Redecl) { + return Redecl->isInlineSpecified(); + }; + if (any_of(FD->redecls(), CheckRedeclForInline)) + return true; + const FunctionDecl *Pattern = FD->getTemplateInstantiationPattern(); + if (!Pattern) + return false; + return any_of(Pattern->redecls(), CheckRedeclForInline); + }; + if (CheckForInline(FD)) { B.addAttribute(llvm::Attribute::InlineHint); } else if (CodeGenOpts.getInlining() == CodeGenOptions::OnlyHintInlining && |