[lld] r248560 - [ELF2] Make buildbot happy again.

[Rafael Espíndola via llvm-commits]

Is this function known to be hot? Why not just keep declaring it in
the header as before but defining it in a .cpp file?

On 25 September 2015 at 00:22, Davide Italiano via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
> Author: davide
> Date: Thu Sep 24 23:22:13 2015
> New Revision: 248560
>
> URL: http://llvm.org/viewvc/llvm-project?rev=248560&view=rev
> Log:
> [ELF2] Make buildbot happy again.
>
> Modified:
>     lld/trunk/ELF/OutputSections.h
>
> Modified: lld/trunk/ELF/OutputSections.h
> URL: http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/OutputSections.h?rev=248560&r1=248559&r2=248560&view=diff
> ==============================================================================
> --- lld/trunk/ELF/OutputSections.h (original)
> +++ lld/trunk/ELF/OutputSections.h Thu Sep 24 23:22:13 2015
> @@ -43,7 +43,7 @@ getLocalSymVA(const typename llvm::objec
>  bool includeInSymtab(const SymbolBody &B);
>  bool includeInDynamicSymtab(const SymbolBody &B);
>
> -static bool shouldKeepInSymtab(StringRef SymName) {
> +inline bool shouldKeepInSymtab(StringRef SymName) {
>    if (Config->DiscardNone)
>      return true;
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits