• Ng Zhi An's avatar
    Fix assembler for sqrtpd · d05d335e
    Ng Zhi An authored
    The assembly of sqrtpd when using Sqrtpd macro was wrong, since
    Sqrtpd(xmm1, xmm1) will incorrect generated vsqrtpd(xmm1, xmm1, xmm1),
    which is nonsensical, since vsqrtpd only takes two operands. The
    expected instruction should be vsqrtpd(xmm1, xmm0, xmm1) in terms of the
    encoding, which is vsqrtpd(xmm1, xmm1).
    
    So, move sqrtpd and cvtps2dq out into their own macro list, because
    they have two operands in their AVX form, unlike the rest of the
    instructions in SSE2_INSTRUCTION_LIST.
    
    Also updated disasm and tests to use this new list.
    
    Fixed: v8:10170
    Change-Id: Ia9343c9a3ae64596bbc876744556e1dcea2a443b
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2032195Reviewed-by: 's avatarDeepti Gandluri <gdeepti@chromium.org>
    Commit-Queue: Zhi An Ng <zhin@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#66088}
    d05d335e
disasm-x64.cc 101 KB