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

Roland McGrath via llvm-commits llvm-commits at lists.llvm.org
Thu May 25 14:12:33 PDT 2017


Look, we have a new system, and we're defining what the ABI will be
for our system.
We've decided we want .dynamic to be read-only and we are confident in
that choice.
It's OK if you aren't convinced that's the decision you would make in
your ABI, but it's the decision we've made for our ABI.

I don't see why you feel the need to push back on a new platform's ABI
decisions when we've done the implementation
work for LLD to support the ABI and the actual impact on the LLD code
base is tiny.

Since LLD doesn't support selecting its behavior based on target
triple, we've added a switch.
If you'd prefer this switch were spelled --target=fuchsia rather than
-z rodynamic, that's fine with us.


Thanks,
Roland


More information about the llvm-commits mailing list