On Fri, 6 Nov 2026 05:23:53 GMT, Vladimir Ivanov <[email protected]> wrote:

>> Jatin Bhateja has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Review comments resolution
>
> src/hotspot/cpu/x86/x86.ad line 21236:
> 
>> 21234:   match(Set dst ( RShiftVB src shift));
>> 21235:   match(Set dst (URShiftVB src shift));
>> 21236:   effect(TEMP dst, TEMP rtmp, TEMP rbase);
> 
> It still kills flags in debug build, doesn't it?

Hi @iwanowww , preserving flags so the rule does not need KILL cr.
Let me know if there are other comments.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/33113#discussion_r4489903147

Reply via email to