[Patch, libgomp] libgomp.fortran/fortran.exp - add -fintrinsic-modules-path ${blddir}

[Thomas Schwinge]

Hi!

On Thu, 3 Jul 2014 13:35:15 +0200, Jakub Jelinek <jakub@redhat.com> wrote:
> On Thu, Jul 03, 2014 at 12:19:15PM +0200, Thomas Schwinge wrote:
> > I found the following to work (but so far only did libgomp testing), but
> > that is a little bit more intrusive, but may actually be the right thing
> > to do.  (Possibly also in additional places where ${tool}_target_compile
> > is used?  CCing testsuite maintainers.)  Comments?
> 
> What about this instead, pass it only for Fortran tests and nothing else?
> Only very lightly tested so far.

Confirming that this does the right thing for my case.

> 	* testsuite/lib/libgomp.exp (libgomp_target_compile): If $source
> 	matches regex $lang_source_re, add $lang_include_flags to options.
> 	* testsuite/libgomp.c/c.exp: Unset lang_include_flags.
> 	* testsuite/libgomp.c++/c++.exp: Likewise.
> 	* testsuite/libgomp.fortran/fortran.exp: Likewise.  Set lang_source_re
> 	and lang_include_flags instead of adding -fintrinsic-modules-path= to
> 	ALWAYS_CFLAGS.
> 	* testsuite/libgomp.graphite/graphite.exp: Unset lang_include_flags.

Thanks!  Yes, that looks less intrusive than my patch.


Grüße,
 Thomas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 472 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20140703/f2ddffe8/attachment.sig>