If I understand correctly the assembly this change is not related to clobber, but to the input register. Moreover, I don't think the patch is correct. More to come...
george. On Feb 23, 2011, at 16:54 , brbar...@osl.iu.edu wrote: > Author: brbarret > Date: 2011-02-23 16:54:07 EST (Wed, 23 Feb 2011) > New Revision: 24449 > URL: https://svn.open-mpi.org/trac/ompi/changeset/24449 > > Log: > Fix register clobber list for x86 assembly. Thanks to Jay Fenlason for the > patch. > > Text files modified: > trunk/opal/config/opal_config_asm.m4 | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > Modified: trunk/opal/config/opal_config_asm.m4 > ============================================================================== > --- trunk/opal/config/opal_config_asm.m4 (original) > +++ trunk/opal/config/opal_config_asm.m4 2011-02-23 16:54:07 EST (Wed, > 23 Feb 2011) > @@ -867,7 +867,7 @@ > ompi_cv_asm_arch="AMD64" > fi > OPAL_ASM_SUPPORT_64BIT=1 > - OMPI_GCC_INLINE_ASSIGN='"xaddl %1,%0" : "=m"(ret), "+r"(negone)' > + OMPI_GCC_INLINE_ASSIGN='"xaddl %1,%0" : "=m"(ret), "+r"(negone) > : "m"(ret)' > ;; > > ia64-*) > _______________________________________________ > svn-full mailing list > svn-f...@open-mpi.org > http://www.open-mpi.org/mailman/listinfo.cgi/svn-full "I disapprove of what you say, but I will defend to the death your right to say it" -- Evelyn Beatrice Hall
