supersize: Fix path counts, missing section in clusters

tools/binary_size/libsupersize/diff.py

 # Copyright 2017 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.
 """Logic for diffing two SizeInfo objects."""
 
 import collections
 import re
 
 import models
 
@@ skipping 14 matching lines @@
     ".L__unnamed_1193", ".L__unnamed_712"
   """
   name = symbol.full_name
   clone_idx = name.find(' [clone ')
   if clone_idx != -1:
     name = name[:clone_idx]
   if name.startswith('*'):
     # "symbol gap 3 (bar)" -> "symbol gaps"
     name = re.sub(r'\s+\d+( \(.*\))?$', 's', name)
 
+  # Use section rather than section_name since clang & gcc use
+  # .data.rel.ro vs .data.rel.ro.local.
   if '.' not in name:
-    return (symbol.section_name, name)
+    return (symbol.section, name)
+
   # Compiler or Linker generated symbol.
   name = re.sub(r'[.0-9]', '', name)  # Strip out all numbers and dots.
-  return (symbol.section_name, name, symbol.object_path)
+  return (symbol.section, name, symbol.object_path)
 
 
 def _CloneSymbol(sym, size):
   """Returns a copy of |sym| with an updated |size|.
 
   Padding and aliases are not copied.
   """
   return models.Symbol(
       sym.section_name, size, address=sym.address, full_name=sym.full_name,
       template_name=sym.template_name, name=sym.name,
@@ skipping 134 matching lines @@
   assert isinstance(after, models.SizeInfo)
   section_sizes = {k: after.section_sizes.get(k, 0) - v
                    for k, v in before.section_sizes.iteritems()}
   for k, v in after.section_sizes.iteritems():
     if k not in section_sizes:
       section_sizes[k] = v
 
   symbol_diff = _DiffSymbolGroups(before.raw_symbols, after.raw_symbols)
   return models.SizeInfoDiff(section_sizes, symbol_diff, before.metadata,
                              after.metadata)