<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hi Dmitrij,<div class=""><br class=""></div><div class="">the fix looks good to me.</div><div class=""><br class=""></div><div class="">--</div><div class="">Thanks, </div><div class="">Igor<br class=""><div class=""><br class=""><div class=""><div><blockquote type="cite" class=""><div class="">On Oct 13, 2015, at 11:15 PM, Dmitrij Pochepko <<a href="mailto:dmitrij.pochepko@oracle.com" class="">dmitrij.pochepko@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">
  

    <meta http-equiv="content-type" content="text/html; charset=utf-8" class="">
  
  <div text="#000000" bgcolor="#FFFFFF" class="">
    Hi,
    <br class="">
    <br class="">
    please review a small fix for test:
    compiler/jvmci/compilerToVM/MaterializeVirtualObjectTest.java
    <meta http-equiv="content-type" content="text/html; charset=utf-8" class="">
    <br class="">
    <br class="">
    Description:<br class="">
    <br class="">
    A test is changed to run testcases in separate VM launch instead of
    consecutive execution, which had wrong assumption about method
    compilation state in case of -XX:-TieredCompilation.<br class="">
    So, a boolean system property is added into test to control test
    execution logic externally.<br class="">
    I've also added minor code cleanup.<br class="">
    <br class="">
    webrev: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~dpochepk/8139438/webrev.01/">http://cr.openjdk.java.net/~dpochepk/8139438/webrev.01/</a><a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/%7Edpochepk/8136718/webrev.8136718.01/"></a>
    <br class="">
    <br class="">
    Thanks,
    <br class="">
    Dmitrij
  </div>

</div></blockquote></div><br class=""></div></div></div></body></html>