Ok, there was a need for OPENSSL_cleanse() instead of bzero() to prevent

supposedly smart compilers from optimizing memory cleanups away. Understood.

Ok, in case of an hypothetically super smart compiler, OPENSSL_cleanse() had
to be convoluted enough for the compiler not to recognize that this was
actually bzero() in disguise. Understood.

But then why there had been optimized assembler versions of OPENSSL_cleanse()
is beyond me. Did someone not trust the C obfuscation?

1 files changed, 0 insertions, 83 deletions

diff --git a/src/lib/libcrypto/sparccpuid.S b/src/lib/libcrypto/sparccpuid.S
index 0cc247e489..d8b44af2f0 100644
--- a/src/lib/libcrypto/sparccpuid.S
+++ b/src/lib/libcrypto/sparccpuid.S
@@ -314,89 +314,6 @@ _sparcv9_fmadd_probe:
 .type   _sparcv9_fmadd_probe,#function
 .size   _sparcv9_fmadd_probe,.-_sparcv9_fmadd_probe
 
-.global OPENSSL_cleanse
-.align  32
-OPENSSL_cleanse:
-        cmp     %o1,14
-        nop
-#ifdef ABI64
-        bgu     %xcc,.Lot
-#else
-        bgu     .Lot
-#endif
-        cmp     %o1,0
-        bne     .Little
-        nop
-        retl
-        nop
-
-.Little:
-        stb     %g0,[%o0]
-        subcc   %o1,1,%o1
-        bnz     .Little
-        add     %o0,1,%o0
-        retl
-        nop
-.align  32
-.Lot:
-#ifndef ABI64
-        subcc   %g0,1,%g1
-        ! see above for explanation
-        .word   0x83408000      !rd     %ccr,%g1
-        cmp     %g1,0x99
-        bne     .v8lot
-        nop
-#endif
-
-.v9lot: andcc   %o0,7,%g0
-        bz      .v9aligned
-        nop
-        stb     %g0,[%o0]
-        sub     %o1,1,%o1
-        ba      .v9lot
-        add     %o0,1,%o0
-.align  16,0x01000000
-.v9aligned:
-        .word   0xc0720000      !stx    %g0,[%o0]
-        sub     %o1,8,%o1
-        andcc   %o1,-8,%g0
-#ifdef ABI64
-        .word   0x126ffffd      !bnz    %xcc,.v9aligned
-#else
-        .word   0x124ffffd      !bnz    %icc,.v9aligned
-#endif
-        add     %o0,8,%o0
-
-        cmp     %o1,0
-        bne     .Little
-        nop
-        retl
-        nop
-#ifndef ABI64
-.v8lot: andcc   %o0,3,%g0
-        bz      .v8aligned
-        nop
-        stb     %g0,[%o0]
-        sub     %o1,1,%o1
-        ba      .v8lot
-        add     %o0,1,%o0
-        nop
-.v8aligned:
-        st      %g0,[%o0]
-        sub     %o1,4,%o1
-        andcc   %o1,-4,%g0
-        bnz     .v8aligned
-        add     %o0,4,%o0
-
-        cmp     %o1,0
-        bne     .Little
-        nop
-        retl
-        nop
-#endif
-.type   OPENSSL_cleanse,#function
-.size   OPENSSL_cleanse,.-OPENSSL_cleanse
-
 .section        ".init",#alloc,#execinstr
         call    OPENSSL_cpuid_setup
         nop