]> sourceware.org Git - glibc.git/commitdiff
Remove configure tests for assembler CFI support.
authorJoseph Myers <joseph@codesourcery.com>
Thu, 26 Jun 2014 17:33:32 +0000 (17:33 +0000)
committerJoseph Myers <joseph@codesourcery.com>
Thu, 26 Jun 2014 17:33:32 +0000 (17:33 +0000)
This patch removes configure tests for assembler CFI support (and
thereby eliminates an architecture-specific case in the main
configure.ac), instead assuming that support is present
unconditionally.

The main test was added in 2003 around the time CFI support was added
to the assembler.  cfi_personality and cfi_lsda support were added to
the assembler in 2006.  cfi_sections support was added in 2009, a few
weeks before binutils 2.20 was released; it's in 2.20, the minimum
supported version, so even that configure test is obsolete.

Tested x86_64 that the installed shared libraries are unchanged by
this patch.

* configure.ac (libc_cv_asm_cfi_directives): Remove configure
test.
* configure: Regenerated.
* config.h.in (HAVE_ASM_CFI_DIRECTIVES): Remove macro undefine.
* sysdeps/arm/configure.ac (libc_cv_asm_cfi_directive_sections):
Remove configure test.
* sysdeps/arm/configure: Regenerated.
* sysdeps/nptl/configure.ac: Do not check
libc_cv_asm_cfi_directives.
* sysdeps/nptl/configure: Regenerated.
* sysdeps/x86_64/nptl/configure.ac: Remove file.
* sysdeps/x86_64/nptl/configure: Remove generated file.
* b/sysdeps/generic/sysdep.h [HAVE_ASM_CFI_DIRECTIVES]: Make code
unconditional.
[!HAVE_ASM_CFI_DIRECTIVES]: Remove conditional code.

ChangeLog
config.h.in
configure
configure.ac
sysdeps/arm/configure
sysdeps/arm/configure.ac
sysdeps/generic/sysdep.h
sysdeps/nptl/configure
sysdeps/nptl/configure.ac
sysdeps/x86_64/nptl/configure [deleted file]
sysdeps/x86_64/nptl/configure.ac [deleted file]

index c159faa45af9573a8d5522e648632dd31f6e685b..e90652fba1c1469fc0e7f4a0490720460234ad98 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,21 @@
+2014-06-26  Joseph Myers  <joseph@codesourcery.com>
+
+       * configure.ac (libc_cv_asm_cfi_directives): Remove configure
+       test.
+       * configure: Regenerated.
+       * config.h.in (HAVE_ASM_CFI_DIRECTIVES): Remove macro undefine.
+       * sysdeps/arm/configure.ac (libc_cv_asm_cfi_directive_sections):
+       Remove configure test.
+       * sysdeps/arm/configure: Regenerated.
+       * sysdeps/nptl/configure.ac: Do not check
+       libc_cv_asm_cfi_directives.
+       * sysdeps/nptl/configure: Regenerated.
+       * sysdeps/x86_64/nptl/configure.ac: Remove file.
+       * sysdeps/x86_64/nptl/configure: Remove generated file.
+       * b/sysdeps/generic/sysdep.h [HAVE_ASM_CFI_DIRECTIVES]: Make code
+       unconditional.
+       [!HAVE_ASM_CFI_DIRECTIVES]: Remove conditional code.
+
 2014-06-26  Siddhesh Poyarekar  <siddhesh@redhat.com>
 
        * posix/fnmatch.c: Define WIDE_CHAR_VERSION.
index 38d4aa4b6abeb3a5cb20f471585bad914bcfa183..2dcd1351c7bf3bb170f8ff71e3b4e9ef3a968b95 100644 (file)
@@ -24,9 +24,6 @@
 /* Define if weak symbols are available via the `.weakext' directive.  */
 #undef HAVE_ASM_WEAKEXT_DIRECTIVE
 
-/* Define if CFI directives are available.  */
-#undef HAVE_ASM_CFI_DIRECTIVES
-
 /* Define to the assembler line separator character for multiple
    assembler instructions per line.  Default is `;'  */
 #undef ASM_LINE_SEP
index ac5a67cf9d9e62caf24df726bf5e892bd69185ba..eefc43001d36b8287b0e08b4198626589e24cab9 100755 (executable)
--- a/configure
+++ b/configure
@@ -6494,43 +6494,6 @@ elif test $libc_cv_asm_weakext_directive = yes; then
 
 fi
 
-{ $as_echo "$as_me:${as_lineno-$LINENO}: checking whether CFI directives are supported" >&5
-$as_echo_n "checking whether CFI directives are supported... " >&6; }
-if ${libc_cv_asm_cfi_directives+:} false; then :
-  $as_echo_n "(cached) " >&6
-else
-  case $machine in
-  sparc/sparc64*) cfi_offset=2047;;
-  *) cfi_offset=0;;
-esac
-cat > conftest.s <<EOF
-       .text
-       .type   func,%function
-func:
-       .cfi_startproc
-       .cfi_remember_state
-       .cfi_rel_offset 1, $cfi_offset
-       .cfi_endproc
-EOF
-if { ac_try='${CC-cc} $ASFLAGS -c conftest.s 1>&5'
-  { { eval echo "\"\$as_me\":${as_lineno-$LINENO}: \"$ac_try\""; } >&5
-  (eval $ac_try) 2>&5
-  ac_status=$?
-  $as_echo "$as_me:${as_lineno-$LINENO}: \$? = $ac_status" >&5
-  test $ac_status = 0; }; }; then
-  libc_cv_asm_cfi_directives=yes
-else
-  libc_cv_asm_cfi_directives=no
-fi
-rm -f conftest*
-fi
-{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $libc_cv_asm_cfi_directives" >&5
-$as_echo "$libc_cv_asm_cfi_directives" >&6; }
-if test $libc_cv_asm_cfi_directives = yes; then
-  $as_echo "#define HAVE_ASM_CFI_DIRECTIVES 1" >>confdefs.h
-
-fi
-
 { $as_echo "$as_me:${as_lineno-$LINENO}: checking for ld --no-whole-archive" >&5
 $as_echo_n "checking for ld --no-whole-archive... " >&6; }
 if ${libc_cv_ld_no_whole_archive+:} false; then :
index bd3e8296662f33799e11cc8fbf93540835858e7b..22c2b4c376497638838e7ab5d354e216d801d468 100644 (file)
@@ -1696,30 +1696,6 @@ elif test $libc_cv_asm_weakext_directive = yes; then
   AC_DEFINE(HAVE_ASM_WEAKEXT_DIRECTIVE)
 fi
 
-AC_CACHE_CHECK(whether CFI directives are supported, libc_cv_asm_cfi_directives, [dnl
-case $machine in
-  sparc/sparc64*) cfi_offset=2047;;
-  *) cfi_offset=0;;
-esac
-cat > conftest.s <<EOF
-       .text
-       .type   func,%function
-func:
-       .cfi_startproc
-       .cfi_remember_state
-       .cfi_rel_offset 1, $cfi_offset
-       .cfi_endproc
-EOF
-if AC_TRY_COMMAND(${CC-cc} $ASFLAGS -c conftest.s 1>&AS_MESSAGE_LOG_FD); then
-  libc_cv_asm_cfi_directives=yes
-else
-  libc_cv_asm_cfi_directives=no
-fi
-rm -f conftest*])
-if test $libc_cv_asm_cfi_directives = yes; then
-  AC_DEFINE(HAVE_ASM_CFI_DIRECTIVES)
-fi
-
 AC_CACHE_CHECK(for ld --no-whole-archive, libc_cv_ld_no_whole_archive, [dnl
 cat > conftest.c <<\EOF
 _start () {}
index 953ef4406dfadc1c04abfbba20687789c65b9424..860f41289d123ca7704b2f4e0498d18ccda484a2 100644 (file)
@@ -3,35 +3,6 @@
 
 #AC_DEFINE(PI_STATIC_AND_HIDDEN)
 
-{ $as_echo "$as_me:${as_lineno-$LINENO}: checking whether the CFI directive .cfi_sections is supported" >&5
-$as_echo_n "checking whether the CFI directive .cfi_sections is supported... " >&6; }
-if ${libc_cv_asm_cfi_directive_sections+:} false; then :
-  $as_echo_n "(cached) " >&6
-else
-  cat > conftest.s <<EOF
-       .text
-       .cfi_sections .debug_frame, .eh_frame
-       .cfi_startproc
-       .cfi_endproc
-EOF
-  if { ac_try='${CC-cc} $ASFLAGS -c conftest.s 1>&5'
-  { { eval echo "\"\$as_me\":${as_lineno-$LINENO}: \"$ac_try\""; } >&5
-  (eval $ac_try) 2>&5
-  ac_status=$?
-  $as_echo "$as_me:${as_lineno-$LINENO}: \$? = $ac_status" >&5
-  test $ac_status = 0; }; }; then
-    libc_cv_asm_cfi_directive_sections=yes
-  else
-    libc_cv_asm_cfi_directive_sections=no
-  fi
-  rm -f conftest*
-fi
-{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $libc_cv_asm_cfi_directive_sections" >&5
-$as_echo "$libc_cv_asm_cfi_directive_sections" >&6; }
-if test $libc_cv_asm_cfi_directive_sections != yes; then
-  as_fn_error $? "need .cfi_sections in this configuration" "$LINENO" 5
-fi
-
 # We check to see if the compiler and flags are
 # selecting the hard-float ABI and if they are then
 # we set libc_cv_arm_pcs_vfp to yes which causes
index fdc52c04082a0b931cb643cfe7646d0b1f2e574f..f1d980436b9afbd1277237e1f2fa8e2b01a41c04 100644 (file)
@@ -7,24 +7,6 @@ dnl NOTE: This feature was added by the GCC TLS patches.  We should test for
 dnl it.  Until we do, don't define it.
 #AC_DEFINE(PI_STATIC_AND_HIDDEN)
 
-AC_CACHE_CHECK([whether the CFI directive .cfi_sections is supported],
-  [libc_cv_asm_cfi_directive_sections],
-  [cat > conftest.s <<EOF
-       .text
-       .cfi_sections .debug_frame, .eh_frame
-       .cfi_startproc
-       .cfi_endproc
-EOF
-  if AC_TRY_COMMAND(${CC-cc} $ASFLAGS -c conftest.s 1>&AS_MESSAGE_LOG_FD); then
-    libc_cv_asm_cfi_directive_sections=yes
-  else
-    libc_cv_asm_cfi_directive_sections=no
-  fi
-  rm -f conftest*])
-if test $libc_cv_asm_cfi_directive_sections != yes; then
-  AC_MSG_ERROR([need .cfi_sections in this configuration])
-fi
-
 # We check to see if the compiler and flags are
 # selecting the hard-float ABI and if they are then
 # we set libc_cv_arm_pcs_vfp to yes which causes
index d1104efdf44c3a3a59e990a8a4bac9d51883a5c9..583696c8c93257eb3703b8d49957349bb1323737 100644 (file)
 #endif
 
 /* Makros to generate eh_frame unwind information.  */
-#ifdef HAVE_ASM_CFI_DIRECTIVES
-# ifdef __ASSEMBLER__
-#  define cfi_startproc                        .cfi_startproc
-#  define cfi_endproc                  .cfi_endproc
-#  define cfi_def_cfa(reg, off)                .cfi_def_cfa reg, off
-#  define cfi_def_cfa_register(reg)    .cfi_def_cfa_register reg
-#  define cfi_def_cfa_offset(off)      .cfi_def_cfa_offset off
-#  define cfi_adjust_cfa_offset(off)   .cfi_adjust_cfa_offset off
-#  define cfi_offset(reg, off)         .cfi_offset reg, off
-#  define cfi_rel_offset(reg, off)     .cfi_rel_offset reg, off
-#  define cfi_register(r1, r2)         .cfi_register r1, r2
-#  define cfi_return_column(reg)       .cfi_return_column reg
-#  define cfi_restore(reg)             .cfi_restore reg
-#  define cfi_same_value(reg)          .cfi_same_value reg
-#  define cfi_undefined(reg)           .cfi_undefined reg
-#  define cfi_remember_state           .cfi_remember_state
-#  define cfi_restore_state            .cfi_restore_state
-#  define cfi_window_save              .cfi_window_save
-#  define cfi_personality(enc, exp)    .cfi_personality enc, exp
-#  define cfi_lsda(enc, exp)           .cfi_lsda enc, exp
+#ifdef __ASSEMBLER__
+# define cfi_startproc                 .cfi_startproc
+# define cfi_endproc                   .cfi_endproc
+# define cfi_def_cfa(reg, off)         .cfi_def_cfa reg, off
+# define cfi_def_cfa_register(reg)     .cfi_def_cfa_register reg
+# define cfi_def_cfa_offset(off)       .cfi_def_cfa_offset off
+# define cfi_adjust_cfa_offset(off)    .cfi_adjust_cfa_offset off
+# define cfi_offset(reg, off)          .cfi_offset reg, off
+# define cfi_rel_offset(reg, off)      .cfi_rel_offset reg, off
+# define cfi_register(r1, r2)          .cfi_register r1, r2
+# define cfi_return_column(reg)        .cfi_return_column reg
+# define cfi_restore(reg)              .cfi_restore reg
+# define cfi_same_value(reg)           .cfi_same_value reg
+# define cfi_undefined(reg)            .cfi_undefined reg
+# define cfi_remember_state            .cfi_remember_state
+# define cfi_restore_state             .cfi_restore_state
+# define cfi_window_save               .cfi_window_save
+# define cfi_personality(enc, exp)     .cfi_personality enc, exp
+# define cfi_lsda(enc, exp)            .cfi_lsda enc, exp
 
-# else /* ! ASSEMBLER */
+#else /* ! ASSEMBLER */
 
-#  define CFI_STRINGIFY(Name) CFI_STRINGIFY2 (Name)
-#  define CFI_STRINGIFY2(Name) #Name
-#  define CFI_STARTPROC        ".cfi_startproc"
-#  define CFI_ENDPROC  ".cfi_endproc"
-#  define CFI_DEF_CFA(reg, off)        \
+# define CFI_STRINGIFY(Name) CFI_STRINGIFY2 (Name)
+# define CFI_STRINGIFY2(Name) #Name
+# define CFI_STARTPROC ".cfi_startproc"
+# define CFI_ENDPROC   ".cfi_endproc"
+# define CFI_DEF_CFA(reg, off) \
    ".cfi_def_cfa " CFI_STRINGIFY(reg) "," CFI_STRINGIFY(off)
-#  define CFI_DEF_CFA_REGISTER(reg) \
+# define CFI_DEF_CFA_REGISTER(reg) \
    ".cfi_def_cfa_register " CFI_STRINGIFY(reg)
-#  define CFI_DEF_CFA_OFFSET(off) \
+# define CFI_DEF_CFA_OFFSET(off) \
    ".cfi_def_cfa_offset " CFI_STRINGIFY(off)
-#  define CFI_ADJUST_CFA_OFFSET(off) \
+# define CFI_ADJUST_CFA_OFFSET(off) \
    ".cfi_adjust_cfa_offset " CFI_STRINGIFY(off)
-#  define CFI_OFFSET(reg, off) \
+# define CFI_OFFSET(reg, off) \
    ".cfi_offset " CFI_STRINGIFY(reg) "," CFI_STRINGIFY(off)
-#  define CFI_REL_OFFSET(reg, off) \
+# define CFI_REL_OFFSET(reg, off) \
    ".cfi_rel_offset " CFI_STRINGIFY(reg) "," CFI_STRINGIFY(off)
-#  define CFI_REGISTER(r1, r2) \
+# define CFI_REGISTER(r1, r2) \
    ".cfi_register " CFI_STRINGIFY(r1) "," CFI_STRINGIFY(r2)
-#  define CFI_RETURN_COLUMN(reg) \
+# define CFI_RETURN_COLUMN(reg) \
    ".cfi_return_column " CFI_STRINGIFY(reg)
-#  define CFI_RESTORE(reg) \
+# define CFI_RESTORE(reg) \
    ".cfi_restore " CFI_STRINGIFY(reg)
-#  define CFI_UNDEFINED(reg) \
+# define CFI_UNDEFINED(reg) \
    ".cfi_undefined " CFI_STRINGIFY(reg)
-#  define CFI_REMEMBER_STATE \
+# define CFI_REMEMBER_STATE \
    ".cfi_remember_state"
-#  define CFI_RESTORE_STATE \
+# define CFI_RESTORE_STATE \
    ".cfi_restore_state"
-#  define CFI_WINDOW_SAVE \
+# define CFI_WINDOW_SAVE \
    ".cfi_window_save"
-#  define CFI_PERSONALITY(enc, exp) \
+# define CFI_PERSONALITY(enc, exp) \
    ".cfi_personality " CFI_STRINGIFY(enc) "," CFI_STRINGIFY(exp)
-#  define CFI_LSDA(enc, exp) \
+# define CFI_LSDA(enc, exp) \
    ".cfi_lsda " CFI_STRINGIFY(enc) "," CFI_STRINGIFY(exp)
-# endif
-
-#else
-
-# define CFI_STARTPROC
-# define CFI_ENDPROC
-# define CFI_DEF_CFA(reg, off)
-# define CFI_DEF_CFA_REGISTER(reg)
-# define CFI_DEF_CFA_OFFSET(off)
-# define CFI_ADJUST_CFA_OFFSET(off)
-# define CFI_OFFSET(reg, off)
-# define CFI_REL_OFFSET(reg, off)
-# define CFI_REGISTER(r1, r2)
-# define CFI_RETURN_COLUMN(reg)
-# define CFI_RESTORE(reg)
-# define CFI_UNDEFINED(reg)
-# define CFI_REMEMBER_STATE
-# define CFI_RESTORE_STATE
-# define CFI_WINDOW_SAVE
-# define CFI_PERSONALITY(enc, exp)
-# define CFI_LSDA(enc, exp)
 #endif
 
 #include "dwarf2.h"
index 239a039e3aefec7edb55faf026353ae1a362b827..65ed16400c7a53cd79b263dc13c860fa5040e6d2 100644 (file)
@@ -9,14 +9,6 @@ if test "x${libc_cv_visibility_attribute}" != xyes ||
   as_fn_error $? "working compiler support for visibility attribute is required" "$LINENO" 5
 fi
 
-if test "x$libc_cv_asm_cfi_directives" != xyes; then
-    case "$base_machine" in
-    i386 | x86_64 | powerpc | s390)
-      as_fn_error $? "CFI directive support in assembler is required" "$LINENO" 5 ;;
-    *) ;;
-  esac
-fi
-
 
 { $as_echo "$as_me:${as_lineno-$LINENO}: checking for forced unwind support" >&5
 $as_echo_n "checking for forced unwind support... " >&6; }
index 413af96799995c5979d12da1bb62f4acee757f60..ab9d5e624a0eca94e606459c2f626530916871f3 100644 (file)
@@ -10,15 +10,6 @@ if test "x${libc_cv_visibility_attribute}" != xyes ||
   AC_MSG_ERROR(working compiler support for visibility attribute is required)
 fi
 
-if test "x$libc_cv_asm_cfi_directives" != xyes; then
-  dnl We need this only for some architectures.
-  case "$base_machine" in
-    i386 | x86_64 | powerpc | s390)
-      AC_MSG_ERROR(CFI directive support in assembler is required) ;;
-    *) ;;
-  esac
-fi
-
 dnl Iff <unwind.h> is available, make sure it is the right one and it
 dnl contains struct _Unwind_Exception.
 AC_CACHE_CHECK(dnl
diff --git a/sysdeps/x86_64/nptl/configure b/sysdeps/x86_64/nptl/configure
deleted file mode 100644 (file)
index c9dea5a..0000000
+++ /dev/null
@@ -1,34 +0,0 @@
-# This file is generated from configure.ac by Autoconf.  DO NOT EDIT!
- # Local configure fragment for sysdeps/i386.
-
-{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for .cfi_personality and .cfi_lsda pseudo-ops" >&5
-$as_echo_n "checking for .cfi_personality and .cfi_lsda pseudo-ops... " >&6; }
-if ${libc_cv_asm_cfi_personality+:} false; then :
-  $as_echo_n "(cached) " >&6
-else
-    cat > conftest.s <<EOF
-${libc_cv_dot_text}
-foo:
-       .cfi_startproc
-       .cfi_personality 0, foo
-       .cfi_lsda 0, foo
-       .cfi_endproc
-EOF
-  if { ac_try='${CC-cc} $ASFLAGS -c conftest.s 1>&5'
-  { { eval echo "\"\$as_me\":${as_lineno-$LINENO}: \"$ac_try\""; } >&5
-  (eval $ac_try) 2>&5
-  ac_status=$?
-  $as_echo "$as_me:${as_lineno-$LINENO}: \$? = $ac_status" >&5
-  test $ac_status = 0; }; }; then
-    libc_cv_asm_cfi_personality=yes
-  else
-    libc_cv_asm_cfi_personality=no
-  fi
-  rm -f conftest*
-
-fi
-{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $libc_cv_asm_cfi_personality" >&5
-$as_echo "$libc_cv_asm_cfi_personality" >&6; }
-if test x"$libc_cv_asm_cfi_personality" != xyes; then
-  as_fn_error $? "assembler too old, .cfi_personality support missing" "$LINENO" 5
-fi
diff --git a/sysdeps/x86_64/nptl/configure.ac b/sysdeps/x86_64/nptl/configure.ac
deleted file mode 100644 (file)
index 0ba0cc3..0000000
+++ /dev/null
@@ -1,23 +0,0 @@
-GLIBC_PROVIDES dnl See aclocal.m4 in the top level source directory.
-# Local configure fragment for sysdeps/i386.
-
-AC_CACHE_CHECK([for .cfi_personality and .cfi_lsda pseudo-ops],
-              libc_cv_asm_cfi_personality, [dnl
-  cat > conftest.s <<EOF
-${libc_cv_dot_text}
-foo:
-       .cfi_startproc
-       .cfi_personality 0, foo
-       .cfi_lsda 0, foo
-       .cfi_endproc
-EOF
-  if AC_TRY_COMMAND(${CC-cc} $ASFLAGS -c conftest.s 1>&AS_MESSAGE_LOG_FD); then
-    libc_cv_asm_cfi_personality=yes
-  else
-    libc_cv_asm_cfi_personality=no
-  fi
-  rm -f conftest*
-])
-if test x"$libc_cv_asm_cfi_personality" != xyes; then
-  AC_MSG_ERROR([assembler too old, .cfi_personality support missing])
-fi
This page took 0.133021 seconds and 5 git commands to generate.