<div>
                    Thanks Vladimir and Tom!
                </div>
                <div><div><br></div>igor<div><br></div></div>
                 
                <p style="color: #A0A0A8;">On Wednesday, November 16, 2011 at 4:55 PM, Tom Rodriguez wrote:</p>
                <blockquote type="cite" style="border-left-style:solid;border-width:1px;margin-left:0px;padding-left:10px;">
                    <span><div><div><div>Looks good.</div><div><br></div><div>tom</div><div><br></div><div>On Nov 16, 2011, at 3:18 PM, Igor Veresov wrote:</div><div><br></div><blockquote type="cite"><div><div>On Wednesday, November 16, 2011 at 12:46 PM, Vladimir Kozlov wrote:</div><blockquote type="cite"><div>Looks good.</div></blockquote><div>Thank you. </div><blockquote type="cite"><div><div>Did it pass all compiler regression tests with -XX:+TieredCompilation</div><div>-XX:+DeoptimizeALot?</div></div></blockquote><div>Yes. I just reran the tests again to make sure. I also noticed a problem with the test for 6792161. It increases the inlining size so much that it makes the fastdebug C1 very slow. I'll have to investigate why, but for now I decide to increase the timeout. Webrev updated.</div><div><br></div><div>igor</div><div> </div><blockquote type="cite"><div><div>Thanks,</div><div>Vladimir</div><div><br></div><div>Igor Veresov wrote:</div><blockquote type="cite"><div><div>The runtime call to the policy from the inlinee invocation had an</div><div>incorrect jvm state. As a result the bytecode preceding the invoke of</div><div>the inlinee was executed twice in case we deoptimized. Since it's</div><div>currently not quite easy to set the reexecute flag in C1, I moved the</div><div>notification call into the preamble of the inlinee so that I could use</div><div>InvocationEntryBci in the state specification, which would deopt correctly.</div><div><br></div><div>Also made some minor changes to printing of tiered events.</div><div><br></div><div>Webrev: <a href="http://cr.openjdk.java.net/~iveresov/7112085/webrev.00/">http://cr.openjdk.java.net/~iveresov/7112085/webrev.00/</a></div><div><br></div><div>Thanks!</div><div>igor</div></div></blockquote></div></blockquote></div></blockquote></div></div></span>
                 
                 
                 
                 
                </blockquote>
                 
                <div>
                    <br>
                </div>