diff mbox series

AArch64: Fix typo in sve configure check (BZ# 29394)

Message ID AM5PR0801MB166893E2869BF1BBFCB234BD83649@AM5PR0801MB1668.eurprd08.prod.outlook.com
State New
Headers show
Series AArch64: Fix typo in sve configure check (BZ# 29394) | expand

Commit Message

Wilco Dijkstra Aug. 11, 2022, 4:25 p.m. UTC 
Fix a typo in the SVE configure check. This fixes BZ# 29394.

---

Comments

Adhemerval Zanella Netto Aug. 11, 2022, 4:42 p.m. UTC | #1 
LGTM, thanks.

On 11/08/22 13:25, Wilco Dijkstra via Libc-alpha wrote:
> Fix a typo in the SVE configure check. This fixes BZ# 29394.
> 
> ---
> 
> diff --git a/sysdeps/aarch64/configure b/sysdeps/aarch64/configure
> index bf972122b199e1dd7ef04c58dee99ed864274174..2130f6b8f8b423fbdc97b066e9cfe2cd977aa792 100644
> --- a/sysdeps/aarch64/configure
> +++ b/sysdeps/aarch64/configure
> @@ -303,7 +303,7 @@ aarch64-variant-pcs = $libc_cv_aarch64_variant_pcs"
>  # Check if asm support armv8.2-a+sve
>  { $as_echo "$as_me:${as_lineno-$LINENO}: checking for SVE support in assembler" >&5
>  $as_echo_n "checking for SVE support in assembler... " >&6; }
> -if ${libc_cv_asm_sve+:} false; then :
> +if ${libc_cv_aarch64_sve_asm+:} false; then :
>    $as_echo_n "(cached) " >&6
>  else
>    cat > conftest.s <<\EOF
> @@ -321,8 +321,8 @@ else
>  fi
>  rm -f conftest*
>  fi
> -{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $libc_cv_asm_sve" >&5
> -$as_echo "$libc_cv_asm_sve" >&6; }
> +{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $libc_cv_aarch64_sve_asm" >&5
> +$as_echo "$libc_cv_aarch64_sve_asm" >&6; }
>  if test $libc_cv_aarch64_sve_asm = yes; then
>    $as_echo "#define HAVE_AARCH64_SVE_ASM 1" >>confdefs.h
>  
> diff --git a/sysdeps/aarch64/configure.ac b/sysdeps/aarch64/configure.ac
> index 51253d98025b6f5956575558e459d72e0763c674..85c6f765082811f5e8312eebda8d615eabcf39a1 100644
> --- a/sysdeps/aarch64/configure.ac
> +++ b/sysdeps/aarch64/configure.ac
> @@ -88,7 +88,7 @@ EOF
>  LIBC_CONFIG_VAR([aarch64-variant-pcs], [$libc_cv_aarch64_variant_pcs])
>  
>  # Check if asm support armv8.2-a+sve
> -AC_CACHE_CHECK(for SVE support in assembler, libc_cv_asm_sve, [dnl
> +AC_CACHE_CHECK([for SVE support in assembler], [libc_cv_aarch64_sve_asm], [dnl
>  cat > conftest.s <<\EOF
>          ptrue p0.b
>  EOF
>
Andreas Schwab Aug. 16, 2022, 1 p.m. UTC | #2 
On Aug 11 2022, Wilco Dijkstra via Libc-alpha wrote:

> Fix a typo in the SVE configure check. This fixes BZ# 29394.

Ok.
diff mbox series

Patch

diff --git a/sysdeps/aarch64/configure b/sysdeps/aarch64/configure
index bf972122b199e1dd7ef04c58dee99ed864274174..2130f6b8f8b423fbdc97b066e9cfe2cd977aa792 100644
--- a/sysdeps/aarch64/configure
+++ b/sysdeps/aarch64/configure
@@ -303,7 +303,7 @@  aarch64-variant-pcs = $libc_cv_aarch64_variant_pcs"
 # Check if asm support armv8.2-a+sve
 { $as_echo "$as_me:${as_lineno-$LINENO}: checking for SVE support in assembler" >&5
 $as_echo_n "checking for SVE support in assembler... " >&6; }
-if ${libc_cv_asm_sve+:} false; then :
+if ${libc_cv_aarch64_sve_asm+:} false; then :
   $as_echo_n "(cached) " >&6
 else
   cat > conftest.s <<\EOF
@@ -321,8 +321,8 @@  else
 fi
 rm -f conftest*
 fi
-{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $libc_cv_asm_sve" >&5
-$as_echo "$libc_cv_asm_sve" >&6; }
+{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $libc_cv_aarch64_sve_asm" >&5
+$as_echo "$libc_cv_aarch64_sve_asm" >&6; }
 if test $libc_cv_aarch64_sve_asm = yes; then
   $as_echo "#define HAVE_AARCH64_SVE_ASM 1" >>confdefs.h
 
diff --git a/sysdeps/aarch64/configure.ac b/sysdeps/aarch64/configure.ac
index 51253d98025b6f5956575558e459d72e0763c674..85c6f765082811f5e8312eebda8d615eabcf39a1 100644
--- a/sysdeps/aarch64/configure.ac
+++ b/sysdeps/aarch64/configure.ac
@@ -88,7 +88,7 @@  EOF
 LIBC_CONFIG_VAR([aarch64-variant-pcs], [$libc_cv_aarch64_variant_pcs])
 
 # Check if asm support armv8.2-a+sve
-AC_CACHE_CHECK(for SVE support in assembler, libc_cv_asm_sve, [dnl
+AC_CACHE_CHECK([for SVE support in assembler], [libc_cv_aarch64_sve_asm], [dnl
 cat > conftest.s <<\EOF
         ptrue p0.b
 EOF