[9] RFR (L): 8057042: LambdaFormEditor: derive new LFs from a base LF

[Vladimir Ivanov]

Peter,

Yes, it's a known problem [1].
There are other caches (in MethodTypeForm, for example), which shouldn't 
retain their elements.

Best regards,
Vladimir Ivanov

[1] https://bugs.openjdk.java.net/browse/JDK-8057020

On 9/3/14, 3:43 PM, Peter Levart wrote:
> Hi Vladimir,
>
> I'm sure you and John have thought about it through, but I'll ask
> anyway. Are cached LambdaForms going to stay around forever? What about
> using a WeakReference<LambdaForm> (LambdaFormEditor.Transform could
> extend WeakReference). This way unused LambdaForms would get GCed.
>
> Regards, Peter
>
> On 09/02/2014 03:59 PM, Vladimir Ivanov wrote:
>> Webrev: http://cr.openjdk.java.net/~vlivanov/8057042/webrev.00
>>
>> Best regards,
>> Vladimir Ivanov
>>
>> On 9/2/14, 5:57 PM, Vladimir Ivanov wrote:
>>> https://bugs.openjdk.java.net/browse/JDK-8057042
>>>
>>> LambdaFormEditor provides an API to transform LambdaForms. Deriving new
>>> LambdaForms from a base one allows to cache and reuse results of
>>> repeated transformations.
>>>
>>> BMH binding is rewritten to use LambdaFormEditor.
>>>
>>> Testing: jdk/java/lang/invoke, jdk/java/util/streams, nashorn, octane w/
>>> "-ea -esa" and COMPILE_THRESHOLD={0,30}.
>>>
>>> Reviewed-by: vlivanov, ?
>>> Contributed-by: john.r.rose at oracle.com
>>>
>>> Thanks!
>>>
>>> Best regards,
>>> Vladimir Ivanov
>>> _______________________________________________
>>> mlvm-dev mailing list
>>> mlvm-dev at openjdk.java.net
>>> http://mail.openjdk.java.net/mailman/listinfo/mlvm-dev
>> _______________________________________________
>> mlvm-dev mailing list
>> mlvm-dev at openjdk.java.net
>> http://mail.openjdk.java.net/mailman/listinfo/mlvm-dev
>