Patchwork PATCH COMMITTED: Try again to get libgcc -fsplit-stack in sync

login
register
mail settings
Submitter Ian Taylor
Date Sept. 29, 2010, 12:55 a.m.
Message ID <mcrfwwtqsgq.fsf@google.com>
Download mbox | patch
Permalink /patch/66032/
State New
Headers show

Comments

Ian Taylor - Sept. 29, 2010, 12:55 a.m.
This is another attempt to get the libgcc build in sync with gcc with
respect to -fsplit-stack.  This changes the test in the libgcc configure
script to match the test done in ix86_supports_split_stack.
Bootstrapped on x86_64-unknown-linux-gnu.  Committed to mainline.

Ian


2010-09-28  Ian Lance Taylor  <iant@google.com>

	* configure.ac: Adjust CFI test to test assembler directly rather
	than checking gcc preprocessor macro.
	* configure: Rebuild.

Patch

Index: configure.ac
===================================================================
--- configure.ac	(revision 164696)
+++ configure.ac	(working copy)
@@ -137,9 +137,14 @@  AC_SUBST(fixed_point)
 
 # Check for assembler CFI support.
 AC_CACHE_CHECK([whether assembler supports CFI directives], [libgcc_cv_cfi],
-	       [AC_COMPILE_IFELSE([int i = __GCC_HAVE_DWARF2_CFI_ASM;],
-				  [libgcc_cv_cfi=yes],
-				  [libgcc_cv_cfi=no])])
+	       [AC_COMPILE_IFELSE(
+[asm("\n\
+	.text\n\
+	.cfi_startproc\n\
+	.cfi_personality 0, symbol\n\
+	.cfi_endproc");],
+  [libgcc_cv_cfi=yes],
+  [libgcc_cv_cfi=no])])
 
 # Check 32bit or 64bit for x86.
 case ${host} in