[PATCH] D33251: [lld][ELF]Add option to make .dynamic read only

Jake Ehrlich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 25 19:12:04 PDT 2017


jakehehrlich updated this revision to Diff 100355.
jakehehrlich added a comment.

Right! I forgot to check that DEBUG wasn't being emitted in the test. I also forgot to check the default behavior. The other part of the test ensures that the .dynamic section is in fact read-only and that in the default case it is read-write. I had to produce a shared library to make the executable have a .dynamic section as well so I added a simple file to Inputs.


Repository:
  rL LLVM

https://reviews.llvm.org/D33251

Files:
  ELF/Config.h
  ELF/Driver.cpp
  ELF/SyntheticSections.cpp
  test/ELF/Inputs/rodynamic.s
  test/ELF/rodynamic.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D33251.100355.patch
Type: text/x-patch
Size: 3669 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170526/420e3b86/attachment.bin>


More information about the llvm-commits mailing list