RFR: 8339790: Support Intel APX setzucc instruction [v4]

Jatin Bhateja jbhateja at openjdk.org
Tue Sep 17 08:49:24 UTC 2024


On Fri, 13 Sep 2024 20:37:27 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:

>> - Support APX variant of SETcc, which supports zero-upper semantics (full register writer). Sets the destination operand to 0 or 1 depending on the settings of the status flags (CF, SF, OF, ZF, and PF) in the EFLAGS register. The destination operand points to a byte register or a byte in memory. The
>> condition code suffix (cc) indicates the condition being tested for. Additionally, if ND = 1 and the destination is a GPR, then also set the upper 56 bits of the GPR to 0.
>> - This saves emitting an explicit MOVZX instruction after setCC.
>> - These new instructions are encoded using 4 byte Extended EVEX encoding.
>> 
>> Validation performed over stand alone test point using Intel SDE.
>> 
>> Best Regards,
>> Jatin
>
> Jatin Bhateja has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Review comments resolution.

src/hotspot/cpu/x86/assembler_x86.cpp line 16052:

> 16050: 
> 16051:   // Encoding Format : eevex_prefix | opcode_cc | modrm
> 16052:   int encode = vex_prefix_and_encode(dst->encoding(), 0, 0, VEX_SIMD_F2, /* MAP4 */VEX_OPCODE_0F_3C, &attributes);

Suggestion:

  int encode = vex_prefix_and_encode(0, 0, dst->encoding(), VEX_SIMD_F2, /* MAP4 */VEX_OPCODE_0F_3C, &attributes);

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/20920#discussion_r1762668007


More information about the hotspot-dev mailing list