Re: [Testsuite] Fix Cilk's exp to add -B for libcilkrts (was: Re: [Build, Driver] Add -lcilkrts for -fcilkplus)

2014-03-27 Thread Rainer Orth
Tobias Burnus writes: > Rainer Orth wrote: >> Tobias Burnus writes: >>> H.J. Lu wrote: xgcc: error: libcilkrts.spec: No such file or directory >>> Hmm, I really wonder why it fails for you while it works for me: >> Do you happen to have the same/a recent version installed at the same >> pre

Re: [Testsuite] Fix Cilk's exp to add -B for libcilkrts (was: Re: [Build, Driver] Add -lcilkrts for -fcilkplus)

2014-03-26 Thread H.J. Lu
On Wed, Mar 26, 2014 at 4:55 PM, Tobias Burnus wrote: > > Rainer Orth wrote: >> >> Tobias Burnus writes: >>> >>> H.J. Lu wrote: xgcc: error: libcilkrts.spec: No such file or directory >>> >>> Hmm, I really wonder why it fails for you while it works for me: >> >> Do you happen to have th

[Testsuite] Fix Cilk's exp to add -B for libcilkrts (was: Re: [Build, Driver] Add -lcilkrts for -fcilkplus)

2014-03-26 Thread Tobias Burnus
Rainer Orth wrote: Tobias Burnus writes: H.J. Lu wrote: xgcc: error: libcilkrts.spec: No such file or directory Hmm, I really wonder why it fails for you while it works for me: Do you happen to have the same/a recent version installed at the same prefix your build under test is configured f

Re: [Build, Driver] Add -lcilkrts for -fcilkplus

2014-03-26 Thread Rainer Orth
Tobias Burnus writes: > H.J. Lu wrote: >> All cilk-plus link tests failed with >> xgcc: error: libcilkrts.spec: No such file or directory >> Don't you need to add -B/libcilkrts for this? > > Hmm, I really wonder why it fails for you while it works for me: [...] > I cross-checked: I ha

Re: [Build, Driver] Add -lcilkrts for -fcilkplus

2014-03-26 Thread H.J. Lu
On Wed, Mar 26, 2014 at 3:35 PM, Tobias Burnus wrote: > H.J. Lu wrote: >> >> All cilk-plus link tests failed with >> xgcc: error: libcilkrts.spec: No such file or directory >> Don't you need to add -B/libcilkrts for this? > > > Hmm, I really wonder why it fails for you while it works f

Re: [Build, Driver] Add -lcilkrts for -fcilkplus

2014-03-26 Thread Tobias Burnus
H.J. Lu wrote: All cilk-plus link tests failed with xgcc: error: libcilkrts.spec: No such file or directory Don't you need to add -B/libcilkrts for this? Hmm, I really wonder why it fails for you while it works for me: Running /home/tob/projects/gcc/gcc/testsuite/gcc.dg/cilk-plus/

Re: [Build, Driver] Add -lcilkrts for -fcilkplus

2014-03-26 Thread H.J. Lu
On Mon, Mar 10, 2014 at 11:42 PM, Tobias Burnus wrote: > When using Cilk Plus (-fcilkplus), it makes sense to automatically link the > run-time library (-lcilkrts). > > This patch mimics libgomp by adding a .spec file; I am not 100% sure whether > the .spec file is needed, but the pthread tests in

Re: [Build, Driver] Add -lcilkrts for -fcilkplus

2014-03-26 Thread Paolo Bonzini
Il 26/03/2014 00:28, Tobias Burnus ha scritto: Paolo Bonzini wrote: Il 11/03/2014 07:42, Tobias Burnus ha scritto: +XPCFLAGS="" +CFLAGS="$CFLAGS -pthread" +AC_LINK_IFELSE( ... + [XPCFLAGS=" -Wc,-pthread"], XPCFLAGS is dead, I think? Yes - contrary to libgomp, from which I have taken that cod

Re: [Build, Driver] Add -lcilkrts for -fcilkplus

2014-03-25 Thread Tobias Burnus
Paolo Bonzini wrote: Il 11/03/2014 07:42, Tobias Burnus ha scritto: +XPCFLAGS="" +CFLAGS="$CFLAGS -pthread" +AC_LINK_IFELSE( ... + [XPCFLAGS=" -Wc,-pthread"], XPCFLAGS is dead, I think? Yes - contrary to libgomp, from which I have taken that code block. I have now removed it. Also, should

Re: [Build, Driver] Add -lcilkrts for -fcilkplus

2014-03-25 Thread Paolo Bonzini
Il 11/03/2014 07:42, Tobias Burnus ha scritto: +# Check to see if -pthread or -lpthread is needed. Prefer the former. +# In case the pthread.h system header is not found, this test will fail. +XPCFLAGS="" +CFLAGS="$CFLAGS -pthread" +AC_LINK_IFELSE( + [AC_LANG_PROGRAM( + [#include + void *g(

Build *PING* Re: [Build, Driver] Add -lcilkrts for -fcilkplus

2014-03-24 Thread Tobias Burnus
*PING* for the build part as the driver part has been already approved. http://gcc.gnu.org/ml/gcc-patches/2014-03/msg00492.html Also pinging another one-line libcilkrts/Makefile.am patch: http://gcc.gnu.org/ml/gcc-patches/2014-03/msg0.html On March 11, 2014, Tobias Burnus wrote: When using

[build]-PING - Re: [Build, Driver] Add -lcilkrts for -fcilkplus

2014-03-16 Thread Tobias Burnus
PING for the build part; Joseph has already approved the driver part. Tobias Burnus wrote: When using Cilk Plus (-fcilkplus), it makes sense to automatically link the run-time library (-lcilkrts). This patch mimics libgomp by adding a .spec file; I am not 100% sure whether the .spec file is n

Re: [Build, Driver] Add -lcilkrts for -fcilkplus

2014-03-11 Thread Joseph S. Myers
On Tue, 11 Mar 2014, Tobias Burnus wrote: > When using Cilk Plus (-fcilkplus), it makes sense to automatically link the > run-time library (-lcilkrts). > > This patch mimics libgomp by adding a .spec file; I am not 100% sure whether > the .spec file is needed, but the pthread tests in libgomp imp

[Build, Driver] Add -lcilkrts for -fcilkplus

2014-03-10 Thread Tobias Burnus
When using Cilk Plus (-fcilkplus), it makes sense to automatically link the run-time library (-lcilkrts). This patch mimics libgomp by adding a .spec file; I am not 100% sure whether the .spec file is needed, but the pthread tests in libgomp imply that it makes sense. (libgomp also checks for