<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 5, 2017 at 8:18 AM, Florian Hahn via Phabricator via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">fhahn updated this revision to Diff 101408.<br>
fhahn marked 2 inline comments as done.<br>
fhahn added a comment.<br>
<br>
Update CHECK- line.<br>
<br>
@evandro I’m sorry, I cannot share the exact details for various reasons, but it was over 40% on a proprietary benchmark.<br></blockquote><div><br></div><div>If you are going to say that you are doing a change for performance reasons, you really need to be able to motivate it with actual data.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
@joelkevinjones unfortunately I cannot share the benchmark we used either, and I’m not aware of a publicly-available one.<br>
<div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div>Fine to not be able to share benchmarks, but if you are going to make a generic change, you need to make more of an effort than say "can't share our numbers, don't know of any others".</div><div> </div></div></div></div>