This patch removes what I think is extraneous code in the atomic compare-and-swap function for sparcs. Since it's assembly, I'd like to get a second opinion before I commit it.
Thanks, --Brian
Index: atomic/solaris_sparc/apr_atomic_sparc.s
===================================================================
RCS file: /home/cvs/apr/atomic/solaris_sparc/apr_atomic_sparc.s,v
retrieving revision 1.4
diff -u -r1.4 apr_atomic_sparc.s
--- atomic/solaris_sparc/apr_atomic_sparc.s 13 Mar 2002 20:39:08 -0000 1.4
+++ atomic/solaris_sparc/apr_atomic_sparc.s 20 Apr 2002 09:20:17 -0000
@@ -110,12 +110,6 @@
ENTRY(apr_atomic_casptr_sparc)
cas [%o0], %o2, %o1
- cmp %o1, %o2 ! if o1 == o2 values weren't swapped
- bne,a _apr_atomic_cas_ne
- mov %o2, %o0
- retl
- mov %o2, %o0
-_apr_atomic_cas_ne:
retl
mov %o1, %o0
