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
pgpYvMXhxiAob.pgp
Description: PGP signature