[PATCH] D51607: [ELF] Fix bugzilla #38748 for option no-rosegment

PeiHsiangHung via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 3 18:39:23 PDT 2018


NickHung created this revision.
NickHung added a reviewer: lld.
Herald added subscribers: llvm-commits, arichardson, emaste.
Herald added a reviewer: espindola.

  Fix option no-rosegment for bugzilla #38784
  
  * Prior to this revision, lld always groups read-only sections into
  executable segments. This decision might cause a huge executable size as
  described in the bugzilla.
  * This revision tries to group the following sections with the current
  segment if no-rosegment is on.
  
  * add a new test case: no-rosegment.s
  * modify 4 test cases accordingly.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D51607

Files:
  ELF/Writer.cpp
  test/ELF/linkerscript/at-self-reference.s
  test/ELF/linkerscript/at.s
  test/ELF/linkerscript/overlay.test
  test/ELF/no-rosegment.s
  test/ELF/phdr-align.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51607.163754.patch
Type: text/x-patch
Size: 5008 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180904/4c7f2965/attachment.bin>


More information about the llvm-commits mailing list