Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(161)

Side by Side Diff: tools/binary_size/README.md

Issue 2795593005: //tools/binary_size: Various enhancements to console.py (Closed)
Patch Set: Review fixes Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | tools/binary_size/console.py » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # map2size.py 1 # map2size.py
2 2
3 Parses a linker .map(.gz) file and outputs the result as a .size(.gz) file. 3 Parses a linker .map(.gz) file and outputs the result as a .size(.gz) file.
4 4
5 ## Example Usage: 5 ## Example Usage:
6 6
7 # Android: 7 # Android:
8 gn gen out/Release --args='target_os="android" is_official_build=true' 8 gn gen out/Release --args='target_os="android" is_official_build=true'
9 ninja -C out/Release -j 1000 libchrome.so 9 ninja -C out/Release -j 1000 libchrome.so
10 tools/binary_size/map2size.py out/Release/lib.unstripped/libchrome.so.map.gz chrome.size -v 10 tools/binary_size/map2size.py out/Release/lib.unstripped/libchrome.so.map.gz chrome.size -v
(...skipping 27 matching lines...) Expand all
38 38
39 Tracked in https://crbug.com/681694 39 Tracked in https://crbug.com/681694
40 40
41 1. More console.py features: 41 1. More console.py features:
42 * Template Symbols - shows when templates lead to code bloat. 42 * Template Symbols - shows when templates lead to code bloat.
43 * Duplicate Symbols - shows when statics in headers are an issue. 43 * Duplicate Symbols - shows when statics in headers are an issue.
44 * Overloaded Symbols - shows when overloads are excessive. 44 * Overloaded Symbols - shows when overloads are excessive.
45 * Per-class / namespace size (no way to distinguish class vs namespace). 45 * Per-class / namespace size (no way to distinguish class vs namespace).
46 * Per-Chrome package (Chrome-specific grouping. e.g. name prefixes). 46 * Per-Chrome package (Chrome-specific grouping. e.g. name prefixes).
47 * CSV output (for pasting into a spreadsheet). 47 * CSV output (for pasting into a spreadsheet).
48 * Allow Print() to not sort.
49 1. More create_html_breakdown.py features: 48 1. More create_html_breakdown.py features:
50 * Break down by other groupings (e.g. create from nested `SymbolGroups`) 49 * Break down by other groupings (e.g. create from nested `SymbolGroups`)
51 1. More `map2size.py` features: 50 1. More `map2size.py` features:
52 * Find out more about 0xffffffffffffffff addresses, and why such large 51 * Find out more about 0xffffffffffffffff addresses, and why such large
53 gaps exist after them. 52 gaps exist after them.
54 * Use nm to get the full list of symbols that share the same address. 53 * Use nm to get the full list of symbols that share the same address.
55 1. Integrate with `resource_sizes.py` so that it tracks size of major 54 1. Integrate with `resource_sizes.py` so that it tracks size of major
56 components separately: chrome vs blink vs skia vs v8. 55 components separately: chrome vs blink vs skia vs v8.
57 1. Speed up some steps (like normalizing names) via multiprocessing. 56 1. Speed up some steps (like normalizing names) via multiprocessing.
58 1. Use resource whitelist information to attribute .pak file size to .o files. 57 1. Use resource whitelist information to attribute .pak file size to .o files.
59 1. Add dependency graph info, perhaps just on a per-file basis. 58 1. Add dependency graph info, perhaps just on a per-file basis.
OLDNEW
« no previous file with comments | « no previous file | tools/binary_size/console.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698