This is the mail archive of the
libstdc++@gcc.gnu.org
mailing list for the libstdc++ project.
Re: [PATCH] libstdc++/testsuite/experimental/algorithm link errors
- From: Jonathan Wakely <jwakely at redhat dot com>
- To: David Edelsohn <dje dot gcc at gmail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, libstdc++ at gcc dot gnu dot org
- Date: Tue, 26 Jun 2018 20:13:07 +0100
- Subject: Re: [PATCH] libstdc++/testsuite/experimental/algorithm link errors
- References: <CAGWvnykn2z6dSdjU+kWzNOw+qBefMqcxiadAyhO4_SjMTmJy-g@mail.gmail.com>
On 26/06/18 15:09 -0400, David Edelsohn wrote:
The recent addition of
testsuite/experimental/algorithm/sample-2.cc
testsuite/experimental/algorithm/shuffle.cc
introduced link errors on AIX. AIX (and Solaris) require additional
options for TLS in some situations.
This patch adds the dejagnu TLS directives.
Was this dependency intended? Is this patch okay?
Yes, it's intended: those two tests depend on the per-thread random
number engine in <experimental/random>, and so the tests need the same
directives as testsuite/experimental/random/randint.cc
OK for trunk, thanks.