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 20:48:59 UTC 2025
On Tue, 20 May 2025 16:28:26 GMT, Srinivas Vamsi Parasa <sparasa at openjdk.org> wrote:
>> src/hotspot/cpu/x86/assembler_x86.cpp line 13002:
>>
>>> 13000: }
>>> 13001:
>>> 13002: int Assembler::evex_prefix_and_encode_ndd(int dst_enc, int nds_enc, VexSimdPrefix pre, VexOpcode opc,
>>
>> Suggestion:
>>
>> int Assembler::emit_eevex_prefix_or_demote_ndd(int dst_enc, int nds_enc, VexSimdPrefix pre, VexOpcode opc,
>>
>>
>> Nameing suggetion
>
> There are different overloaded functions with the name evex_prefix_and_encode_ndd. Just want to confirm that this renaming suggestion must be applied to all of them with the same name (which also do demotion), right?
This suggestion was incorporated.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24431#discussion_r2098846647
More information about the hotspot-compiler-dev
mailing list