Skip to content
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

Jump to bottom

Update the XLA pin #7254

Merged
merged 1 commit into from
Jun 12, 2024
Merged

Update the XLA pin #7254

merged 1 commit into from
Jun 12, 2024

Conversation

vanbasten23
Copy link
Collaborator

No description provided.

@vanbasten23 vanbasten23 marked this pull request as ready for review June 12, 2024 18:44
@vanbasten23
Copy link
Collaborator Author

I did a quick run on resnet with fakedata and didn't find any obvious speed regression: before this pin update https://gist.github.com/vanbasten23/23e2b6eea8fcc0fcea9c2506edfcb195, after this pin update https://gist.github.com/vanbasten23/e76a3b4b935ef3004f89622dd8b78f4b

@vanbasten23
Copy link
Collaborator Author

Thanks Jack for the review!

@vanbasten23 vanbasten23 merged commit a2edc21 into master Jun 12, 2024
22 checks passed
@vanbasten23 vanbasten23 mentioned this pull request Jun 12, 2024
Merged
vanbasten23 added a commit that referenced this pull request Jun 12, 2024
@vanbasten23
Update the XLA pin (#7254)
45f7f63
This was referenced Jun 12, 2024
Merged
Open
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JackCaoG JackCaoG JackCaoG approved these changes

@zpcore zpcore Awaiting requested review from zpcore

Assignees

@vanbasten23 vanbasten23

Labels
tpuci
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vanbasten23 @JackCaoG