Patchwork allow setting LDFLAGS_FOR_TARGET in top-level configuration.

login
register
mail settings
Submitter Doug Kwan (關振德)
Date May 9, 2011, 2:38 a.m.
Message ID <BANLkTimagY7GweUuCeBWN=bWesKbuGmd-Q@mail.gmail.com>
Download mbox | patch
Permalink /patch/94728/
State New
Headers show

Comments

Doug Kwan (關振德) - May 9, 2011, 2:38 a.m.
Sorry, forgot the patch and the ChangeLog

2011-05-08  Doug Kwan  <dougkwan@google.com>

	* configure.ac: Propagate LDFLAGS_FOR_TARGET.
	* configure: Regenerated.
	* Makefile.tpl (LDFLAGS_FOR_TARGET): Use LDFLAGS_FOR_TARGET
	value from configure.
	* Makefile.in: Regenerated.


On Sun, May 8, 2011 at 7:31 PM, Doug Kwan (關振德) <dougkwan@google.com> wrote:
> Hi,
>
>    The x86 Android toolchain needs setting LDFLAGS_FOR_TARGET to
> build.  This patch does that.  The patch was tested by bootstrapping
> natively on x86_64 linux.  Do I also need to submit this to binutils
> as well?
>
> -Doug
>
Paolo Bonzini - May 9, 2011, 7:32 a.m.
On 05/09/2011 04:38 AM, Doug Kwan (關振德) wrote:
> Sorry, forgot the patch and the ChangeLog
>
> 2011-05-08  Doug Kwan<dougkwan@google.com>
>
> 	* configure.ac: Propagate LDFLAGS_FOR_TARGET.
> 	* configure: Regenerated.
> 	* Makefile.tpl (LDFLAGS_FOR_TARGET): Use LDFLAGS_FOR_TARGET
> 	value from configure.
> 	* Makefile.in: Regenerated.

Ok.

Paolo

Patch

Index: configure.ac
===================================================================
--- configure.ac	(revision 173552)
+++ configure.ac	(working copy)
@@ -2110,6 +2110,8 @@  if test "x$CXXFLAGS_FOR_TARGET" = x; then
 fi
 AC_SUBST(CXXFLAGS_FOR_TARGET)
 
+AC_SUBST(LDFLAGS_FOR_TARGET)
+
 # Handle --with-headers=XXX.  If the value is not "yes", the contents of
 # the named directory are copied to $(tooldir)/sys-include.
 if test x"${with_headers}" != x && test x"${with_headers}" != xno ; then
Index: Makefile.tpl
===================================================================
--- Makefile.tpl	(revision 173552)
+++ Makefile.tpl	(working copy)
@@ -494,7 +494,7 @@  CXXFLAGS_FOR_TARGET = @CXXFLAGS_FOR_TARGET@
 
 LIBCFLAGS_FOR_TARGET = $(CFLAGS_FOR_TARGET)
 LIBCXXFLAGS_FOR_TARGET = $(CXXFLAGS_FOR_TARGET) -fno-implicit-templates
-LDFLAGS_FOR_TARGET = 
+LDFLAGS_FOR_TARGET = @LDFLAGS_FOR_TARGET@
 GOCFLAGS_FOR_TARGET = -O2 -g
 
 FLAGS_FOR_TARGET = @FLAGS_FOR_TARGET@