On Tue, Jan 15, 2019 at 08:05:59AM +0100, Richard Biener wrote: > >It either should > >#include "../../../libgfortran/ISO_Fortran_binding.h" > >instead or the Fortran *.exp files should arrange for > >-I.../libgfortran/ > >to be added to all gfortran tests. Because right now it FAILs if you > >don't > >have ISO_Fortran_binding.h header installed, or succeeds, but includes > >header from some other compiler version or even other compiler > >altogether.
This still needs to be fixed. > >Where is that header installed BTW? > >Would be best if it got installed in directories like: > >$prefix/lib/gcc/$target/$version/include > > > >See e.g. libssp or libsanitizer, both have something like > >target_noncanonical = @target_noncanonical@ > >libsubincludedir = > >$(libdir)/gcc/$(target_noncanonical)/$(gcc_version)/include > >nobase_libsubinclude_HEADERS = ssp/ssp.h ssp/string.h ssp/stdio.h > >ssp/unistd.h > > > >You probably want it to go directly in the include dir, so without the > >ssp/ > >or whatever else prefixes. > > It's there, but also in the multilib locations (which is dubious? Not sure if > we ever search tose include paths) Yeah, for -m32 it is in .../lib/gcc/x86_64-pc-linux-gnu/9.0.0/32/include/ which isn't that useful; while the finclude/ in there is needed, because those are target specific, this header is the same and so it could be just in .../9.0.0/include/ always (like e.g. the std*.h headers, intrinsics etc.). Jakub