Skip to content

[BOLT] Increase nfc-test coverage #467

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 1 commit into
base: users/paschalis-mpeis/set-nfc-tests-once
Choose a base branch
from

Conversation

paschalis-mpeis
Copy link
Member

Use --check-bolt-sources from nfc-check-setup.py to run tests when source files change between the current and previous revisions.

This triggers only in-tree tests. When the llvm-bolt binary changes, both in-tree and out-of-tree tests are run.

@paschalis-mpeis
Copy link
Member Author

paschalis-mpeis commented Jun 10, 2025

This PR triggers in-tree tests for additional source changes. Running out-of-tree tests is not needed.

The main motivation is to cover patches #1, #2, and #3 (see table below). It also catches other cases, like driver-file modifications (e.g., breaking heatmap.cpp without altering llvm-bolt).

Once merged, any changes to nfc-check-setup.py itself will automatically re-trigger tests, as an extra sanity check to the configuration.

I've verified these internally and got expected behaviour in the below patches:

# PATCH CODE CHANGE In-Tree Out-Of-Tree
1 e24d86 Modifies only a BOLT test. Draft Draft
2 6090c0 Modifies only a BOLT test. Draft Draft
3 686ec6 Modifies only a BOLT test. Draft Draft
4 b037d0 Modifies bolt heatmap docs. Draft Draft
5 7a688c Modifies LLVM: an unsupported backend (ARM). Draft Draft
6 3a9893 Modifies LLVM: a supported backend (AArch64). Draft Draft

Use `--check-bolt-sources` from nfc-check-setup.py to run tests
when source files change between the current and previous revisions.

This triggers only in-tree tests. When the llvm-bolt binary changes,
both in-tree and out-of-tree tests are run.
@paschalis-mpeis paschalis-mpeis force-pushed the users/paschalis-mpeis/increase-test-coverage branch from 8f4aaf1 to 326abf3 Compare June 13, 2025 09:57
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.

1 participant