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

[Aleksey Shipilev]

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