On Wed, 6 May 2026 08:23:31 GMT, Aleksey Shipilev <[email protected]> wrote:
>> Andrew Haley has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 111 commits: >> >> - Merge branch 'master' into JDK-8134940 >> - Review comments >> - Review comments >> - Add FrameMap::last_fpu_reg() >> - Review comments >> - Review comments >> - Review comments >> - Merge branch 'JDK-8134940' of https://github.com/theRealAph/jdk into >> JDK-8134940 >> - More >> - Merge branch 'JDK-8134940' of https://github.com/theRealAph/jdk into >> JDK-8134940 >> - ... and 101 more: https://git.openjdk.org/jdk/compare/3e1d0b8e...4610777b > > src/hotspot/share/c1/c1_LIR.cpp line 904: > >> 902: do_input(opr->_step); do_temp(opr->_step); >> 903: if (opr->_result->is_valid()) { >> 904: do_temp(opr->_result); do_output(opr->_result); > > Analysis flags that it is unusual to have an operand declared both `temp` and > `output`. Not sure if C1 treats this well. Indeed, but it works fine. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/28541#discussion_r3335889805
