On Wed, Apr 4, 2012 at 8:47 PM, H.J. Lu <hjl.to...@gmail.com> wrote:
> On Wed, Apr 4, 2012 at 11:08 AM, Uros Bizjak <ubiz...@gmail.com> wrote:
>> Hello!
>>> This patch defines TRY_EMPTY_VM_SPACE for Linux/x32.  Tested on Linux/x32.
>>> OK for trunk?
>>> 2012-04-03  H.J. Lu  <hongjiu...@intel.com>
>>>       * config/host-linux.c (TRY_EMPTY_VM_SPACE): Defined to
>>>       0x60000000 for x32.
>> I think we can simply check for __LP64__, without version check, as is
>> the case with SPARC and MIPS targets.
> When you compile GCC 4.8 with GCC 3.2 on Linux/x86-64,
> __LP64__ won't be defined and TRY_EMPTY_VM_SPACE
> will be 0x60000000 instead of 0x1000000000.

Looking at how other targets implement this check, I don't think that
this is a problem at all. This issue only shows on a non-bootstrapped
build. A full bootstrap will use correct address.


Reply via email to