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

Add repo for Rust-specific soft fork of Enzyme #1665

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

traviscross
Copy link
Contributor

@traviscross traviscross commented Feb 23, 2025

Currently our submodule for Enzyme in rust-lang/rust points to an external repository outside of our organization. As we do for LLVM, it would be useful for us to maintain a Rust-specific soft fork of Enzyme within rust-lang/. Let's create a repository for this.

In particular, this would unblock some work being held up by CI issues on the upstream project, and it'd allow us to make some changes that might better facilitate upstreaming of this work into LLVM.

Probably we should also create a wg-enzyme or whatnot team that could maintain this. Alternatively or in addition, we could formalize wg-autodiff, for which we have a Zulip stream but no entry here. But let's handle that separately. For the moment, we just give access to the normal teams that would be likely to need to touch it, review PRs for it, manage issues on it, etc.

CC @oli-obk who seconded the compiler MCP for this work and is overseeing it on that side.

I'm the lang liaison for our approved experiment for this work.

CC @Mark-Simulacrum, @cuviper, @ehuss for thoughts on release and bootstrap and whatnot here.

CC @nikic on the LLVM aspect.

CC @ZuseZ4 who is the owner for this work (and for whom we should find a team at some point).

CC @wsmoses from upstream Enzyme with whom @ZuseZ4 talked this over.

Currently our submodule for Enzyme in `rust-lang/rust` points to an
external repository outside of our organization.  As we do for LLVM,
it would be useful for us to maintain a Rust-specific soft fork of
Enzyme within `rust-lang/`.  Let's create a repository for this.

In particular, this would unblock some work being held up by CI issues
on the upstream project, and it'd allow us to make some changes that
might better facilitate upstreaming of this work into LLVM.

Probably we should also create a `wg-enzyme` or whatnot team that
could maintain this.  Alternatively or in addition, we could formalize
`wg-autodiff`, for which we have a Zulip stream but no entry here.
But let's handle that separately.  For the moment, we just give access
to the normal teams that would be likely to need to touch it, review
PRs for it, manage issues on it, etc.
Comment on lines +16 to +22
[[branch-protections]]
pattern = "rustc/*"
pr-required = false

[[branch-protections]]
pattern = "master"
pr-required = false
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The bits here I arranged in the same way as our llvm-project repo in the expectation we'd want to manage it in a roughly similar way.

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.

1 participant