[8u] approval request for 8145550: Megamorphic invoke should use CompiledFunction variants without any LinkLogic, 8145669: apply2call optimized callsite fails after becoming megamorphic

[Seán Coffey]

Approved for jdk8u-dev.

Regards,
Sean.

On 17/12/2015 15:05, Jim Laskey (Oracle) wrote:
> +1
>
>> On Dec 17, 2015, at 10:58 AM, Sundararajan Athijegannathan <sundararajan.athijegannathan at oracle.com> wrote:
>>
>> Please approve the backport of the following fixes:
>>
>> 8145550: Megamorphic invoke should use CompiledFunction variants without any LinkLogic
>> https://bugs.openjdk.java.net/browse/JDK-8145550
>> jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-December/005736.html
>>
>> 8145669: apply2call optimized callsite fails after becoming megamorphic
>> https://bugs.openjdk.java.net/browse/JDK-8145669
>> jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-December/005747.html
>>
>> jdk8u-dev combined webrev:
>> http://cr.openjdk.java.net/~sundar/8145550_8145669/8u/webrev.00/
>>
>> Apart from the modular source layout changes, the tests have to be adjusted slightly for jdk8u -- as tests in jdk9 use a new Nashorn engine option that does not exist in jdk8u. For jdk8u, the tests use System property to configure megamorphic threshold  and @fork option. I'm CC'ing nashorn-dev for review.
>>
>> Thanks,
>> -Sundar