<div dir="ltr"><div dir="ltr" style="font-family:arial,sans-serif;font-size:13px"><a href="http://cr.openjdk.java.net/~rasbold/8061259/webrev.06/" target="_blank">http://cr.openjdk.java.net/~rasbold/8061259/webrev.06/</a><br></div><div><br></div><div>PTAL. </div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 17, 2014 at 7:57 AM, Kim Barrett <span dir="ltr"><<a href="mailto:kim.barrett@oracle.com" target="_blank">kim.barrett@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Nov 17, 2014, at 5:11 AM, Bengt Rutisson <<a href="mailto:bengt.rutisson@oracle.com">bengt.rutisson@oracle.com</a>> wrote:<br>
><br>
>> I think this does leave open the possibility of moving the<br>
>> _promotion_failed flag check earlier in copy_to_survivor_space_XXX(),<br>
</span>>> […]<br>
>><br>
>> I'm not sure it would be overall beneficial though, […]<br>
<span class="">><br>
> Yes, there is probably room for more improvement here. I think adding the check in the copy_to_survivor_space_XXX_undo() methods is a clear improvement in many cases. So, doing that as a first step is a good start. We can consider moving it earlier as a future enhancement.<br>
<br>
</span>Agreed.<br>
<br>
</blockquote></div><br></div>