diff mbox

[PATCH/AARCH64] Handle ILP32 multi-arch

Message ID CA+=Sn1kwDEfEA3sqOnm5nZZfxsg4JuqRrYhJzFywJRbsrD9GGA@mail.gmail.com
State New
Headers show

Commit Message

Andrew Pinski Oct. 7, 2016, 9:08 p.m. UTC
Hi,
  This patch adds ilp32 multi-arch support.  This is needed to support
multi-arch on Debian like systems.

OK?  Bootstrapped and tested on aarch64-linux-gnu with no regressions.
Also tested with ilp32 with a newly built toolchain that supports
ILP32 with Ubuntu 1604 base.

Thanks,
Andrew

ChangeLog:
* config/aarch64/t-aarch64-linux (MULTILIB_OSDIRNAMES): Handle
multi-arch for ilp32.

Comments

Andrew Pinski Oct. 21, 2016, 8:24 a.m. UTC | #1
On Fri, Oct 7, 2016 at 2:08 PM, Andrew Pinski <pinskia@gmail.com> wrote:
> Hi,
>   This patch adds ilp32 multi-arch support.  This is needed to support
> multi-arch on Debian like systems.
>
> OK?  Bootstrapped and tested on aarch64-linux-gnu with no regressions.
> Also tested with ilp32 with a newly built toolchain that supports
> ILP32 with Ubuntu 1604 base.

Ping?

>
> Thanks,
> Andrew
>
> ChangeLog:
> * config/aarch64/t-aarch64-linux (MULTILIB_OSDIRNAMES): Handle
> multi-arch for ilp32.
Andrew Pinski Oct. 25, 2016, 5:35 p.m. UTC | #2
On Fri, Oct 21, 2016 at 1:24 AM, Andrew Pinski <pinskia@gmail.com> wrote:
> On Fri, Oct 7, 2016 at 2:08 PM, Andrew Pinski <pinskia@gmail.com> wrote:
>> Hi,
>>   This patch adds ilp32 multi-arch support.  This is needed to support
>> multi-arch on Debian like systems.
>>
>> OK?  Bootstrapped and tested on aarch64-linux-gnu with no regressions.
>> Also tested with ilp32 with a newly built toolchain that supports
>> ILP32 with Ubuntu 1604 base.
>
> Ping?
>
>>
>> Thanks,
>> Andrew
>>
>> ChangeLog:
>> * config/aarch64/t-aarch64-linux (MULTILIB_OSDIRNAMES): Handle
>> multi-arch for ilp32.
Matthias Klose Oct. 25, 2016, 10:25 p.m. UTC | #3
On 07.10.2016 23:08, Andrew Pinski wrote:
> Hi,
>   This patch adds ilp32 multi-arch support.  This is needed to support
> multi-arch on Debian like systems.
> 
> OK?  Bootstrapped and tested on aarch64-linux-gnu with no regressions.
> Also tested with ilp32 with a newly built toolchain that supports
> ILP32 with Ubuntu 1604 base.
> 
> Thanks,
> Andrew
> 
> ChangeLog:
> * config/aarch64/t-aarch64-linux (MULTILIB_OSDIRNAMES): Handle
> multi-arch for ilp32.

I can't approve that, but it looks like a reasonable change, but we should
document the multiarch triplet at https://wiki.debian.org/Multiarch/Tuples

Matthias
Andrew Pinski Nov. 11, 2016, 2:58 a.m. UTC | #4
On Tue, Oct 25, 2016 at 3:25 PM, Matthias Klose <doko@debian.org> wrote:
> On 07.10.2016 23:08, Andrew Pinski wrote:
>> Hi,
>>   This patch adds ilp32 multi-arch support.  This is needed to support
>> multi-arch on Debian like systems.
>>
>> OK?  Bootstrapped and tested on aarch64-linux-gnu with no regressions.
>> Also tested with ilp32 with a newly built toolchain that supports
>> ILP32 with Ubuntu 1604 base.
>>
>> Thanks,
>> Andrew
>>
>> ChangeLog:
>> * config/aarch64/t-aarch64-linux (MULTILIB_OSDIRNAMES): Handle
>> multi-arch for ilp32.
>
> I can't approve that, but it looks like a reasonable change, but we should
> document the multiarch triplet at https://wiki.debian.org/Multiarch/Tuples


Ping?

Thanks,
Andrew

>
> Matthias
>
Andrew Pinski Dec. 10, 2016, 9:24 p.m. UTC | #5
On Thu, Nov 10, 2016 at 6:58 PM, Andrew Pinski <pinskia@gmail.com> wrote:
> On Tue, Oct 25, 2016 at 3:25 PM, Matthias Klose <doko@debian.org> wrote:
>> On 07.10.2016 23:08, Andrew Pinski wrote:
>>> Hi,
>>>   This patch adds ilp32 multi-arch support.  This is needed to support
>>> multi-arch on Debian like systems.
>>>
>>> OK?  Bootstrapped and tested on aarch64-linux-gnu with no regressions.
>>> Also tested with ilp32 with a newly built toolchain that supports
>>> ILP32 with Ubuntu 1604 base.
>>>
>>> Thanks,
>>> Andrew
>>>
>>> ChangeLog:
>>> * config/aarch64/t-aarch64-linux (MULTILIB_OSDIRNAMES): Handle
>>> multi-arch for ilp32.
>>
>> I can't approve that, but it looks like a reasonable change, but we should
>> document the multiarch triplet at https://wiki.debian.org/Multiarch/Tuples
>
>
> Ping?

Ping?  This is the only outstanding GCC ILP32 related patch.

>
> Thanks,
> Andrew
>
>>
>> Matthias
>>
Andrew Pinski Jan. 3, 2017, 9:04 p.m. UTC | #6
Ping?

On Sat, Dec 10, 2016 at 1:24 PM, Andrew Pinski <pinskia@gmail.com> wrote:
> On Thu, Nov 10, 2016 at 6:58 PM, Andrew Pinski <pinskia@gmail.com> wrote:
>> On Tue, Oct 25, 2016 at 3:25 PM, Matthias Klose <doko@debian.org> wrote:
>>> On 07.10.2016 23:08, Andrew Pinski wrote:
>>>> Hi,
>>>>   This patch adds ilp32 multi-arch support.  This is needed to support
>>>> multi-arch on Debian like systems.
>>>>
>>>> OK?  Bootstrapped and tested on aarch64-linux-gnu with no regressions.
>>>> Also tested with ilp32 with a newly built toolchain that supports
>>>> ILP32 with Ubuntu 1604 base.
>>>>
>>>> Thanks,
>>>> Andrew
>>>>
>>>> ChangeLog:
>>>> * config/aarch64/t-aarch64-linux (MULTILIB_OSDIRNAMES): Handle
>>>> multi-arch for ilp32.
>>>
>>> I can't approve that, but it looks like a reasonable change, but we should
>>> document the multiarch triplet at https://wiki.debian.org/Multiarch/Tuples
>>
>>
>> Ping?
>
> Ping?  This is the only outstanding GCC ILP32 related patch.
>
>>
>> Thanks,
>> Andrew
>>
>>>
>>> Matthias
>>>
Andrew Pinski Feb. 7, 2017, 11:21 p.m. UTC | #7
On Tue, Jan 3, 2017 at 1:04 PM, Andrew Pinski <pinskia@gmail.com> wrote:
> Ping?
>
> On Sat, Dec 10, 2016 at 1:24 PM, Andrew Pinski <pinskia@gmail.com> wrote:
>> On Thu, Nov 10, 2016 at 6:58 PM, Andrew Pinski <pinskia@gmail.com> wrote:
>>> On Tue, Oct 25, 2016 at 3:25 PM, Matthias Klose <doko@debian.org> wrote:
>>>> On 07.10.2016 23:08, Andrew Pinski wrote:
>>>>> Hi,
>>>>>   This patch adds ilp32 multi-arch support.  This is needed to support
>>>>> multi-arch on Debian like systems.
>>>>>
>>>>> OK?  Bootstrapped and tested on aarch64-linux-gnu with no regressions.
>>>>> Also tested with ilp32 with a newly built toolchain that supports
>>>>> ILP32 with Ubuntu 1604 base.
>>>>>
>>>>> Thanks,
>>>>> Andrew
>>>>>
>>>>> ChangeLog:
>>>>> * config/aarch64/t-aarch64-linux (MULTILIB_OSDIRNAMES): Handle
>>>>> multi-arch for ilp32.
>>>>
>>>> I can't approve that, but it looks like a reasonable change, but we should
>>>> document the multiarch triplet at https://wiki.debian.org/Multiarch/Tuples
>>>
>>>
>>> Ping?
>>
>> Ping?  This is the only outstanding GCC ILP32 related patch.

Ping x4?

>>
>>>
>>> Thanks,
>>> Andrew
>>>
>>>>
>>>> Matthias
>>>>
diff mbox

Patch

Index: gcc/config/aarch64/t-aarch64-linux
===================================================================
--- gcc/config/aarch64/t-aarch64-linux	(revision 240867)
+++ gcc/config/aarch64/t-aarch64-linux	(working copy)
@@ -25,4 +25,4 @@  AARCH_BE = $(if $(findstring TARGET_BIG_
 MULTILIB_OSDIRNAMES = mabi.lp64=../lib64$(call if_multiarch,:aarch64$(AARCH_BE)-linux-gnu)
 MULTIARCH_DIRNAME = $(call if_multiarch,aarch64$(AARCH_BE)-linux-gnu)
 
-MULTILIB_OSDIRNAMES += mabi.ilp32=../libilp32
+MULTILIB_OSDIRNAMES += mabi.ilp32=../libilp32$(call if_multiarch,:aarch64$(AARCH_BE)_ilp32-linux-gnu)