RFR: 8370405: C2: mismatched store from MergeStores wrongly scalarized in allocation elimination

Emanuel Peter epeter at openjdk.org
Wed Oct 29 12:48:15 UTC 2025


On Wed, 29 Oct 2025 10:27:26 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> src/hotspot/share/runtime/deoptimization.cpp line 1393:
>> 
>>> 1391:             tty->print_cr("Deopt rematerialization found [int, long] in a int/flat array.");
>>> 1392:             sv->print_fields_on(tty);
>>> 1393:             assert(false, "never hit this case in testing, seems to be a strange case");
>> 
>> Looks like this code came from https://openjdk.org/jeps/243, so it's worth double-checking with the Graal team if it's still needed.
>
> Right. I though I would just put an assert here, and if none of our testing fails, we can eventually remove it.

@dougxc It seems in all our testing this assert does not fail. Do you think it may impact Graal?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27997#discussion_r2472885290


More information about the hotspot-compiler-dev mailing list