Re: [PATCH] Fix PR52297

2012-02-29 Thread Richard Guenther
On Wed, 29 Feb 2012, Rainer Orth wrote: > Richard Guenther writes: > > >> Moving them to libitm/testsuite is an option, but that might be more > >> difficult for the LTO case. I thought about adding > >> dg-require-effective-target tm_runtime (to check for working > >> -litm/libitm.spec) and dg

Re: [PATCH] Fix PR52297

2012-02-29 Thread Jakub Jelinek
On Wed, Feb 29, 2012 at 11:26:54AM +0100, Eric Botcazou wrote: > > It's still used by the working trans-mem-3_[01].c testcase. > > That test is a little puzzling: > > /* Test that we can build one object file with -fgnu-tm >(trans-mem-3_1.c), but do the final link of all objects without >

Re: [PATCH] Fix PR52297

2012-02-29 Thread Eric Botcazou
> It's still used by the working trans-mem-3_[01].c testcase. That test is a little puzzling: /* Test that we can build one object file with -fgnu-tm (trans-mem-3_1.c), but do the final link of all objects without -fgnu-tm. */ So, in the end, we cannot do that without the trans-mem.h klud

Re: [PATCH] Fix PR52297

2012-02-29 Thread Rainer Orth
Richard Guenther writes: >> Moving them to libitm/testsuite is an option, but that might be more >> difficult for the LTO case. I thought about adding >> dg-require-effective-target tm_runtime (to check for working >> -litm/libitm.spec) and dg-add-options tm_runtime to add the necessary >> -B/-L

Re: [PATCH] Fix PR52297

2012-02-29 Thread Richard Guenther
On Wed, 29 Feb 2012, Rainer Orth wrote: > Richard Guenther writes: > > > This fixes the LTO TM testcases by removing them. TM testcases > > belong in libitm testsuite (but dg-additional-sources does not > > work there currently for some weird reasons, so I gave up > > trying). > > Moving them

Re: [PATCH] Fix PR52297

2012-02-29 Thread Richard Guenther
On Wed, 29 Feb 2012, Eric Botcazou wrote: > > 2012-02-29 Richard Guenther > > > > PR testsuite/52297 > > * gcc.dg/lto/trans-mem-1_0.c: Remove. > > * gcc.dg/lto/trans-mem-1_1.c: Likewise. > > * gcc.dg/lto/trans-mem-2_0.c: Likewise. > > * gcc.dg/lto/trans-mem-2_1.c: Likewise.

Re: [PATCH] Fix PR52297

2012-02-29 Thread Rainer Orth
Richard Guenther writes: > This fixes the LTO TM testcases by removing them. TM testcases > belong in libitm testsuite (but dg-additional-sources does not > work there currently for some weird reasons, so I gave up > trying). Moving them to libitm/testsuite is an option, but that might be more

Re: [PATCH] Fix PR52297

2012-02-29 Thread Eric Botcazou
> 2012-02-29 Richard Guenther > > PR testsuite/52297 > * gcc.dg/lto/trans-mem-1_0.c: Remove. > * gcc.dg/lto/trans-mem-1_1.c: Likewise. > * gcc.dg/lto/trans-mem-2_0.c: Likewise. > * gcc.dg/lto/trans-mem-2_1.c: Likewise. > * gcc.dg/lto/trans-mem-4_0.c: Likewise.

[PATCH] Fix PR52297

2012-02-29 Thread Richard Guenther
This fixes the LTO TM testcases by removing them. TM testcases belong in libitm testsuite (but dg-additional-sources does not work there currently for some weird reasons, so I gave up trying). Committed. Richard. 2012-02-29 Richard Guenther PR testsuite/52297 * gcc.dg/lto/t