<div dir="ltr"><div><div><div><span class="gmail-il">> Galina</span>, which machines do we still have available for setting up bots?<br>
<span class="gmail-il">> </span>For this particular case it is critical that the machines be real<br>
<span class="gmail-il">> </span>hardware (no vm) and not in use for anything else. Having many cores is<br>
<span class="gmail-il">> </span>not important for now since we will be benchmarking single threaded<br>
<span class="gmail-il">> </span>performance. Lots (32 GB) of ram would be nice, but not critical.<br><br></div>I have vacant machine, will set up it for the bot.<br><br></div>Thanks<br><br></div>Galina<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 3, 2017 at 8:29 AM, Rafael Avila de Espindola <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="">Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> writes:<br>
<br>
> I did not benchmark these particular patches as it seems neutral to<br>
> performance to me. Do you have any concerns? I think as we improved<br>
> performance in the last few months, we actually have some room to aim for<br>
> readability and simplicity even if it slows down the linker a little bit.<br>
<br>
</span>I have no problem with an informed decision about performance versus<br>
readability.<br>
<br>
The key is in the informed part. It is at least non obvious that<br>
checking endianess for so many reads and writes will not have an impact.<br>
<br>
I benchmarked this particular patch and it was fine, but we recently<br>
regressed on linking both the linux kernel and firefox/scylla. Those<br>
regressions were also not completely obvious from reading the patch.<br>
<br>
For this reason I think setting up performance tracking bots should be a<br>
priority for lld.<br>
<br>
Galina, which machines do we still have available for setting up bots?<br>
For this particular case it is critical that the machines be real<br>
hardware (no vm) and not in use for anything else. Having many cores is<br>
not important for now since we will be benchmarking single threaded<br>
performance. Lots (32 GB) of ram would be nice, but not critical.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>