math/mpx-mul4-*-sse2.S: Remove an unhelpful comment.
authorMark Wooding <mdw@distorted.org.uk>
Thu, 7 Nov 2019 01:43:46 +0000 (01:43 +0000)
committerMark Wooding <mdw@distorted.org.uk>
Sat, 9 May 2020 19:57:33 +0000 (20:57 +0100)
It's not actually wrong, but it's misleading because we don't actually
care that the flags are preserved at this point, because the next
instruction clobbers them anyway.  I think this was cut-and-paste
lossage from the earlier code which relies on `mov' preserving the carry
flag.

math/mpx-mul4-amd64-sse2.S
math/mpx-mul4-x86-sse2.S

index 0f2dbd3..a37aba6 100644 (file)
@@ -1262,7 +1262,7 @@ FUNC(mpxmont_redc4_amd64_sse2)
 
        // Deal with the tail end.
 7:     add     [rdi], C
-       mov     C, 0                    // preserves flags
+       mov     C, 0
        add     rdi, 4
        adc     C, 0
        cmp     rdi, DVL
index b1072ff..9e1d478 100644 (file)
@@ -1064,7 +1064,7 @@ FUNC(mpxmont_redc4_x86_sse2)
 
        // Deal with the tail end.
 7:     add     [edi], eax
-       mov     eax, 0                  // preserves flags
+       mov     eax, 0
        add     edi, 4
        adc     eax, 0
        cmp     edi, edx