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

Specify -Z unstable-options in README #163

Closed
wants to merge 119 commits into from
Closed

Specify -Z unstable-options in README #163

wants to merge 119 commits into from

Conversation

samestep
Copy link

@samestep samestep commented Aug 12, 2024

@ZuseZ4 Following up on our conversation in #156 (comment) and in the Differentiable Programming Discord, this PR modifies the README so that its commands should now work without any additional modifications.

ZuseZ4 and others added 26 commits April 19, 2024 18:32
* Update enzyme-ci.yml

We don't need to build LLD (takes 2.5 min), we just want to use some LLD for linking. We also don't need llvm plugins.

* Update enzyme-ci.yml

* Update .github/workflows/enzyme-ci.yml

Co-authored-by: Jed Brown <[email protected]>

* Update enzyme-ci.yml

* Update enzyme-ci.yml

or will it be --enable-use-lld? stay tuned.

* Update enzyme-ci.yml

* Update enzyme-ci.yml

authored-by: @jedbrown

* Update enzyme-ci.yml

* Update enzyme-ci.yml

* Update .github/workflows/enzyme-ci.yml

Co-authored-by: Tim Gymnich <[email protected]>

* Update enzyme-ci.yml

* Use vendored llvm

* Update enzyme-ci.yml

* Update enzyme-ci.yml

* Update enzyme-ci.yml

* Update enzyme-ci.yml

* Revert "Update enzyme-ci.yml"

This reverts commit 9ec1139.

* Revert "Update enzyme-ci.yml"

This reverts commit 84fe269.

* Fxing lld rebuild
Co-authored-by: I-Al-Istannen <[email protected]>

---------

Co-authored-by: I-Al-Istannen <[email protected]>
Co-authored-by: Jed Brown <[email protected]>
Co-authored-by: Tim Gymnich <[email protected]>
Co-authored-by: William Moses <[email protected]>
Co-authored-by: Tim Gymnich <[email protected]>
* Be a bit more aggressive with caching
* adding Fnc param + return attributes
* fix wrapper

* add tt writes into module

* don't pass tt anymore through Enzyme API
* Implement a opt dbg helper to create compiler-explorer MWE bug reproducer.
* fix enzyme library used for build
* add RUSTFLAGS version of enzyme flags

* Remove old env arg checks, use flags now

* small fixups
@ZuseZ4
Copy link
Member

ZuseZ4 commented Aug 15, 2024

As I was hoping for, updating to the latest nightly fixed this, thus closing.

@ZuseZ4 ZuseZ4 closed this Aug 15, 2024
@samestep samestep deleted the readme-unstable-options branch August 15, 2024 11:22
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.

6 participants