This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Patch: move TLS configury to config/


>>>>> "Ranjit" == Ranjit Mathew <rmathew@gmail.com> writes:

>> Index: config/ChangeLog
>> from  Tom Tromey  <tromey@redhat.com>
>> * enable.m4: New file.
>> * tls.m4: New file.

FWIW I think I just copied these from libstdc++...

Ranjit> Don't you think there should be a comma after "_g_switch" in
Ranjit> that use of AC_HELP_STRING? I think this is the cause for
Ranjit> this mangled string in the output of "libjava/configure --help":
Ranjit>   --disable-rpath         do not hardcode runtime library paths
Ranjit>   _g_switchUse thread-local storage
Ranjit>                           [default=yes]

I think the problem was not passing a help-arg argument to
GCC_ENABLE.  This worked for me.  Any comments?

Tom

2006-07-25  Tom Tromey  <tromey@redhat.com>

	* tls.m4 (GCC_CHECK_TLS): Pass empty argument as "help arg" to
	GCC_ENABLE.

Index: config/tls.m4
===================================================================
--- config/tls.m4	(revision 114572)
+++ config/tls.m4	(working copy)
@@ -1,6 +1,6 @@
 dnl Check whether the target supports TLS.
 AC_DEFUN([GCC_CHECK_TLS], [
-  GCC_ENABLE(tls, yes, [Use thread-local storage])
+  GCC_ENABLE(tls, yes, [], [Use thread-local storage])
   AC_CACHE_CHECK([whether the target supports thread-local storage],
 		 have_tls, [
     AC_RUN_IFELSE([__thread int a; int b; int main() { return a = b; }],


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]