<div dir="ltr">Go for it.  Somebody could play around with protected ELF visibility if they want to win back the ~3%.<div><a href="http://www.airs.com/blog/archives/307">http://www.airs.com/blog/archives/307</a><br></div></div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Dec 4, 2013 at 2:56 PM, Will Dietz <span dir="ltr"><<a href="mailto:w@wdtz.org" target="_blank">w@wdtz.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Interesting.<br>
<br>
Before this change 'ninja check' takes 1:47 on my machine, after the<br>
patch takes 1:50.<br>
(Minimum of 5 runs, times were pleasantly consistent, "-j1" added to LIT_ARGS)<br>
<br>
Note that in Make we presently export symbols in llc and lli, so we<br>
already take this hit on<br>
'make check' when not using CMake.  llvm-stress doesn't currently<br>
export, but it should<br>
(since it supports plugins) but that shouldn't impact the testing use case.<br>
<br>
Accordingly IMO a configuration option for managing test times isn't worth it,<br>
but it might be useful for making the installed (stripped) binaries smaller.<br>
If this route is taken, I'd suggest disabling plugin loading in a friendlier<br>
way than removing -export-dynamic and giving undefined reference errors<br>
should a user attempt to load a plugin.<br>
<br>
Anyway, updated patch attached to address the inconsistencies.  Okay to commit?<br>
<span class="HOEnZb"><font color="#888888"><br>
~Will<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On Wed, Dec 4, 2013 at 12:58 PM, Rafael Espíndola<br>
<<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
> On 4 December 2013 13:49, Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br>
>> Can you make sure that this doesn't slow down 'make check' on Linux too<br>
>> much?  I worry that we --export-dynamic for llc, lots of symbol uses will<br>
>> become dynamic PLT lookups and startup time will regress.<br>
>><br>
><br>
> Good point. If it does, we should probably add a --enable-plugins (and<br>
> cmake equivalent) and replace CLANG_IS_PRODUCTION with it.<br>
><br>
> Cheers,<br>
> Rafael<br>
</div></div></blockquote></div><br></div>