[mpich-commits] [mpich] MPICH primary repository branch, master, updated. v3.0.4-231-g0388a85

mysql vizuser noreply at mpich.org
Sat May 18 23:49:29 CDT 2013


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "MPICH primary repository".

The branch, master has been updated
       via  0388a85b1a6dfe8772766c67e9793266452e6c2c (commit)
       via  9d909feee1bfa4337f1bceabc7d3f42e7e2cd3fb (commit)
       via  ccf7f70c3252d4e38933de00dd2f4bb49a389a61 (commit)
       via  dfad7fdfc307ca74184a133d2921b8bc6d7e5023 (commit)
      from  01e95cbfc0dde9fac3180addb7981f4d199a235c (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
http://git.mpich.org/mpich.git/commitdiff/0388a85b1a6dfe8772766c67e9793266452e6c2c

commit 0388a85b1a6dfe8772766c67e9793266452e6c2c
Author: Pavan Balaji <balaji at mcs.anl.gov>
Date:   Sat May 18 23:48:33 2013 -0500

    Warning squash.
    
    No reviewer.

diff --git a/src/pm/hydra/pm/pmiserv/pmip_cb.c b/src/pm/hydra/pm/pmiserv/pmip_cb.c
index d8681eb..e284028 100644
--- a/src/pm/hydra/pm/pmiserv/pmip_cb.c
+++ b/src/pm/hydra/pm/pmiserv/pmip_cb.c
@@ -462,7 +462,7 @@ static int local_to_global_id(int local_id)
 
 static HYD_status launch_procs(void)
 {
-    int i, j, arg, process_id;
+    int i, j, process_id;
     int using_pmi_port = 0;
     char *str, *envstr, *list, *pmi_port;
     struct HYD_string_stash stash;
@@ -685,7 +685,7 @@ static HYD_status launch_procs(void)
             }
 
             HYD_STRING_STASH_INIT(stash);
-            for (j = 0, arg = 0; exec->exec[j]; j++)
+            for (j = 0; exec->exec[j]; j++)
                 HYD_STRING_STASH(stash, HYDU_strdup(exec->exec[j]), status);
 
             status = HYDU_create_process(stash.strlist, force_env,

http://git.mpich.org/mpich.git/commitdiff/9d909feee1bfa4337f1bceabc7d3f42e7e2cd3fb

commit 9d909feee1bfa4337f1bceabc7d3f42e7e2cd3fb
Author: Pavan Balaji <balaji at mcs.anl.gov>
Date:   Sat May 18 23:27:47 2013 -0500

    Remove accidental debug statements.
    
    No reviewer.

diff --git a/confdb/aclocal_runlog.m4 b/confdb/aclocal_runlog.m4
index a538975..6446af8 100644
--- a/confdb/aclocal_runlog.m4
+++ b/confdb/aclocal_runlog.m4
@@ -183,8 +183,6 @@ dnl Save a copy of ac_link on a stack
 dnl which is safe through nested invocations of this macro.
 PAC_VAR_PUSHVAL([ac_link])
 dnl Modify ac_link based on the unmodified ac_link.
-echo "ac_link: $ac_link"
-echo "pac_FirstSavedValueOf_ac_link: $pac_FirstSavedValueOf_ac_link"
 ac_link="`echo $pac_FirstSavedValueOf_ac_link | sed -e 's|>.*$|> $1 2>\&1|g'`"
 dnl
 AC_LINK_IFELSE([$2],[

http://git.mpich.org/mpich.git/commitdiff/ccf7f70c3252d4e38933de00dd2f4bb49a389a61

commit ccf7f70c3252d4e38933de00dd2f4bb49a389a61
Author: Pavan Balaji <balaji at mcs.anl.gov>
Date:   Sat May 18 22:38:40 2013 -0500

    Warning squash for clang.
    
    For the VA_ARGS test, clang was accepting it in C89 mode, but throwing
    warnings.  This causes configure to enable it, but the build to throw
    many warnings.  This patch will disable VA_ARGS support unless the
    compiler can accept it without warnings.
    
    No reviewer.

diff --git a/confdb/aclocal_cc.m4 b/confdb/aclocal_cc.m4
index 10964e1..47d58da 100644
--- a/confdb/aclocal_cc.m4
+++ b/confdb/aclocal_cc.m4
@@ -1625,19 +1625,29 @@ AC_DEFUN([PAC_STRUCT_ALIGNMENT],[
 	   pac_cv_struct_alignment="eight"
 	fi
 ])
-dnl
+
 dnl PAC_C_MACRO_VA_ARGS
 dnl
 dnl will AC_DEFINE([HAVE_MACRO_VA_ARGS]) if the compiler supports C99 variable
 dnl length argument lists in macros (#define foo(...) bar(__VA_ARGS__))
 AC_DEFUN([PAC_C_MACRO_VA_ARGS],[
     AC_MSG_CHECKING([for variable argument list macro functionality])
-    AC_LINK_IFELSE([AC_LANG_PROGRAM([
+
+    # check if the program links correctly
+    rm -f pac_test.log
+    PAC_LINK_IFELSE_LOG([pac_test.log],[AC_LANG_PROGRAM([
         #include <stdio.h>
         #define conftest_va_arg_macro(...) printf(__VA_ARGS__)
     ],
     [conftest_va_arg_macro("a test %d", 3);])],
-    [AC_DEFINE([HAVE_MACRO_VA_ARGS],[1],[Define if C99-style variable argument list macro functionality])
-     AC_MSG_RESULT([yes])],
-    [AC_MSG_RESULT([no])])
+    prog_links=yes,prog_links=no)
+
+    # If the program linked OK, make sure there were no warnings
+    if test "$prog_links" = "yes" -a "`cat pac_test.log`" = "" ; then
+       AC_DEFINE([HAVE_MACRO_VA_ARGS],[1],[Define if C99-style variable argument list macro functionality])
+       AC_MSG_RESULT([yes])
+    else
+       AC_MSG_RESULT([no])
+    fi
+    rm -f pac_test.log
 ])dnl

http://git.mpich.org/mpich.git/commitdiff/dfad7fdfc307ca74184a133d2921b8bc6d7e5023

commit dfad7fdfc307ca74184a133d2921b8bc6d7e5023
Author: Pavan Balaji <balaji at mcs.anl.gov>
Date:   Sat May 18 22:55:56 2013 -0500

    Missed check for inline in mpl.
    
    Added a check for inline in the MPL configure.  Also made C89 headers
    available by default.  We should not be checking for them.
    
    No reviewer.

diff --git a/src/mpl/configure.ac b/src/mpl/configure.ac
index bc0322c..0ea3ab1 100644
--- a/src/mpl/configure.ac
+++ b/src/mpl/configure.ac
@@ -47,6 +47,7 @@ m4_ifdef([AM_SILENT_RULES], [AM_SILENT_RULES([yes])])
 
 AC_C_CONST
 AC_C_RESTRICT
+AC_C_INLINE
 
 PAC_C_MACRO_VA_ARGS
 
diff --git a/src/mpl/include/mpl.h b/src/mpl/include/mpl.h
index dee4743..3a33f0a 100644
--- a/src/mpl/include/mpl.h
+++ b/src/mpl/include/mpl.h
@@ -9,6 +9,12 @@
 
 #include "mplconfig.h"
 
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
+#include <stdarg.h>
+#include <stdint.h>
+
 #if defined _mpl_restrict
 #define mpl_restrict _mpl_restrict
 #else
@@ -21,21 +27,11 @@
 #define mpl_const const
 #endif /* _mpl_const */
 
-#if defined MPL_HAVE_STDIO_H
-#include <stdio.h>
-#endif /* MPL_HAVE_STDIO_H */
-
-#if defined MPL_HAVE_STDLIB_H
-#include <stdlib.h>
-#endif /* MPL_HAVE_STDLIB_H */
-
-#if defined MPL_HAVE_STRING_H
-#include <string.h>
-#endif /* MPL_HAVE_STRING_H */
-
-#if defined MPL_HAVE_STDARG_H
-#include <stdarg.h>
-#endif /* MPL_HAVE_STDARG_H */
+#if defined _mpl_inline
+#define mpl_inline _mpl_inline
+#else
+#define mpl_inline inline
+#endif /* _mpl_inline */
 
 #if defined MPL_HAVE_CTYPE_H
 #include <ctype.h>
@@ -45,10 +41,6 @@
 #include <inttypes.h>
 #endif /* MPL_HAVE_INTTYPES_H */
 
-#if defined(MPL_HAVE_STDINT_H)
-#include <stdint.h>
-#endif /* MPL_HAVE_STDINT_H */
-
 #if !defined ATTRIBUTE
 #  if defined MPL_HAVE_GCC_ATTRIBUTE
 #    define ATTRIBUTE(a_) __attribute__(a_)
diff --git a/src/mpl/include/mpl_valgrind.h b/src/mpl/include/mpl_valgrind.h
index 84fe057..4d36a9e 100644
--- a/src/mpl/include/mpl_valgrind.h
+++ b/src/mpl/include/mpl_valgrind.h
@@ -158,7 +158,7 @@
 #    define MPL_VG_PRINTF_BACKTRACE(...)              do {} while (0)
 #  else
 #    define MPL_VG_PRINTF_BACKTRACE MPL_VG_printf_do_nothing_func
-static inline void MPL_VG_printf_do_nothing_func(char *fmt, ...)
+static mpl_inline void MPL_VG_printf_do_nothing_func(char *fmt, ...)
 {
     /* do nothing */
 }

-----------------------------------------------------------------------

Summary of changes:
 confdb/aclocal_cc.m4              |   20 +++++++++++++++-----
 confdb/aclocal_runlog.m4          |    2 --
 src/mpl/configure.ac              |    1 +
 src/mpl/include/mpl.h             |   30 +++++++++++-------------------
 src/mpl/include/mpl_valgrind.h    |    2 +-
 src/pm/hydra/pm/pmiserv/pmip_cb.c |    4 ++--
 6 files changed, 30 insertions(+), 29 deletions(-)


hooks/post-receive
-- 
MPICH primary repository


More information about the commits mailing list