<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2014-06-25 18:19 GMT-07:00 Justin Bogner <span dir="ltr"><<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">Alex L <<a href="mailto:arphaman@gmail.com">arphaman@gmail.com</a>> writes:<br>
> There is a link to phabricator or is it not visible?<br>
<br>
</div>The canonical place for code reviews is on this list, so the patch<br>
should be sent here.<br></blockquote><div><br></div><div>I've attached the patch with this email.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I often see phabricator sending the patch to the list when you create a<br>
review - from [1] it looks like this is done by adding llvm-commits as a<br>
reviewer. This may also be needed for comments on phabricator to come to<br>
the list, which is important so that developers that don't use<br>
phabricator can participate and see what's happening.<br>
<br>
[1]: <a href="http://llvm.org/docs/Phabricator.html" target="_blank">http://llvm.org/docs/Phabricator.html</a><br></blockquote><div><br></div><div>Thanks, I didn't add llvm-commits yesterday, I will do it now. <br></div>
</div><br></div></div>