RFR(M) 8173846: [AOT] Stubs hang onto intermediate compiler state forever

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

RFR(M) 8173846: [AOT] Stubs hang onto intermediate compiler state forever

Igor Veresov
This a port of Graal change that alters the API that is used by the jaotc. In order to maintain drop-in compatibility with Graal, it's important to follow this API change in jaotc. The change essentially gets rid of Stub.compResult and Stub.compiledCode fields that hang on to the compilation state.

Webrev: http://cr.openjdk.java.net/~never/aot-stub-changes/webrev

Thanks,
igor
Reply | Threaded
Open this post in threaded view
|

Re: RFR(M) 8173846: [AOT] Stubs hang onto intermediate compiler state forever

Igor Veresov
Forgot to mention that the code is contributed by Tom Rodriguez and the Graal part will be pushed to graal-core.

igor

> On Feb 2, 2017, at 11:14 AM, Igor Veresov <[hidden email]> wrote:
>
> This a port of Graal change that alters the API that is used by the jaotc. In order to maintain drop-in compatibility with Graal, it's important to follow this API change in jaotc. The change essentially gets rid of Stub.compResult and Stub.compiledCode fields that hang on to the compilation state.
>
> Webrev: http://cr.openjdk.java.net/~never/aot-stub-changes/webrev
>
> Thanks,
> igor

Reply | Threaded
Open this post in threaded view
|

Re: RFR(M) 8173846: [AOT] Stubs hang onto intermediate compiler state forever

Vladimir Kozlov
Looks good.

Thanks,
Vladimir

On 2/2/17 2:22 PM, Igor Veresov wrote:

> Forgot to mention that the code is contributed by Tom Rodriguez and the Graal part will be pushed to graal-core.
>
> igor
>
>> On Feb 2, 2017, at 11:14 AM, Igor Veresov <[hidden email]> wrote:
>>
>> This a port of Graal change that alters the API that is used by the jaotc. In order to maintain drop-in compatibility with Graal, it's important to follow this API change in jaotc. The change essentially gets rid of Stub.compResult and Stub.compiledCode fields that hang on to the compilation state.
>>
>> Webrev: http://cr.openjdk.java.net/~never/aot-stub-changes/webrev
>>
>> Thanks,
>> igor
>
Reply | Threaded
Open this post in threaded view
|

Re: RFR(M) 8173846: [AOT] Stubs hang onto intermediate compiler state forever

Igor Veresov
Thanks, Vladimir!

igor

> On Feb 2, 2017, at 2:43 PM, Vladimir Kozlov <[hidden email]> wrote:
>
> Looks good.
>
> Thanks,
> Vladimir
>
> On 2/2/17 2:22 PM, Igor Veresov wrote:
>> Forgot to mention that the code is contributed by Tom Rodriguez and the Graal part will be pushed to graal-core.
>>
>> igor
>>
>>> On Feb 2, 2017, at 11:14 AM, Igor Veresov <[hidden email]> wrote:
>>>
>>> This a port of Graal change that alters the API that is used by the jaotc. In order to maintain drop-in compatibility with Graal, it's important to follow this API change in jaotc. The change essentially gets rid of Stub.compResult and Stub.compiledCode fields that hang on to the compilation state.
>>>
>>> Webrev: http://cr.openjdk.java.net/~never/aot-stub-changes/webrev
>>>
>>> Thanks,
>>> igor
>>