<div dir="ltr"><div>As an interesting coincidence and data point, my <a href="http://sony.com" target="_blank">sony.com</a> email just got CC'ed to a new patch request, posted directly on llvm-commits. I'm guessing the email  actually bounced or got stuck in some sort of moderation queue, as I haven't seen it appear in the archives to link to it directly. Anyway, it appears the user didn't know about Phabricator being the preferred method (not helped due to the previously mentioned documentation no doubt), so I've pointed them at it, along with the relevant <a href="http://llvm.org" target="_blank">llvm.org</a> pages to help them out.</div><div><br></div><div>(+1 to the side point of switching entirely to Phabricator reviews by the way)<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 23 Apr 2021 at 21:43, Martin Storsjö via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">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">On Fri, 23 Apr 2021, Renato Golin via llvm-dev wrote:<br>
<br>
> Yeah, I can't remember the last time I did a review by email. But I'm sure<br>
> that they were replies to a commit that broke our bots, and not an actual<br>
> review of a pull request.<br>
> Worse, I think if someone does a review by email and doesn't copy me<br>
> directly, I won't even notice. <br>
<br>
Especially for cases where a commit didn't go through review on <br>
Phabricator, it's probably common to react to it via e.g. a reply to the <br>
commit mail, as it both goes to the list and hopefully reaches the author <br>
that way.<br>
<br>
// Martin<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>