[PATCH] SubRegister liveness tracking

Matthias Braun mbraun at apple.com
Tue Oct 8 08:54:35 PDT 2013


On Oct 8, 2013, at 7:00 AM, Renato Golin <renato.golin at linaro.org> wrote:

> Hi Mathias,
> 
> I'm not sure how's the best way to review your patches. There are too many, and they're all related...
> 
> Would that be better on Phabricator? GitHub had a good way to review a series of patches on the same pull request, not sure Phabricator does the same thing.
Phabricator merged all commits into a single one last time I tried it. Creating multiple phabricator items would just end up in many mails again.

> 
> I'm looking at them in order, but the emails will soon entangle when we start reviewing comments from different patches…
I don’t think it will be that bad…

Thanks for the review,
	Matthias





More information about the llvm-commits mailing list