RFR: 8307513: C2: intrinsify Math.max(long,long) and Math.min(long,long)

Emanuel Peter epeter at openjdk.org
Thu Jan 9 12:21:44 UTC 2025


On Thu, 9 Jan 2025 12:03:55 GMT, Galder Zamarreño <galder at openjdk.org> wrote:

>> @galderz Nice, thanks for the updates. I gave the patch a quick scan and I think it looks really good. Just ping me again when you are done with your aarch64 investigations, and you think I should review again :)
>
> @eme64 aarch64 work for this is now complete. I tweaked the `applyIf` condition `MinMaxRed_Long` to make sure `MaxVectorSize` is 32 or higher. I verified this on both Graviton 3 (256 bit register, `MaxVectorSize=32`) and an AVX-512 intel (512 bit register, `MaxVectorSize=64`)

@galderz So you want me to review again?

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

PR Comment: https://git.openjdk.org/jdk/pull/20098#issuecomment-2580018680


More information about the graal-dev mailing list