[PATCH] D27042: lld: Default image base address to 0x400000

Ed Maste via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 23 06:31:38 PST 2016


On 23 November 2016 at 09:19, Rafael EspĂ­ndola
<rafael.espindola at gmail.com> wrote:
> I would have a small preference for 2MB on X86_64 and 4MB on x86 (if
> anyone cares about x86).

2MB is common to arm64 and x86-64 (my platforms of interest), and I
think MIPS will be 2MB as well. Having x86 be an eventual outlier at
4MB seems fine.

> also, please change only x86_64 for now. You can set DefaultImageBase
> in the constructor.

OK, good point: that will certainly make it easier to update and
review a subset of tests. However, I will want to update arm64 very
shortly thereafter.


More information about the llvm-commits mailing list