[PATCH] D42037: [PATCH 2/6] [zorg] Use slave's "jobs" setting for ARM 32-bit bots.

Diana Picus via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 15 05:20:23 PST 2018


rovka accepted this revision.
rovka added a comment.
This revision is now accepted and ready to land.

LGTM, but I have a couple of nitpicks:

- Why is this PATCH 2/6? It doesn't seem related to either 1/6 or 3/6 and can be committed independently
- The summary just repeats the title. A better summary would be something along the lines of "Don't force the number of jobs on the ARM builders. This makes it possible to fall back on the number of jobs configured for the slave instead."


Repository:
  rL LLVM

https://reviews.llvm.org/D42037





More information about the llvm-commits mailing list