<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Nov 7, 2019 at 1:09 PM Nemanja Ivanovic 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">TL; DR;<br><div>As it was outlined by previous comments in the thread, the GitHub PR review tool is inferior to Phabricator (and I would argue that this is an objective statement). Those that are only familiar with GitHub may not ascribe much value to capabilities such as commenting on unchanged lines, expanding whatever context you're interested in, seeing the diff of two revisions, etc. But for a good portion of the LLVM community, these things are important.</div></div></blockquote><div>A strong -1 to moving away from Phab.<br></div><div>+1 to Nemanja's comment re: the (lack of) quality in the GitHub PR experience.</div></div><div class="gmail_quote"><br></div><div class="gmail_quote">As for lowering the cost of entry for new developers, I will point out that a bad review experience for reviewers does not improve communication between existing community members and the new community members.<br></div></div>