[opensource-dev] Review Request: VWR-24312: Massively duplicated objects (part 2)
Aleric Inglewood
Aleric.Inglewood at gmail.com
Sun Jan 16 05:53:19 PST 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/81/
-----------------------------------------------------------
(Updated Jan. 16, 2011, 5:53 a.m.)
Review request for Viewer.
Changes
-------
Changed tab's into spaces in the part that Boroondas pointed out (in indra/llcharacter/llanimationstates.cpp).
Summary
-------
Turns out that most of my SNOW-800 patch was included in Viewer 2 (albeit without crediting me).
However, not everything was used and some more cleaning up was possible.
After this patch, and when compiling with optimization, there are no duplicates left
anymore that shouldn't be there in the first place: apart from the debug stream
iostream guard variable, there are several static variables with the same name (r, r1,
r2, etc) but that indeed actually different symbol objects. Then there are a few
constant POD arrays that are duplicated a hand full of times because they are
accessed with a variable index (so optimizing them away is not possible). I left them
like that (although defining those as extern as well would have been more consistent
and not slower; in fact it would be faster theoretically because those arrays could
share the same cache page then).
This addresses bug VWR-24312.
http://jira.secondlife.com/browse/VWR-24312
Diffs (updated)
-----
doc/contributions.txt 422f636c3343
indra/llcharacter/llanimationstates.cpp 422f636c3343
indra/llcommon/llavatarconstants.h 422f636c3343
indra/llcommon/lllslconstants.h 422f636c3343
indra/llcommon/llmetricperformancetester.h 422f636c3343
indra/llmath/llcamera.h 422f636c3343
indra/llmath/llcamera.cpp 422f636c3343
indra/newview/llviewerobject.cpp 422f636c3343
indra/newview/llvoavatar.cpp 422f636c3343
indra/newview/llvosky.h 422f636c3343
indra/newview/llvosky.cpp 422f636c3343
Diff: http://codereview.secondlife.com/r/81/diff
Testing
-------
Compiled with optimization and then running readelf on the executable to find duplicated symbols.
Thanks,
Aleric
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110116/49f50523/attachment.htm
More information about the opensource-dev
mailing list