RFR (XXS): 8067012 : Don't create MDO for constant getters

Pavel Punegov pavel.punegov at oracle.com
Wed Jan 28 18:32:35 UTC 2015


Vladimir,

thank you for review

On 28.01.2015 20:55, Vladimir Kozlov wrote:
> Okay.
>
> Thanks,
> Vladimir
>
> On 1/28/15 9:47 AM, Pavel Punegov wrote:
>> Vladimir,
>>
>> It doesn't affect any test. There is a 
>> compiler/tiered/ConstantGettersTransitionsTest.java that checks that 
>> such methods
>> will go to the 1st level only, but they go there anyway regardless 
>> they have or not MDO.
>>
>> On 28.01.2015 20:07, Vladimir Kozlov wrote:
>>> Looks good. Does this change affect any tests which have to be 
>>> modified too? I remember some WB tests use method which
>>> return only constant to test compilation.
>>>
>>> Thanks,
>>> Vladimir
>>>
>>> On 1/28/15 4:18 AM, Pavel Punegov wrote:
>>>> Hi,
>>>>
>>>> please review the following small change.
>>>>
>>>> With the fix for JDK-8056071 [*]  constant getters are now compiled 
>>>> on level 1,
>>>> so there are no need to create MDO for constant getter methods in 
>>>> TieredCompilation
>>>>
>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8067012
>>>> webrev: http://cr.openjdk.java.net/~ppunegov/8067012/webrev/
>>>>
>>>> Testing: done locally and with JPRT.
>>>>
>>>> -- 
>>>> [*]  https://bugs.openjdk.java.net/browse/JDK-8056071
>>>>
>>

-- 
Thanks,
Pavel Punegov



More information about the hotspot-compiler-dev mailing list