[all-commits] [llvm/llvm-project] 242002: [llvm-objcopy][MachO] Fix symbol order in the symb...

[Fangrui Song via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 242002770ba00516339875a064f9ae0efd78a8dc
      https://github.com/llvm/llvm-project/commit/242002770ba00516339875a064f9ae0efd78a8dc
  Author: Fangrui Song <i at maskray.me>
  Date:   2019-11-21 (Thu, 21 Nov 2019)

  Changed paths:
    M llvm/test/tools/llvm-objcopy/MachO/Inputs/strip-all.yaml
    M llvm/test/tools/llvm-objcopy/MachO/strip-all.test
    M llvm/tools/llvm-objcopy/MachO/MachOLayoutBuilder.cpp

  Log Message:
  -----------
  [llvm-objcopy][MachO] Fix symbol order in the symbol table

Only consider isUndefinedSymbol() when the symbol is not local. This
fixes an assert failure when copying the symbol table, if a n_type=0x20
symbol is followed by a n_type=0x64 symbol.

Reviewed By: alexshap, seiya

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