RFR: 8138799 - improve tests for CompilerToVM::MaterializeVirtualObjectTest

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

RFR: 8138799 - improve tests for CompilerToVM::MaterializeVirtualObjectTest

dmitrij pochepko
Hi,

please review patch for 8138799 - improve tests for
CompilerToVM::MaterializeVirtualObjectTest

This patch adds logic which checks that frame materialization doesn't
affect previous or next frame.


webrev: http://cr.openjdk.java.net/~dpochepk/8138799/webrev.01/

CR: https://bugs.openjdk.java.net/browse/JDK-8138799

I've tested this patch on linux-x64.


Thanks,

Dmitrij

Reply | Threaded
Open this post in threaded view
|

Re: RFR: 8138799 - improve tests for CompilerToVM::MaterializeVirtualObjectTest

Vladimir Kozlov
Okay.

thanks,
Vladimir

On 2/14/17 6:37 AM, Dmitrij Pochepko wrote:

> Hi,
>
> please review patch for 8138799 - improve tests for CompilerToVM::MaterializeVirtualObjectTest
>
> This patch adds logic which checks that frame materialization doesn't affect previous or next frame.
>
>
> webrev: http://cr.openjdk.java.net/~dpochepk/8138799/webrev.01/
>
> CR: https://bugs.openjdk.java.net/browse/JDK-8138799
>
> I've tested this patch on linux-x64.
>
>
> Thanks,
>
> Dmitrij
>
Reply | Threaded
Open this post in threaded view
|

Re: RFR: 8138799 - improve tests for CompilerToVM::MaterializeVirtualObjectTest

dmitrij pochepko
Thank you!


On 14.02.2017 22:13, Vladimir Kozlov wrote:

> Okay.
>
> thanks,
> Vladimir
>
> On 2/14/17 6:37 AM, Dmitrij Pochepko wrote:
>> Hi,
>>
>> please review patch for 8138799 - improve tests for
>> CompilerToVM::MaterializeVirtualObjectTest
>>
>> This patch adds logic which checks that frame materialization doesn't
>> affect previous or next frame.
>>
>>
>> webrev: http://cr.openjdk.java.net/~dpochepk/8138799/webrev.01/
>>
>> CR: https://bugs.openjdk.java.net/browse/JDK-8138799
>>
>> I've tested this patch on linux-x64.
>>
>>
>> Thanks,
>>
>> Dmitrij
>>