[llvm-dev] RFC: Adding a staging branch (temporarily) to facilitate upstreaming
Duncan Exon Smith via llvm-dev
llvm-dev at lists.llvm.org
Mon Jun 29 20:28:44 PDT 2020
To facilitate collaboration on an upstreaming effort (see "More context" below), we'd like to push a branch (with history) called "staging/apple" to github.com/llvm/llvm-project to serve as an official contribution to the LLVM project. This enables motivated parties to work with us to craft incremental patches for review on Phabricator. This branch would live during the effort and then be deleted after. It would not be merged.
Does this seem fine? If you have a strong objection, please share your concern.
For reference, I ran some experiments:
A `--bare` clone (just the Git database) I have of github.com/llvm/llvm-project was around ~1GB. Fetching this branch from github.com/apple/llvm-project increased it to ~1.2GB. Running `git gc --aggressive` brought it down to ~850MB.
The worktree of the "master" branch is ~1GB. Adding the Git database gives ~2GB, ~2.2GB, and ~1.9GB.
The diff of the proposed staging/apple branch is 3.1MB at `-U0`, 4.1MB at `-U3`, and 32MB at `-U999999` (Phabricator settings).
We're making a major push over the next few months to upstream changes that have accumulated over time in the branch called "apple/master" at github.com/apple/llvm-project. It has always been a non-goal for us to have changes, but over the years we've accumulated a non-trivial diff vs. github.com/llvm/llvm-project. This includes (but is not limited to) tweaks/features related to tuple hashing, modules hashing, source attributes, API notes, pointer authentication, indexing-while-building, and local refactoring.
Our goal is to eliminate this difference. Besides paying off some debt, this upstreaming effort unblocks the Swift compiler (github.com/apple/swift) from building directly against an upstream checkout of LLVM. That's why non-Apple contributors are motivated to help craft incremental patches.
As an alternative, we could post a GitHub pull request and close it without merging. From our perspective this would serve the same purpose. However, pull requests are contentious in LLVM.
Another alternative is to post a bulk Phabricator review and then "abandon" it. However, this has the disadvantage of not contributing the history (~30k commits).
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-dev