Reviewers: Erik Corry, Message: Hi Erik,
This is the change that we've discussed some time ago. Dean had pointed out that when calling 'exp' library function, many ticks are assigned to 'fegetexcept' function which isn't really called. This was due to a fact that 'fegetexcept' itself is tiny but under it there is a big chunk of unnamed code, that owned reported PC values. Now such executions of unnamed code are attributed to a library itself. Description: TickProcessor: more accurate mapping of statically compiled code on Linux. 'nm' is now called with an option to report function code sizes. Static code entries are restricted to the sizes reported, and the remaining unnamed code is attributed to a library as a whole. This makes reports more accurate, as some functions are tiny, but has chunks of unnamed code behind them. This change doesn't affect reporting on Windows, as in .map files function code sizes aren't specified. Please review this at http://codereview.chromium.org/149513 Affected files: M test/mjsunit/tools/codemap.js M test/mjsunit/tools/profile.js M test/mjsunit/tools/tickprocessor-test.default M test/mjsunit/tools/tickprocessor-test.ignore-unknown M test/mjsunit/tools/tickprocessor-test.separate-ic M test/mjsunit/tools/tickprocessor.js M tools/codemap.js M tools/profile.js M tools/tickprocessor.js --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
