diff options
author | Simon Marchi <simon.marchi@polymtl.ca> | 2020-10-31 08:30:58 -0400 |
---|---|---|
committer | Simon Marchi <simon.marchi@polymtl.ca> | 2020-10-31 08:30:58 -0400 |
commit | 7e8c7130fe59473bcda83fca345907be7c43936b (patch) | |
tree | 532904594e160fbbdf2fa2d265b9a2fe171497af /gdbserver/configure.ac | |
parent | e41fda1d5fd0c6751591a547924d5013f267aa80 (diff) | |
download | binutils-7e8c7130fe59473bcda83fca345907be7c43936b.zip binutils-7e8c7130fe59473bcda83fca345907be7c43936b.tar.gz binutils-7e8c7130fe59473bcda83fca345907be7c43936b.tar.bz2 |
gdbserver: modernize configure.ac
Run autoupdate on gdbserver/configure.ac and then tweak it to use easier
to read indentation. This removes a few warnings when running
`autoreconf -vf -Wall`.
* Replace AC_INIT with AC_INIT and no arguments plus AC_CONFIG_SRCDIR.
* Replace AC_GNU_SOURCE with AC_USE_SYSTEM_EXTENSIONS.
* Replace AC_TRY_COMPILE with AC_COMPILE_IFELSE.
* Replace AC_TRY_LINK with AC_LINK_IFELSE.
autoupdate gets it right, except this one here:
--- a/gdbserver/configure.ac
+++ b/gdbserver/configure.ac
@@ -304,7 +304,7 @@ if test "$srv_linux_thread_db" = "yes"; then
AC_LINK_IFELSE([AC_LANG_PROGRAM([[]], [[]])],[found="-Wl,--dynamic-list"
RDYNAMIC='-Wl,--dynamic-list=$(srcdir)/proc-service.list'],[RDYNAMIC="-rdynamic"
LDFLAGS="$old_LDFLAGS $RDYNAMIC"
- AC_TRY_LINK([], [],
+ _au_m4_changequote([,])AC_TRY_LINK([], [],
[found="-rdynamic"],
[found="no"
RDYNAMIC=""])])
... which I had to convert manually.
The changes in the generated configure file only contain insignificant
whitespace changes, so that gives confidence that the conversion is
correct.
gdbserver/ChangeLog:
* configure.ac: Modernize.
* configure: Re-generate.
Change-Id: Ia769aaec2aafac595504f477da955e91dffa4d8f
Diffstat (limited to 'gdbserver/configure.ac')
-rw-r--r-- | gdbserver/configure.ac | 99 |
1 files changed, 65 insertions, 34 deletions
diff --git a/gdbserver/configure.ac b/gdbserver/configure.ac index c474870..dd646cf 100644 --- a/gdbserver/configure.ac +++ b/gdbserver/configure.ac @@ -18,14 +18,15 @@ dnl along with this program. If not, see <http://www.gnu.org/licenses/>. dnl Process this file with autoconf to produce a configure script. -AC_INIT(server.cc) +AC_INIT +AC_CONFIG_SRCDIR([server.cc]) AC_CONFIG_HEADERS(config.h:config.in, [echo > stamp-h]) AM_MAINTAINER_MODE AC_PROG_CC AC_PROG_CXX -AC_GNU_SOURCE +AC_USE_SYSTEM_EXTENSIONS AC_SYS_LARGEFILE AM_PROG_INSTALL_STRIP @@ -124,12 +125,19 @@ if test "x$with_ust" != "xno"; then saved_CFLAGS="$CFLAGS" CFLAGS="$CFLAGS $ustinc" AC_MSG_CHECKING([for ust]) - AC_TRY_COMPILE([ + AC_COMPILE_IFELSE( + [AC_LANG_PROGRAM( + [ #define CONFIG_UST_GDB_INTEGRATION #include <ust/ust.h> - ],[], - [AC_MSG_RESULT([yes]); AC_DEFINE(HAVE_UST, 1, [Define if UST is available])], - [AC_MSG_RESULT([no]); ustlibs= ; ustinc= ]) + ], + [] + )], + [AC_MSG_RESULT([yes]); + AC_DEFINE(HAVE_UST, 1, [Define if UST is available])], + [AC_MSG_RESULT([no]) + ustlibs= ; ustinc= ] + ) CFLAGS="$saved_CFLAGS" fi @@ -249,11 +257,14 @@ if test "${srv_linux_regsets}" = "yes"; then [Define if the target supports register sets.]) AC_MSG_CHECKING(for PTRACE_GETREGS) - AC_CACHE_VAL(gdbsrv_cv_have_ptrace_getregs, - [AC_TRY_COMPILE([#include <sys/ptrace.h>], - [PTRACE_GETREGS;], - [gdbsrv_cv_have_ptrace_getregs=yes], - [gdbsrv_cv_have_ptrace_getregs=no])]) + AC_CACHE_VAL( + [gdbsrv_cv_have_ptrace_getregs], + [AC_COMPILE_IFELSE( + [AC_LANG_PROGRAM([#include <sys/ptrace.h>], [PTRACE_GETREGS;])], + [gdbsrv_cv_have_ptrace_getregs=yes], + [gdbsrv_cv_have_ptrace_getregs=no] + )] + ) AC_MSG_RESULT($gdbsrv_cv_have_ptrace_getregs) if test "${gdbsrv_cv_have_ptrace_getregs}" = "yes"; then AC_DEFINE(HAVE_PTRACE_GETREGS, 1, @@ -262,11 +273,14 @@ if test "${srv_linux_regsets}" = "yes"; then fi AC_MSG_CHECKING(for PTRACE_GETFPXREGS) - AC_CACHE_VAL(gdbsrv_cv_have_ptrace_getfpxregs, - [AC_TRY_COMPILE([#include <sys/ptrace.h>], - [PTRACE_GETFPXREGS;], - [gdbsrv_cv_have_ptrace_getfpxregs=yes], - [gdbsrv_cv_have_ptrace_getfpxregs=no])]) + AC_CACHE_VAL( + [gdbsrv_cv_have_ptrace_getfpxregs], + [AC_COMPILE_IFELSE( + [AC_LANG_PROGRAM([#include <sys/ptrace.h>], [PTRACE_GETFPXREGS;])], + [gdbsrv_cv_have_ptrace_getfpxregs=yes], + [gdbsrv_cv_have_ptrace_getfpxregs=no] + )] + ) AC_MSG_RESULT($gdbsrv_cv_have_ptrace_getfpxregs) if test "${gdbsrv_cv_have_ptrace_getfpxregs}" = "yes"; then AC_DEFINE(HAVE_PTRACE_GETFPXREGS, 1, @@ -308,15 +322,19 @@ if test "$srv_linux_thread_db" = "yes"; then # supported there. RDYNAMIC="-Wl,--dynamic-list=${srcdir}/proc-service.list" LDFLAGS="$LDFLAGS $RDYNAMIC" - AC_TRY_LINK([], [], - [found="-Wl,--dynamic-list" - RDYNAMIC='-Wl,--dynamic-list=$(srcdir)/proc-service.list'], - [RDYNAMIC="-rdynamic" - LDFLAGS="$old_LDFLAGS $RDYNAMIC" - AC_TRY_LINK([], [], - [found="-rdynamic"], - [found="no" - RDYNAMIC=""])]) + AC_LINK_IFELSE( + [AC_LANG_PROGRAM([],[])], + [found="-Wl,--dynamic-list" + RDYNAMIC='-Wl,--dynamic-list=$(srcdir)/proc-service.list'], + [RDYNAMIC="-rdynamic" + LDFLAGS="$old_LDFLAGS $RDYNAMIC" + AC_LINK_IFELSE( + [AC_LANG_PROGRAM([], [])], + [found="-rdynamic"], + [found="no" + RDYNAMIC=""] + )] + ) AC_SUBST(RDYNAMIC) LDFLAGS="$old_LDFLAGS" AC_MSG_RESULT($found) @@ -326,10 +344,15 @@ if test "$srv_linux_thread_db" = "yes"; then srv_thread_depfiles="thread-db.o proc-service.o" AC_DEFINE(USE_THREAD_DB, 1, [Define if we should use libthread_db.]) - AC_CACHE_CHECK([for TD_VERSION], gdbsrv_cv_have_td_version, - [AC_TRY_COMPILE([#include <thread_db.h>], [TD_VERSION;], - [gdbsrv_cv_have_td_version=yes], - [gdbsrv_cv_have_td_version=no])]) + AC_CACHE_CHECK( + [for TD_VERSION], + [gdbsrv_cv_have_td_version], + [AC_COMPILE_IFELSE( + [AC_LANG_PROGRAM([#include <thread_db.h>], [TD_VERSION;])], + [gdbsrv_cv_have_td_version=yes], + [gdbsrv_cv_have_td_version=no] + )] + ) if test "$gdbsrv_cv_have_td_version" = yes; then AC_DEFINE(HAVE_TD_VERSION, 1, [Define if TD_VERSION is available.]) fi @@ -360,11 +383,19 @@ GDBSERVER_DEPFILES="$srv_regobj $srv_tgtobj $srv_hostio_err_objs $srv_thread_dep GDBSERVER_LIBS="$srv_libs" dnl Check whether the target supports __sync_*_compare_and_swap. -AC_CACHE_CHECK([whether the target supports __sync_*_compare_and_swap], - gdbsrv_cv_have_sync_builtins, [ -AC_TRY_LINK([], [int foo, bar; bar = __sync_val_compare_and_swap(&foo, 0, 1);], - gdbsrv_cv_have_sync_builtins=yes, - gdbsrv_cv_have_sync_builtins=no)]) +AC_CACHE_CHECK( + [whether the target supports __sync_*_compare_and_swap], + [gdbsrv_cv_have_sync_builtins], + [AC_LINK_IFELSE( + [AC_LANG_PROGRAM( + [], + [int foo, bar; + bar = __sync_val_compare_and_swap(&foo, 0, 1);] + )], + [gdbsrv_cv_have_sync_builtins=yes], + [gdbsrv_cv_have_sync_builtins=no] + )] +) if test "$gdbsrv_cv_have_sync_builtins" = yes; then AC_DEFINE(HAVE_SYNC_BUILTINS, 1, [Define to 1 if the target supports __sync_*_compare_and_swap]) |