<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On Dec 7, 2012, at 1:45 PM, Jiangli Zhou wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><span class="Apple-style-span" style="border-collapse: separate; font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; ">Could I have another reviewer for the change? Either official or non-official reviewer is okay.<br></span></blockquote><br></div><div>I have a comment on this one. &nbsp;(And you can use me as a reviewer.)</div><br><div>In the sparc code, I would prefer to see the Address constants kept around, because they make the code easier to read.</div><div><br></div><div><div>- &nbsp;const Address size_of_parameters(G5_method, Method::size_of_parameters_offset());</div><div>- &nbsp;const Address size_of_locals &nbsp; &nbsp;(G5_method, Method::size_of_locals_offset());</div><div>+ &nbsp;const Register G3_constMethod =&nbsp;Glocals_size;</div><div>+ &nbsp;const Address size_of_parameters(G3_ConstMethod, ConstMethod::size_of_parameters_offset());</div><div>+ &nbsp;const Address size_of_locals &nbsp; &nbsp;(G3_ConstMethod,&nbsp;ConstMethod::size_of_locals_offset());</div><div>&nbsp; &nbsp;const Address constMethod &nbsp; &nbsp; &nbsp; (G5_method, Method::const_offset());</div><div>&nbsp; &nbsp;int rounded_vm_local_words = round_to( frame::interpreter_frame_vm_local_words, WordsPerLong );</div><div>&nbsp;</div><div>@@ -511,7 +509,8 @@</div><div>&nbsp; &nbsp;// Lscratch can't be used as a temporary because the call_stub uses</div><div>&nbsp; &nbsp;// it to assert that the stack frame was setup correctly.</div><div>&nbsp;</div><div>+ &nbsp;__ ld_ptr( constMethod,&nbsp;G3_constMethod&nbsp;);</div><div>- &nbsp;__ lduh( size_of_parameters, Glocals_size);</div><div><br></div></div><div>Maybe the details make it difficult, but I think it's possible in most places.</div><div><br></div><div>As a side effect, using a fixed register (like G3) for the ConstMethod will make debugging easier.</div><div><br></div><div>Same comment for x86/64.</div><div><br></div><div>This may be more difficult to do on x86/32, since there are fewer temp registers. &nbsp;But that's an oddball case.</div><div><br></div><div>Thanks,</div><div>— John</div></body></html>