[PATCH] D128645: Update developer policy.

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

The changes LGTM, but you should wait a bit before landing in case other reviewers/subscribers want to chime in if they still have opinions.



================
Comment at: llvm/docs/DeveloperPolicy.rst:88
+#. Patches should be unified diffs with "infinte context" (i.e. using something
+   like `git diff -U 999999 main`).
+
----------------
xbolva00 wrote:
> vext01 wrote:
> > xbolva00 wrote:
> > > vext01 wrote:
> > > > xbolva00 wrote:
> > > > > main is weird example here imho.
> > > > How comes? The docs told me to make my diffs off `main`, so that's what you'd be diffing against typically. No?
> > > Ah, okay. I usually use it without it.
> > That would mean you haven't committed your changes, and `git clang-fomat` wont work in that case, so committing is best IMHO.
> I use arc land :)
FWIW, I also never put `main` explicitly on my diff lines; and I don't use `arc` (I just use `git diff`). Not everyone's workflow involves carrying around git branches for everything. That said, I don't have an issue with being explicit here, either.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128645/new/

https://reviews.llvm.org/D128645