RFR: 8351414: C2: MergeStores must happen after RangeCheck smearing [v2]
Christian Hagedorn
chagedorn at openjdk.org
Mon Mar 10 15:12:53 UTC 2025
On Mon, 10 Mar 2025 15:00:48 GMT, Emanuel Peter <epeter at openjdk.org> wrote:
>> With [JDK-8348959](https://bugs.openjdk.org/browse/JDK-8348959) we see that there can be some issues when RangeCheck smearing happens in the same IGVN phase as MergeStores. It means that some RangeChecks are still around as we do MergeStores, and then we cannot merge as many stores as we would like. We should ensure that RangeCheck smearing happens during post-loop-opts, and then MergeStores happens in a separate dedicated IGVN round afterwards.
>
> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>
> For Christian, add comment
Marked as reviewed by chagedorn (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/23944#pullrequestreview-2671335086
More information about the hotspot-compiler-dev
mailing list