google / bloaty

Bloaty: a size profiler for binaries
Apache License 2.0
4.79k stars 346 forks source link

Size diff broken after PR #313 "Preserve Size Information During Diff" #361

Closed carlosperate closed 1 year ago

carlosperate commented 1 year ago

Building the parent commit before the merge f01ea59bdda11708d74a3826c23d6e2db6c996f0 and comparing an elf file with itself, we get the expected 0 size diff:

$ ./bloaty  MICROBIT.elf -- MICROBIT.elf
    FILE SIZE        VM SIZE    
 --------------  -------------- 
  [ = ]       0  [ = ]       0    TOTAL

Building the merge commit a1bbc93f5f6f969242046dffd9deb379f6735020 of https://github.com/google/bloaty/pull/313, and we get:

$ ./bloaty2  MICROBIT.elf -- MICROBIT.elf
    FILE SIZE        VM SIZE    
 --------------  -------------- 
  +100% +2.15Mi  +100%  +211Ki    TOTAL

Elf file used for testing was built with arm-none-eabi-gcc v12.3.1: