[PATCH] D53790: [llvm-objcopy] Move elf-specific code into subfolder

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 29 03:53:55 PDT 2018


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

LGTM, assuming my question below is answered positively. I think this might be a good start towards librarifying things too. I envisage a similar structure to LLD's lldELF/lldCOFF/lldCore long-term, with each of the different libraries in a different directory.

Have you checked that the include list is the right set for the source files, i.e. that you haven't moved across unnecessary includes?


Repository:
  rL LLVM

https://reviews.llvm.org/D53790





More information about the llvm-commits mailing list