Skip to content

Commit

Permalink
Another movd/movq inconsistency.
Browse files Browse the repository at this point in the history
 - Visual Studio <= 2015 does not support AVX-512 either way;
 - Visual Studio 2017 does not tolerate vmovd with 64-bit operands;
 - Visual Studio 2019 does not care.
  • Loading branch information
sneves authored and oconnor663-zoom committed May 20, 2021
1 parent 0cceb2e commit 49cf7a5
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions c/blake3_avx512_x86-64_windows_msvc.asm
Original file line number Diff line number Diff line change
Expand Up @@ -2421,8 +2421,8 @@ _blake3_compress_in_place_avx512 PROC
movzx r8d, r8b
shl rax, 32
add r8, rax
vmovd xmm3, r9
vmovd xmm4, r8
vmovq xmm3, r9
vmovq xmm4, r8
vpunpcklqdq xmm3, xmm3, xmm4
vmovaps xmm2, xmmword ptr [BLAKE3_IV]
vmovups xmm8, xmmword ptr [rdx]
Expand Down Expand Up @@ -2516,8 +2516,8 @@ _blake3_compress_xof_avx512 PROC
mov r10, qword ptr [rsp+78H]
shl rax, 32
add r8, rax
vmovd xmm3, r9
vmovd xmm4, r8
vmovq xmm3, r9
vmovq xmm4, r8
vpunpcklqdq xmm3, xmm3, xmm4
vmovaps xmm2, xmmword ptr [BLAKE3_IV]
vmovups xmm8, xmmword ptr [rdx]
Expand Down

0 comments on commit 49cf7a5

Please sign in to comment.