RFR: JDK-8153837: AArch64: Handle special cases for MaxINode & MinINode [v4]

Thu Jan 12 13:45:55 UTC 2023

On Mon, 9 Jan 2023 05:52:45 GMT, Fei Yang <fyang at openjdk.org> wrote:

>> Dmitry Chuyko has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   392806Register, iRegIorL2I matched, m4 cleanup
>
> src/hotspot/cpu/aarch64/aarch64.ad line 13828:
> 
>> 13826: %{
>> 13827:   match(Set dst (MinI src imm));
>> 13828:   ins_cost(INSN_COST * 2);
> 
> I think the cost of the node before expansion should be the sum of the new nodes expanded. So the cost here and other places should be INSN_COST * 3 instead.

Agree, corrected in a new version.

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

PR: https://git.openjdk.org/jdk/pull/11570