[PATCH] D48739: add a new ppc64le OpenMP buildbot
Samer Yacob via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 17 12:16:59 PDT 2018
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/2f451fa8/attachment.html>
More information about the llvm-commits
mailing list