<div dir="ltr">Agreed with Renato - for the avoidance of doubt, my rejection is simply "rejected for now", not "rejected in concept". I'm not convinced about the numbers that were given and therefore the conclusion from those numbers.<div><br></div><div>I'd be happy to see a new set of numbers.</div><div><br></div><div>James</div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, 7 Jan 2016 at 12:14 Renato Golin via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">rengolin added a comment.<br>
<br>
In <a href="http://reviews.llvm.org/D15792#321176" rel="noreferrer" target="_blank">http://reviews.llvm.org/D15792#321176</a>, @flyingforyou wrote:<br>
<br>
> James, I still think this commit might be worth for Cortex-A57. Could you give me a second chance, please?<br>
<br>
<br>
The status is "needs revision", not "refused". :)<br>
<br>
If you get new numbers that prove this is a good move, you should definitely try again.<br>
<br>
The hint is to do yourself the analysis that James did. Look at the numbers with care, make sure that they're consistent and if they're not, investigate why not, and only publicise your findings when you're sure that every corner was swept and you have an answer to all issues with the data.<br>
<br>
In a nutshell, always distrust your own numbers until you prove yourself that they're good.<br>
<br>
cheers,<br>
--renato<br>
<br>
<br>
<a href="http://reviews.llvm.org/D15792" rel="noreferrer" target="_blank">http://reviews.llvm.org/D15792</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>