Skip to content

Commit

Permalink
[Size reports] Remove .heap from reports (#13056)
Browse files Browse the repository at this point in the history
#### Problem

For platforms that use it, the `.heap` section indicates memory
available for the heap, that is, RAM not otherwise in use.
Including it in size reports is confusing because a `.heap` increase
is good.

#### Change overview

Remove `.heap` from the default section list of the platforms
where it currently appears, namely Mbed and P6.

#### Testing

Manually checked on a P6 binary.
  • Loading branch information
kpschoedel authored and pull[bot] committed Mar 9, 2022
1 parent 479a30f commit 1598772
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion scripts/tools/memory/platform/mbed.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
'section': {
# By default, only these sections will be included
# when operating by sections.
'default': ['.text', '.data', '.bss', '.heap'],
'default': ['.text', '.data', '.bss'],
},
# 'symbol': {
# 'free': {
Expand Down
2 changes: 1 addition & 1 deletion scripts/tools/memory/platform/p6.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
'section': {
# By default, only these sections will be included
# when operating by sections.
'default': ['.text', '.data', '.bss', '.heap'],
'default': ['.text', '.data', '.bss'],
},
# 'symbol': {
# 'free': {
Expand Down

0 comments on commit 1598772

Please sign in to comment.