[llvm-dev] How to deal with multiple patches to the same file

Robinson, Paul via llvm-dev llvm-dev at lists.llvm.org
Wed Aug 12 09:34:09 PDT 2020



> -----Original Message-----
> From: Paul C. Anagnostopoulos <paul at windfall.com>
> Sent: Wednesday, August 12, 2020 11:52 AM
> To: Robinson, Paul <paul.robinson at sony.com>
> Cc: 'llvm-dev at lists.llvm.org' <llvm-dev at lists.llvm.org>
> Subject: RE: [llvm-dev] How to deal with multiple patches to the same file
> 
> I didn't think my plan sounded right. I understand what you are saying,
> but once the first patch is accepted, I have to remember to rebase the
> other branch and resubmit the patch. No? 

Optional.  Phab works off the diff file you give it, not by comparing to
the source in git.  (This is why we ask for the full-context diff, so
reviewers can look anywhere in the file you were working from.)  If the 
patches were dependent, you'd really want to rebase, but for independent 
patches it doesn't matter.  (Well, if there are a *lot* of changes, then
people might ask for a rebase just to make sure nothing was affected.
And if you're going to make changes in response to comments, my feeling
is you might as well rebase at the same time.)
--paulr


More information about the llvm-dev mailing list