[lworld] RFR: 8372113: [lworld] Fix various issues with TypeAryPtr and TypeAryKlassPtr [v7]
Quan Anh Mai
qamai at openjdk.org
Mon Dec 8 14:25:26 UTC 2025
On Mon, 8 Dec 2025 10:24:42 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:
>> Quan Anh Mai has updated the pull request incrementally with one additional commit since the last revision:
>>
>> reviews
>
> src/hotspot/share/opto/memnode.cpp line 1355:
>
>> 1353: Node* init_value = ld_alloc->in(AllocateNode::InitValue);
>> 1354: if (init_value != nullptr) {
>> 1355: // TODO 8350865 Scalar replacement does not work well for flat arrays.
>
> Can you re-enable the tests in `TestArrays.java` marked with `// TODO 8350865 Scalar replacement does not work well for flat arrays` now? There are a few more places in the code marked with `// TODO 8350865 Scalar replacement ...` as well. We might just want to leave this to [JDK-8350865](https://bugs.openjdk.org/browse/JDK-8350865) though.
Simply re-enabling the tests results in failures of scenario 2 due to the returned oop being kept alive. I will need to look into this a little further.
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/1755#discussion_r2598828728
More information about the valhalla-dev
mailing list