[jdk11u-dev] RFR: 8258894: C2: Forbid GCM to move stores into loops [v2]

[Goetz Lindenmaier]

On Fri, 21 Oct 2022 07:00:15 GMT, Goetz Lindenmaier <goetz at openjdk.org> wrote:

>> I backport this for parity with 11.0.18-oracle.
>> 
>> Two copyrights had to be resolved.
>> 
>> I had to resolve two hunks in generateOptoStub.cpp. Simple.
>> There is #ifdef SPARC code that is left out by the
>> change. 
>> I edited it canonically (quite obvious), but did not test it.
>
> Goetz Lindenmaier has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains one additional commit since the last revision:
> 
>   Backport f353fcf2569f5435d1410a83265c425d6385fc3f

C2 asserts that an incI_mem node does not have a region input while it is a memory node.
IIrc, in jdk17, it is an invariant that all memory nodes have a region input. In 11 it is not (yet).
Thus, we can either remove the assertion or backport the changes needed to make this invariant hold.

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

PR: https://git.openjdk.org/jdk11u-dev/pull/1401