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