From patchwork Thu Jul 15 17:19:58 2010
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: [trans-mem] Re: EH TM bug
Date: Thu, 15 Jul 2010 07:19:58 -0000
From: Richard Henderson <rth@redhat.com>
X-Patchwork-Id: 58997
Message-Id: <4C3F433E.4050405@redhat.com>
To: Aldy Hernandez <aldyh@redhat.com>, GCC Patches <gcc-patches@gcc.gnu.org>

[ TM+EH fails for i686-linux ]

Wow, this is silly.  When I converted to C++, I added -fno-exceptions
to get better code in the library, since the library itself does not
use exceptions.

Except there are a few routines involved in the *user's* exception
handling, and we need the unwind tables generated for that.  Of course
I did all the previous testing with x86_64-linux, which builds the
tables by default.


r~


	* Makefile.am (AM_CXXFLAGS): Add -funwind-tables.
	* Makefile.in: Rebuild.


diff --git a/libitm/Makefile.am b/libitm/Makefile.am
index fef2a13..8ad4e5f 100644
--- a/libitm/Makefile.am
+++ b/libitm/Makefile.am
@@ -18,7 +18,8 @@ vpath % $(strip $(search_path))
 
 AM_CPPFLAGS = $(addprefix -I, $(search_path))
 AM_CFLAGS = $(XCFLAGS)
-AM_CXXFLAGS = -std=gnu++0x -fno-exceptions -fno-rtti $(XCFLAGS) $(abi_version)
+AM_CXXFLAGS = -std=gnu++0x -funwind-tables -fno-exceptions -fno-rtti \
+	$(XCFLAGS) $(abi_version)
 AM_CCASFLAGS = $(XCFLAGS)
 AM_LDFLAGS = $(XLDFLAGS) $(SECTION_LDFLAGS) $(OPT_LDFLAGS)