[PATCH] D39132: [llvm-objcopy] Fix bug in how segment alignment was being handled

[Jake Ehrlich via Phabricator via llvm-commits]

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

1. Added test for -O binary case
2. That test still made no sense. It should test the fact that even though the second segment has alignment 0x1000 it can't still have offset 0x1008.


Repository:
  rL LLVM

https://reviews.llvm.org/D39132

Files:
  test/tools/llvm-objcopy/check-addr-offset-align-binary.test
  test/tools/llvm-objcopy/check-addr-offset-align.test
  tools/llvm-objcopy/Object.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D39132.121096.patch
Type: text/x-patch
Size: 4929 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171101/0b7f8ffc/attachment.bin>