
<html><head></head><body>Lnt has some logic for deciding if the change is significant or not.<br>

<br>

Looks like only clang has enough symbols (because of export dynamic) for the patch to make a difference.<br>

<br>

I will attach a table with all the numbers.<br>

<br>

Cheers,<br>

Rafael<br><br><div class="gmail_quote">On December 1, 2017 3:34:29 PM PST, Rui Ueyama <ruiu@google.com> wrote:<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<div dir="ltr">Does the attached image show the benchmark results? I wonder what are numbers in the other rows than clang and clang-fsds.</div><div class="gmail_extra"><br /><div class="gmail_quote">On Fri, Dec 1, 2017 at 3:31 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br /><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">>> OK, that makes it more likely that this is useful, but I still think<br />

>> we should fully benchmark every single optimization.<br />

><br />

><br />

> It depends. Your test cases do not cover all possible use cases, and since<br />

> this is an optimization for that particular code path, it actually makes<br />

> more sense to only measure the speed of that particular path. The benchmark<br />

> numbers would be more noisy if you measure the entire execution time.<br />

<br />

</span>If the results don't show up in a full run then it is probably not<br />

something that is worth optimizing for.<br />

<span class=""><br />

> It still make some sense to run some benchmarks just in case, because some<br />

> seemingly unrelated change could results in a surprising performance<br />

> degradation, but for that purpose, running one or two benchmarks should<br />

> enough.<br />

<br />

</span>I still disagree. We had bad performance regressions from testing just<br />

clang in the part and running all the benchmarks is pretty easy now<br />

that the script is in lld.<br />

<br />

In any case, the results look good, so LGTM.<br />

<br />

<br />

Cheers,<br />

Rafael<br />

</blockquote></div><br /></div>

</blockquote></div><br>

-- <br>

Sent from my Android device with K-9 Mail. Please excuse my brevity.</body></html>