RFR: 8263781: C2: Cannot hoist independent load above arraycopy [v2]

[Richard Reingruber]

On Fri, 19 Mar 2021 16:33:47 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Richard Reingruber has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Improve comment.
>
> src/hotspot/share/opto/memnode.cpp line 536:
> 
>> 534: // (b) can_see_stored_value=false and ac could have set the value for this load or if
>> 535: // (c) can_see_stored_value=false and ac cannot have set the value for this load.
>> 536: // In (c) also change the parameter mem to the memory input of ac.
> 
> I suggest to modify last line to something like to make it clear why we do that:
> `In (c) case change the parameter mem to the memory input of ac to skip it when searching stored value.`

Sure, I've done that now.

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

PR: https://git.openjdk.java.net/jdk/pull/3070