RFR: 8336860: x86: Change integer src operand for CMoveL of 0 and 1 to long [v2]
Jasmine Karthikeyan
jkarthikeyan at openjdk.org
Thu Jul 25 14:39:34 UTC 2024
On Thu, 25 Jul 2024 14:28:35 GMT, Emanuel Peter <epeter at openjdk.org> wrote:
>> @eme64 or @TobiHartmann might take a look too, I guess?
>
> @liach @jaskarth I'll run some testing.
>
> Can you point me to the "base case" you mention in your PR description?
Thanks for running testing @eme64!
The base case I mention is this:
https://github.com/openjdk/jdk/blob/34ee06f51122e8afb875cc3b75f513912272fd9b/src/hotspot/cpu/x86/x86_64.ad#L6405-L6415
Instead of matching the flow-free `SETcc` idiom, we fall back to using the default `cmovq` behavior.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/20275#issuecomment-2250493196
More information about the hotspot-compiler-dev
mailing list