powerpc64le: Fix typo in configure

Message ID 20210706201936.78d71219@kryten.localdomain
State Committed
Commit 01d780628278972cfa3630feeb74d7861b0fd5fc
Delegated to: Tulio Magno Quites Machado Filho
Headers
Series powerpc64le: Fix typo in configure |

Checks

Context Check Description
dj/TryBot-apply_patch success Patch applied to master at the time it was sent
dj/TryBot-32bit success Build for i686

Commit Message

Anton Blanchard July 6, 2021, 10:19 a.m. UTC
  The configure script checks for -mlong-double-128 but mentions -mlongdouble
when it fails.
---
 sysdeps/powerpc/powerpc64/le/configure    | 2 +-
 sysdeps/powerpc/powerpc64/le/configure.ac | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
  

Comments

Adhemerval Zanella Netto July 6, 2021, 3 p.m. UTC | #1
On 06/07/2021 07:19, Anton Blanchard via Libc-alpha wrote:
> The configure script checks for -mlong-double-128 but mentions -mlongdouble
> when it fails.

LGTM, thanks.

Reviewed-by: Adhemerval Zanella  <adhemerval.zanella@linaro.org>

> ---
>  sysdeps/powerpc/powerpc64/le/configure    | 2 +-
>  sysdeps/powerpc/powerpc64/le/configure.ac | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/sysdeps/powerpc/powerpc64/le/configure b/sysdeps/powerpc/powerpc64/le/configure
> index 98a4889945..35950a6ddd 100644
> --- a/sysdeps/powerpc/powerpc64/le/configure
> +++ b/sysdeps/powerpc/powerpc64/le/configure
> @@ -114,7 +114,7 @@ fi
>  { $as_echo "$as_me:${as_lineno-$LINENO}: result: $libc_cv_compiler_powerpc64le_ldbl128_mabi" >&5
>  $as_echo "$libc_cv_compiler_powerpc64le_ldbl128_mabi" >&6; }
>  if test "$libc_cv_compiler_powerpc64le_ldbl128_mabi" == "no"; then :
> -  critic_missing="$critic_missing The compiler must support -mabi=ieeelongdouble and -mlongdouble simultaneously."
> +  critic_missing="$critic_missing The compiler must support -mabi=ieeelongdouble and -mlong-double-128 simultaneously."
>  fi
>  
>  for ac_prog in $OBJCOPY
> diff --git a/sysdeps/powerpc/powerpc64/le/configure.ac b/sysdeps/powerpc/powerpc64/le/configure.ac
> index 769c06709e..9f0423ede2 100644
> --- a/sysdeps/powerpc/powerpc64/le/configure.ac
> +++ b/sysdeps/powerpc/powerpc64/le/configure.ac
> @@ -64,7 +64,7 @@ long double x;
>  		  [libc_cv_compiler_powerpc64le_ldbl128_mabi=no])
>  CFLAGS="$save_CFLAGS"])
>  AS_IF([test "$libc_cv_compiler_powerpc64le_ldbl128_mabi" == "no"],
> -      [critic_missing="$critic_missing The compiler must support -mabi=ieeelongdouble and -mlongdouble simultaneously."])
> +      [critic_missing="$critic_missing The compiler must support -mabi=ieeelongdouble and -mlong-double-128 simultaneously."])
>  
>  dnl objcopy (binutils) 2.26 or newer required to support the --update-section
>  dnl feature for fixing up .gnu.attribute section with IEEE ldbl.
>
  
Tulio Magno Quites Machado Filho July 9, 2021, 1:01 a.m. UTC | #2
Adhemerval Zanella via Libc-alpha <libc-alpha@sourceware.org> writes:

> On 06/07/2021 07:19, Anton Blanchard via Libc-alpha wrote:
>> The configure script checks for -mlong-double-128 but mentions -mlongdouble
>> when it fails.
>
> LGTM, thanks.
>
> Reviewed-by: Adhemerval Zanella  <adhemerval.zanella@linaro.org>

Pushed as 01d780628278972cfa3630feeb74d7861b0fd5fc.

Thanks!
  

Patch

diff --git a/sysdeps/powerpc/powerpc64/le/configure b/sysdeps/powerpc/powerpc64/le/configure
index 98a4889945..35950a6ddd 100644
--- a/sysdeps/powerpc/powerpc64/le/configure
+++ b/sysdeps/powerpc/powerpc64/le/configure
@@ -114,7 +114,7 @@  fi
 { $as_echo "$as_me:${as_lineno-$LINENO}: result: $libc_cv_compiler_powerpc64le_ldbl128_mabi" >&5
 $as_echo "$libc_cv_compiler_powerpc64le_ldbl128_mabi" >&6; }
 if test "$libc_cv_compiler_powerpc64le_ldbl128_mabi" == "no"; then :
-  critic_missing="$critic_missing The compiler must support -mabi=ieeelongdouble and -mlongdouble simultaneously."
+  critic_missing="$critic_missing The compiler must support -mabi=ieeelongdouble and -mlong-double-128 simultaneously."
 fi
 
 for ac_prog in $OBJCOPY
diff --git a/sysdeps/powerpc/powerpc64/le/configure.ac b/sysdeps/powerpc/powerpc64/le/configure.ac
index 769c06709e..9f0423ede2 100644
--- a/sysdeps/powerpc/powerpc64/le/configure.ac
+++ b/sysdeps/powerpc/powerpc64/le/configure.ac
@@ -64,7 +64,7 @@  long double x;
 		  [libc_cv_compiler_powerpc64le_ldbl128_mabi=no])
 CFLAGS="$save_CFLAGS"])
 AS_IF([test "$libc_cv_compiler_powerpc64le_ldbl128_mabi" == "no"],
-      [critic_missing="$critic_missing The compiler must support -mabi=ieeelongdouble and -mlongdouble simultaneously."])
+      [critic_missing="$critic_missing The compiler must support -mabi=ieeelongdouble and -mlong-double-128 simultaneously."])
 
 dnl objcopy (binutils) 2.26 or newer required to support the --update-section
 dnl feature for fixing up .gnu.attribute section with IEEE ldbl.