Skip to content

[BOLT] Add nfc-check-validation step #462

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: users/paschalis-mpeis/drop-meta-bolt-buildbot-aarch64
Choose a base branch
from
+21 −0
Open

[BOLT] Add nfc-check-validation step #462

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
22d4eb1
[BOLT] Add nfc-check-validation step
paschalis-mpeis Jun 6, 2025
697e2c1
Fix case when unique ids are not present
paschalis-mpeis Jun 6, 2025
55a3b78
nits
paschalis-mpeis Jun 12, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 21 additions & 0 deletions zorg/buildbot/builders/BOLTBuilder.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
from buildbot.plugins import steps
from buildbot.process.results import SUCCESS, FAILURE, WARNINGS
from buildbot.steps.shell import ShellCommand
from zorg.buildbot.builders.UnifiedTreeBuilder import getLLVMBuildFactoryAndSourcecodeSteps, addCmakeSteps, addNinjaSteps
from zorg.buildbot.commands.LitTestCommand import LitTestCommand
Expand Down Expand Up @@ -92,6 +93,26 @@ def getBOLTCmakeBuildFactory(
haltOnFailure=False,
flunkOnFailure=False,
env=env),
ShellCommand(
name='nfc-check-validation',
command=(
"info=$(bin/llvm-bolt.new --version | grep 'BOLT revision' "
"| grep -q '<unknown>' || echo 'bolt-revision'); "
"info=$info$(readelf --notes bin/llvm-bolt.new "
"| grep -q 'Build ID:' && echo ' GNU-build-id'); "
"info=$(echo \"$info\" | sed 's/^ //'); "
"[ ! -z \"$info\" ] || return 0 && "
"(printf \"NFC-Mode WARNING: unique IDs found in binaries ($info)"
". This means tests will run at all times.\"; return 2)"
),
description=('Check that nfc-mode works as intended when '
'comparing with the previous commit.'),
haltOnFailure=False,
warnOnFailure=True,
warnOnWarnings=True,
decodeRC={0: SUCCESS, 1: FAILURE, 2: WARNINGS},
descriptionDone=["NFC-Mode unique IDs in binaries"],
env=env),
ShellCommand(
name='check-bolt-different',
command=('find -name timing.log -delete; '
Expand Down