----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/48/#review62 -----------------------------------------------------------
Ship it! I haven't tried building on LINUX (I recommend whoever does the integration to run a TC build on a test repo for safety before merging in lindenlab/viewer-development) but everything seems sound to me. Ship it! - Merov On 2010-12-21 07:59:33, Aleric Inglewood wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/48/ > ----------------------------------------------------------- > > (Updated 2010-12-21 07:59:33) > > > Review request for Viewer. > > > Summary > ------- > > To use ld.gold configure with: > -DCMAKE_EXE_LINKER_FLAGS:STRING="-Wl,-use-gold". > > See VWR-24254 for more details. > > > This addresses bug VWR-24254. > http://jira.secondlife.com/browse/VWR-24254 > > > Diffs > ----- > > indra/cmake/LLCommon.cmake b0689af42a71 > > Diff: http://codereview.secondlife.com/r/48/diff > > > Testing > ------- > > ld.gold links the viewer on my machine in 8 seconds, as > opposed to 19 seconds with ld.bfd. Moreover, it uses a > LOT less memory during linking (about 750 MB instead of > 2.5 GB! (for viewer 1)). > > Since my machine locked up hard when I run out of memory > (something with using an encrypted RAID for my swap), > and compiling viewer 2 uses more than 3 GB, I couldn't > compile viewer 2 at all anymore without this patch (and > using ld.gold). And OMG this is fast! > > > Thanks, > > Aleric > >
_______________________________________________ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
