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.

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 README.md #35

Merged
merged 1 commit into from
Jan 14, 2021
Merged

Update README.md #35

merged 1 commit into from
Jan 14, 2021

Conversation

ViralBShah
Copy link
Contributor

No description provided.

@vchuravy
Copy link
Member

Thanks Viral,

know I just need to know how CI broke :)

@vchuravy vchuravy merged commit 438610f into EnzymeAD:master Jan 14, 2021
@ViralBShah
Copy link
Contributor Author

You may already know - but the example crashed for me on a recent 1.6 build, but worked fine on 1.5.3 on mac.

@vchuravy
Copy link
Member

Yeah 1.6 is not supported yet, we are not building Enzyme itself for LLVM 11.

@wsmoses
Copy link
Member

wsmoses commented Jan 14, 2021

I'm amused that of all the platforms and headaches it gave us, MacOS decides to be fine right now.

@vchuravy do we want to add an LLVM 11 build on Yggdrassil (or does that require some more julia's version of LLVM setup that isn't worth doing at the moment). We do test LLVM 11 in https://github.com/wsmoses/Enzyme proper

@ViralBShah
Copy link
Contributor Author

Maybe note the supported Julia versions in the README? Perhaps the Julia dependency in Project.toml should be stricter to restrict to 1.5 only?

@vchuravy
Copy link
Member

I was hoping that we would get JuliaPackaging/JLLWrappers.jl#27 before we have to compile for multiple Julia versions.

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.

None yet

3 participants