[PATCH] D42040: [PATCH 5/6] [zorg] Add linaro-armv8-01-arm-* slaves

[Diana Picus via Phabricator via llvm-commits]

rovka added inline comments.


================
Comment at: buildbot/osuosl/master/config/builders.py:226
         {'name' : "clang-native-arm-lnt",
-         'slavenames':["linaro-tk1-03"],
+         'slavenames':["linaro-tk1-03", "linaro-armv8-01-arm-lnt"],
          'builddir':"clang-native-arm-lnt",
----------------
If we're not going to connect both of these at the same time, then why don't we remove the TK1 from the slave list and only commit this when the armv8 slaves are ready to take over? Then we can safely disconnect the TK1s any time afterwards. 


================
Comment at: buildbot/osuosl/master/config/builders.py:235
                       runTestSuite=True,
                       testsuite_flags=['--cppflags', '-mcpu=cortex-a15 -marm',
                                        '--threads=4', '--build-threads=4'],
----------------
Should this still be cortex-a15, or should we be using a different combination of flags?


Repository:
  rL LLVM

https://reviews.llvm.org/D42040