[PATCH] D67700: [Object] Extend MachOUniversalBinary::getObjectForArch

Saleem Abdulrasool via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 18 15:00:55 PDT 2019


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

Okay, both @mtrent and @alexshap bring up good points.  I *suppose* that is okay, but I shall string grumble.  @mtrent, I think we could also simply make `getObjectForArch` be internal to the type, and force users to go down either `getMachOArchiveForArch` or `getMachOObjectForArch`.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D67700





More information about the llvm-commits mailing list