[PATCH] D58426: llvm-objcopy: Change sectionWithinSegment() to use virtual addresses instead of file offsets for SHT_NOBITS sections.
Jake Ehrlich via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 23 17:15:58 PDT 2019
jakehehrlich accepted this revision.
jakehehrlich added a comment.
This revision is now accepted and ready to land.
I think this is good. The current behavior treats sectionWithinSegment as not mattering in this case because this predicate is used for getting file offsets right and ignores the SHT_NOBITS case. This is a strict improvement that just sets the predicate in such a way that llvm-objcopy will output the conventional offset now.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58426/new/
https://reviews.llvm.org/D58426
More information about the llvm-commits
mailing list