<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jun 25, 2020 at 12:29 PM Mehdi AMINI via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr"><br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Re: patch chains - perhaps we can ask github on support for that?</div><div>I think what would help is somebody providing a doc with the features that phab provides that github PR doesn't, so we can get some consensus on what the diff is.</div></div></blockquote><div><br></div><div>Indeed, I'll try to start a doc on this.<br></div></div></div></div></blockquote><div>Thanks. Some things that are high on my list:</div><div>Phabricator has persistent per-user inline comment collapsing.</div><div>
<div>Phabricator can show all inline comments over the lifetime of the patch in-line (even if the placement is not perfect) in the full diff view.</div><div><br></div>

</div></div></div>