[zorg] r284153 - Added support for ThinLTO.

[Davide Italiano via llvm-commits]

On Thu, Oct 13, 2016 at 12:32 PM, Galina Kistanova via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
> Author: gkistanova
> Date: Thu Oct 13 14:32:23 2016
> New Revision: 284153
>
> URL: http://llvm.org/viewvc/llvm-project?rev=284153&view=rev
> Log:
> Added support for ThinLTO.
>

Thanks. I may miss something, but where are you actually setting `-flto=thin` ?

> Modified:
>     zorg/trunk/zorg/buildbot/builders/ClangLTOBuilder.py
>
> Modified: zorg/trunk/zorg/buildbot/builders/ClangLTOBuilder.py
> URL: http://llvm.org/viewvc/llvm-project/zorg/trunk/zorg/buildbot/builders/ClangLTOBuilder.py?rev=284153&r1=284152&r2=284153&view=diff
> ==============================================================================
> --- zorg/trunk/zorg/buildbot/builders/ClangLTOBuilder.py (original)
> +++ zorg/trunk/zorg/buildbot/builders/ClangLTOBuilder.py Thu Oct 13 14:32:23 2016
> @@ -233,6 +233,7 @@ def getClangWithLTOBuildFactory(
>             jobs  = None,
>             extra_configure_args = None,
>             compare_last_2_stages = True,
> +           lto = None, # The string gets passed to -flto flag as is. Like -flto=thin.
>             env = None):
>
>      # Set defaults
> @@ -305,11 +306,16 @@ def getClangWithLTOBuildFactory(
>      # Build all the remaining stages with exactly the same configuration.
>
>      # Set proper compile and link flags.
> +    if lto:
> +        lto = '-flto=%s' % lto
> +    else:
> +        lto = '-flto'
> +
>      CmakeCommand.appendFlags(extra_configure_args, [
> -        ('-DCMAKE_CXX_FLAGS=',           ['-flto']),
> -        ('-DCMAKE_EXE_LINKER_FLAGS=',    ['-flto', '-fuse-ld=lld']),
> -        ('-DCMAKE_MODULE_LINKER_FLAGS=', ['-flto', '-fuse-ld=lld']),
> -        ('-DCMAKE_SHARED_LINKER_FLAGS=', ['-flto', '-fuse-ld=lld']),
> +        ('-DCMAKE_CXX_FLAGS=',           [lto]),
> +        ('-DCMAKE_EXE_LINKER_FLAGS=',    [lto, '-fuse-ld=lld']),
> +        ('-DCMAKE_MODULE_LINKER_FLAGS=', [lto, '-fuse-ld=lld']),
> +        ('-DCMAKE_SHARED_LINKER_FLAGS=', [lto, '-fuse-ld=lld']),
>          ])
>
>      # The rest are test stages, which depend on the staged compiler we are ultimately after.
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits

-- 
Davide

"There are no solved problems; there are only problems that are more
or less solved" -- Henri Poincare