RFR: 8324517: C2: crash in compiled code because of dependency on removed range check CastIIs [v2]

Thu Mar 28 14:14:57 UTC 2024

On Thu, 28 Mar 2024 11:56:53 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> test/hotspot/jtreg/compiler/rangechecks/TestArrayAccessAboveRCAfterRCCastIIEliminated.java line 39:
>> 
>>> 37:  *                   -XX:+StressIGVN -XX:StressSeed=94546681 TestArrayAccessAboveRCAfterRCCastIIEliminated
>>> 38:  *
>>> 39:  */
>> 
>> Can you please add a "vanilla" run like this:
>> `@run main TestArrayAccessAboveRCAfterRCCastIIEliminated`
>> That would allow us to run the test with any flag combination from the outside.
>
> And maybe one that only does `-XX:CompileCommand=dontinline,TestArrayAccessAboveRCAfterRCCastIIEliminated::notInlined`, since that seems important for reproducing the interesting patterns.

Done in new commit.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18377#discussion_r1543051958