On Fri, 28 Jan 2022 07:43:45 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:

>> This test is failing in the loom repo when using -Xcomp. The reason is 
>> because loom introduced doing a full GC in the codecache sweeper, which 
>> causes the large array that the test allocates to be GC'd before the heap 
>> dump is done. The fix is to make the large array static rather than a local 
>> variable.
>> 
>> I'm choosing to fix this in the jdk repo rather than the loom repo since it 
>> is a latent bug that theoretically could occur even without the loom 
>> changes, and also to help reduce the amount of changes to be reviewed when 
>> loom is integrated into jdk.
>
> Chris Plummer has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Use Reference.reachabilityFence() instead of static field.

Marked as reviewed by alanb (Reviewer).

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

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

Reply via email to