[zorg] r309970 - [zorg] Remove test_polly_format step from Polly buildbot

[Pengxuan Zheng via llvm-commits]

Author: pzheng
Date: Thu Aug  3 11:15:30 2017
New Revision: 309970

URL: http://llvm.org/viewvc/llvm-project?rev=309970&view=rev
Log:
[zorg] Remove test_polly_format step from Polly buildbot

Summary:
The test_polly_format does "make polly-check-format" which is already covered by
the "make check-polly" step.

Reviewers: mgrang, zinob, grosser, gkistanova, vleschuk, bollu

Reviewed By: grosser

Subscribers: llvm-commits

Differential Revision: https://reviews.llvm.org/D36276

Modified:
    zorg/trunk/buildbot/osuosl/master/config/builders.py
    zorg/trunk/zorg/buildbot/builders/AOSPBuilder.py
    zorg/trunk/zorg/buildbot/builders/PollyBuilder.py

Modified: zorg/trunk/buildbot/osuosl/master/config/builders.py
URL: http://llvm.org/viewvc/llvm-project/zorg/trunk/buildbot/osuosl/master/config/builders.py?rev=309970&r1=309969&r2=309970&view=diff
==============================================================================
--- zorg/trunk/buildbot/osuosl/master/config/builders.py (original)
+++ zorg/trunk/buildbot/osuosl/master/config/builders.py Thu Aug  3 11:15:30 2017
@@ -803,7 +803,6 @@ def _get_polly_builders():
                 install=True,
                 make='ninja',
                 jobs=16,
-                checkFormat=False,
                 extraCmakeArgs=["-G", "Ninja",
                                 "-DLLVM_TARGETS_TO_BUILD='ARM;AArch64'",
                                 "-DLLVM_DEFAULT_TARGET_TRIPLE=arm-linux-gnueabi",

Modified: zorg/trunk/zorg/buildbot/builders/AOSPBuilder.py
URL: http://llvm.org/viewvc/llvm-project/zorg/trunk/zorg/buildbot/builders/AOSPBuilder.py?rev=309970&r1=309969&r2=309970&view=diff
==============================================================================
--- zorg/trunk/zorg/buildbot/builders/AOSPBuilder.py (original)
+++ zorg/trunk/zorg/buildbot/builders/AOSPBuilder.py Thu Aug  3 11:15:30 2017
@@ -68,7 +68,6 @@ def getAOSPBuildFactory(
                                               install=True,
                                               make='ninja',
                                               jobs=jobs,
-                                              checkFormat=False,
                                               extraCmakeArgs=extra_cmake_args)
         clang_dir = 'llvm.inst/bin'
 

Modified: zorg/trunk/zorg/buildbot/builders/PollyBuilder.py
URL: http://llvm.org/viewvc/llvm-project/zorg/trunk/zorg/buildbot/builders/PollyBuilder.py?rev=309970&r1=309969&r2=309970&view=diff
==============================================================================
--- zorg/trunk/zorg/buildbot/builders/PollyBuilder.py (original)
+++ zorg/trunk/zorg/buildbot/builders/PollyBuilder.py Thu Aug  3 11:15:30 2017
@@ -14,7 +14,6 @@ def getPollyBuildFactory(
     install=False,
     make='make',
     jobs=None,
-    checkFormat=True,
     extraCmakeArgs=[]):
     llvm_srcdir = "llvm.src"
     llvm_objdir = "llvm.obj"
@@ -27,7 +26,6 @@ def getPollyBuildFactory(
     build_cmd = [make] + jobs_cmd
     install_cmd = [make, 'install'] + jobs_cmd
     check_cmd = [make, 'check-polly'] + jobs_cmd
-    check_format_cmd = [make, 'polly-check-format'] + jobs_cmd
     cmake_install = []
     if install:
         cmake_install = ["-DCMAKE_INSTALL_PREFIX=../%s" % llvm_instdir]
@@ -106,17 +104,10 @@ def getPollyBuildFactory(
     # Test Polly
     f.addStep(ShellCommand(name="test_polly",
                            command=check_cmd,
-                           haltOnFailure=True,
+                           haltOnFailure=False,
                            description=["test polly"],
                            workdir=llvm_objdir))
 
-    # Check formatting
-    if checkFormat:
-        f.addStep(ShellCommand(name="test_polly_format",
-                               command=check_format_cmd,
-                               haltOnFailure=False,
-                               description=["check formatting"],
-                               workdir=llvm_objdir))
     return f
 
 def AddExternalPollyBuildFactory(f, llvm_installdir, build_type = "Release"):