[PATCH] D56828: [ELF] Make RW PT_LOAD start with PT_GNU_RELRO sections

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 1 17:10:07 PST 2019


pcc added a comment.

Well, with the old layout you could still place `.data.rel.ro` and `.bss.rel.ro` on the same page, but with the new one `.data` and `.bss` can now be placed together. That's probably where you get the largest benefit since they're more commonly used.

In terms of mappings I guess you save a mapping unless `.bss.rel.ro` is large enough to require another page, in which case you need another PT_LOAD (and another mapping).


Repository:
  rLLD LLVM Linker

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56828/new/

https://reviews.llvm.org/D56828





More information about the llvm-commits mailing list