[PATCH] Test for lazy tls descriptor initialization

Mike Frysinger vapier@gentoo.org
Tue Dec 29 19:53:00 GMT 2015


On 17 Nov 2015 16:51, Szabolcs Nagy wrote:
> +AC_CACHE_CHECK(for compiler option to enable TLS descriptors,
> +	       libc_cv_cc_tlsdesc, [dnl
> +libc_cv_cc_tlsdesc=-no-tlsdesc-support

should use "no" rather than an arbitrary nonsense string.  this is a cache
var, so people running configure would have to do:
	libc_cv_cc_tlsdesc=-no-tlsdesc-support
instead of the more common:
	libc_cv_cc_tlsdesc=no

> --- /dev/null
> +++ b/elf/tst-lazytls.h
> @@ -0,0 +1,11 @@
> +#define A(n) \

all these files need proper comment block headers

> +#include <assert.h>

you can't use assert in tests because it writes to stderr.
you'll need to use printf/exit yourself.

> +	extern void f##n (void); \

style on these files is wrong -- indentation needs to use spaces rather
than tabs
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://sourceware.org/pipermail/libc-alpha/attachments/20151229/433f8394/attachment.sig>


More information about the Libc-alpha mailing list