Patchwork PATCH COMMITTED: Fix test for CFI directives

login
register
mail settings
Submitter Ian Taylor
Date Sept. 28, 2010, 6:26 p.m.
Message ID <mcrpqvxsp1g.fsf@google.com>
Download mbox | patch
Permalink /patch/66012/
State New
Headers show

Comments

Ian Taylor - Sept. 28, 2010, 6:26 p.m.
In the patch I committed yesterday, I foolishly tested
dwarf2out_do_cfi_asm when all I cared about was whether the assembler
supported the CFI directives.  This patch corrects that error.
Bootstrapped on x86_64-unknown-linux-gnu with a partial testsuite run.
Committed to mainline.

Ian


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

	* config/i386/i386.c (ix86_supports_split_stack): Test
	HAVE_GAS_CFI_PERSONALITY_DIRECTIVE rather than
	dwarf2out_do_cfi_asm.

Patch

Index: config/i386/i386.c
===================================================================
--- config/i386/i386.c	(revision 164695)
+++ config/i386/i386.c	(working copy)
@@ -8380,7 +8380,7 @@  ix86_supports_split_stack (bool report A
     error ("%<-fsplit-stack%> currently only supported on GNU/Linux");
   ret = false;
 #else
-  if (!dwarf2out_do_cfi_asm ())
+  if (!HAVE_GAS_CFI_PERSONALITY_DIRECTIVE)
     {
       if (report)
 	error ("%<-fsplit-stack%> requires "