Patchwork [libitm] minor adjustment to make flags.

login
register
mail settings
Submitter IainS
Date Nov. 22, 2011, 3:07 p.m.
Message ID <C4D48BB9-54C0-4203-88CC-E8E4F90CEFD5@sandoe-acoustics.co.uk>
Download mbox | patch
Permalink /patch/127102/
State New
Headers show

Comments

IainS - Nov. 22, 2011, 3:07 p.m.
Hi,
since:
libitm_la_LDFLAGS
are now correctly applied Darwin no longer gets the "-Wl,- 
undefined,dynamic_lookup"
(which I had mentioned before was surprising me with -no-undefined in  
libitm_la_LDFLAGS).

Anyway, to make weak refs work with Darwin, we need that - and it  
turns out that other GCC libs are built without "-no-undefined".

is the following OK for trunk?
Iain

libitm:

	* Makefile.am (libitm_la_LDFLAGS): Remove "-no-undefined".
	* Makefile.in: Regenerate.
Richard Henderson - Nov. 22, 2011, 5:23 p.m.
On 11/22/2011 07:07 AM, Iain Sandoe wrote:
>     * Makefile.am (libitm_la_LDFLAGS): Remove "-no-undefined".
>     * Makefile.in: Regenerate.

Ok.


r~

Patch

Index: libitm/Makefile.am
===================================================================
--- libitm/Makefile.am	(revision 181618)
+++ libitm/Makefile.am	(working copy)
@@ -54,8 +54,7 @@  libitm_version_info = -version-info $(libtool_VERS
  # want or need libstdc++.
  libitm_la_DEPENDENCIES = $(libitm_version_dep)
  libitm_la_LINK = $(LINK) $(libitm_la_LDFLAGS)
-libitm_la_LDFLAGS = $(libitm_version_info) $(libitm_version_script) \
-        -no-undefined
+libitm_la_LDFLAGS = $(libitm_version_info) $(libitm_version_script)

  libitm_la_SOURCES = \
  	aatree.cc alloc.cc alloc_c.cc alloc_cpp.cc barrier.cc beginend.cc \