<div dir="ltr">On 12 August 2013 20:30, Jeroen Hofstee <span dir="ltr"><<a href="mailto:llvm@myspectrum.nl" target="_blank">llvm@myspectrum.nl</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

  
    
  
  <div text="#000000" bgcolor="#FFFFFF">
    <div>Don't worry nor fear, see
    <a href="http://lists.denx.de/pipermail/u-boot/2013-August/160604.html" target="_blank">http://lists.denx.de/pipermail/u-boot/2013-August/160604.html</a></div></div></blockquote><div><br></div><div>Good!</div><div><br>
</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div text="#000000" bgcolor="#FFFFFF"><div>Yes, that is why proposed to send both patches through llvm-commits,
    so it<br></div>
    can be applied at the same time to keep the bots and users happy.</div></blockquote><div><br></div><div>You can't commit *at the same time*, and it's possible that some bots will fail if they get one and not the other. It'll also be harder to revert the patch in case there is something wrong in the future. But it's not the end of the world. Just remember to add comments on both commits pointing to each other (with maybe the same header on both Clang and LLVM, to make it easier to revert).</div>
<div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div text="#000000" bgcolor="#FFFFFF"><div class="im"><span style="color:rgb(34,34,34)">I did on IRC and is on cc. And besides the rename of the hidden
    flag, nothing changed for IOS.</span></div></div></blockquote><div><br></div><div>I'm glad Jim checked it. I'm happy with the change, too.</div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div text="#000000" bgcolor="#FFFFFF"><div class="im"><span style="color:rgb(34,34,34)">make check-all output:</span><br></div>
    <br>
    Testing Time: 40.08s<br>
      Expected Passes    : 13181<br>
      Expected Failures  : 53<br>
      Unsupported Tests  : 1724</div></blockquote><div><br></div><div>Excellent!</div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div text="#000000" bgcolor="#FFFFFF">
<div class="im"><span style="color:rgb(34,34,34)">mmm, this is a bit more difficult since the target has no compiler
    and only 512Mb flash.</span></div></div></blockquote><div><br></div><div>Ok, not to worry, that's why we've got buildbots for. Since this is a small change, I think you can go ahead and commit, and let's see what happens with the test-suite bot. I'm not expecting much, to be honest...</div>
<div><br></div><div>cheers,</div><div>--renato</div></div></div></div>