[PATCH] D39207: [llvm-objcopy] Add support for dwarf fission

[Jake Ehrlich via Phabricator via llvm-commits]

jakehehrlich updated this revision to Diff 121089.
jakehehrlich added a comment.

1. Added section count to tests to avoid false positives.
2. Fixed option descriptions.
3. Repalced "Dwo" with "DWO" everywhere including in cl::opt types.
4. Changed "-gsplit-dwarf" to "-split-dwo" but I think this should actually be removed. I'm waiting to hear back before I remove it.
5. Moved some comments around and altered them.
6. Removed extraneous passing of Obj and additionally removed the template.


Repository:
  rL LLVM

https://reviews.llvm.org/D39207

Files:
  test/tools/llvm-objcopy/Inputs/dwarf.dwo
  test/tools/llvm-objcopy/drawf-fission.test
  tools/llvm-objcopy/Object.h
  tools/llvm-objcopy/llvm-objcopy.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D39207.121089.patch
Type: text/x-patch
Size: 5987 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171101/493cbd3f/attachment.bin>