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

Srinivas Vamsi Parasa sparasa at openjdk.org
Thu May 15 17:52:13 UTC 2025


On Wed, 14 May 2025 15:46:48 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:
>> 
>>   eimull revert fully to original version
>
> src/hotspot/cpu/x86/assembler_x86.cpp line 4068:
> 
>> 4066: void Assembler::enegl(Register dst, Register src, bool no_flags) {
>> 4067:   InstructionAttr attributes(AVX_128bit, /* vex_w */ false, /* legacy_mode */ false, /* no_mask_reg */ true, /* uses_vl */ false);
>> 4068:   int encode = evex_prefix_and_encode_ndd(dst->encoding(), src->encoding(), VEX_SIMD_NONE, VEX_OPCODE_0F_3C, &attributes, no_flags);
> 
> Since VEX_OPCODE_OF_3C is not a standard MAP4 encoding please add /* MAP4 */ after it in the places.

Please see the update code with the change `VEX_OPCODE_OF_3C /* MAP4 */`

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

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


More information about the hotspot-compiler-dev mailing list