Skip to content

update llvm revision #189

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

Merged
merged 6 commits into from
Jul 30, 2024
Merged

update llvm revision #189

merged 6 commits into from
Jul 30, 2024

Conversation

ZhennanQin
Copy link
Contributor

No description provided.

@ZhennanQin
Copy link
Contributor Author

Local build & test pass.

@ZhennanQin ZhennanQin requested a review from kurapov-peter July 29, 2024 04:45
@WangJialei-A
Copy link
Contributor

WangJialei-A commented Jul 29, 2024

@ZhennanQin
image

@leshikus has upgraded the runner to version 0.0.2.
Please help to upgrade the filter too. We need to rebuild the llvm for this PR

runs-on: [self-hosted, 0.0.1]

@ZhennanQin
Copy link
Contributor Author

@kurapov-peter Need your help here.

@kurapov-peter kurapov-peter requested a review from leshikus July 29, 2024 09:37
Copy link
Contributor

@leshikus leshikus left a comment

Choose a reason for hiding this comment

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

LGTM

@leshikus
Copy link
Contributor

leshikus commented Jul 30, 2024

The requested feature to support multiple llvm hashes is implemented. In rare cases it may require re-running some workflows of correspondent PR more than once (just re-running the same workflow should be enough). @kurapov-peter

@leshikus leshikus merged commit e2e7149 into main Jul 30, 2024
4 checks passed
@kurapov-peter kurapov-peter deleted the llvm_update branch July 31, 2024 14:44
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.

4 participants