<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On May 1, 2014, at 8:34 AM, Morris Meyer <<a href="mailto:morris.meyer@oracle.com">morris.meyer@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">The parfait error is with the ASSERT define blocking out initialization of a the fpu structures when that code path is enabled.<br><br>I've removed the ifdef such that both paths initialized their cpu and fpu variables, and the assertion is always checked.<br></blockquote><div><br></div>But the code path using the uninitialized variables is also under:</div><div><br></div><div><pre style="background-color: rgb(238, 238, 238);">1648   if (has_fpu_registers()) {
</pre><div><br></div><div>Am I missing something?  Anyway, during an earlier internal review I said we should keep the assert; I take that back.  We should move the variables and initialization down to the use and remove the assert.  It doesnít have much value anyway.</div><div><br></div><blockquote type="cite"><br>--mm<br><br>On 4/30/14, 3:36 PM, Christian Thalinger wrote:<br><blockquote type="cite">I donít understand the Parfait error here.  Could you explain?<br><br>On Apr 30, 2014, at 7:40 AM, Morris Meyer <<a href="mailto:morris.meyer@oracle.com">morris.meyer@oracle.com</a>> wrote:<br><br><blockquote type="cite">Folks,<br><br>Could I get a review for this Parfait uninitialized variable issue? This has been tested with JPRT and a clear pass with Parfait.<br><br>Thanks,<br><br>        --mm<br><br>BUG - <a href="https://bugs.openjdk.java.net/browse/JDK-8040921">https://bugs.openjdk.java.net/browse/JDK-8040921</a><br>WEBREV - <a href="http://cr.openjdk.java.net/~morris/JDK-8040921.05">http://cr.openjdk.java.net/~morris/JDK-8040921.05</a><br><br><br></blockquote></blockquote><br></blockquote></div><br></body></html>