This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [RFA:] fix non-tls regression with "PATCH: libstdc++/include/profile bug fixes and cleanup"
- From: Hans-Peter Nilsson <hans-peter dot nilsson at axis dot com>
- To: bkoz at redhat dot com
- Cc: libstdc++ at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org, paolo dot carlini at oracle dot com, rus at google dot com
- Date: Tue, 12 Jan 2010 06:24:59 +0100
- Subject: Re: [RFA:] fix non-tls regression with "PATCH: libstdc++/include/profile bug fixes and cleanup"
> Date: Tue, 12 Jan 2010 03:57:38 +0100
> From: Hans-Peter Nilsson <hp@axis.com>
> There is dg-require-effective-target tls_native (tls_runtime for
> execute tests), but that wasn't my preferred approach, as it
> would restrict the targets for the test where it wasn't
> restricted before. On the other hand, it doesn't test
> compilability of the thread-stuff.
Let's force no-threads only when needed and not restrict the
test to only tls targets. Obvious, right? Committed.
* testsuite/ext/profile/all.cc: Pass -D_GLIBCXX_PROFILE_NO_THREADS
for non-tls_native targets.
Index: testsuite/ext/profile/all.cc
===================================================================
--- testsuite/ext/profile/all.cc (revision 155826)
+++ testsuite/ext/profile/all.cc (working copy)
@@ -1,4 +1,5 @@
// { dg-options "-std=gnu++0x -O0 -D_GLIBCXX_PROFILE" }
+// { dg-options "-std=gnu++0x -O0 -D_GLIBCXX_PROFILE -D_GLIBCXX_PROFILE_NO_THREADS" { target { ! tls_native } } }
// { dg-do compile }
// -*- C++ -*-
brgds, H-P