From 7e8c7130fe59473bcda83fca345907be7c43936b Mon Sep 17 00:00:00 2001 From: Simon Marchi Date: Sat, 31 Oct 2020 08:30:58 -0400 Subject: 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 --- gdbserver/configure.ac | 99 +++++++++++++++++++++++++++++++++----------------- 1 file changed, 65 insertions(+), 34 deletions(-) (limited to 'gdbserver/configure.ac') 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 . 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 - ],[], - [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 ], - [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 ], [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 ], - [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 ], [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 ], [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 ], [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]) -- cgit v1.1