ampi: do not put mpi.h in include/ 45/5045/6
authorMatthias Diener <mdiener@illinois.edu>
Thu, 28 Mar 2019 20:02:13 +0000 (15:02 -0500)
committerMatthias Diener <mdiener@illinois.edu>
Tue, 2 Apr 2019 19:21:00 +0000 (14:21 -0500)
Change-Id: I3b5d3f432a5790be805530782c83be1f5c4abd69

src/libs/ck-libs/ampi/Makefile
src/libs/ck-libs/ampi/ampiCC
src/libs/ck-libs/idxl/Makefile
src/scripts/Makefile

index d4ec7f7a80ccc089d53211fee7995610817efea0..6eafa0400ab599cb9d0aa32c8e60a04f70dc01c7 100644 (file)
@@ -79,7 +79,11 @@ $(COMPATLIB): $(COMPAT)
 
 headers: $(HEADERS)
        cp $(HEADERS) $(CDIR)/include/
-       cp ampi.h $(CDIR)/include/mpi.h
+
+       # Put AMPI's mpi.h in a separate directory to support the MPI machine layer and MPI interop
+       mkdir -p $(CDIR)/include/ampi/
+       cp ampi.h $(CDIR)/include/ampi/mpi.h
+
        cp ampif.h $(CDIR)/include/mpif.h
        cp ampiimpl.h $(CDIR)/include/          # For AMPI + Charm++ interop
        cp ddt.h $(CDIR)/include/               # For AMPI + Charm++ interop
index a317d6ca39a19ea95a592668b7f4ec43d6585e39..3e4cac62189c2de40bdea3fb3c3137cee74ff395 100755 (executable)
@@ -46,15 +46,7 @@ AMPICC_EXTRA_OPTS=''
 
 if test "$AMPICC_MODE" = 'ampi'
 then
-  charmarch=$(cat $CHARMBIN/../tmp/.gdir)
-
-  if test "$charmarch" = "mpi"
-  then
-    dirName=ampicc.inc.$$
-    mkdir $dirName
-    cp -n $CHARMBIN/../include/mpi.h $dirName/mpi.h   2>/dev/null
-    AMPICC_EXTRA_OPTS="$AMPICC_EXTRA_OPTS -I./$dirName"
-  fi
+  AMPICC_EXTRA_OPTS="$AMPICC_EXTRA_OPTS -I$CHARMBIN/../include/ampi"
 fi
 
 [ -n "$ROMIO_CONFIGURE" -a -z "$MPITEST" ] && STANDALONE='true'
@@ -66,15 +58,6 @@ fi
 $CHARMBIN/charmc -language $AMPICC_MODE -default-to-aout $AMPICC_EXTRA_OPTS $ARGS $ROMIO
 status=$?
 
-if test "$AMPICC_MODE" = 'ampi'
-then
-  if test "$charmarch" = "mpi"
-  then
-    /bin/rm -f $dirName/mpi.h 2>/dev/null
-    rmdir $dirName
-  fi
-fi
-
 # Copy ampirun, but do not overwrite it if it already exists.
 [ $status -eq 0 ] && cp -n $CHARMBIN/ampirun .
 
index c7e44329598d50b2a0f43fb1afd1b2c85bd5f086..879d3db8dd048475ed34f3537f5fb80c62c09814 100644 (file)
@@ -1,16 +1,15 @@
 CDIR=../../../..
-CHARMC=$(CDIR)/bin/charmc $(OPTS)
 AMPICC=$(CDIR)/bin/ampicxx $(OPTS)
 
 HEADERS=idxlc.h idxlf.h idxl.h idxl_layout.h idxl_comm.h
 HEADDEP=$(HEADERS)
-OBJS=idxl.o idxl_layout.o idxl_comm.o idxl_api.o 
+OBJS=idxl.o idxl_layout.o idxl_comm.o idxl_api.o
 DEST=$(CDIR)/lib/libidxl.a
 
 all: $(DEST)
 
 $(DEST): $(OBJS) $(COMPAT) headers
-       $(CHARMC) $(OBJS) $(COMPAT) -o $@
+       $(AMPICC) $(OBJS) $(COMPAT) -o $@
 
 headers: $(HEADERS)
        cp $(HEADERS) $(CDIR)/include/
@@ -20,13 +19,13 @@ idxl.o: idxl.C $(HEADDEP)
        $(AMPICC) -c idxl.C
 
 idxl_layout.o: idxl_layout.C $(HEADDEP)
-       $(CHARMC) -c idxl_layout.C
+       $(AMPICC) -c idxl_layout.C
 
 idxl_comm.o: idxl_comm.C $(HEADDEP)
-       $(CHARMC) -c idxl_comm.C
+       $(AMPICC) -c idxl_comm.C
 
 idxl_api.o: idxl_api.C $(HEADDEP)
-       $(CHARMC) -c idxl_api.C
+       $(AMPICC) -c idxl_api.C
 
 clean:
        rm -rf *.a *.def.h *.decl.h *.o SunWS_cache $(DEST) headers
index 822709302f20b7fcb789a218a766291b80db2664..f9ae6200762608440211aa5c0e424e10f638c569 100644 (file)
@@ -568,9 +568,7 @@ $(L)/libconv-machine.a: machine.o
        $(CHARMC) -o $@ $<
 
 machine.o: machine.C $(CVHEADERS)
-       @-test -f $(INC)/mpi.h && mv -f $(INC)/mpi.h $(INC)/mpi.BAK || true
        $(CHARMC) -I. -o $@ $<
-       @-test -f $(INC)/mpi.BAK && mv -f $(INC)/mpi.BAK $(INC)/mpi.h || true
 
 $(L)/libconv-util.a: $(LIBCONV_UTIL)
        $(CHARMC) -o $@ $(LIBCONV_UTIL)