<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">

<html>

<head>

  <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">

</head>

<body bgcolor="#ffffff" text="#000000">

Hi John,<br>

<br>

Sorry I'm late getting back to you. Thanks for the review. Can you

check out the latest webrev based upon feedback from Jon Masmitsu -

when you get a chance?<br>

<br>

Thanks,<br>

<br>

JohnC<br>

<br>

On 10/01/12 16:27, John Coomes wrote:

<blockquote cite="mid:20586.9943.595220.270355@oracle.com" type="cite">

  <pre wrap="">John Cuthbertson (<a class="moz-txt-link-abbreviated" href="mailto:john.cuthbertson@oracle.com">john.cuthbertson@oracle.com</a>) wrote:

  </pre>

  <blockquote type="cite">

    <pre wrap="">Hi Everyone,

Can I have a couple of volunteers look over the changes for this fix? 

The webrev can be found at: 

<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~johnc/8000311/webrev.0/">http://cr.openjdk.java.net/~johnc/8000311/webrev.0/</a>

Summary:

While testing the changes for another CR, I ran into this divide by zero 

error. I decided to make a separate CR so that the change can be 

backported to hs24 - which also has the issue.

Testing:

The original test case with and without PGCT=0 and PrintPLAB enabled.

    </pre>

  </blockquote>

  <pre wrap=""><!---->

Looks good to me.

-John

  </pre>

</blockquote>

<br>

</body>

</html>