[all-commits] [llvm/llvm-project] bdd0ce: [lld-macho] Include symbol sizes in mapfile

[Jez Ng via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: bdd0cec5697f1df988cda1b5305fe9a8b869a680
      https://github.com/llvm/llvm-project/commit/bdd0cec5697f1df988cda1b5305fe9a8b869a680
  Author: Jez Ng <jezng at fb.com>
  Date:   2022-10-13 (Thu, 13 Oct 2022)

  Changed paths:
    M lld/MachO/MapFile.cpp
    M lld/test/MachO/dead-strip.s
    M lld/test/MachO/map-file.s

  Log Message:
  -----------
  [lld-macho] Include symbol sizes in mapfile

This matches ld64's behavior.

Additionally, I edited the "Dead Stripped Symbols" header to omit "Address" --
this also matches ld64.

Reviewed By: #lld-macho, oontvoo

Differential Revision: https://reviews.llvm.org/D135883


  Commit: 922346ef052f05cfef79b184740217b75647fa34
      https://github.com/llvm/llvm-project/commit/922346ef052f05cfef79b184740217b75647fa34
  Author: Jez Ng <jezng at fb.com>
  Date:   2022-10-13 (Thu, 13 Oct 2022)

  Changed paths:
    M lld/MachO/Driver.cpp
    M lld/test/MachO/dead-strip.s
    M lld/test/MachO/map-file.s

  Log Message:
  -----------
  [lld-macho] Preserve the size of common symbols

We never noticed this before because the only time the size gets emitted is via
the linker map...

Reviewed By: #lld-macho, thakis

Differential Revision: https://reviews.llvm.org/D135884


Compare: https://github.com/llvm/llvm-project/compare/aa5bc41b7656...922346ef052f