<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, May 25, 2017 at 2:51 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="">On 25 May 2017 at 13:28, Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br>
> It took for a while due to the size of the archive, but it is now<br>
> downloadable here:<br>
> <a href="https://drive.google.com/open?id=0By41E4A3-q0hZ1hNX05tbDAzQzQ" rel="noreferrer" target="_blank">https://drive.google.com/open?<wbr>id=0By41E4A3-<wbr>q0hZ1hNX05tbDAzQzQ</a><br>
><br>
> Beware: it is 8GB when expanded.<br>
<br>
</span>I tried reverting your patch on top of trunk and the result I get on<br>
this test is:<br>
<br>
clang-dbg<br>
  master 13.262570166<br>
  patch  13.243235112 1.00145999477x faster<br>
<br>
so there is a very small difference, which I guess is expected in a<br>
slow link with debug info. In other links, specially with -O0 I see an<br>
improvement, so I will revert the change.<br></blockquote><div><br></div><div>Thank you for testing, and your explanation makes sense. If we spent a lot of time linking debug info, the cost of handling SHT_GROUP becomes relatively unimportant.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I have added testcase you provided to the set I normally use for<br>
benchmarking.  I will post the link as soon as the upload is<br>
completed.<br>
<br>
Thanks,<br>
Rafael<br>
</blockquote></div><br></div></div>