[PATCH] D84574: [lld-macho] Fix segment filesize calculation
Jez Ng via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 27 18:17:41 PDT 2020
int3 added inline comments.
================
Comment at: lld/MachO/Writer.cpp:137
for (OutputSection *osec : seg->getSections()) {
- c->filesize += osec->getFileSize();
+ // This condition should hold unless we are dealing with a __bss section.
+ if (osec->fileOff >= seg->fileOff)
----------------
compnerd wrote:
> I wonder if that should just be an assert instead.
Agreed :)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D84574/new/
https://reviews.llvm.org/D84574
More information about the llvm-commits
mailing list