RFR: 8332340: Add JavacBench as a test case for CDS [v3]
Matias Saavedra Silva
matsaave at openjdk.org
Tue May 21 21:12:10 UTC 2024
On Mon, 20 May 2024 17:24:22 GMT, Ioi Lam <iklam at openjdk.org> wrote:
>> JavacBench is a test program that compiles 90 Java source files. It uses a fair amount of invokedynamic callsites, so it's good for testing CDS support for indy and lambda expressions.
>>
>> This test was first integrated into the [leyden](https://github.com/openjdk/leyden/tree/premain) repo. Hence some of the files have copyrights in 2023.
>
> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
>
> @calvinccheung comments
LGTM, and thanks for the new utilities! That should make writing CDS tests a lot easier. I have a few style considerations but you can take them or leave them.
test/lib/jdk/test/lib/cds/CDSAppTester.java line 98:
> 96: }
> 97:
> 98: private final String name;
Could these fields and the constructor be moved to the top of the class?
test/lib/jdk/test/lib/cds/CDSAppTester.java line 147:
> 145: }
> 146:
> 147: private OutputAnalyzer dumpStaticArchive() throws Exception {
The code from 156 to 162 is repeated 3 times here, is it worth making another function for this?
-------------
Marked as reviewed by matsaave (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/19256#pullrequestreview-2069620296
PR Review Comment: https://git.openjdk.org/jdk/pull/19256#discussion_r1608953801
PR Review Comment: https://git.openjdk.org/jdk/pull/19256#discussion_r1608949352
More information about the core-libs-dev
mailing list