[11] RFR(S): 8200230: [Graal] Compilations should not be enqueued before Graal is initialized

[Tobias Hartmann]

Thanks Vladimir!

On 26.03.2018 19:01, Vladimir Kozlov wrote:
> Looks good. Did you test these changes when Graal is enabled as JIT?

Yes, I've tested with Graal but since we have lots of known issues, it's a bit hard to filter out
the real ones. I haven't seen any related issues.

Best regards,
Tobias


> On 3/26/18 3:35 AM, Tobias Hartmann wrote:
>> Hi,
>>
>> please review the following patch:
>> https://bugs.openjdk.java.net/browse/JDK-8200230
>> http://cr.openjdk.java.net/~thartmann/8200230/webrev.00/
>>
>> Looking at the PrintCompilation output when running Graal with -Xbatch and -XX:-TieredCompilation
>> shows lots of blocking compilations that time out because Graal is not yet initialized. Execution
>> with -version therefore takes 41 seconds on my machine with a fastdebug build.
>>
>> With -Xbatch, we should only allow compilations to be enqueued when Graal is fully initialized. This
>> reduces execution time with -version to 2.5 seconds on my machine.
>>
>> Thanks to Doug Simon for providing the patch.
>>
>> Best regards,
>> Tobias
>>