Issue 9033 - Remove __thread from the language.

[ghost]

http://d.puremagic.com/issues/show_bug.cgi?id=9033

[ibuclaw]

LGTM

[yebblies]

If we still need to print this, it might be cleaner to leave TOKtls in lexer.h and the mapping in lexer.c rather than special-casing it here.

[ghost]

Well ok, I just wanted to get rid of the token entirely. But I agree, some future function might end up having to special-case as well so I better leave it in.

[yebblies]

I just realised this keeps __thread as a keyword, which we maybe don't want. Sigh. The way it was before would probably be better than that, sorry.
I doubt we need to go through the whole deprecation process, but it should probably be added to deprecate.html.

[ghost]

Ok I've removed it and I'll add a deprecation notice to deprecate.html. Let's wait for all greens.

[WalterBright]

I'd prefer having the parser issue an error "__thread no longer supported, thread local storage is now the default" rather than some other random message. That means leaving it in as a keyword for now.

[ghost]

The thing is __thread was practically never used in D2 except the DMD test-suite. At least I haven't seen it used by anyone or any library. And it's not in D1 either. It's extremely easy to remove it automatically as well. We can add a note about it in the changelog, but I'm 99% certain it won't break any code.

[don-clugston-sociomantic]

I am 100% certain it will not break code. It NEVER worked, except in a couple of specific cases that were in the test suite, and it was never even documented. Just remove it entirely. Supporting deprecated features always comes at a cost. In this case the benefit is zero.

[dnadlinger]

+1 for removing it.