On Tue, May 15, 2012 at 8:06 AM, Nathanael Nerode (GCC)
<ncn_gc...@fastmail.fm> wrote:
> On 05/11/2012 12:50 PM, H.J. Lu wrote:
>> On Mon, Apr 16, 2012 at 10:47 AM, H.J. Lu <hjl.to...@gmail.com> wrote:
>>> On Tue, Apr 3, 2012 at 7:49 AM, H.J. Lu <hjl.to...@gmail.com> wrote:
>>>> On Thu, Mar 29, 2012 at 7:34 AM, H.J. Lu <hjl.to...@gmail.com> wrote:
>>>>> On Sat, Mar 3, 2012 at 9:54 AM, H.J. Lu <hongjiu...@intel.com> wrote:
>>>>>> Hi,
>>>>>>
>>>>>> This patch backports x32 support to libtool:
>>>>>>
>>>>>> http://git.savannah.gnu.org/cgit/libtool.git/commit/?id=88992fe6771ec3258bde1b03314ce579da0ac2d5
>>>>>>
>>>>>> OK to install?
>>>>>>
>>>>>> Thanks.
>>>>>>
>>>>>>
>>>>>> H.J.
>>>>>> ---
>>>>>> ommit 0d8c092cac25c3bce5dbfc1981b84df91b3f6086
>>>>>> Author: H.J. Lu <hjl.to...@gmail.com>
>>>>>> Date:   Mon Dec 12 13:03:14 2011 -0800
>>>>>>
>>>>>>    Add x32 support to libtool.m4
>>>>>>
>>>>>>    2011-12-12  H.J. Lu  <hongjiu...@intel.com>
>>>>>>
>>>>>>        * libtool.m4 (_LT_ENABLE_LOCK): Support x32.
>>>>>>
>>>>>> diff --git a/ChangeLog.x32 b/ChangeLog.x32
>>>>>> new file mode 100644
>>>>>> index 0000000..b6e01ee
>>>>>> --- /dev/null
>>>>>> +++ b/ChangeLog.x32
>>>>>> @@ -0,0 +1,3 @@
>>>>>> +2011-12-12  H.J. Lu  <hongjiu...@intel.com>
>>>>>> +
>>>>>> +       * libtool.m4 (_LT_ENABLE_LOCK): Support x32.
>>>>>> diff --git a/libtool.m4 b/libtool.m4
>>>>>> index 67321a7..a7f99ac 100644
>>>>>> --- a/libtool.m4
>>>>>> +++ b/libtool.m4
>>>>>> @@ -1232,7 +1232,14 @@ s390*-*linux*|s390*-*tpf*|sparc*-*linux*)
>>>>>>            LD="${LD-ld} -m elf_i386_fbsd"
>>>>>>            ;;
>>>>>>          x86_64-*linux*)
>>>>>> -           LD="${LD-ld} -m elf_i386"
>>>>>> +           case `/usr/bin/file conftest.o` in
>>>>>> +             *x86-64*)
>>>>>> +               LD="${LD-ld} -m elf32_x86_64"
>>>>>> +               ;;
>>>>>> +             *)
>>>>>> +               LD="${LD-ld} -m elf_i386"
>>>>>> +               ;;
>>>>>> +           esac
>>>>>>            ;;
>>>>>>          ppc64-*linux*|powerpc64-*linux*)
>>>>>>            LD="${LD-ld} -m elf32ppclinux"
>>>>>
>>>>>
>>>>> Hi Ralf,
>>>>>
>>>>> Can you review this patch?
>>>>>
>>>>> Thanks.
>>>>>
>>>>
>>>> PING.
>>>>
>>>
>>> PING.
>>>
>>
>> PING.
>
> I'm not Ralf, and I don't fully understand the details of what's going
> on here, but here's my review question: you seem to be relying on the
> format of the output of 'file' containing the exact string "x86-64".  Is
> this the right string to look for?  Standard libtool seems to look for
> the phrase "64-bit" or "32-bit" in the output of 'file'.

Here is the original libtool.m4

   case `/usr/bin/file conftest.o` in
      *32-bit*)
        case $host in
          x86_64-*kfreebsd*-gnu)
            LD="${LD-ld} -m elf_i386_fbsd"
            ;;
          x86_64-*linux*)
            LD="${LD-ld} -m elf_i386"
            ;;

In case of "*32-bit", we want

LD="${LD-ld} -m elf32_x86_64"

if `file' also displays "x86-64".  Is my patch OK?

Thanks.

-- 
H.J.

Reply via email to