<div dir="ltr">Hi John,<div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 22, 2014 at 6:50 PM, John Coomes <span dir="ltr"><<a href="mailto:John.Coomes@oracle.com" target="_blank">John.Coomes@oracle.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

opto/runtime.cpp:<br>
<br>
   fields[TypeFunc::Parms+0] = TypeInt::INT; // trap_reason (deopt reason and action)<br>
<br>
   The comment you added is not obviously correct; did someone from<br>
   the compiler team check this?  If the comment is correct, the code<br>
   is crazily obscure!<br>
<br></blockquote><div>Yes, I proposed this change. The old comment had been out of date even before JDK6.</div><div><br></div><div>duke@0:</div><div>line 392 of <a href="http://hg.openjdk.java.net/hsx/hotspot-main/hotspot/file/a61af66fc99e/src/share/vm/opto/runtime.cpp">http://hg.openjdk.java.net/hsx/hotspot-main/hotspot/file/a61af66fc99e/src/share/vm/opto/runtime.cpp</a></div>
<div><br></div><div>The corresponding runtime call goes through SharedRuntime::uncommon_trap_blob to:</div><div><br></div><div>Deoptimization::UnrollBlock* Deoptimization::uncommon_trap(JavaThread* thread, jint trap_request)<br>
</div><div><br></div><div>The "thread" argument came from uncommon_trap_blob; the "trap_request" argument came from constant embedded in compiled code.</div><div>And this "trap_request" is the thing described by OptoRuntime::uncommon_trap_Type().</div>
<div><br></div><div>Back in JDK1.4.2-ish or maybe even earlier, the signature of this function was:</div><div><br></div><div>Deoptimization::UnrollBlock*, Deoptimization::uncommon_trap(JavaThread* thread, jint unloaded_class_index)<br>
</div><div><br></div><div>where "unloaded_class_index" was a constant pool index of a class if >= 0, or other deopt cases if < 0.</div><div><br></div><div>I'm not sure when it changed to its current form... waiting for someone else to share the history :-)</div>
<div>Maybe the comment had been out of date longer then I thought.</div><div><br></div><div>Speaking of this, the comment in SharedRuntime::generate_uncommon_trap_blob() is also out of date:</div><div><br></div><div>e.g. hotspot/src/cpu/x86/vm/sharedRuntime_x86_64.cpp</div>
<div><br></div><div><div>  // compiler left unloaded_class_index in j_rarg0 move to where the</div><div>  // runtime expects it.</div><div>  __ movl(c_rarg1, j_rarg0);</div></div><div><br></div><div>"unloaded_class_index" should be "trap_reason" now.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
-John<br>
<div class=""><div class="h5"><br>
</div></div><span class=""><font color="#888888">--<br>
John Coomes                            Oracle, MS USCA22-3??<br>
<a href="mailto:john.coomes@oracle.com">john.coomes@oracle.com</a>                 4220 Network Circle<br>
<a href="tel:408-276-7048" value="+14082767048">408-276-7048</a>                           Santa Clara, CA 95054-1778<br>
         *** Support GreenPeace and we'll all breathe easier. ***<br>
</font></span></blockquote></div><br></div></div>