On Mon, Jul 18, 2011 at 01:53:40PM +0200, Rainer Orth wrote:
> As described in the PR, the fact that the libgomp testsuite always links
> with -lgomp, even with -fno-openmp, causes testsuite failures on targets
> (like Solaris) that need special options when linking libgomp.  With
> -fgomp, this is taken care of by the relevant specs (like adding
> -pthread), but not so otherwise.

Needing special options to link against -lgomp is unfortunate, -lgomp
is what users want to link against if they have an OpenMP program that calls
some of the omp_* functions unconditionally if they want to compile it
without OpenMP parallelization.  For -static I can understand why special
options are needed, but for dynamic linking it is weird.

> The following patch fixes the failures on Solaris 8 and was tested with
> make check in libgomp on sparc-sun-solaris2.8, i386-pc-solaris2.10, and
> x86_64-unknown-linux-gnu.
> 
> Ok for mainline and 4.6 branch?

In any case, -lgomp explicitly isn't needed in the testsuite, so this is ok
for trunk/4.6.

> 2011-07-15  Rainer Orth  <r...@cebitec.uni-bielefeld.de>
> 
>       PR target/49541
>       * testsuite/lib/libgomp.exp (libgomp_init): Don't add -lgomp to
>       ldflags.

        Jakub

Reply via email to