Only include unique lines when patching orderfile.

tools/cygprofile/patch_orderfile.py

 #!/usr/bin/python
 # Copyright 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Patch an orderfile.
 
 Starting with a list of symbols in a binary and an orderfile (ordered list of
 symbols), matches the symbols in the orderfile and augments each symbol with the
 symbols residing at the same address (due to having identical code).
@@ skipping 84 matching lines @@
   """
   command = 'nm -S -n %s | egrep "( t )|( W )|( T )"' % binary_filename
   p = subprocess.Popen(command, shell=True, stdout=subprocess.PIPE)
   try:
     result = _GetSymbolInfosFromStream(p.stdout)
     return result
   finally:
     p.wait()
 
 
+def _StripPrefix(line):
+  """Get the symbol from a line with a linker section name.
+
+  Args:
+    line: a line from an orderfile, usually in the form:
+          .text.SymbolName
+
+  Returns:
+    The symbol, SymbolName in the example above.
+  """
+  line = line.rstrip('\n')
+  for prefix in _PREFIXES:
+    if line.startswith(prefix):
+      return line[len(prefix):]
+  return line  # Unprefixed case
+
+
 def _GetSymbolsFromStream(lines):
   """Get the symbols from an iterable of lines.
+     Filters out wildcards and lines which do not correspond to symbols.
 
   Args:
     lines: iterable of lines from an orderfile.
 
   Returns:
     Same as GetSymbolsFromOrderfile
   """
   # TODO(lizeb): Retain the prefixes later in the processing stages.
   symbols = []
+  unique_symbols = set()
   for line in lines:
-    line = line.rstrip('\n')
-    for prefix in _PREFIXES:
-      if line.startswith(prefix):
-        line = line[len(prefix):]
-        break
+    line = _StripPrefix(line)
     name = _RemoveClone(line)
-    if name == '':
+    if name == '' or name == '*' or name == '.text':
       continue
-    symbols.append(line)
+    if not line in unique_symbols:
+      symbols.append(line)
+      unique_symbols.add(line)
   return symbols
 
 
 def _GetSymbolsFromOrderfile(filename):
   """Return the symbols from an orderfile.
 
   Args:
     filename: The name of the orderfile.
 
   Returns:
@@ skipping 48 matching lines @@
   for symbol in symbol_infos:
     if symbol.offset not in seen_offsets:
       seen_offsets.add(symbol.offset)
       matching_symbols += [
           s.name for s in offset_to_symbol_infos[symbol.offset]]
   return matching_symbols
 
 
 def _PrintSymbolsWithPrefixes(symbol_names, output_file):
   """For each symbol, outputs it to output_file with the prefixes."""
+  unique_outputs = set()
   for name in symbol_names:
-    output_file.write('\n'.join(prefix + name for prefix in _PREFIXES) + '\n')
+    for prefix in _PREFIXES:
+      linker_section = prefix + name
+      if not linker_section in unique_outputs:
+        output_file.write(linker_section + '\n')
+        unique_outputs.add(linker_section)
 
 
 def main(argv):
   if len(argv) != 3:
     print 'Usage: %s <unpatched_orderfile> <libchrome.so>' % argv[0]
     return 1
   orderfile_filename = argv[1]
   binary_filename = argv[2]
   (offset_to_symbol_infos, name_to_symbol_infos) = _GetSymbolInfosFromBinary(
       binary_filename)
   profiled_symbols = _GetSymbolsFromOrderfile(orderfile_filename)
   matched_symbols = _MatchProfiledSymbols(
       profiled_symbols, name_to_symbol_infos)
   symbols_by_offset = _ExpandSymbolsWithDupsFromSameOffset(
       matched_symbols, offset_to_symbol_infos)
   _PrintSymbolsWithPrefixes(symbols_by_offset, sys.stdout)
   # The following is needed otherwise Gold only applies a partial sort.
   print '.text'    # gets methods not in a section, such as assembly
   print '.text.*'  # gets everything else
   return 0
 
 
 if __name__ == '__main__':
   logging.basicConfig(level=logging.INFO)
   sys.exit(main(sys.argv))