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
