tra added a comment. In https://reviews.llvm.org/D45489#1071044, @yaxunl wrote:
> In https://reviews.llvm.org/D45489#1070929, @yaxunl wrote: > > > In https://reviews.llvm.org/D45489#1070470, @tra wrote: > > > > > I'm getting confused about the order of the patches. > > > The patch stack phabricator displays in this patch is different compared > > > to the stack in https://reviews.llvm.org/D44984. Which one should I trust? > > > > > > Sorry I think I may misunderstand the parent/child relation between > > reviews. I thought a review depends on its parent reviews, i.e., parent > > reviews should be committed first. Is that correct? Thanks. > > > I think it is just visual difference. The relations are the same. Would it be possible to arrange the changes in order in which you apply them in the tree you are working on? If I want to try (partially) apply your parches in my tree, it would help to know that what I get matches your setup. Patch relationships as they are right now make intended order ambiguous. https://reviews.llvm.org/D45489 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
