PATCH COMMITTED: Fix test for CFI directives

Submitted by Ian Taylor on Sept. 28, 2010, 6:26 p.m.

Details

Message ID mcrpqvxsp1g.fsf@google.com
State New
Headers show

Commit Message

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 hide | download patch | download mbox

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 "