Hi!

On Thu, 3 Jul 2014 13:35:15 +0200, Jakub Jelinek <ja...@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

Attachment: pgpYvMXhxiAob.pgp
Description: PGP signature

Reply via email to