<!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 Tom,<br>

<br>

Thanks. I'll make the change (which was what I started with and then

changed it to it's current version).<br>

<br>

JohnC<br>

<br>

On 04/08/11 11:31, Tom Rodriguez wrote:

<blockquote cite="mid:5FAB74B1-C3A6-46C4-ABC3-E58380059D70@oracle.com"

 type="cite">

  <pre wrap="">Actually you want to write this:

  Node* referent_off = __ ConX(java_lang_ref_Reference::referent_offset);

which will pick ConL in 64 bit and ConI in 32 bit.  Sorry I didn't catch this in the review.

tom

On Apr 8, 2011, at 11:04 AM, John Cuthbertson wrote:

  </pre>

  <blockquote type="cite">

    <pre wrap="">Hi Everyone,

Can I have a couple of volunteers to look over the fix for this CR? The webrev can be found at: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~johnc/7035177/webrev.0/">http://cr.openjdk.java.net/~johnc/7035177/webrev.0/</a>.

The problem is that the node representing the offset (in an Unsafe.getObject compilation) could be typed as a long and generating the compare of offset against java_lang_ref_Reference::referent_offset (typed as an int) caused an assertion failure about the mis-matching types.

The fix is to generate a suitably typed constant based upon the type of "offset".

Tested using the failing test case from the nightly tests.

Thanks,

JohnC

    </pre>

  </blockquote>

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

  </pre>

</blockquote>

<br>

</body>

</html>