Re: [PATCH] Simplify dg-options for tests using pthreads

2016-08-23 Thread Jonathan Wakely
On 23/08/16 10:04 +0100, Jonathan Wakely wrote: On 23/08/16 10:45 +0200, Rainer Orth wrote: Btw., there's one last -pthreads reference left in libstdc++-v3/doc/xml/manual/using.xml. Perhaps this should go, too. Ah thanks, I'll update that. Committed to trunk. commit

Re: [PATCH] Simplify dg-options for tests using pthreads

2016-08-23 Thread Jonathan Wakely
On 23/08/16 11:19 +0200, Rainer Orth wrote: Hi Jonathan, The next step would be to replace the explict target list with an effective-target keyword ;-) Yes, I've started doing that in a local branch, so we'd use something like: { dg-do run { target libstdc++-threads-supported } } {

Re: [PATCH] Simplify dg-options for tests using pthreads

2016-08-23 Thread Rainer Orth
Hi Jonathan, >>The next step would be to replace the explict target list with an >>effective-target keyword ;-) > > Yes, I've started doing that in a local branch, so we'd use something > like: > > { dg-do run { target libstdc++-threads-supported } } > { dg-add-options libstdc++-threads-flags } >

Re: [PATCH] Simplify dg-options for tests using pthreads

2016-08-23 Thread Jonathan Wakely
On 23/08/16 10:45 +0200, Rainer Orth wrote: Hi Jonathan, It's been several years now that Solaris supported -pthread as well as -pthreads, so there's no need to have separate dg-options directives for Solaris. This patch removes all the lines: // { dg-options "... -pthreads" { target

Re: [PATCH] Simplify dg-options for tests using pthreads

2016-08-23 Thread Rainer Orth
Hi Jonathan, > It's been several years now that Solaris supported -pthread as well as > -pthreads, so there's no need to have separate dg-options directives > for Solaris. > > This patch removes all the lines: > > // { dg-options "... -pthreads" { target *-*-solaris* } } > > And adds *-*-solaris*

Re: [PATCH] Simplify dg-options for tests using pthreads

2016-08-19 Thread Jonathan Wakely
On 18/08/16 09:06 +0100, Jonathan Wakely wrote: It's been several years now that Solaris supported -pthread as well as -pthreads, so there's no need to have separate dg-options directives for Solaris. This patch removes all the lines: // { dg-options "... -pthreads" { target *-*-solaris* } }