[OMPI devel] [2.0.1rc1] ppc64 atomics (still) broken w/ xlc-12.1

2016-08-27 Thread Paul Hargrove
I didn't get to test 2.0.1rc1 with xlc-12.1 until just now because I need a CRYPTOCard for access (== not fully automated like my other tests). It appears that the problem I reported in 2.0.0rc2 and thought to be as fixed by pr1140 was never /fu

[OMPI devel] [2.0.1rc1] type warnings from clang

2016-08-27 Thread Paul Hargrove
Building w/ clang-3.4.2 on Linux/x86-64: /scratch/phargrov/OMPI/openmpi-2.0.1rc1-linux-x86_64-clang/openmpi-2.0.1rc1/opal/mca/btl/openib/btl_openib_component.c:2158:21: warning: implicit conversion from enumeration type 'btl_openib_receive_queues_source_t' to different enumeration type 'mca_base_v

[OMPI devel] [2.0-latest] unreachable code

2016-08-27 Thread Paul Hargrove
Oracle Studio 12.5 C compiler on a recent v2.0 nightly tarball points out the following unreachable code: "/shared/OMPI/openmpi-2.0-latest-solaris11-x86-ib-ss12u5/openmpi-v2.0.0-227-g917d293/opal/mca/base/mca_base_component_repository.c", line 265: warning: statement not reached "/shared/OMPI/open

[OMPI devel] C89 support

2016-08-27 Thread C Bergström
I realize a number of changes have been made to make the codebase C99. As I'm setting up more testing platforms, I found that this caused Clang (and us) to be broken on SLES10. While I realize that platform is quite *old*, it is still used in production at more than one sight which we support. If t

Re: [OMPI devel] C89 support

2016-08-27 Thread Jeff Squyres (jsquyres)
On Aug 27, 2016, at 8:38 AM, C Bergström wrote: > > I realize a number of changes have been made to make the codebase C99. > As I'm setting up more testing platforms, I found that this caused > Clang (and us) to be broken on SLES10. While I realize that platform > is quite *old*, it is still used

Re: [OMPI devel] C89 support

2016-08-27 Thread Nathan Hjelm
Considering gcc more or less had full C99 support in 3.1 (2002) and SLES10 dates back to 2004 I find this surprising. Clangs goal from the beginning was full C99 support. Checking back it looks like llvm 1.0 (2003) had C99 support. What version of clang/llvm are you using? -Nathan > On Aug 27,

Re: [OMPI devel] C89 support

2016-08-27 Thread Gilles Gouaillardet
out of curiosity, does Open MPI fails in configure (because it does not detect a valid C99 compiler) ? or does it fail in make (because some C99 syntax is not supported, and that was not detected at configure time) ? Cheers, Gilles On Saturday, August 27, 2016, C Bergström wrote: > I realize a

Re: [OMPI devel] C89 support

2016-08-27 Thread cbergstrom
It's well documented that the version of glibc that goes with SLES10 is not c99. As well as that gcc's claimed c99 is not in fact conformant. Newer glibc fixed this but SLES10 is stuck. I can provide exact documentation links if necessary.  Clang and any real c99 compiler fails at link time. T

Re: [OMPI devel] C89 support

2016-08-27 Thread Jeff Squyres (jsquyres)
Can we see the patch? > On Aug 27, 2016, at 9:41 AM, cbergst...@pathscale.com wrote: > > It's well documented that the version of glibc that goes with SLES10 is not > c99. As well as that gcc's claimed c99 is not in fact conformant. Newer glibc > fixed this but SLES10 is stuck. I can provide ex

Re: [OMPI devel] C89 support

2016-08-27 Thread Nathan Hjelm
We do not depend on any C99 specific behavior out of libc that I know of. We depend on the types (stdint.h) and syntax (sub-object naming, variadic macros, etc). A little surprised there are any linking failures with Open MPI even with an ancient glibc. If the patch is simple please send it to

Re: [OMPI devel] [2.0.1rc1] ppc64 atomics (still) broken w/ xlc-12.1

2016-08-27 Thread George Bosilca
Paul, Sorry for the half-fix. I'll submit a patch and PRs to the releases asap. George. On Sat, Aug 27, 2016 at 4:14 AM, Paul Hargrove wrote: > I didn't get to test 2.0.1rc1 with xlc-12.1 until just now because I need > a CRYPTOCard for access (== not fully automated like my other tests). >

Re: [OMPI devel] [2.0.1rc1] ppc64 atomics (still) broken w/ xlc-12.1

2016-08-27 Thread George Bosilca
v2.x: https://github.com/open-mpi/ompi-release/pull/1344 master: https://github.com/open-mpi/ompi/commit/a6d515b Thanks, George. On Sat, Aug 27, 2016 at 12:45 PM, George Bosilca wrote: > Paul, > > Sorry for the half-fix. I'll submit a patch and PRs to the releases asap. > > George. > > > O