RFR: 8336702: C2 compilation fails with "all memory state should have been processed" assert [v2]

[Roland Westrelin]

On Wed, 18 Sep 2024 12:05:57 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Roland Westrelin 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 three additional commits since the last revision:
>> 
>>  - Merge branch 'master' into JDK-8336702
>>  - test indentation
>>  - fix & test
>
> test/hotspot/jtreg/compiler/longcountedloops/TestSafePointWithEAState.java line 59:
> 
>> 57:   float n;
>> 58:   h(float n) { this.n = n; }
>> 59: }
> 
> Java indentation is supposed to be 4 spaces ;)
> Adding some explicit brackets would also be nice, but that is more subjective.

Right. Fixed in new commit.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21009#discussion_r1782784463