RFR: 8351994: Enable Extended EVEX to REX2/REX demotion when src and dst are the same [v27]

Srinivas Vamsi Parasa sparasa at openjdk.org
Tue May 20 23:17:57 UTC 2025


On Sat, 17 May 2025 11:54:29 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:

>> Srinivas Vamsi Parasa has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update x86-asmtest.py to enable demotion by default and make test generation optional
>
> src/hotspot/cpu/x86/assembler_x86.cpp line 1657:
> 
>> 1655: void Assembler::eandl(Register dst, Register src1, Address src2, bool no_flags) {
>> 1656:   InstructionMark im(this);
>> 1657:   evex_prefix_int8_operand(dst, src1, src2, VEX_SIMD_NONE, VEX_OPCODE_0F_3C /* MAP4 */, EVEX_32bit, 0x23, no_flags);
> 
> Nomenclature of routine is not very clear here, what do you mean by int8 operand, this is operating over 32bit word.

Please see the name changed to `evex_opcode_prefix_and_encode` in the updated code.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24431#discussion_r2099012422


More information about the hotspot-compiler-dev mailing list