Skip to content

Conversation

ldionne
Copy link
Member

@ldionne ldionne commented Oct 6, 2025

This will allow the CI to report everything as green as we work on making all the tests green. I also renamed a CHECK- directive in test_suite-c-compiler.shtest because it was conflicting with the recent DEFINE: lit directive, making the test fail as UNRESOLVED.

@ldionne ldionne force-pushed the review/temporary-xfails branch from 6008360 to b443954 Compare October 6, 2025 17:48
@ldionne ldionne marked this pull request as draft October 6, 2025 17:48
Copy link

@boomanaiden154 boomanaiden154 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems to have a bunch of changes lumped into it that should probably be split out?

@ldionne
Copy link
Member Author

ldionne commented Oct 6, 2025

This seems to have a bunch of changes lumped into it that should probably be split out?

Sorry, I was trying to get the CI working. I'll rebase and mark this as ready soon.

This will allow the CI to report everything as green as we work on
making all the tests green. I also renamed a CHECK-<prefix> directive
in test_suite-c-compiler.shtest because it was conflicting with the
recent `DEFINE:` lit directive, making the test fail as UNRESOLVED.
@ldionne ldionne force-pushed the review/temporary-xfails branch from b443954 to ef9a1e7 Compare October 6, 2025 17:55
@ldionne ldionne marked this pull request as ready for review October 6, 2025 17:58
@ldionne
Copy link
Member Author

ldionne commented Oct 6, 2025

This is ready for review. The CI is now all passing with Python 3.10.

Copy link

@boomanaiden154 boomanaiden154 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes LGTM, but it seems like the CI still isn't passing?

@ldionne
Copy link
Member Author

ldionne commented Oct 6, 2025

Changes LGTM, but it seems like the CI still isn't passing?

The CI isn't passing because we still run Python 3.9. But if you look at the Python 3.10 job, you'll see that everything "passed" and then the job was cancelled (due to the Python 3.9 failure).

@ldionne ldionne merged commit ae5049e into llvm:main Oct 6, 2025
1 of 3 checks passed
@ldionne ldionne deleted the review/temporary-xfails branch October 6, 2025 18:25
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