[9] RFR(S): 8153514: Whitebox API should allow compilation of <clinit>

Igor Ignatyev igor.ignatyev at oracle.com
Wed Apr 6 11:38:42 UTC 2016


Hi Tobias,

looks good to me, thanks for implementing this.

Thanks,
— Igor
> On Apr 6, 2016, at 2:05 PM, Tobias Hartmann <tobias.hartmann at oracle.com> wrote:
> 
> Thanks, Zoltan!
> 
> Best regards,
> Tobias
> 
> On 06.04.2016 12:59, Zoltán Majó wrote:
>> Hi Tobias,
>> 
>> 
>> that looks good to me!
>> 
>> Best regards,
>> 
>> 
>> Zoltan
>> 
>> 
>> On 04/06/2016 12:53 PM, Tobias Hartmann wrote:
>>> Hi,
>>> 
>>> please review the following patch that adds support to the Whitebox API to compile the static initializer of a class:
>>> 
>>> https://bugs.openjdk.java.net/browse/JDK-8153514
>>> http://cr.openjdk.java.net/~thartmann/8153514/webrev.root.00/
>>> http://cr.openjdk.java.net/~thartmann/8153514/webrev.00/
>>> 
>>> The static initializer is not accessible through reflection and can therefore not be compiled by using the existing WhiteBox::enqueueMethodForCompilation feature.
>>> 
>>> I did not add tests for the new feature because that would require implementing additional methods to check for a successful <clinit> compilation (or parsing the log compilation output). The feature will be exercised by JDK-8153515 which should be enough for testing.
>>> 
>>> Thanks,
>>> Tobias
>> 



More information about the hotspot-compiler-dev mailing list