<div dir="ltr">On 28 May 2013 12:42, Gabor Ballabas <span dir="ltr"><<a href="mailto:gaborb@inf.u-szeged.hu" target="_blank">gaborb@inf.u-szeged.hu</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
  
    
  
  <div text="#000000" bgcolor="#FFFFFF">You are right I don't really need to set the target. I attached the
    modified patch.<br></div></blockquote><div><br></div><div style>Ok, let's wait for the review on the other patch. This one looks good to me.</div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div text="#000000" bgcolor="#FFFFFF">(I hope these modifications can be followed. BTW, why don't you use
    Bugzilla for the review process?<br>
    I quiet liked it when I was working on WebKit and you do have a LLVM
    Bugzilla.)<br></div></blockquote><div></div></div><br></div><div class="gmail_extra" style>We use the list for historic reasons, but there's a new tool that you can also use:</div><div class="gmail_extra" style><br>
</div><div class="gmail_extra" style>Phabricator:</div><div class="gmail_extra" style><a href="http://llvm-reviews.chandlerc.com/">http://llvm-reviews.chandlerc.com/</a><br></div><div class="gmail_extra" style><br></div><div class="gmail_extra" style>
I believe this is what Facebook uses for internal review and has a similar interface to GitHub, which is quite pleasing.</div><div class="gmail_extra" style><br></div><div class="gmail_extra" style>cheers,</div><div class="gmail_extra" style>
--renato</div></div>