buildman: Include symbols in the read-only data section
When symbols switch between the inited data section and the read-only data section their visbility changes, at present, with the -B option. This is confusing, since adding 'const' to a variable declaration can make it look like a significant improvement in bloat. But in fact nothing has changed. Add 'r' to the list of symbols types that are recorded, to correct this problem. Add a constant to make it easier to find this code next time. Signed-off-by: Simon Glass <sjg@chromium.org> Reported-by: Tom Rini <trini@konsulko.com> Reviewed-by: Tom Rini <trini@konsulko.com>
This commit is contained in:
parent
1f46e8af42
commit
ad8dbabc22
|
@ -35,6 +35,10 @@ from u_boot_pylib.terminal import tprint
|
|||
# which indicates that BREAK_ME has an empty default
|
||||
RE_NO_DEFAULT = re.compile(b'\((\w+)\) \[] \(NEW\)')
|
||||
|
||||
# Symbol types which appear in the bloat feature (-B). Others are silently
|
||||
# dropped when reading in the 'nm' output
|
||||
NM_SYMBOL_TYPES = 'tTdDbBr'
|
||||
|
||||
"""
|
||||
Theory of Operation
|
||||
|
||||
|
@ -693,7 +697,7 @@ class Builder:
|
|||
parts = line.split()
|
||||
if line and len(parts) == 3:
|
||||
size, type, name = line.split()
|
||||
if type in 'tTdDbB':
|
||||
if type in NM_SYMBOL_TYPES:
|
||||
# function names begin with '.' on 64-bit powerpc
|
||||
if '.' in name[1:]:
|
||||
name = 'static.' + name.split('.')[0]
|
||||
|
|
Loading…
Reference in New Issue
Block a user