Skip to content

Commit 9f1fea2

Browse files
Addressing reviewers
1 parent 352259d commit 9f1fea2

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

zorg/buildbot/builders/BOLTBuilder.py

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -130,6 +130,8 @@ def getBOLTCmakeBuildFactory(
130130
haltOnFailure=False,
131131
flunkOnFailure=False,
132132
env=env),
133+
# Verify that the llvm-bolt binary can report its version within a
134+
# reasonable amount of time.
133135
ShellCommand(
134136
name='llvm-bolt-version-check',
135137
command=(f"{boltNew} --version"),
@@ -164,7 +166,7 @@ def getBOLTCmakeBuildFactory(
164166
haltOnFailure=False,
165167
warnOnFailure=True,
166168
warnOnWarnings=True,
167-
maxTime=20,
169+
maxTime=30,
168170
decodeRC={0: SUCCESS, 1: FAILURE, 2: WARNINGS},
169171
descriptionDone=["NFC-Mode Validation"],
170172
env=env),
@@ -185,10 +187,12 @@ def getBOLTCmakeBuildFactory(
185187
haltOnFailure=False,
186188
env=env),
187189
# Run in-tree tests if the llvm-bolt binary has changed, or if
188-
# relevant source code changes are detected.
190+
# relevant source code changes are detected. Lower scheduling
191+
# priority with nice to reduce CPU contention in virtualized
192+
# environments.
189193
LitTestCommand(
190194
name='nfc-check-bolt',
191-
command=("LIT_OPTS='-j2' nice -n 5 ninja check-bolt"),
195+
command=("nice -n 5 ninja check-bolt"),
192196
description=["running", "NFC", "check-bolt"],
193197
descriptionDone=["NFC", "check-bolt", "completed"],
194198
warnOnFailure=True,
@@ -197,6 +201,7 @@ def getBOLTCmakeBuildFactory(
197201
doStepIf=FileDoesNotExist(f"build/{skipInTree}"),
198202
env=env),
199203
# Run out-of-tree large tests if the llvm-bolt binary has changed.
204+
# Lower scheduling priority, as above.
200205
LitTestCommand(
201206
name='nfc-check-large-bolt',
202207
command=('nice -n 5 bin/llvm-lit -sv -j2 tools/bolttests'),

0 commit comments

Comments
 (0)