[all-commits] [llvm/llvm-project] ca518c: [llvm-install-name-tool] Add -id option

[Sameer Arora via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: ca518c49183aa3bace0def1827c1def7a28fa293
      https://github.com/llvm/llvm-project/commit/ca518c49183aa3bace0def1827c1def7a28fa293
  Author: Sameer Arora <sameerarora101 at fb.com>
  Date:   2020-06-30 (Tue, 30 Jun 2020)

  Changed paths:
    A llvm/test/tools/llvm-objcopy/MachO/install-name-tool-id.test
    M llvm/tools/llvm-objcopy/CopyConfig.cpp
    M llvm/tools/llvm-objcopy/CopyConfig.h
    M llvm/tools/llvm-objcopy/InstallNameToolOpts.td
    M llvm/tools/llvm-objcopy/MachO/MachOObjcopy.cpp

  Log Message:
  -----------
  [llvm-install-name-tool] Add -id option

Implement `-id` option for install-name-tool. Differences from cctool's
behavior:
 - Does **NOT** throw an error if multiple -id options are specified.
    Instead, picks the last one.
 - Throws an error in case empty id is specified.

Reviewed By: jhenderson, smeenai

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


  Commit: 2bdcd8b14f470b06fa6cf48972902e4b8030b088
      https://github.com/llvm/llvm-project/commit/2bdcd8b14f470b06fa6cf48972902e4b8030b088
  Author: Sameer Arora <sameerarora101 at fb.com>
  Date:   2020-06-30 (Tue, 30 Jun 2020)

  Changed paths:
    A llvm/test/tools/llvm-objcopy/MachO/install-name-tool-change.test
    M llvm/tools/llvm-objcopy/CopyConfig.cpp
    M llvm/tools/llvm-objcopy/CopyConfig.h
    M llvm/tools/llvm-objcopy/InstallNameToolOpts.td
    M llvm/tools/llvm-objcopy/MachO/MachOObjcopy.cpp

  Log Message:
  -----------
  [llvm-install-name-tool] Add -change option

Implement `-change` option for install-name-tool. The behavior exactly
matches that of cctools. Depends on D82410.

Reviewed By: jhenderson, smeenai

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


Compare: https://github.com/llvm/llvm-project/compare/b7402edce315...2bdcd8b14f47