This is an archive of the discontinued LLVM Phabricator instance.

[ELF] Set p_memsz to p_filesz when aligning the last segment to page boundary
ClosedPublic

Authored by phosek on Aug 31 2017, 7:33 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

phosek created this revision.Aug 31 2017, 7:33 PM
grimar added a subscriber: grimar.Sep 1 2017, 12:42 AM

Looks good. Minor nit.

ELF/Writer.cpp
1935

Looks unformatted.

grimar added inline comments.Sep 1 2017, 1:18 AM
ELF/Writer.cpp
1935

It is formatted. Sorry for noise,

ruiu accepted this revision.Sep 1 2017, 1:53 PM

LGTM

This revision is now accepted and ready to land.Sep 1 2017, 1:53 PM
This revision was automatically updated to reflect the committed changes.