FYI, several developers already use Rietveld for code reviews: <a href="http://codereview.appspot.com/">http://codereview.appspot.com/</a><div><br></div><div>I'd rather focus on one that we already have a base of familiarity with if we're going to use something more than email.<br>
<div><br><div class="gmail_quote">On Wed, Nov 10, 2010 at 8:56 AM, Bjorn Reese <span dir="ltr"><<a href="mailto:breese@mail1.stofanet.dk">breese@mail1.stofanet.dk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On 2010-11-10 12:58, Renato Golin wrote:<br>
<br>
> In a nutshell, the design decisions should be communicated more<br>
> effectively, and a Wiki is a great place to start. Peer reviewers<br>
> should communicate via the Wiki, so patchers could learn and plan<br>
> before the next iteration and reduce the cost for everybody.<br>
<br>
</div>You may consider using a review tool rather than (or in addition to)<br>
llvm-commit. For instance:<br>
<br>
   <a href="http://www.reviewboard.org/" target="_blank">http://www.reviewboard.org/</a><br>
<br>
With this tool you can assign review responsibility of certain parts of<br>
the code to different reviewers, so if somebody submits a patch of some<br>
source code files, the responsible reviewers will automatically get a<br>
review request in their inbox.<br>
<div><div></div><div class="h5">_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
</div></div></blockquote></div><br></div></div>