[ARC] Allow more ABIs in GLIBC_DYNAMIC_LINKER

Message ID 20200329000503.27897-1-vgupta@synopsys.com
State New
Headers show
Series
  • [ARC] Allow more ABIs in GLIBC_DYNAMIC_LINKER
Related show

Commit Message

Jakub Jelinek via Gcc-patches March 29, 2020, 12:05 a.m.
Enable big-endian suffixed dynamic linker per glibc multi-abi support.

And to avoid a future churn and version pairingi hassles, also allow
arc700 although glibc for ARC currently doesn't support it.

gcc/
xxxx-xx-xx  Vineet Gupta <vgupta@synopsys.com>
+
+       * config/arc/linux.h: GLIBC_DYNAMIC_LINKER support BE/arc700

Signed-off-by: Vineet Gupta <vgupta@synopsys.com>

---
 gcc/ChangeLog          | 4 ++++
 gcc/config/arc/linux.h | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

-- 
2.20.1

Comments

Jakub Jelinek via Gcc-patches March 31, 2020, 10:06 a.m. | #1
Pushed.

Thank you,
Claudiu

On Sun, Mar 29, 2020 at 2:05 AM Vineet Gupta via Gcc-patches
<gcc-patches@gcc.gnu.org> wrote:
>

> Enable big-endian suffixed dynamic linker per glibc multi-abi support.

>

> And to avoid a future churn and version pairingi hassles, also allow

> arc700 although glibc for ARC currently doesn't support it.

>

> gcc/

> xxxx-xx-xx  Vineet Gupta <vgupta@synopsys.com>

> +

> +       * config/arc/linux.h: GLIBC_DYNAMIC_LINKER support BE/arc700

>

> Signed-off-by: Vineet Gupta <vgupta@synopsys.com>

> ---

>  gcc/ChangeLog          | 4 ++++

>  gcc/config/arc/linux.h | 2 +-

>  2 files changed, 5 insertions(+), 1 deletion(-)

>

> diff --git a/gcc/ChangeLog b/gcc/ChangeLog

> index 86ad683a6cb0..c26a748fd51b 100644

> --- a/gcc/ChangeLog

> +++ b/gcc/ChangeLog

> @@ -1,3 +1,7 @@

> +2020-03-28  Vineet Gupta <vgupta@synopsys.com>

> +

> +       * config/arc/linux.h: GLIBC_DYNAMIC_LINKER support BE/arc700

> +

>  2020-03-28  Jakub Jelinek  <jakub@redhat.com>

>

>         PR c/93573

> diff --git a/gcc/config/arc/linux.h b/gcc/config/arc/linux.h

> index 0b99da3fcdaf..1bbeccee7115 100644

> --- a/gcc/config/arc/linux.h

> +++ b/gcc/config/arc/linux.h

> @@ -29,7 +29,7 @@ along with GCC; see the file COPYING3.  If not see

>      }                                          \

>    while (0)

>

> -#define GLIBC_DYNAMIC_LINKER   "/lib/ld-linux-arc.so.2"

> +#define GLIBC_DYNAMIC_LINKER   "/lib/ld-linux-arc%{mbig-endian:eb}%{mcpu=arc700:700}.so.2"

>  #define UCLIBC_DYNAMIC_LINKER  "/lib/ld-uClibc.so.0"

>

>  /* Note that the default is to link against dynamic libraries, if they are

> --

> 2.20.1

>
Jakub Jelinek via Gcc-patches March 31, 2020, 4:26 p.m. | #2
Hi Claus,

> -----Original Message-----

> From: linux-snps-arc <linux-snps-arc-bounces@lists.infradead.org> On Behalf Of Claudiu Zissulescu

> Ianculescu

> Sent: Tuesday, March 31, 2020 1:07 PM

> To: Vineet Gupta <vgupta@synopsys.com>

> Cc: linux-snps-arc@lists.infradead.org; gcc-patches@gcc.gnu.org; Claudiu Zissulescu

> <claziss@synopsys.com>

> Subject: Re: [PATCH] [ARC] Allow more ABIs in GLIBC_DYNAMIC_LINKER

> 

> Pushed.


Is this one eligible for being back-ported to older GCCs?
At least GCC 9.x would be really good.

-Alexey
Jakub Jelinek via Gcc-patches March 31, 2020, 5:57 p.m. | #3
Well its a hard requirement considering glibc is still using gcc-9 !

Thx,
-Vineet

On 3/31/20 9:26 AM, Alexey Brodkin wrote:
> Hi Claus,

>

>> -----Original Message-----

>> From: linux-snps-arc <linux-snps-arc-bounces@lists.infradead.org> On Behalf Of Claudiu Zissulescu

>> Ianculescu

>> Sent: Tuesday, March 31, 2020 1:07 PM

>> To: Vineet Gupta <vgupta@synopsys.com>

>> Cc: linux-snps-arc@lists.infradead.org; gcc-patches@gcc.gnu.org; Claudiu Zissulescu

>> <claziss@synopsys.com>

>> Subject: Re: [PATCH] [ARC] Allow more ABIs in GLIBC_DYNAMIC_LINKER

>>

>> Pushed.

> Is this one eligible for being back-ported to older GCCs?

> At least GCC 9.x would be really good.

>

> -Alexey

>
Jakub Jelinek via Gcc-patches April 1, 2020, 12:52 a.m. | #4
FWIW this change needs a pairing glibc change so must NOT be included for upcoming
2020.x release which still has old version of glibc !

-Vineet


On 3/31/20 10:57 AM, Vineet Gupta wrote:
> Well its a hard requirement considering glibc is still using gcc-9 !

>

> Thx,

> -Vineet

>

> On 3/31/20 9:26 AM, Alexey Brodkin wrote:

>> Hi Claus,

>>

>>> -----Original Message-----

>>> From: linux-snps-arc <linux-snps-arc-bounces@lists.infradead.org> On Behalf Of Claudiu Zissulescu

>>> Ianculescu

>>> Sent: Tuesday, March 31, 2020 1:07 PM

>>> To: Vineet Gupta <vgupta@synopsys.com>

>>> Cc: linux-snps-arc@lists.infradead.org; gcc-patches@gcc.gnu.org; Claudiu Zissulescu

>>> <claziss@synopsys.com>

>>> Subject: Re: [PATCH] [ARC] Allow more ABIs in GLIBC_DYNAMIC_LINKER

>>>

>>> Pushed.

>> Is this one eligible for being back-ported to older GCCs?

>> At least GCC 9.x would be really good.

>>

>> -Alexey

>>  

> _______________________________________________

> linux-snps-arc mailing list

> linux-snps-arc@lists.infradead.org

> https://urldefense.proofpoint.com/v2/url?u=http-3A__lists.infradead.org_mailman_listinfo_linux-2Dsnps-2Darc&d=DwICAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=c14YS-cH-kdhTOW89KozFhBtBJgs1zXscZojEZQ0THs&m=xnl4FBxLN70UDZIvD2NTxyfPAenrfQsdij0DBuPImCI&s=YnG2GxYtFIbUU0FesR19TV7fz0ILe8xncKLnRiqBhOc&e=
Jakub Jelinek via Gcc-patches April 8, 2020, 11:37 p.m. | #5
Hi Claudiu,

For glibc needs can this be backported to gcc-9 please !

Thx,
-Vineet

On 3/31/20 3:06 AM, Claudiu Zissulescu Ianculescu wrote:
> Pushed.

>

> Thank you,

> Claudiu

>

> On Sun, Mar 29, 2020 at 2:05 AM Vineet Gupta via Gcc-patches

> <gcc-patches@gcc.gnu.org> wrote:

>> Enable big-endian suffixed dynamic linker per glibc multi-abi support.

>>

>> And to avoid a future churn and version pairingi hassles, also allow

>> arc700 although glibc for ARC currently doesn't support it.

>>

>> gcc/

>> xxxx-xx-xx  Vineet Gupta <vgupta@synopsys.com>

>> +

>> +       * config/arc/linux.h: GLIBC_DYNAMIC_LINKER support BE/arc700

>>

>> Signed-off-by: Vineet Gupta <vgupta@synopsys.com>

>> ---

>>  gcc/ChangeLog          | 4 ++++

>>  gcc/config/arc/linux.h | 2 +-

>>  2 files changed, 5 insertions(+), 1 deletion(-)

>>

>> diff --git a/gcc/ChangeLog b/gcc/ChangeLog

>> index 86ad683a6cb0..c26a748fd51b 100644

>> --- a/gcc/ChangeLog

>> +++ b/gcc/ChangeLog

>> @@ -1,3 +1,7 @@

>> +2020-03-28  Vineet Gupta <vgupta@synopsys.com>

>> +

>> +       * config/arc/linux.h: GLIBC_DYNAMIC_LINKER support BE/arc700

>> +

>>  2020-03-28  Jakub Jelinek  <jakub@redhat.com>

>>

>>         PR c/93573

>> diff --git a/gcc/config/arc/linux.h b/gcc/config/arc/linux.h

>> index 0b99da3fcdaf..1bbeccee7115 100644

>> --- a/gcc/config/arc/linux.h

>> +++ b/gcc/config/arc/linux.h

>> @@ -29,7 +29,7 @@ along with GCC; see the file COPYING3.  If not see

>>      }                                          \

>>    while (0)

>>

>> -#define GLIBC_DYNAMIC_LINKER   "/lib/ld-linux-arc.so.2"

>> +#define GLIBC_DYNAMIC_LINKER   "/lib/ld-linux-arc%{mbig-endian:eb}%{mcpu=arc700:700}.so.2"

>>  #define UCLIBC_DYNAMIC_LINKER  "/lib/ld-uClibc.so.0"

>>

>>  /* Note that the default is to link against dynamic libraries, if they are

>> --

>> 2.20.1

>>

> _______________________________________________

> linux-snps-arc mailing list

> linux-snps-arc@lists.infradead.org

> https://urldefense.proofpoint.com/v2/url?u=http-3A__lists.infradead.org_mailman_listinfo_linux-2Dsnps-2Darc&d=DwICAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=7FgpX6o3vAhwMrMhLh-4ZJey5kjdNUwOL2CWsFwR4T8&m=MrObyH2ki95_7m_xHpnWX-k9eIMOsxMuSa48qhxYOCY&s=3ggbGwaiJuSFnFECy0ItuwBBMDAcriwCdSc3GA0UFig&e=
Jakub Jelinek via Gcc-patches April 10, 2020, 12:25 p.m. | #6
Done.

Thank you for your support,
Claudiu

On Thu, Apr 9, 2020 at 2:38 AM Vineet Gupta <Vineet.Gupta1@synopsys.com> wrote:
>

> Hi Claudiu,

>

> For glibc needs can this be backported to gcc-9 please !

>

> Thx,

> -Vineet

>

> On 3/31/20 3:06 AM, Claudiu Zissulescu Ianculescu wrote:

> > Pushed.

> >

> > Thank you,

> > Claudiu

> >

> > On Sun, Mar 29, 2020 at 2:05 AM Vineet Gupta via Gcc-patches

> > <gcc-patches@gcc.gnu.org> wrote:

> >> Enable big-endian suffixed dynamic linker per glibc multi-abi support.

> >>

> >> And to avoid a future churn and version pairingi hassles, also allow

> >> arc700 although glibc for ARC currently doesn't support it.

> >>

> >> gcc/

> >> xxxx-xx-xx  Vineet Gupta <vgupta@synopsys.com>

> >> +

> >> +       * config/arc/linux.h: GLIBC_DYNAMIC_LINKER support BE/arc700

> >>

> >> Signed-off-by: Vineet Gupta <vgupta@synopsys.com>

> >> ---

> >>  gcc/ChangeLog          | 4 ++++

> >>  gcc/config/arc/linux.h | 2 +-

> >>  2 files changed, 5 insertions(+), 1 deletion(-)

> >>

> >> diff --git a/gcc/ChangeLog b/gcc/ChangeLog

> >> index 86ad683a6cb0..c26a748fd51b 100644

> >> --- a/gcc/ChangeLog

> >> +++ b/gcc/ChangeLog

> >> @@ -1,3 +1,7 @@

> >> +2020-03-28  Vineet Gupta <vgupta@synopsys.com>

> >> +

> >> +       * config/arc/linux.h: GLIBC_DYNAMIC_LINKER support BE/arc700

> >> +

> >>  2020-03-28  Jakub Jelinek  <jakub@redhat.com>

> >>

> >>         PR c/93573

> >> diff --git a/gcc/config/arc/linux.h b/gcc/config/arc/linux.h

> >> index 0b99da3fcdaf..1bbeccee7115 100644

> >> --- a/gcc/config/arc/linux.h

> >> +++ b/gcc/config/arc/linux.h

> >> @@ -29,7 +29,7 @@ along with GCC; see the file COPYING3.  If not see

> >>      }                                          \

> >>    while (0)

> >>

> >> -#define GLIBC_DYNAMIC_LINKER   "/lib/ld-linux-arc.so.2"

> >> +#define GLIBC_DYNAMIC_LINKER   "/lib/ld-linux-arc%{mbig-endian:eb}%{mcpu=arc700:700}.so.2"

> >>  #define UCLIBC_DYNAMIC_LINKER  "/lib/ld-uClibc.so.0"

> >>

> >>  /* Note that the default is to link against dynamic libraries, if they are

> >> --

> >> 2.20.1

> >>

> > _______________________________________________

> > linux-snps-arc mailing list

> > linux-snps-arc@lists.infradead.org

> > https://urldefense.proofpoint.com/v2/url?u=http-3A__lists.infradead.org_mailman_listinfo_linux-2Dsnps-2Darc&d=DwICAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=7FgpX6o3vAhwMrMhLh-4ZJey5kjdNUwOL2CWsFwR4T8&m=MrObyH2ki95_7m_xHpnWX-k9eIMOsxMuSa48qhxYOCY&s=3ggbGwaiJuSFnFECy0ItuwBBMDAcriwCdSc3GA0UFig&e=

>

Patch

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 86ad683a6cb0..c26a748fd51b 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,7 @@ 
+2020-03-28  Vineet Gupta <vgupta@synopsys.com>
+
+	* config/arc/linux.h: GLIBC_DYNAMIC_LINKER support BE/arc700
+
 2020-03-28  Jakub Jelinek  <jakub@redhat.com>
 
 	PR c/93573
diff --git a/gcc/config/arc/linux.h b/gcc/config/arc/linux.h
index 0b99da3fcdaf..1bbeccee7115 100644
--- a/gcc/config/arc/linux.h
+++ b/gcc/config/arc/linux.h
@@ -29,7 +29,7 @@  along with GCC; see the file COPYING3.  If not see
     }						\
   while (0)
 
-#define GLIBC_DYNAMIC_LINKER   "/lib/ld-linux-arc.so.2"
+#define GLIBC_DYNAMIC_LINKER   "/lib/ld-linux-arc%{mbig-endian:eb}%{mcpu=arc700:700}.so.2"
 #define UCLIBC_DYNAMIC_LINKER  "/lib/ld-uClibc.so.0"
 
 /* Note that the default is to link against dynamic libraries, if they are