[PATCH roland/Wundef] Compile with -Wundef.

Thomas Schwinge thomas@codesourcery.com
Mon Mar 3 09:03:00 GMT 2014


Hi!

On Fri, 28 Feb 2014 15:28:05 -0800 (PST), Roland McGrath <roland@hack.frob.com> wrote:
> --- a/sysdeps/mach/hurd/i386/tls.h
> +++ b/sysdeps/mach/hurd/i386/tls.h
> @@ -26,6 +26,7 @@
>  /* The TCB can have any size and the memory following the address the
>     thread pointer points to is unspecified.  Allocate the TCB there.  */
>  #define TLS_TCB_AT_TP	1
> +#define TLS_TCB_AT_TP	0

:-) Hmm?  (Should that have been TLS_DTV_AT_TP, I think?)


Also, not relevant for Hurd but for NPTL: don't all »#ifdef
TLS_TCB_AT_TP« need to be adapted now?  And, all the instances of »#error
"Either TLS_TCB_AT_TP or TLS_DTV_AT_TP must be defined"« are not longer
appropriate in this wording, too.


Generally, of course, thumbs up for stricter compile-time checks.


Grüße,
 Thomas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 489 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/libc-alpha/attachments/20140303/251e1b20/attachment.sig>


More information about the Libc-alpha mailing list