<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On May 2, 2016, at 11:53 AM, Deshpande, Vivek R <<a href="mailto:vivek.r.deshpande@intel.com" class="">vivek.r.deshpande@intel.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="WordSection1" style="page: WordSection1; font-family: Verdana; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 8.5pt; font-family: Monaco, serif;" class="">Hi Christian<o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 8.5pt; font-family: Monaco, serif;" class=""><o:p class=""> </o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 8.5pt; font-family: Monaco, serif;" class="">I had tried using call_VM_leaf() which you mentioned.<o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 8.5pt; font-family: Monaco, serif;" class=""><o:p class=""> </o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 8.5pt; font-family: Monaco, serif;" class="">But this function<o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 8.5pt; font-family: Monaco, serif; color: rgb(147, 26, 104);" class=""><o:p class=""> </o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 8.5pt; font-family: Monaco, serif; color: rgb(147, 26, 104);" class="">void</span><span style="font-size: 8.5pt; font-family: Monaco, serif;" class=""><span class="Apple-converted-space"> </span>MacroAssembler::call_VM_leaf(<span style="color: rgb(0, 97, 65);" class="">address</span><span class="Apple-converted-space"> </span>entry_point,<span class="Apple-converted-space"> </span><span style="color: rgb(147, 26, 104);" class="">int</span><span class="Apple-converted-space"> </span>number_of_arguments) {<o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 8.5pt; font-family: Monaco, serif;" class="">  call_VM_leaf_base(entry_point, number_of_arguments);<o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 8.5pt; font-family: Monaco, serif;" class="">}<o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 8.5pt; font-family: Monaco, serif;" class=""><o:p class=""> </o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 8.5pt; font-family: Monaco, serif;" class="">ends up calling<o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 8.5pt; font-family: Monaco, serif;" class=""> <o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 9.5pt; font-family: Consolas; color: blue; background-color: white; background-position: initial initial; background-repeat: initial initial;" class="">void<span class="Apple-converted-space"> </span></span><span style="font-size: 9.5pt; font-family: Consolas; color: rgb(43, 145, 175); background-color: white; background-position: initial initial; background-repeat: initial initial;" class="">InterpreterMacroAssembler</span><span style="font-size: 9.5pt; font-family: Consolas; background-color: white;" class="">::call_VM_leaf_base(</span><span style="font-size: 9.5pt; font-family: Consolas; color: rgb(43, 145, 175); background-color: white; background-position: initial initial; background-repeat: initial initial;" class="">address</span><span style="font-size: 9.5pt; font-family: Consolas; background-color: white;" class=""><span class="Apple-converted-space"> </span></span><span style="font-size: 9.5pt; font-family: Consolas; color: gray; background-color: white; background-position: initial initial; background-repeat: initial initial;" class="">entry_point</span><span style="font-size: 9.5pt; font-family: Consolas; background-color: white;" class="">,<o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 9.5pt; font-family: Consolas; background-color: white;" class="">                                                 <span class="Apple-converted-space"> </span></span><span style="font-size: 9.5pt; font-family: Consolas; color: blue; background-color: white; background-position: initial initial; background-repeat: initial initial;" class="">int</span><span style="font-size: 9.5pt; font-family: Consolas; background-color: white;" class=""><span class="Apple-converted-space"> </span></span><span style="font-size: 9.5pt; font-family: Consolas; color: gray; background-color: white; background-position: initial initial; background-repeat: initial initial;" class="">number_of_arguments</span><span style="font-size: 9.5pt; font-family: Consolas; background-color: white;" class="">) {</span><span style="font-size: 9.5pt; font-family: Consolas;" class=""><span class="Apple-converted-space"> </span>...<o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 9.5pt; font-family: Consolas;" class="">from interp_masm_x86.cpp instead of<o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 9.5pt; font-family: Consolas;" class=""><o:p class=""> </o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 9.5pt; font-family: Consolas; color: blue; background-color: white; background-position: initial initial; background-repeat: initial initial;" class="">void</span><span style="font-size: 9.5pt; font-family: Consolas; background-color: white;" class=""><span class="Apple-converted-space"> </span></span><span style="font-size: 9.5pt; font-family: Consolas; color: rgb(43, 145, 175); background-color: white; background-position: initial initial; background-repeat: initial initial;" class="">MacroAssembler</span><span style="font-size: 9.5pt; font-family: Consolas; background-color: white;" class="">::call_VM_leaf_base(</span><span style="font-size: 9.5pt; font-family: Consolas; color: rgb(43, 145, 175); background-color: white; background-position: initial initial; background-repeat: initial initial;" class="">address</span><span style="font-size: 9.5pt; font-family: Consolas; background-color: white;" class=""><span class="Apple-converted-space"> </span></span><span style="font-size: 9.5pt; font-family: Consolas; color: gray; background-color: white; background-position: initial initial; background-repeat: initial initial;" class="">entry_point</span><span style="font-size: 9.5pt; font-family: Consolas; background-color: white;" class="">,<span class="Apple-converted-space"> </span></span><span style="font-size: 9.5pt; font-family: Consolas; color: blue; background-color: white; background-position: initial initial; background-repeat: initial initial;" class="">int</span><span style="font-size: 9.5pt; font-family: Consolas; background-color: white;" class=""><span class="Apple-converted-space"> </span></span><span style="font-size: 9.5pt; font-family: Consolas; color: gray; background-color: white; background-position: initial initial; background-repeat: initial initial;" class="">num_args</span><span style="font-size: 9.5pt; font-family: Consolas; background-color: white;" class="">) {</span><span style="font-size: 9.5pt; font-family: Consolas;" class=""><span class="Apple-converted-space"> </span>…</span></div></div></div></blockquote><div><br class=""></div>Frankly, I didn’t know that there is an overload for call_VM_leaf_base in InterpreterMacroAssembler.  So this means there are two options:</div><div><br class=""></div><div>a)  Add a method in MacroAssembler to call MacroAssembler::call_VM_leaf_base (what you did) or</div><div><br class=""></div><div>b)  Add InterpreterMacroAssembler::call_VM_leaf and change MacroAssembler::call_VM_leaf to do:</div><div><br class=""></div><div><div style="margin: 0px; font-size: 11px; line-height: normal; font-family: Monaco;" class=""><span style="color: #931a68" class="">void</span> MacroAssembler::call_VM_leaf(<span style="color: #006141" class="">address</span> entry_point, <span style="color: #931a68" class="">int</span> number_of_arguments) {</div><div style="margin: 0px; font-size: 11px; line-height: normal; font-family: Monaco;" class="">  MacroAssembler::call_VM_leaf_base(entry_point, number_of_arguments);</div><div style="margin: 0px; font-size: 11px; line-height: normal; font-family: Monaco;" class="">}</div><div class=""><br class=""></div><div class="">I will let the runtime team decide.</div><div class=""><br class=""></div></div><div><blockquote type="cite" class=""><div class=""><div class="WordSection1" style="page: WordSection1; font-family: Verdana; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 9.5pt; font-family: Consolas;" class=""><o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 9.5pt; font-family: Consolas;" class=""><o:p class=""> </o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 9.5pt; font-family: Consolas;" class="">So I had put mathfunc() to call the masm version of call_VM_leaf_base().<o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 9.5pt; font-family: Consolas;" class=""><o:p class=""> </o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 9.5pt; font-family: Consolas;" class="">Let me know what you think.<o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 9.5pt; font-family: Consolas;" class=""><o:p class=""> </o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 9.5pt; font-family: Consolas;" class="">Thanks and regards,<br class="">Vivek<o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""><o:p class=""> </o:p></span></div><div class=""><div style="border-style: solid none none; border-top-color: rgb(181, 196, 223); border-top-width: 1pt; padding: 3pt 0in 0in;" class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><b class=""><span style="font-size: 10pt; font-family: Tahoma, sans-serif;" class="">From:</span></b><span style="font-size: 10pt; font-family: Tahoma, sans-serif;" class=""><span class="Apple-converted-space"> </span>Christian Thalinger [<a href="mailto:christian.thalinger@oracle.com" class="">mailto:christian.thalinger@oracle.com</a>]<span class="Apple-converted-space"> </span><br class=""><b class="">Sent:</b><span class="Apple-converted-space"> </span>Monday, May 02, 2016 1:50 PM<br class=""><b class="">To:</b><span class="Apple-converted-space"> </span>Deshpande, Vivek R<br class=""><b class="">Cc:</b><span class="Apple-converted-space"> </span><a href="mailto:hotspot-compiler-dev@openjdk.java.net" class="">hotspot-compiler-dev@openjdk.java.net</a><br class=""><b class="">Subject:</b><span class="Apple-converted-space"> </span>Re: RFR (S): 8154473: Update for CompilerDirectives to control stub generation and intrinsics<o:p class=""></o:p></span></div></div></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><o:p class=""> </o:p></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><o:p class=""> </o:p></div><div class=""><blockquote style="margin-top: 5pt; margin-bottom: 5pt;" class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class="">On Apr 26, 2016, at 8:53 PM, Deshpande, Vivek R <<a href="mailto:vivek.r.deshpande@intel.com" style="color: purple; text-decoration: underline;" class="">vivek.r.deshpande@intel.com</a>> wrote:<o:p class=""></o:p></div></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><o:p class=""> </o:p></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">Hi Christian</span><o:p class=""></o:p></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""> </span><o:p class=""></o:p></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">I have updated the webrev and link for the same is here:</span><o:p class=""></o:p></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""><a href="http://cr.openjdk.java.net/~vdeshpande/CompilerDirectives/8154473/webrev.03/" style="color: purple; text-decoration: underline;" class=""><span style="color: purple;" class="">http://cr.openjdk.java.net/~vdeshpande/CompilerDirectives/8154473/webrev.03/</span></a></span><o:p class=""></o:p></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">I am using mathfunc() to call the masm version of call_VM_leaf_base() and not the InterpreterMacroAssembler version.</span><o:p class=""></o:p></div></div></div></blockquote><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><o:p class=""> </o:p></div></div><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class="">That’s better but, again, there is nothing math-y about this method:<o:p class=""></o:p></div></div><div class=""><pre style="margin: 0in 0in 0.0001pt; font-size: 10pt; font-family: 'Courier New'; background-color: rgb(238, 238, 238);" class=""><span class="changed"><span style="color: blue;" class="">! void MacroAssembler::mathfunc(address runtime_entry) {</span></span><o:p class=""></o:p></pre><pre style="margin: 0in 0in 0.0001pt; font-size: 10pt; font-family: 'Courier New'; background-color: rgb(238, 238, 238);" class="">    MacroAssembler::call_VM_leaf_base(runtime_entry, 0);<o:p class=""></o:p></pre><pre style="margin: 0in 0in 0.0001pt; font-size: 10pt; font-family: 'Courier New'; background-color: rgb(238, 238, 238);" class="">  }<o:p class=""></o:p></pre><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class="">Also, there is this method:<o:p class=""></o:p></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><o:p class=""> </o:p></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 8.5pt; font-family: Monaco, serif; color: rgb(147, 26, 104);" class="">void</span><span style="font-size: 8.5pt; font-family: Monaco, serif;" class=""><span class="Apple-converted-space"> </span>MacroAssembler::call_VM_leaf(<span style="color: rgb(0, 97, 65);" class="">address</span><span class="Apple-converted-space"> </span>entry_point,<span class="Apple-converted-space"> </span><span style="color: rgb(147, 26, 104);" class="">int</span><span class="Apple-converted-space"> </span>number_of_arguments) {<o:p class=""></o:p></span></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 8.5pt; font-family: Monaco, serif;" class="">  call_VM_leaf_base(entry_point, number_of_arguments);<o:p class=""></o:p></span></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 8.5pt; font-family: Monaco, serif;" class="">}<o:p class=""></o:p></span></div></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><o:p class=""> </o:p></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class="">which has:<o:p class=""></o:p></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><o:p class=""> </o:p></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 8.5pt; font-family: Monaco, serif;" class="">  <span style="color: rgb(147, 26, 104);" class="">void</span><span class="Apple-converted-space"> </span>call_VM_leaf(<span style="color: rgb(0, 97, 65);" class="">address</span><span class="Apple-converted-space"> </span>entry_point,<o:p class=""></o:p></span></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 8.5pt; font-family: Monaco, serif;" class="">                   <span class="Apple-converted-space"> </span><span style="color: rgb(147, 26, 104);" class="">int</span><span class="Apple-converted-space"> </span>number_of_arguments = 0);<o:p class=""></o:p></span></div></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><o:p class=""> </o:p></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class="">Get rid of mathfunc completely and use call_VM_leaf directly.<o:p class=""></o:p></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><o:p class=""> </o:p></div></div><blockquote style="margin-top: 5pt; margin-bottom: 5pt;" class=""><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""> </span><o:p class=""></o:p></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">Regards,</span><o:p class=""></o:p></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">Vivek</span><o:p class=""></o:p></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 8.5pt; font-family: Monaco, serif;" class=""> </span><o:p class=""></o:p></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><a name="_MailEndCompose" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""> </span></a><o:p class=""></o:p></div></div><div class=""><div style="border-style: solid none none; border-top-color: rgb(225, 225, 225); border-top-width: 1pt; padding: 3pt 0in 0in;" class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><a name="_____replyseparator" class=""></a><b class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class="">From:</span></b><span class="apple-converted-space"><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class=""> </span></span><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class="">Christian Thalinger [<a href="mailto:christian.thalinger@oracle.com" style="color: purple; text-decoration: underline;" class="">mailto:christian.thalinger@oracle.com</a>]<span class="apple-converted-space"> </span><br class=""><b class="">Sent:</b><span class="apple-converted-space"> </span>Thursday, April 21, 2016 2:35 PM<br class=""><b class="">To:</b><span class="apple-converted-space"> </span>Deshpande, Vivek R <<a href="mailto:vivek.r.deshpande@intel.com" style="color: purple; text-decoration: underline;" class="">vivek.r.deshpande@intel.com</a>><br class=""><b class="">Cc:</b><span class="apple-converted-space"> </span>Nils Eliasson <<a href="mailto:nils.eliasson@oracle.com" style="color: purple; text-decoration: underline;" class="">nils.eliasson@oracle.com</a>>;<span class="Apple-converted-space"> </span><a href="mailto:hotspot-compiler-dev@openjdk.java.net" style="color: purple; text-decoration: underline;" class="">hotspot-compiler-dev@openjdk.java.net</a>; Vladimir Kozlov <<a href="mailto:vladimir.kozlov@oracle.com" style="color: purple; text-decoration: underline;" class="">vladimir.kozlov@oracle.com</a>><br class=""><b class="">Subject:</b><span class="apple-converted-space"> </span>Re: RFR (S): 8154473: Update for CompilerDirectives to control stub generation and intrinsics</span><o:p class=""></o:p></div></div></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""> <o:p class=""></o:p></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""> <o:p class=""></o:p></div></div><div class=""><blockquote style="margin-top: 5pt; margin-bottom: 5pt;" class=""><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class="">On Apr 20, 2016, at 2:13 PM, Deshpande, Vivek R <<a href="mailto:vivek.r.deshpande@intel.com" style="color: purple; text-decoration: underline;" class=""><span style="color: purple;" class="">vivek.r.deshpande@intel.com</span></a>> wrote:<o:p class=""></o:p></div></div></div><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""> <o:p class=""></o:p></div></div><div class=""><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">Hi</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""> </span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">The correct URL for the updated webrev is this.</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""><a href="http://cr.openjdk.java.net/~vdeshpande/CompilerDirectives/8154473/webrev.02/" style="color: purple; text-decoration: underline;" class=""><span style="color: purple;" class="">http://cr.openjdk.java.net/~vdeshpande/CompilerDirectives/8154473/webrev.02/</span></a></span><o:p class=""></o:p></div></div></div></div></blockquote><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""> <o:p class=""></o:p></div></div></div><pre style="margin: 0in 0in 0.0001pt; font-size: 10pt; font-family: 'Courier New'; background-color: rgb(238, 238, 238); background-position: initial initial; background-repeat: initial initial;" class=""><span class="new"><span style="color: blue;" class="">+void MacroAssembler::mathfunc(address runtime_entry) {</span></span><o:p class=""></o:p></pre><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class="">I don’t like the name of this method.  Mainly because it’s only aligning the stack (shouldn’t that happen somewhere else?) and doing this 0x20 stack frame thing which I still don’t understand.<o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""> <o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class="">Right, this is the one I was thinking about:<o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""> <o:p class=""></o:p></div></div></div><div class=""><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><span style="font-size: 8.5pt; font-family: Monaco, serif; color: rgb(147, 26, 104);" class="">void</span><span class="apple-converted-space"><span style="font-size: 8.5pt; font-family: Monaco, serif;" class=""> </span></span><span style="font-size: 8.5pt; font-family: Monaco, serif;" class="">MacroAssembler::call_VM_leaf_base(<span style="color: rgb(0, 97, 65);" class="">address</span><span class="apple-converted-space"> </span>entry_point,<span class="apple-converted-space"> </span><span style="color: rgb(147, 26, 104);" class="">int</span><span class="apple-converted-space"> </span>num_args) {</span><o:p class=""></o:p></div></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif;" class=""><br class=""><br class=""><br class=""><o:p class=""></o:p></div></div><blockquote style="margin-top: 5pt; margin-bottom: 5pt;" class=""><div class=""><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">Sorry for the spam.</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""> </span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">Regards,</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">Vivek</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""> </span><o:p class=""></o:p></div></div></div><div class=""><div style="border-style: solid none none; border-top-color: rgb(181, 196, 223); border-top-width: 1pt; padding: 3pt 0in 0in;" class=""><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><b class=""><span style="font-size: 10pt; font-family: Tahoma, sans-serif;" class="">From:</span></b><span class="apple-converted-space"><span style="font-size: 10pt; font-family: Tahoma, sans-serif;" class=""> </span></span><span style="font-size: 10pt; font-family: Tahoma, sans-serif;" class="">Deshpande, Vivek R<span class="apple-converted-space"> </span><br class=""><b class="">Sent:</b><span class="apple-converted-space"> </span>Wednesday, April 20, 2016 5:10 PM<br class=""><b class="">To:</b><span class="apple-converted-space"> </span>Deshpande, Vivek R; 'Nils Eliasson'; '<a href="mailto:hotspot-compiler-dev@openjdk.java.net" style="color: purple; text-decoration: underline;" class=""><span style="color: purple;" class="">hotspot-compiler-dev@openjdk.java.net</span></a>'<br class=""><b class="">Cc:</b><span class="apple-converted-space"> </span>'Vladimir Kozlov'; 'Volker Simonis'; 'Christian Thalinger'; Viswanathan, Sandhya<br class=""><b class="">Subject:</b><span class="apple-converted-space"> </span>RE: RFR (S): 8154473: Update for CompilerDirectives to control stub generation and intrinsics</span><o:p class=""></o:p></div></div></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class=""> </span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">Sent out the wrong link by mistake.</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""> </span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 9.5pt; font-family: Arial, sans-serif; color: rgb(51, 51, 51); background-color: rgb(245, 245, 245); background-position: initial initial; background-repeat: initial initial;" class="">updated webrev:<span class="apple-converted-space"> </span></span><span style="font-size: 9.5pt; font-family: Arial, sans-serif; color: rgb(51, 51, 51);" class=""><br class=""></span><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class=""><a href="http://cr.openjdk.java.net/~vdeshpande/CompilerDirectives/8154473/webrev.01/" style="color: purple; text-decoration: underline;" class=""><span style="font-size: 9.5pt; font-family: Arial, sans-serif; color: rgb(59, 115, 175); background-color: rgb(245, 245, 245); text-decoration: none; background-position: initial initial; background-repeat: initial initial;" class="">http://cr.openjdk.java.net/~vdeshpande/CompilerDirectives/8154473/webrev.02/</span></a></span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class=""> </span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class="">Regards</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class="">Vivek</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""> </span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""> </span><o:p class=""></o:p></div></div></div><div class=""><div style="border-style: solid none none; border-top-color: rgb(181, 196, 223); border-top-width: 1pt; padding: 3pt 0in 0in;" class=""><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><b class=""><span style="font-size: 10pt; font-family: Tahoma, sans-serif;" class="">From:</span></b><span class="apple-converted-space"><span style="font-size: 10pt; font-family: Tahoma, sans-serif;" class=""> </span></span><span style="font-size: 10pt; font-family: Tahoma, sans-serif;" class="">Deshpande, Vivek R<span class="apple-converted-space"> </span><br class=""><b class="">Sent:</b><span class="apple-converted-space"> </span>Wednesday, April 20, 2016 5:07 PM<br class=""><b class="">To:</b><span class="apple-converted-space"> </span>'Nils Eliasson';<span class="apple-converted-space"> </span><a href="mailto:hotspot-compiler-dev@openjdk.java.net" style="color: purple; text-decoration: underline;" class=""><span style="color: purple;" class="">hotspot-compiler-dev@openjdk.java.net</span></a><br class=""><b class="">Cc:</b><span class="apple-converted-space"> </span>Vladimir Kozlov; Volker Simonis; Christian Thalinger; Viswanathan, Sandhya<br class=""><b class="">Subject:</b><span class="apple-converted-space"> </span>RE: RFR (S): 8154473: Update for CompilerDirectives to control stub generation and intrinsics</span><o:p class=""></o:p></div></div></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class=""> </span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">Hi Nils</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""> </span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">I have updated the webrev with all the suggestions.</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 9.5pt; font-family: Arial, sans-serif; color: rgb(51, 51, 51); background-color: rgb(245, 245, 245); background-position: initial initial; background-repeat: initial initial;" class="">updated webrev:<span class="apple-converted-space"> </span></span><span style="font-size: 9.5pt; font-family: Arial, sans-serif; color: rgb(51, 51, 51);" class=""><br class=""></span><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class=""><a href="http://cr.openjdk.java.net/~vdeshpande/CompilerDirectives/8154473/webrev.01/" style="color: purple; text-decoration: underline;" class=""><span style="font-size: 9.5pt; font-family: Arial, sans-serif; color: rgb(59, 115, 175); background-color: rgb(245, 245, 245); text-decoration: none; background-position: initial initial; background-repeat: initial initial;" class="">http://cr.openjdk.java.net/~vdeshpande/CompilerDirectives/8154473/webrev.01/</span></a></span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class="">Thanks for your comments and review.</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class=""> </span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class="">@Vladimir,</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class="">I have taken care of all the comments. Would you please review and sponsor the patch.</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class=""> </span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class="">Thanks and regards,</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class="">Vivek<span class="apple-converted-space"> </span></span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""> </span><o:p class=""></o:p></div></div></div><div class=""><div style="border-style: solid none none; border-top-color: rgb(181, 196, 223); border-top-width: 1pt; padding: 3pt 0in 0in;" class=""><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><b class=""><span style="font-size: 10pt; font-family: Tahoma, sans-serif;" class="">From:</span></b><span class="apple-converted-space"><span style="font-size: 10pt; font-family: Tahoma, sans-serif;" class=""> </span></span><span style="font-size: 10pt; font-family: Tahoma, sans-serif;" class="">Nils Eliasson [<a href="mailto:nils.eliasson@oracle.com" style="color: purple; text-decoration: underline;" class=""><span style="color: purple;" class="">mailto:nils.eliasson@oracle.com</span></a>]<span class="apple-converted-space"> </span><br class=""><b class="">Sent:</b><span class="apple-converted-space"> </span>Wednesday, April 20, 2016 12:27 PM<br class=""><b class="">To:</b><span class="apple-converted-space"> </span>Deshpande, Vivek R;<span class="apple-converted-space"> </span><a href="mailto:hotspot-compiler-dev@openjdk.java.net" style="color: purple; text-decoration: underline;" class=""><span style="color: purple;" class="">hotspot-compiler-dev@openjdk.java.net</span></a><br class=""><b class="">Cc:</b><span class="apple-converted-space"> </span>Vladimir Kozlov; Volker Simonis; Christian Thalinger; Viswanathan, Sandhya<br class=""><b class="">Subject:</b><span class="apple-converted-space"> </span>Re: RFR (S): 8154473: Update for CompilerDirectives to control stub generation and intrinsics</span><o:p class=""></o:p></div></div></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class=""> </span><o:p class=""></o:p></div></div></div><p class="MsoNormal" style="margin: 0in 0in 12pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;"><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class="">In vmSymbols.cpp together with the other flag checks.<br class=""><br class="">Regards,<br class="">Nils</span><o:p class=""></o:p></p><div class=""><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class="">On 2016-04-20 02:44, Deshpande, Vivek R wrote:</span><o:p class=""></o:p></div></div></div></div><blockquote style="margin-top: 5pt; margin-bottom: 5pt;" class=""><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">HI Nils</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""> </span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">Yes you are right the function accesses the command line flag DisableIntrinsic and changes are static.</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">Could you point me the right location for the function ?</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">Also I have updated the webrev with rest of the comments here:</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""><a href="http://cr.openjdk.java.net/~vdeshpande/CompilerDirectives/8154473/webrev.01/" style="color: purple; text-decoration: underline;" class=""><span style="color: purple;" class="">http://cr.openjdk.java.net/~vdeshpande/CompilerDirectives/8154473/webrev.01/</span></a></span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""> </span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">Regards,</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">Vivek</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""> </span><o:p class=""></o:p></div></div></div><div class=""><div style="border-style: solid none none; border-top-color: rgb(181, 196, 223); border-top-width: 1pt; padding: 3pt 0in 0in;" class=""><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><b class=""><span style="font-size: 10pt; font-family: Tahoma, sans-serif;" class="">From:</span></b><span class="apple-converted-space"><span style="font-size: 10pt; font-family: Tahoma, sans-serif;" class=""> </span></span><span style="font-size: 10pt; font-family: Tahoma, sans-serif;" class="">hotspot-compiler-dev [<a href="mailto:hotspot-compiler-dev-bounces@openjdk.java.net" style="color: purple; text-decoration: underline;" class=""><span style="color: purple;" class="">mailto:hotspot-compiler-dev-bounces@openjdk.java.net</span></a>]<b class="">On Behalf Of<span class="apple-converted-space"> </span></b>Nils Eliasson<br class=""><b class="">Sent:</b><span class="apple-converted-space"> </span>Tuesday, April 19, 2016 5:55 AM<br class=""><b class="">To:</b><span class="apple-converted-space"> </span><a href="mailto:hotspot-compiler-dev@openjdk.java.net" style="color: purple; text-decoration: underline;" class=""><span style="color: purple;" class="">hotspot-compiler-dev@openjdk.java.net</span></a><br class=""><b class="">Subject:</b><span class="apple-converted-space"> </span>Re: RFR (S): 8154473: Update for CompilerDirectives to control stub generation and intrinsics</span><o:p class=""></o:p></div></div></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class=""> </span><o:p class=""></o:p></div></div></div><p class="MsoNormal" style="margin: 0in 0in 12pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;"><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class="">Hi Vivek,<br class=""><br class="">The changes in is_intrinsic_disabled in compilerDirectives.* are static and only access the command line flag DisableIntrinsics. As long as stubs are only generated during startup and don't have a method context - that is ok - but it doesn't belong in the compilerDirectives-files if it doens't use directives. <span class="apple-converted-space"> </span><br class=""><br class="">Regards,<br class="">Nils</span><o:p class=""></o:p></p><div class=""><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif;" class="">On 2016-04-18 19:38, Deshpande, Vivek R wrote:</span><o:p class=""></o:p></div></div></div></div><blockquote style="margin-top: 5pt; margin-bottom: 5pt;" class=""><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">Hi all</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""> </span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">I would like to contribute a patch which helps to control<span class="apple-converted-space"> </span><span class="il">the intrinsics in interpreter, c1 and c2 by disabling the stub generation.</span></span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span class="il"><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">This uses -XX:DisableIntrinsic option to achieve the same.</span></span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">Could you please review and sponsor this patch.</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""> </span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class="">Bug-id: </span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""><a href="https://bugs.openjdk.java.net/browse/JDK-8154473" style="color: purple; text-decoration: underline;" class=""><span style="color: purple;" class="">https://bugs.openjdk.java.net/browse/JDK-8154473</span></a><br class="">webrev:</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125);" class=""><a href="http://cr.openjdk.java.net/%7Evdeshpande/CompilerDirectives/8154473/webrev.00/" style="color: purple; text-decoration: underline;" class=""><span style="color: purple;" class="">http://cr.openjdk.java.net/~vdeshpande/CompilerDirectives/8154473/webrev.00/</span></a></span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(34, 34, 34);" class=""> </span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125); background-color: white; background-position: initial initial; background-repeat: initial initial;" class="">Thanks and regards,</span><o:p class=""></o:p></div></div></div><div class=""><div class=""><div style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; background-color: white;" class=""><span style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, 125); background-color: white;" class="">Vivek</span></div></div></div></blockquote></blockquote></div></blockquote></div></div></blockquote></div></div></div></blockquote></div><br class=""></body></html>