RFR(XS) : 8195724 compiler.calls.common.InvokeDynamicPatcher can be run as driver

[Vladimir Kozlov]

Wow. Very nice!

Thanks,
Vladimir

On 1/18/18 3:10 PM, Igor Ignatyev wrote:
> Hi Vladimir,
> 
> Thank you for your review. w/ -Xcomp specified, InvokeDynamicPatcher action used to take around 8 seconds to finish, now it takes less than 0.1
> 
> Thanks,
> -- Igor
> 
>> On Jan 18, 2018, at 2:52 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
>>
>> Thank you, Igor for doing it.
>> I also noticed that running these tests with Xcomp takes hours.
>> How much execution time is improved with this fix?
>>
>> Thanks,
>> Vladimir
>>
>> On 1/18/18 2:03 PM, Igor Ignatyev wrote:
>>> http://cr.openjdk.java.net/~iignatyev/8195724/webrev.00/
>>>> 12 lines changed: 0 ins; 0 del; 12 mod;
>>> Hi all,
>>> could you please review this small fix for compiler(/aot)/calls tests? this patch changes execution mode of c.c.c.InvokeDynamicPatcher aux class to 'driver', so jtreg won't use external vm flags and will be able to share agent JVM for execution.
>>> webrev: http://cr.openjdk.java.net/~iignatyev/8195724/webrev.00/
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8195724
>>> testing: compiler/calls and compiler/aot/calls
>>> Thanks,
>>> -- Igor
>