RFR: 8351414: C2: MergeStores must happen after RangeCheck smearing
Quan Anh Mai
qamai at openjdk.org
Mon Mar 10 13:04:02 UTC 2025
On Fri, 7 Mar 2025 15:07:37 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.
Thanks for the elaboration. LGTM.
-------------
Marked as reviewed by qamai (Committer).
PR Review: https://git.openjdk.org/jdk/pull/23944#pullrequestreview-2670878613
More information about the hotspot-compiler-dev
mailing list