This is an archive of the discontinued LLVM Phabricator instance.

[ELF] - Round executable PT_LOAD to page size and pad it with traps
AbandonedPublic

Authored by grimar on Jul 20 2017, 5:38 AM.

Details

Summary

This is PR33819,

Patch attempts to simplify outdated D33630.

Diff Detail

Event Timeline

grimar created this revision.Jul 20 2017, 5:38 AM
grimar retitled this revision from {ELF] - Round executable PT_LOAD to page size and pad it with traps to [ELF] - Round executable PT_LOAD to page size and pad it with traps.Jul 20 2017, 5:39 AM
grimar updated this revision to Diff 108435.Jul 27 2017, 2:09 AM
grimar edited the summary of this revision. (Show Details)
  • Rebased.
  • Simplified.
grimar abandoned this revision.Aug 3 2017, 1:47 AM

r309829

joerg added a subscriber: joerg.Aug 3 2017, 4:53 AM

I'm not sure I want anything like this unconditionally. It is going to waste quite a bit of space, i.e. 2KB per executable and shared library sums up.

grimar added a comment.Aug 3 2017, 4:56 AM

I'm not sure I want anything like this unconditionally. It is going to waste quite a bit of space, i.e. 2KB per executable and shared library sums up.

Just in case: this patch was abandoned, D33630 was landed. It do something about the same though, but probably more correct to discuss in its thread.