RFR: 8203925: tools/javac/importscope/T8193717.java ran out of java heap [v2]
Vicente Romero
vromero at openjdk.java.net
Thu Apr 15 09:22:46 UTC 2021
On Thu, 15 Apr 2021 01:42:06 GMT, Guoxiong Li <github.com+13688759+lgxbslgx at openjdk.org> wrote:
>> Hi all,
>>
>> Because [JDK-8225054-PATCH](https://github.com/openjdk/jdk/commit/827e5e32264666639d36990edd5e7d0b7e7c78a9) modified `T8193717.java`, changing `private static final int CLASSES = 500000;` to `private static final int CLASSES = 50000;`(500K -> 50K). The error, `out of java heap`, no longer occurs now. And the pre-submit tests of this patch passed. I think `tools/javac/importscope/T8193717.java` could be removed from `ProblemList.txt`.
>>
>> Thank you for taking the time to review.
>>
>> Best Regards.
>
> Guoxiong Li 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 two additional commits since the last revision:
>
> - Merge branch 'master' into JDK-8203925
> - 8203925: tools/javac/importscope/T8193717.java ran out of java heap
Marked as reviewed by vromero (Reviewer).
-------------
PR: https://git.openjdk.java.net/jdk/pull/1998
More information about the compiler-dev
mailing list