Re: [OMPI devel] Master broken for ILP32

2016-05-09 Thread Hjelm, Nathan Thomas
11:12:16 AM To: Open MPI Developers Subject: Re: [OMPI devel] Master broken for ILP32 Regarding "distro": This was happening, for instance, on OpenBSD and NetBSD (32-bit kernels on 64-bit h/w) when testing your PR1643. However, it sounds like Nathan knows how/where to fix this

Re: [OMPI devel] Master broken for ILP32

2016-05-09 Thread Paul Hargrove
Regarding "distro": This was happening, for instance, on OpenBSD and NetBSD (32-bit kernels on 64-bit h/w) when testing your PR1643. However, it sounds like Nathan knows how/where to fix this. HOWEVER, that is not the only issue here. Why is master is picking the BUILTIN (__sync) atomics (as

Re: [OMPI devel] Master broken for ILP32

2016-05-09 Thread Hjelm, Nathan Thomas
Nevermind. Looks like there are two different macros for 64-bit and one is wrong in this case. Fix incoming. From: devel on behalf of Gilles Gouaillardet Sent: Monday, May 09, 2016 2:22:24 AM To: Open MPI Developers Subject: Re: [OMPI devel] Master

Re: [OMPI devel] Master broken for ILP32

2016-05-09 Thread Hjelm, Nathan Thomas
problem in vader but it looks like there are more places that need to be fixed. From: devel on behalf of Gilles Gouaillardet Sent: Monday, May 09, 2016 2:22:24 AM To: Open MPI Developers Subject: Re: [OMPI devel] Master broken for ILP32 Paul, on which

Re: [OMPI devel] Master broken for ILP32

2016-05-09 Thread Gilles Gouaillardet
Paul, on which distro are you running ? are you compiling on a 64 bit distro to generate a 32 bit library ? it seems we are currently only testing a atomic on a long (32 bits on a 32 bits arch) and then incorrectly assume it works also on 64 bits (!) Cheers, Gilles On 5/9/2016 3:59

[OMPI devel] Master broken for ILP32

2016-05-09 Thread Paul Hargrove
Perhaps this is already known. Several builds I've tried recently from the ompi (not ompi-release) repo are failing on 32-bit platforms with ../../../opal/.libs/libopen-pal.so: undefined reference to `__sync_add_and_fetch_8' This is impacting PRs that I am being asked to test (e.g. 1643).