[PATCH] D22116: [ELF] Support for setting the base address

Rafael EspĂ­ndola via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 11 12:32:33 PDT 2016


On 8 July 2016 at 21:21, Petr Hosek <phosek at google.com> wrote:
> phosek added a comment.
>
> In general, this is an option supported by all existing linkers and it's used by the existing software; having to use a custom linker script just to change the start address is a bit of an overkill.

I am sympathetic to this. In fact, I think that is why -T* were fist
implemented in gold: to avoid needing to use a linker script for a few
basic cases.

The cost of supporting the option as defined is also really small
since it plugs in the support we need to have for different targets.

So this LGTM, but wait for Rui's to approve it too.

Cheers,
Rafael


More information about the llvm-commits mailing list