Integrated: 8351414: C2: MergeStores must happen after RangeCheck smearing
Emanuel Peter
epeter at openjdk.org
Tue Mar 11 07:13:05 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.
This pull request has now been integrated.
Changeset: 4cf63160
Author: Emanuel Peter <epeter at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/4cf63160ad575d49dbe70f128cd36aba22b8f2ff
Stats: 110 lines in 8 files changed: 106 ins; 0 del; 4 mod
8351414: C2: MergeStores must happen after RangeCheck smearing
Reviewed-by: chagedorn, qamai
-------------
PR: https://git.openjdk.org/jdk/pull/23944
More information about the hotspot-compiler-dev
mailing list