public inbox for newlib-cvs@sourceware.org
help / color / mirror / Atom feed
From: Michael Frysinger <vapier@sourceware.org>
To: newlib-cvs@sourceware.org
Subject: [newlib-cygwin] newlib: add AC_CACHE_CHECK sugar around preprocessor checks
Date: Tue,  1 Feb 2022 23:17:50 +0000 (GMT)	[thread overview]
Message-ID: <20220201231750.39EF63858437@sourceware.org> (raw)

https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;h=041dd7ff265a15096f995396c0005cea19336819

commit 041dd7ff265a15096f995396c0005cea19336819
Author: Mike Frysinger <vapier@gentoo.org>
Date:   Fri Jan 28 05:17:46 2022 -0500

    newlib: add AC_CACHE_CHECK sugar around preprocessor checks
    
    This isn't strictly necessary, but it makes for much clearer logs as
    to what the target is doing, and provides cache vars for anyone who
    wants to force the test a different way, and it lets the build cache
    its own results when rerunning config.status.

Diff:
---
 newlib/libc/configure                  | 48 ++++++++++++++++++++++++----------
 newlib/libc/machine/nds32/acinclude.m4 |  7 ++---
 newlib/libc/machine/sh/acinclude.m4    |  7 ++---
 newlib/libc/machine/spu/acinclude.m4   |  9 +++----
 newlib/libm/configure                  | 32 +++++++++++++++++------
 newlib/libm/machine/nds32/acinclude.m4 | 14 +++++-----
 6 files changed, 78 insertions(+), 39 deletions(-)

diff --git a/newlib/libc/configure b/newlib/libc/configure
index 30e8bc55f..5f433dca3 100755
--- a/newlib/libc/configure
+++ b/newlib/libc/configure
@@ -12467,7 +12467,12 @@ fi
 
 
 if test "${machine_dir}" = "nds32"; then
-    cat confdefs.h - <<_ACEOF >conftest.$ac_ext
+    { $as_echo "$as_me:${as_lineno-$LINENO}: checking for nds32 V3M ISA" >&5
+$as_echo_n "checking for nds32 V3M ISA... " >&6; }
+if ${newlib_cv_nds32_isa_v3m+:} false; then :
+  $as_echo_n "(cached) " >&6
+else
+      cat confdefs.h - <<_ACEOF >conftest.$ac_ext
 /* end confdefs.h.  */
 #ifdef __NDS32_ISA_V3M__
 # error "This is nds32_isa_v3m."
@@ -12482,14 +12487,17 @@ main ()
 }
 _ACEOF
 if ac_fn_c_try_cpp "$LINENO"; then :
-  is_nds32_isa_v3m="no"
+  newlib_cv_nds32_isa_v3m="no"
 else
-  is_nds32_isa_v3m="yes"
+  newlib_cv_nds32_isa_v3m="yes"
 fi
 rm -f conftest.err conftest.i conftest.$ac_ext
+fi
+{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $newlib_cv_nds32_isa_v3m" >&5
+$as_echo "$newlib_cv_nds32_isa_v3m" >&6; }
 fi
 
- if test "$is_nds32_isa_v3m" = "yes"; then
+ if test "$newlib_cv_nds32_isa_v3m" = "yes"; then
   IS_NDS32_ISA_V3M_TRUE=
   IS_NDS32_ISA_V3M_FALSE='#'
 else
@@ -12524,7 +12532,12 @@ fi
 
 
 if test "${machine_dir}" = "sh"; then
-  cat confdefs.h - <<_ACEOF >conftest.$ac_ext
+  { $as_echo "$as_me:${as_lineno-$LINENO}: checking for SH5 (64-bit)" >&5
+$as_echo_n "checking for SH5 (64-bit)... " >&6; }
+if ${newlib_cv_sh64+:} false; then :
+  $as_echo_n "(cached) " >&6
+else
+      cat confdefs.h - <<_ACEOF >conftest.$ac_ext
 /* end confdefs.h.  */
 #if !defined(__SH5__)
 # error "not SH5"
@@ -12539,14 +12552,17 @@ main ()
 }
 _ACEOF
 if ac_fn_c_try_cpp "$LINENO"; then :
-  have_sh64=yes
+  newlib_cv_sh64=yes
 else
-  have_sh64=no
+  newlib_cv_sh64=no
 fi
 rm -f conftest.err conftest.i conftest.$ac_ext
+fi
+{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $newlib_cv_sh64" >&5
+$as_echo "$newlib_cv_sh64" >&6; }
 fi
 
- if test "$have_sh64" = yes; then
+ if test "$newlib_cv_sh64" = yes; then
   SH64_TRUE=
   SH64_FALSE='#'
 else
@@ -12558,7 +12574,10 @@ fi
 if test "${machine_dir}" = "spu"; then
   { $as_echo "$as_me:${as_lineno-$LINENO}: checking whether the compiler supports __ea" >&5
 $as_echo_n "checking whether the compiler supports __ea... " >&6; }
-  cat confdefs.h - <<_ACEOF >conftest.$ac_ext
+if ${newlib_cv_spu_compiler_has_ea+:} false; then :
+  $as_echo_n "(cached) " >&6
+else
+      cat confdefs.h - <<_ACEOF >conftest.$ac_ext
 /* end confdefs.h.  */
 #if !defined (__EA32__) && !defined (__EA64__)
 # error "__ea not supported"
@@ -12573,15 +12592,16 @@ main ()
 }
 _ACEOF
 if ac_fn_c_try_cpp "$LINENO"; then :
-  spu_compiler_has_ea=yes
+  newlib_cv_spu_compiler_has_ea=yes
 else
-  spu_compiler_has_ea=no
+  newlib_cv_spu_compiler_has_ea=no
 fi
 rm -f conftest.err conftest.i conftest.$ac_ext
-  { $as_echo "$as_me:${as_lineno-$LINENO}: result: $spu_compiler_has_ea" >&5
-$as_echo "$spu_compiler_has_ea" >&6; }
 fi
- if test x${spu_compiler_has_ea} != xno; then
+{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $newlib_cv_spu_compiler_has_ea" >&5
+$as_echo "$newlib_cv_spu_compiler_has_ea" >&6; }
+fi
+ if test "$newlib_cv_spu_compiler_has_ea" = yes; then
   HAVE_SPU_EA_TRUE=
   HAVE_SPU_EA_FALSE='#'
 else
diff --git a/newlib/libc/machine/nds32/acinclude.m4 b/newlib/libc/machine/nds32/acinclude.m4
index 35c2afe99..ae8ea5d91 100644
--- a/newlib/libc/machine/nds32/acinclude.m4
+++ b/newlib/libc/machine/nds32/acinclude.m4
@@ -1,10 +1,11 @@
 if test "${machine_dir}" = "nds32"; then
   dnl Use builtin macro to detect if this is for "AndeStar ISA V3m".
-  AC_PREPROC_IFELSE([AC_LANG_PROGRAM(
+  AC_CACHE_CHECK([for nds32 V3M ISA], newlib_cv_nds32_isa_v3m, [dnl
+    AC_PREPROC_IFELSE([AC_LANG_PROGRAM(
 [[#ifdef __NDS32_ISA_V3M__
 # error "This is nds32_isa_v3m."
 #endif
-]])], [is_nds32_isa_v3m="no"], [is_nds32_isa_v3m="yes"])
+]])], [newlib_cv_nds32_isa_v3m="no"], [newlib_cv_nds32_isa_v3m="yes"])])
 fi
 
-AM_CONDITIONAL(IS_NDS32_ISA_V3M, test "$is_nds32_isa_v3m" = "yes")
+AM_CONDITIONAL(IS_NDS32_ISA_V3M, test "$newlib_cv_nds32_isa_v3m" = "yes")
diff --git a/newlib/libc/machine/sh/acinclude.m4 b/newlib/libc/machine/sh/acinclude.m4
index 60af7b1f1..66ec3b4e1 100644
--- a/newlib/libc/machine/sh/acinclude.m4
+++ b/newlib/libc/machine/sh/acinclude.m4
@@ -1,9 +1,10 @@
 if test "${machine_dir}" = "sh"; then
-  AC_PREPROC_IFELSE([AC_LANG_PROGRAM(
+  AC_CACHE_CHECK([for SH5 (64-bit)], newlib_cv_sh64, [dnl
+    AC_PREPROC_IFELSE([AC_LANG_PROGRAM(
 [[#if !defined(__SH5__)
 # error "not SH5"
 #endif
-]])], [have_sh64=yes], [have_sh64=no])
+]])], [newlib_cv_sh64=yes], [newlib_cv_sh64=no])])
 fi
 
-AM_CONDITIONAL(SH64, [test "$have_sh64" = yes])
+AM_CONDITIONAL(SH64, [test "$newlib_cv_sh64" = yes])
diff --git a/newlib/libc/machine/spu/acinclude.m4 b/newlib/libc/machine/spu/acinclude.m4
index 4e5fb363c..bca70e309 100644
--- a/newlib/libc/machine/spu/acinclude.m4
+++ b/newlib/libc/machine/spu/acinclude.m4
@@ -1,10 +1,9 @@
 if test "${machine_dir}" = "spu"; then
-  AC_MSG_CHECKING([whether the compiler supports __ea])
-  AC_PREPROC_IFELSE([AC_LANG_PROGRAM(
+  AC_CACHE_CHECK([whether the compiler supports __ea], newlib_cv_spu_compiler_has_ea, [dnl
+    AC_PREPROC_IFELSE([AC_LANG_PROGRAM(
 [[#if !defined (__EA32__) && !defined (__EA64__)
 # error "__ea not supported"
 #endif
-]])], [spu_compiler_has_ea=yes], [spu_compiler_has_ea=no])
-  AC_MSG_RESULT($spu_compiler_has_ea)
+]])], [newlib_cv_spu_compiler_has_ea=yes], [newlib_cv_spu_compiler_has_ea=no])])
 fi
-AM_CONDITIONAL(HAVE_SPU_EA, test x${spu_compiler_has_ea} != xno)
+AM_CONDITIONAL(HAVE_SPU_EA, test "$newlib_cv_spu_compiler_has_ea" = yes)
diff --git a/newlib/libm/configure b/newlib/libm/configure
index fb7f415f8..d9b3cffd0 100755
--- a/newlib/libm/configure
+++ b/newlib/libm/configure
@@ -12120,7 +12120,12 @@ fi
 
 
 if test "${libm_machine_dir}" = "nds32"; then
-    cat confdefs.h - <<_ACEOF >conftest.$ac_ext
+    { $as_echo "$as_me:${as_lineno-$LINENO}: checking for nds32 FPU SP extension" >&5
+$as_echo_n "checking for nds32 FPU SP extension... " >&6; }
+if ${newlib_cv_nds32_fpu_sp+:} false; then :
+  $as_echo_n "(cached) " >&6
+else
+      cat confdefs.h - <<_ACEOF >conftest.$ac_ext
 /* end confdefs.h.  */
 #if (__NDS32_EXT_FPU_SP__)
 # error "Has nds32 FPU SP extension support"
@@ -12135,13 +12140,21 @@ main ()
 }
 _ACEOF
 if ac_fn_c_try_cpp "$LINENO"; then :
-  has_nds32_fpu_sp="no"
+  newlib_cv_nds32_fpu_sp="no"
 else
-  has_nds32_fpu_sp="yes"
+  newlib_cv_nds32_fpu_sp="yes"
 fi
 rm -f conftest.err conftest.i conftest.$ac_ext
+fi
+{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $newlib_cv_nds32_fpu_sp" >&5
+$as_echo "$newlib_cv_nds32_fpu_sp" >&6; }
 
-  cat confdefs.h - <<_ACEOF >conftest.$ac_ext
+  { $as_echo "$as_me:${as_lineno-$LINENO}: checking for nds32 FPU DP extension" >&5
+$as_echo_n "checking for nds32 FPU DP extension... " >&6; }
+if ${newlib_cv_nds32_fpu_dp+:} false; then :
+  $as_echo_n "(cached) " >&6
+else
+      cat confdefs.h - <<_ACEOF >conftest.$ac_ext
 /* end confdefs.h.  */
 #if (__NDS32_EXT_FPU_DP__)
 # error "Has nds32 FPU DP extension support"
@@ -12156,14 +12169,17 @@ main ()
 }
 _ACEOF
 if ac_fn_c_try_cpp "$LINENO"; then :
-  has_nds32_fpu_dp="no"
+  newlib_cv_nds32_fpu_dp="no"
 else
-  has_nds32_fpu_dp="yes"
+  newlib_cv_nds32_fpu_dp="yes"
 fi
 rm -f conftest.err conftest.i conftest.$ac_ext
+fi
+{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $newlib_cv_nds32_fpu_dp" >&5
+$as_echo "$newlib_cv_nds32_fpu_dp" >&6; }
 fi
 
- if test "$has_nds32_fpu_sp" = "yes"; then
+ if test "$newlib_cv_nds32_fpu_sp" = "yes"; then
   HAS_NDS32_FPU_SP_TRUE=
   HAS_NDS32_FPU_SP_FALSE='#'
 else
@@ -12171,7 +12187,7 @@ else
   HAS_NDS32_FPU_SP_FALSE=
 fi
 
- if test "$has_nds32_fpu_dp" = "yes"; then
+ if test "$newlib_cv_nds32_fpu_dp" = "yes"; then
   HAS_NDS32_FPU_DP_TRUE=
   HAS_NDS32_FPU_DP_FALSE='#'
 else
diff --git a/newlib/libm/machine/nds32/acinclude.m4 b/newlib/libm/machine/nds32/acinclude.m4
index 6363df8a3..781fa9145 100644
--- a/newlib/libm/machine/nds32/acinclude.m4
+++ b/newlib/libm/machine/nds32/acinclude.m4
@@ -1,17 +1,19 @@
 if test "${libm_machine_dir}" = "nds32"; then
   dnl Use builtin macro to detect if FPU extension support is on.
-  AC_PREPROC_IFELSE([AC_LANG_PROGRAM(
+  AC_CACHE_CHECK([for nds32 FPU SP extension], newlib_cv_nds32_fpu_sp, [dnl
+    AC_PREPROC_IFELSE([AC_LANG_PROGRAM(
 [[#if (__NDS32_EXT_FPU_SP__)
 # error "Has nds32 FPU SP extension support"
 #endif
-]])], [has_nds32_fpu_sp="no"], [has_nds32_fpu_sp="yes"])
+]])], [newlib_cv_nds32_fpu_sp="no"], [newlib_cv_nds32_fpu_sp="yes"])])
 
-  AC_PREPROC_IFELSE([AC_LANG_PROGRAM(
+  AC_CACHE_CHECK([for nds32 FPU DP extension], newlib_cv_nds32_fpu_dp, [dnl
+    AC_PREPROC_IFELSE([AC_LANG_PROGRAM(
 [[#if (__NDS32_EXT_FPU_DP__)
 # error "Has nds32 FPU DP extension support"
 #endif
-]])], [has_nds32_fpu_dp="no"], [has_nds32_fpu_dp="yes"])
+]])], [newlib_cv_nds32_fpu_dp="no"], [newlib_cv_nds32_fpu_dp="yes"])])
 fi
 
-AM_CONDITIONAL(HAS_NDS32_FPU_SP, test "$has_nds32_fpu_sp" = "yes")
-AM_CONDITIONAL(HAS_NDS32_FPU_DP, test "$has_nds32_fpu_dp" = "yes")
+AM_CONDITIONAL(HAS_NDS32_FPU_SP, test "$newlib_cv_nds32_fpu_sp" = "yes")
+AM_CONDITIONAL(HAS_NDS32_FPU_DP, test "$newlib_cv_nds32_fpu_dp" = "yes")


                 reply	other threads:[~2022-02-01 23:17 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220201231750.39EF63858437@sourceware.org \
    --to=vapier@sourceware.org \
    --cc=newlib-cvs@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).