On Thu, 25 Aug 2022 09:14:07 GMT, Sean Mullan <[email protected]> wrote:
>> Eric Caspole has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Updates from reviewers comments
>
> test/micro/org/openjdk/bench/java/security/ProtectionDomainBench.java line 90:
>
>> 88: for (int i = 0; i < numberOfClasses; i++) {
>> 89: classNames[i] = "B" + i;
>> 90: compiledClasses[i] =
>> InMemoryJavaCompiler.compile(classNames[i], B(i));
>
> Is it possible to use the existing
> `jdk.test.lib.compiler.InMemoryJavaCompiler` class instead of writing a new
> one?
It needs to be added into the benchmarks.jar, and there are already 2 copies of
it in different test suites so it seems like this is how it's done.
-------------
PR: https://git.openjdk.org/jdk/pull/9950