RFR (XXS): 8025845: Default methods are unnecessarily marked w/ force_inline directive

[Aleksey Shipilev]

I'll meet the fix there. Thanks!

-Aleksey.

On 10/07/2013 02:32 PM, Vladimir Ivanov wrote:
> FYI, the fix is in JPRT queue heading hotspot-comp.
> 
> Best regards,
> Vladimir Ivanov
> 
> On 10/7/13 12:46 PM, Aleksey Shipilev wrote:
>> Hi Vladimir(s)!
>>
>> Can we please get this fix in as soon as possible? This has the
>> unfortunate effect to distort the inlining decisions in Stream ops,
>> which makes it hard to conduct the performance work.
>>
>> Thanks,
>> -Aleksey.
>>
>> On 10/03/2013 08:34 PM, Vladimir Ivanov wrote:
>>> Thank you, Vladimir.
>>>
>>> Best regards,
>>> Vladimir Ivanov
>>>
>>> On 10/3/13 8:23 PM, Vladimir Kozlov wrote:
>>>> Good.
>>>>
>>>> Vladimir
>>>>
>>>> On 10/2/13 1:34 PM, Vladimir Ivanov wrote:
>>>>> http://cr.openjdk.java.net/~vlivanov/8025845/webrev.00/
>>>>> 1 line changed: 0 ins; 1 del
>>>>>
>>>>> Default methods may be marked with force_inline directive, which
>>>>> forces JIT compiler to unconditionally inline them
>>>>> during compilation. It may disturb inlining decisions JIT compiler
>>>>> makes causing generation of suboptimal code and
>>>>> performance regressions.
>>>>>
>>>>> Testing: verified that a default method isn't unconditionally inlined
>>>>> which was inlined before the fix.
>>>>>
>>>>> Thanks!
>>>>>
>>>>> Best regards,
>>>>> Vladimir Ivanov
>>>>>
>>>>> https://bugs.openjdk.java.net/browse/JDK-8025845
>>