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.

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

Conversation

paschalis-mpeis
Copy link
Member

NFC-mode runs in-tree and out-of-tree (large BOLT tests) when there are changes
affecting llvm-bolt between the current revision and the previous one.

However, the revision SHA itself and the GNU build-id may be two unique
differences when doing such a comparison.

This patch introduces a lightweight sanity check step that reports a warning on
the Buildbot so that we are aware when this occurs.

@paschalis-mpeis paschalis-mpeis force-pushed the users/paschalis-mpeis/drop-meta-bolt-buildbot-aarch64 branch from a8f7936 to c9403e5 Compare June 6, 2025 12:23
@paschalis-mpeis paschalis-mpeis force-pushed the users/paschalis-mpeis/bolt-nfc-check-validation branch from 072b153 to 93d801d Compare June 6, 2025 12:40
@paschalis-mpeis paschalis-mpeis marked this pull request as ready for review June 10, 2025 15:05
@paschalis-mpeis paschalis-mpeis requested a review from aaupov June 10, 2025 15:05
@paschalis-mpeis paschalis-mpeis force-pushed the users/paschalis-mpeis/drop-meta-bolt-buildbot-aarch64 branch from c9403e5 to d9f62ef Compare June 12, 2025 07:50
@paschalis-mpeis paschalis-mpeis force-pushed the users/paschalis-mpeis/bolt-nfc-check-validation branch from 86e9c12 to 94d5651 Compare June 12, 2025 13:59
@paschalis-mpeis
Copy link
Member Author

Forced push to rebase, as base branch now renames the existing Meta buildbots instead of dropping them.
I also slightly reformatting of the output warning message.

NFC-mode runs in-tree and out-of-tree (large BOLT tests) when there are changes
affecting llvm-bolt between the current revision and the previous one.

However, the revision SHA itself and the GNU build-id may be two unique
differences when doing such a comparison.

This patch introduces a lightweight sanity check step that reports a warning on
the Buildbot so that we are aware when this occurs.
@paschalis-mpeis paschalis-mpeis force-pushed the users/paschalis-mpeis/bolt-nfc-check-validation branch from 94d5651 to 55a3b78 Compare June 13, 2025 09:29
@paschalis-mpeis
Copy link
Member Author

(forced push to rebase on the parent branch. no other changes here)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants