kris/boringssl
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

chacha20_poly1305_x86_64.pl: Use |imulq| instead of |imul|.

Browse Source 
Perlasm requires the size suffix when targeting NASM and Yasm; without
it, the resulting .asm file has |imu| instead of |imul|.

Change-Id: Icb95b8c0b68cf4f93becdc1930dc217398f56bec
Reviewed-on: https://boringssl-review.googlesource.com/13381
Reviewed-by: Adam Langley <agl@google.com>
This commit is contained in:
Brian Smith 2017-01-26 22:54:51 -10:00 committed by Adam Langley
parent 3416d28a57
commit 357a9f23fe
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
crypto/cipher/asm/chacha20_poly1305_x86_64.pl
View File

@ -138,7 +138,7 @@ $code.="mov 0+$r_store, %rax
mov %rdx, $t1 mov %rdx, $t1
mov 0+$r_store, %rax mov 0+$r_store, %rax
mul $acc1 mul $acc1
imul $acc2, $t2 imulq $acc2, $t2
add %rax, $t1 add %rax, $t1
adc %rdx, $t2\n"; adc %rdx, $t2\n";
} }
@ -157,7 +157,7 @@ $code.="mov 8+$r_store, %rax
} }
sub poly_stage3 { sub poly_stage3 {
$code.="imul $acc2, $t3 $code.="imulq $acc2, $t3
add $acc0, $t2 add $acc0, $t2
adc %rdx, $t3\n"; adc %rdx, $t3\n";
} }
@ -1378,7 +1378,7 @@ $code.=<<___;
mov %rdx, $t2 mov %rdx, $t2
mulx $acc0, $t0, $t1 mulx $acc0, $t0, $t1
mulx $acc1, %rax, %rdx mulx $acc1, %rax, %rdx
imul $acc2, $t2 imulq $acc2, $t2
add %rax, $t1 add %rax, $t1
adc %rdx, $t2 adc %rdx, $t2
___ ___
@ -1392,7 +1392,7 @@ $code.=<<___;
mulx $acc1, $acc1, $t3 mulx $acc1, $acc1, $t3
adc $acc1, $t2 adc $acc1, $t2
adc \$0, $t3 adc \$0, $t3
imul $acc2, %rdx imulq $acc2, %rdx
___ ___
} }