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

[Alexander Richardson via Phabricator via llvm-commits]

arichardson accepted this revision.
arichardson added a comment.
This revision is now accepted and ready to land.

Thanks for fixing this! LGTM



================
Comment at: llvm/tools/llvm-objcopy/CopyConfig.cpp:277
     // Name, {EMachine, OSABI, 64bit, LittleEndian}
     {"elf32-i386", {ELF::EM_386, ELF::ELFOSABI_NONE, false, true}},
+    {"elf32-iamcu", {ELF::EM_IAMCU, ELF::ELFOSABI_NONE, false, true}},
----------------
All of these now use `ELF::ELFOSABI_NONE`. I wonder if we can remove the `ELF::ELFOSABI_NONE` from the initializer by adding a constructor with three arguments or making it the last member of the struct and adding a default initializer.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60773