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

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 16 12:48:09 PDT 2017


ruiu added a comment.

I have a couple of questions.

- Why can't .dynamic be read-only for all output?
- .dynamic is already covered by PT_GNU_RELRO segments. Does it make sense to make it read-only from beginning?

It seems that you are proposing a new command line option that even GNU linkers don't have, so this patch needs extra attention. Are you adding that option to GNU linkers too?


Repository:
  rL LLVM

https://reviews.llvm.org/D33251





More information about the llvm-commits mailing list