I created a new ticket for this issue. It is
https://svn.open-mpi.org/trac/ompi/ticket/2560
Please go there for future details on this issue. But to quickly
summarize, I would like to
go with George's first recommendation that uses the "addl" command as
the "xaddl"
creates a second set of p
Hello Paul,
thanks for testing so thoroughly.
On Wednesday 25 August 2010 02:40:22 Paul H. Hargrove wrote:
> In addition to the atomic.h and Anachronism warnings seen w/ 1.4.3rc1
> (http://www.open-mpi.org/community/lists/devel/2010/08/8322.php), I find
> some "new" warnings in 1.5rc5.
>
> $ unam
Right, that was a pretty intense discussion. However, I don't think we talked
about replacing the = by a +. The difference is that = means write and + means
read&write. Here is the assembly output from gcc -O3:
with output "=m" (*v) | with output "+m" (*v)
and input "ir"
With respect to the warnings with atomic.h, we have been down this road
before.
Here is the ticket with the background information.
https://svn.open-mpi.org/trac/ompi/ticket/1500
Eventually, we decided to just live with the warnings. However, I will
take a look at George's two suggestions.
Following George's advice, I made the following change (NOT yet
conditional on Sun compiler), and can confirm that it eliminates the
warnings in atomic.h and "make check" still passes.
-Paul
--- openmpi-1.5rc5/opal/include/opal/sys/ia32/atomic.h.orig Tue Aug
24 18:40:55 2010
+++ openmpi-1
On Aug 24, 2010, at 20:40 , Paul H. Hargrove wrote:
> "../../../openmpi-1.5rc5/opal/include/opal/sys/ia32/atomic.h", line 170:
> warning: impossible constraint for "%1" asm operand
__asm__ __volatile__(
SMPLOCK "addl %1,%0"
:"=m" (*v)
In addition to the atomic.h and Anachronism warnings seen w/ 1.4.3rc1
(http://www.open-mpi.org/community/lists/devel/2010/08/8322.php), I find
some "new" warnings in 1.5rc5.
$ uname -a
SunOS osol-x64 5.11 snv_111b i86pc i386 i86pc
$ cc -V
cc: Sun C 5.10 SunOS_i386 2009/06/03
usage: cc [ option
