[PATCH] D60773: [llvm-objcopy] Support full list of bfd targets that lld uses.

Wed Apr 17 00:47:17 PDT 2019

rupprecht added inline comments.

================
Comment at: llvm/tools/llvm-objcopy/CopyConfig.h:33-34
 struct MachineInfo {
+  MachineInfo(uint16_t EM, uint8_t ABI, bool Is64, bool IsLittle)
+      : EMachine(EM), OSABI(ABI), Is64Bit(Is64), IsLittleEndian(IsLittle) {}
+  // Alternative constructor that defaults to NONE for OSABI.
----------------
jhenderson wrote:
> Where is this constructor still used?
Missed this when committing: this is shared by the two constructors below. It's a bit speculative; it assumes one might want to explicitly create a MachineInfo with a non-default OSABI. Perhaps we should remove it when extracting this+the same logic in lld into libObject.

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60773/new/

https://reviews.llvm.org/D60773