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: C++ PATCH for target/54908 (thread_local vs emutls)

From: Jason Merrill <jason at redhat dot com>
To: Jack Howarth <howarth at bromo dot med dot uc dot edu>
Cc: gcc-patches List <gcc-patches at gcc dot gnu dot org>, Dominique Dhumieres <dominiq at lps dot ens dot fr>
Date: Sat, 19 Jan 2013 00:15:58 -0500
Subject: Re: C++ PATCH for target/54908 (thread_local vs emutls)
References: <507BBBDE.1090909@redhat.com> <50F9CCAE.7070409@redhat.com> <20130119022016.GA28778@bromo.med.uc.edu>

On 01/18/2013 09:20 PM, Jack Howarth wrote:

    The proposed patch eliminates all of the previous failures from PR54908
at both -m32/-m64 on x86_64-apple-darwin12 but seems to introduce a new set of
failures...

FAIL: g++.dg/tls/thread_local-wrap3.C scan-assembler _ZTH1i
FAIL: g++.dg/gomp/tls-wrap3.C -std=c++98  scan-assembler _ZTH1i
FAIL: g++.dg/gomp/tls-wrap3.C -std=c++11  scan-assembler _ZTH1i

Ah, yes; those are specifically testing for the aliases that we can't generate on darwin. I'll add dg-require-alias to those tests. Thanks for the testing.

Recompiling the testcase with -fextern-tls-init doesn't produce the missing _ZTH1i.

Hmm, compiling with -fextern-tls-init should give an error for you. Doesn't it?

Jason

References:
- Re: C++ PATCH for target/54908 (thread_local vs emutls)
  - From: Jason Merrill
- Re: C++ PATCH for target/54908 (thread_local vs emutls)
  - From: Jack Howarth

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