On Thu, 25 Aug 2022 09:14:07 GMT, Sean Mullan <mul...@openjdk.org> 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

Reply via email to