From: Ian Lance Taylor <iant@google.com>
Date: Fri, 13 Jun 2014 22:30:47 +0000 (+0000)
Subject: re PR go/61496 (Many Go tests FAIL with ld warning about go$zerovalue)
X-Git-Tag: releases/gcc-5.1.0~6880
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=bbeaf05dc0f845cc9d5ccef2e5db6d4e98255580;p=thirdparty%2Fgcc.git

re PR go/61496 (Many Go tests FAIL with ld warning about go$zerovalue)

	PR go/61496
	* gospec.c (lang_specific_driver): On Solaris, when not using GNU
	ld, add -t option to avoid warning about common symbol changing
	size.

From-SVN: r211658
---

diff --git a/gcc/go/ChangeLog b/gcc/go/ChangeLog
index 16d13409cde4..6b535851d6d8 100644
--- a/gcc/go/ChangeLog
+++ b/gcc/go/ChangeLog
@@ -1,3 +1,10 @@
+2014-06-13  Ian Lance Taylor  <iant@google.com>
+
+	PR go/61496
+	* gospec.c (lang_specific_driver): On Solaris, when not using GNU
+	ld, add -t option to avoid warning about common symbol changing
+	size.
+
 2014-06-10  Jan Hubicka  <hubicka@ucw.cz>
 
 	* go/go-gcc.cc (Gcc_backend::global_variable_set_init): Use
diff --git a/gcc/go/gospec.c b/gcc/go/gospec.c
index 02d5842352f5..8b2d8afda72d 100644
--- a/gcc/go/gospec.c
+++ b/gcc/go/gospec.c
@@ -395,6 +395,15 @@ lang_specific_driver (struct cl_decoded_option **in_decoded_options,
     }
 #endif
 
+#if defined(TARGET_SOLARIS) && !defined(USE_GLD)
+  /* We use a common symbol for go$zerovalue.  On Solaris, when not
+     using the GNU linker, the Solaris linker needs an option to not
+     warn about this.  Everything works without this option, but you
+     get unsightly warnings at link time.  */
+  generate_option (OPT_Wl_, "-t", 1, CL_DRIVER, &new_decoded_options[j]);
+  j++;
+#endif
+
   *in_decoded_options_count = j;
   *in_decoded_options = new_decoded_options;
   *in_added_libraries = added_libraries;