Patch testsuite/39790: gcc.dg/tls/alias-1.c,opt-2.c fail if TLS is not supported by C library

Joseph S. Myers joseph@codesourcery.com
Fri Apr 17 09:59:00 GMT 2009


On Thu, 16 Apr 2009, Jing Yu wrote:

> +# This won't change for different subtargets so cache the result.

I don't believe this comment in this case; you can have both glibc and 
uClibc multilibs in one toolchain, for example, with the glibc multilibs 
supporting linking TLS code and the uClibc multilibs not supporting doing 
so.

Since the function does caching indirectly through functions that check 
for changes of target, I don't actually think there is a problem; avoiding 
any mention of caching here might be better (and in many of the other 
places that currently repeat the same comment).

-- 
Joseph S. Myers
joseph@codesourcery.com



More information about the Gcc-patches mailing list