[all-commits] [llvm/llvm-project] 163dcd: [lld-macho] Associate each Symbol with an InputFile

Jez Ng via All-commits all-commits at lists.llvm.org
Wed Feb 3 10:44:12 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 163dcd851309d0bfbbbe0b7810335f30d88ee879
      https://github.com/llvm/llvm-project/commit/163dcd851309d0bfbbbe0b7810335f30d88ee879
  Author: Jez Ng <jezng at fb.com>
  Date:   2021-02-03 (Wed, 03 Feb 2021)

  Changed paths:
    M lld/MachO/Driver.cpp
    M lld/MachO/ExportTrie.cpp
    M lld/MachO/InputFiles.cpp
    M lld/MachO/SymbolTable.cpp
    M lld/MachO/SymbolTable.h
    M lld/MachO/Symbols.cpp
    M lld/MachO/Symbols.h
    M lld/MachO/SyntheticSections.cpp
    M lld/MachO/Writer.cpp
    M lld/test/MachO/invalid/abs-duplicate.s
    M lld/test/MachO/invalid/duplicate-symbol.s

  Log Message:
  -----------
  [lld-macho] Associate each Symbol with an InputFile

This makes our error messages more informative. But the bigger motivation is for
LTO symbol resolution, which will be in an upcoming diff. The changes in this
one are largely mechanical.

Reviewed By: #lld-macho, smeenai

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


  Commit: f843bb82c042f9e669135957bb5bd1c233c5c316
      https://github.com/llvm/llvm-project/commit/f843bb82c042f9e669135957bb5bd1c233c5c316
  Author: Jez Ng <jezng at fb.com>
  Date:   2021-02-03 (Wed, 03 Feb 2021)

  Changed paths:
    M lld/MachO/CMakeLists.txt
    M lld/MachO/Driver.cpp
    M lld/MachO/Driver.h
    M lld/MachO/DriverUtils.cpp
    M lld/MachO/InputFiles.cpp
    M lld/test/MachO/invalid/bad-archive-member.s
    M lld/test/MachO/invalid/bad-archive.s
    M lld/test/MachO/lto-archive.ll

  Log Message:
  -----------
  [lld-macho] Force-loading should share code path with regular archive loads

This extends {D92539} to work even when we are loading archive
members via `-force_load`. I uncovered this issue while trying to
force-load archives containing bitcode -- we were segfaulting.

In addition to fixing the `-force_load` case, this diff also addresses
the behavior of `-ObjC` when LTO bitcode is involved -- we need to
force-load those archive members if they contain ObjC categories.

Reviewed By: #lld-macho, smeenai

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


  Commit: 4b2169fb6b3ce75e946806070debb9f0d38e415a
      https://github.com/llvm/llvm-project/commit/4b2169fb6b3ce75e946806070debb9f0d38e415a
  Author: Jez Ng <jezng at fb.com>
  Date:   2021-02-03 (Wed, 03 Feb 2021)

  Changed paths:
    M lld/MachO/Writer.cpp

  Log Message:
  -----------
  [lld-macho] Remove stray ehFrame change

Per https://reviews.llvm.org/D95121#inline-897943.


  Commit: 2d2e0000d31fb76f5201e3b40bc33790a1086450
      https://github.com/llvm/llvm-project/commit/2d2e0000d31fb76f5201e3b40bc33790a1086450
  Author: Jez Ng <jezng at fb.com>
  Date:   2021-02-03 (Wed, 03 Feb 2021)

  Changed paths:
    M lld/MachO/Options.td

  Log Message:
  -----------
  [lld-macho] Rename VERSION CONTROL to VERSION TARGETING in helptext

Per https://reviews.llvm.org/D94938#inline-896740.


Compare: https://github.com/llvm/llvm-project/compare/d886da042c65...2d2e0000d31f


More information about the All-commits mailing list