diff options
author | azarchs <azarchs@chromium.org> | 2015-02-10 06:06:52 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-02-10 14:07:29 +0000 |
commit | 8de686f6e9920431b41b93fd4211e6d2119189a3 (patch) | |
tree | d52cff58e0c038cfeacc9c15921e9f551c77c8ab | |
parent | 011b6d7dc582002027dad48b1a8dcf3c9f9de4d6 (diff) | |
download | chromium_src-8de686f6e9920431b41b93fd4211e6d2119189a3.zip chromium_src-8de686f6e9920431b41b93fd4211e6d2119189a3.tar.gz chromium_src-8de686f6e9920431b41b93fd4211e6d2119189a3.tar.bz2 |
Don't output duplicate section names in unpatched orderfile.
Currently at least half the lines in the ~17MB unpatched orderfile are
useless duplicates. Since we eventually want builders to download the
unpatched orderfile for repatching, this will become meaninful.
Review URL: https://codereview.chromium.org/912793002
Cr-Commit-Position: refs/heads/master@{#315554}
-rwxr-xr-x | tools/cygprofile/cyglog_to_orderfile.py | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/tools/cygprofile/cyglog_to_orderfile.py b/tools/cygprofile/cyglog_to_orderfile.py index db12328..e81dac1 100755 --- a/tools/cygprofile/cyglog_to_orderfile.py +++ b/tools/cygprofile/cyglog_to_orderfile.py @@ -201,12 +201,16 @@ def _OutputOrderfile(offsets, offset_to_symbol_infos, symbol_to_section_map, success = True unknown_symbol_warnings = WarningCollector(300) symbol_not_found_warnings = WarningCollector(300) + output_sections = set() for offset in offsets: try: symbol_infos = _FindSymbolInfosAtOffset(offset_to_symbol_infos, offset) for symbol_info in symbol_infos: if symbol_info.name in symbol_to_section_map: - output_file.write(symbol_to_section_map[symbol_info.name] + '\n') + section = symbol_to_section_map[symbol_info.name] + if not section in output_sections: + output_file.write(section + '\n') + output_sections.add(section) else: unknown_symbol_warnings.Write( 'No known section for symbol ' + symbol_info.name) |