Hi! On Mon, 10 Nov 2014 17:30:38 +0300, Ilya Verbin <iver...@gmail.com> wrote: > --- /dev/null > +++ b/liboffloadmic/plugin/Makefile.am > @@ -0,0 +1,123 @@ > +# Plugin for offload execution on Intel MIC devices.
> +libgomp_src_dir = $(top_srcdir)/../../libgomp > +libgomp_dir = $(build_dir)/../../libgomp Hmm, I'm not too happy about external (to libgomp) files using (for example, #include) stuff from libgomp, for the reason given in <http://news.gmane.org/find-root.php?message_id=%3C87ioishf5z.fsf%40kepler.schwinge.homeip.net%3E>: it can then easily happen that any such files depend on, for example, Autoconf definitions which are provided in only one of the instances. That said, libgomp_target.h as well as omp.h currently are self-contained (the latter file after having been created from omp.h.in by libgomp's configure script), so this currently is not an actual problem. > + AM_LDFLAGS = -L$(liboffload_dir)/.libs -L$(libgomp_dir)/.libs > -loffloadmic_target -lcoi_device -lmyo-service -lgomp -rdynamic Given that this plugin wishes to link against libgomp, don't we have to make sure that libgomp has actually been built before that is attempted, and the following (untested) patch would be required? diff --git Makefile.def Makefile.def index 7c8761a..f0a3a91 100644 --- Makefile.def +++ Makefile.def @@ -550,7 +550,7 @@ dependencies = { module=configure-target-libvtv; on=all-target-libstdc++-v3; }; // generated by the libgomp configure. Unfortunately, due to the use of // recursive make, we can't be that specific. dependencies = { module=all-target-libstdc++-v3; on=configure-target-libgomp; }; -dependencies = { module=all-target-liboffloadmic; on=configure-target-libgomp; }; +dependencies = { module=all-target-liboffloadmic; on=all-target-libgomp; }; dependencies = { module=install-target-libgo; on=install-target-libatomic; }; dependencies = { module=install-target-libgfortran; on=install-target-libquadmath; }; diff --git Makefile.in Makefile.in index ba5ae4c..8c060b9 100644 --- Makefile.in +++ Makefile.in @@ -48884,7 +48884,7 @@ all-stage3-target-libstdc++-v3: maybe-configure-stage3-target-libgomp all-stage4-target-libstdc++-v3: maybe-configure-stage4-target-libgomp all-stageprofile-target-libstdc++-v3: maybe-configure-stageprofile-target-libgomp all-stagefeedback-target-libstdc++-v3: maybe-configure-stagefeedback-target-libgomp -all-target-liboffloadmic: maybe-configure-target-libgomp +all-target-liboffloadmic: maybe-all-target-libgomp install-target-libgo: maybe-install-target-libatomic install-target-libgfortran: maybe-install-target-libquadmath install-target-libgfortran: maybe-install-target-libgcc Grüße, Thomas
signature.asc
Description: PGP signature
