[zorg] r284767 - A bit different handling for the lto param.

Galina Kistanova via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 20 12:54:34 PDT 2016


Author: gkistanova
Date: Thu Oct 20 14:54:33 2016
New Revision: 284767

URL: http://llvm.org/viewvc/llvm-project?rev=284767&view=rev
Log:
A bit different handling for the lto param.

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=284767&r1=284766&r2=284767&view=diff
==============================================================================
--- zorg/trunk/zorg/buildbot/builders/ClangLTOBuilder.py (original)
+++ zorg/trunk/zorg/buildbot/builders/ClangLTOBuilder.py Thu Oct 20 14:54:33 2016
@@ -244,6 +244,9 @@ def getClangWithLTOBuildFactory(
         # By default we link with LLD.
         depends_on_projects = ['llvm', 'clang', 'lld']
 
+    if lto is None:
+        lto = 'ON'
+
     if jobs is None:
         jobs = "%(jobs)s"
 
@@ -315,7 +318,7 @@ def getClangWithLTOBuildFactory(
     # Build all the remaining stages with exactly the same configuration.
 
     CmakeCommand.applyRequiredOptions(extra_configure_args, [
-        ('-DLLVM_ENABLE_LTO=', lto.upper() or 'ON'),
+        ('-DLLVM_ENABLE_LTO=', lto),
         ])
 
     # If we build LLD, we would link with LLD.




More information about the llvm-commits mailing list