<!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">
<br>
While you are there, I wonder if you can't just get rid of the btos,
ctos, and stos entries in EntryPoint and save some footprint too.&nbsp; I am
not sure if it's worth it: 2 tables (normal+safepoint) * 256 entries *
8 tos states * sizeof (address).&nbsp; 4096 pointers vs. 2560 pointers.&nbsp; Not
counting the safepoint and other tables of EntryPoints.<br>
<br>
Coleen<br>
<br>
On 11/18/09 12:12, Christian Thalinger wrote:
<blockquote cite="mid:1258564327.855.33.camel@macbook" type="cite">
  <pre wrap="">On Wed, 2009-11-18 at 08:56 -0800, Vladimir Kozlov wrote:
  </pre>
  <blockquote type="cite">
    <pre wrap="">Changes are fine but I would place ShouldNotReachHere on separate line.
I think it would be more clear this way that it applies to all previous cases.
    </pre>
  </blockquote>
  <pre wrap=""><!---->
You're right.  I also missed a break:

<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~twisti/6902000/webrev.02/">http://cr.openjdk.java.net/~twisti/6902000/webrev.02/</a>

-- Christian

  </pre>
</blockquote>
</body>
</html>