[PATCH] LoongArch: Don't falsely claim gold supported in toplevel configure

chenglulu chenglulu@loongson.cn
Fri Feb 23 03:16:51 GMT 2024


在 2024/2/22 下午5:17, Xi Ruoyao 写道:
> The gold linker has never been ported to LoongArch (and it seems
> unlikely to be ported in the future as the new architectures are
> focusing on lld and/or mold for fast linkers).
>
> ChangeLog:
>
> 	* configure.ac (ENABLE_GOLD): Remove loongarch*-*-* from target
> 	list.
> 	* configure: Regenerate.
> ---
>
> Ok for GCC trunk (to get synced into Binutils later)?

I have no problem. But I have a question. Is this modification simply 
because we don’t

support it or is there an error somewhere?

>
>   configure    | 2 +-
>   configure.ac | 2 +-
>   2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/configure b/configure
> index 874966fb9f0..02b435c1163 100755
> --- a/configure
> +++ b/configure
> @@ -3092,7 +3092,7 @@ case "${ENABLE_GOLD}" in
>         # Check for target supported by gold.
>         case "${target}" in
>           i?86-*-* | x86_64-*-* | sparc*-*-* | powerpc*-*-* | arm*-*-* \
> -        | aarch64*-*-* | tilegx*-*-* | mips*-*-* | s390*-*-* | loongarch*-*-*)
> +        | aarch64*-*-* | tilegx*-*-* | mips*-*-* | s390*-*-*)
>   	  configdirs="$configdirs gold"
>   	  if test x${ENABLE_GOLD} = xdefault; then
>   	    default_ld=gold
> diff --git a/configure.ac b/configure.ac
> index 4f34004a072..1a19c07a27b 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -364,7 +364,7 @@ case "${ENABLE_GOLD}" in
>         # Check for target supported by gold.
>         case "${target}" in
>           i?86-*-* | x86_64-*-* | sparc*-*-* | powerpc*-*-* | arm*-*-* \
> -        | aarch64*-*-* | tilegx*-*-* | mips*-*-* | s390*-*-* | loongarch*-*-*)
> +        | aarch64*-*-* | tilegx*-*-* | mips*-*-* | s390*-*-*)
>   	  configdirs="$configdirs gold"
>   	  if test x${ENABLE_GOLD} = xdefault; then
>   	    default_ld=gold



More information about the Binutils mailing list