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] 25816 Add configure option to disable TLS...


On Jan 17, 2006, David Daney <ddaney@avtrex.com> wrote:

> I have no system that supports TLS where configure guesses incorrectly
> about assembler TLS support.

Just set the config.cache variable to the wrong result and you'll have
such a system :-)

> In the mean time I would like to see if my patch can be committed to
> 4.1 as is

I don't think we should introduce a patch with functionality that is
known-not-good and likely to change.  Let's at least try to get it to
what we've now agreed to be the good, even if it takes a few minutes
longer.

If you want an actual example, try binutils and gcc from CVS for
arm-linux-gnu.  GCC says it needs binutils 2.17, but 2.16.9* already
have it, so configure gets it wrong.

Also, please check in AC_ARG_ENABLE that $enable_tls expands to either
yes or no, and error out (or at least print a warning) otherwise.

-- 
Alexandre Oliva         http://www.lsd.ic.unicamp.br/~oliva/
Red Hat Compiler Engineer   aoliva@{redhat.com, gcc.gnu.org}
Free Software Evangelist  oliva@{lsd.ic.unicamp.br, gnu.org}


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