This is an archive of the discontinued LLVM Phabricator instance.

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

Authored by maxim-kuvyrkov on Jan 14 2018, 8:07 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

maxim-kuvyrkov created this revision.Jan 14 2018, 8:07 AM
rovka accepted this revision.Jan 15 2018, 5:20 AM

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."
This revision is now accepted and ready to land.Jan 15 2018, 5:20 AM
This revision was automatically updated to reflect the committed changes.