On Fri, 26 Sep 2025 08:27:35 GMT, Marc Chevalier <[email protected]> wrote:

> Details of how we ended up there are in the JBS issue.
> 
> Let's take the mainline version and put it in valhalla... Except that the 
> mainline version doesn't need (and doesn't have) the `loadee` and `storee` 
> parameters, but have the value `IS_REPLACED` for those. It's unfortunate 
> since conflicts may happen again. Should be less bad.
> 
> It now passes the tests of 
> `testlibrary_tests/ir_framework/tests/TestPhaseIRMatching.java` again.

Looks good! Thanks for taking care of that test!

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

Marked as reviewed by chagedorn (Committer).

PR Review: 
https://git.openjdk.org/valhalla/pull/1639#pullrequestreview-3272461698

Reply via email to