[PATCH] D48739: add a new ppc64le OpenMP buildbot

Kelvin Li via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 17 12:37:59 PDT 2018


Done in revision 337315.

Kelvin




From:   Samer Yacob/Toronto/IBM
To:     Galina Kistanova <gkistanova at gmail.com>, Kelvin 
Li/Toronto/IBM at IBMCA
Cc:     guansong.zhang at amd.com, gheorghe-teod.bercea at ibm.com, Artur 
Pilipenko via llvm-commits <llvm-commits at lists.llvm.org>
Date:   2018/07/17 03:17 PM
Subject:        Re: [PATCH] D48739: add a new ppc64le OpenMP buildbot


Hi Galina,
 that's a typo, should have been all dashes ppc64le-nvidia-P100. 

Hi Kelvin,
 can you push this change:

zorg$ git diff
diff --git a/buildbot/osuosl/master/config/builders.py 
b/buildbot/osuosl/master/config/builders.py
index 475b9ad..19918f2 100644
--- a/buildbot/osuosl/master/config/builders.py
+++ b/buildbot/osuosl/master/config/builders.py
@@ -1266,7 +1266,7 @@ def _get_openmp_builders():
 
env={'PATH':'/home/bbot/opt/cmake/bin:/home/bbot/opt/ninja/bin:/usr/local/bin:/usr/bin:/bin'})},

         {'name': "openmp-clang-ppc64le-linux-rhel",
-         'slavenames':["ppc64le_nvidia_P100"],
+         'slavenames':["ppc64le-nvidia-P100"],
          'builddir':"openmp-clang-ppc64le-linux-rhel",
          'factory' : OpenMPBuilder.getOpenMPCMakeBuildFactory(
                          c_compiler="clang",

Thanks.
Regards,
Samer




From:   Galina Kistanova <gkistanova at gmail.com>
To:     Samer Yacob <szyacob at ca.ibm.com>
Cc:     guansong.zhang at amd.com, gheorghe-teod.bercea at ibm.com, Artur 
Pilipenko via llvm-commits <llvm-commits at lists.llvm.org>
Date:   2018/07/17 02:48 PM
Subject:        Re: [PATCH] D48739: add a new ppc64le OpenMP buildbot



Hi Samer,

Please verify the name of your new slave.
The name does not match in slaves.py and  builder.py:

ppc64le_nvidia_P100
ppc64le-nvidia-P100

Thanks

Galina

On Mon, Jul 16, 2018 at 5:47 AM, Samer Yacob via Phabricator <
reviews at reviews.llvm.org> wrote:
szyacob added a comment.

@gkistanova 
Hi Galina,
 just a friendly reminder to review the changes :)

Thanks and regards


Repository:
  rL LLVM

https://reviews.llvm.org/D48739








-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180717/1b781fbf/attachment.html>


More information about the llvm-commits mailing list