-
Notifications
You must be signed in to change notification settings - Fork 19
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
Update to new Enzyme-JAX with new LLVM #975
Conversation
[skip tests]
@avik-pal only test failures are in sharding, can you please take a look? |
locally building to test. |
the non-jll tests can be ignored for now |
Just to know, is this ready to go or do you expect to do something else? Do we want to update Enzyme-JAX again? There are a bunch of more fixes we want soon. CC @wsmoses. Also, I'd prefer to leave the Julia changes for a separate PR, having the "CI" jobs failing is problematic for most of the other PRs, we can live with the local build failing only. |
yeah in that case let's get only the jll changes in and I will do the julia changes with the jll bump |
This reverts commit 4fae3ab.
per latency, concurrently starting the jll in the bg [on main, so we'll discard this PR after checking tests pass] |
so @avik-pal feel free to do the julia changes now |
Superseded by #984. |
Let's see how far we get.