<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">I left this comment in the bug:<div class=""><br class=""></div><div class=""><span style="color: rgb(51, 51, 51); font-family: sans-serif; font-size: 14px; background-color: rgb(245, 245, 245);" class="">I think for the saneness of the macroAssembler_libm_x86_*.cpp files we should put every intrinsic in its own file, like we did for macroAssembler_x86_sha.cpp. They are already too big: </span><br style="color: rgb(51, 51, 51); font-family: sans-serif; font-size: 14px;" class=""><br style="color: rgb(51, 51, 51); font-family: sans-serif; font-size: 14px;" class=""><span style="color: rgb(51, 51, 51); font-family: sans-serif; font-size: 14px; background-color: rgb(245, 245, 245);" class="">$ wc -l hotspot/src/cpu/x86/vm/macroAssembler_libm_x86_*.cpp </span><br style="color: rgb(51, 51, 51); font-family: sans-serif; font-size: 14px;" class=""><span style="color: rgb(51, 51, 51); font-family: sans-serif; font-size: 14px; background-color: rgb(245, 245, 245);" class="">    4571 hotspot/src/cpu/x86/vm/macroAssembler_libm_x86_32.cpp </span><br style="color: rgb(51, 51, 51); font-family: sans-serif; font-size: 14px;" class=""><span style="color: rgb(51, 51, 51); font-family: sans-serif; font-size: 14px; background-color: rgb(245, 245, 245);" class="">    3945 hotspot/src/cpu/x86/vm/macroAssembler_libm_x86_64.cpp </span></div><div class=""><br class=""></div><div class=""><div>Also, can we split out the CompilerDirectives changes?</div><div><br class=""><blockquote type="cite" class=""><div class="">On Mar 28, 2016, at 1:52 PM, 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: 11pt; font-family: Calibri, sans-serif; background-color: white;" class=""><span style="color: rgb(31, 73, 125);" class="">Hi all<o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif; background-color: white; orphans: auto; text-align: start; widows: 1; -webkit-text-stroke-width: 0px; word-spacing: 0px;" class=""><span style="color: rgb(31, 73, 125);" class=""> <o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif; background-color: white; orphans: auto; text-align: start; widows: 1; -webkit-text-stroke-width: 0px; word-spacing: 0px;" class=""><span style="color: rgb(31, 73, 125);" class="">We would like to contribute a patch which optimizes<span class="apple-converted-space"> </span><span class="il">tan and log10 X86</span><span class="apple-converted-space"> </span>architecture using<span class="Apple-converted-space"> </span><span style="background-color: white; background-position: initial initial; background-repeat: initial initial;" class="">Intel LIBM library.</span><o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif; background-color: white; orphans: auto; text-align: start; widows: 1; -webkit-text-stroke-width: 0px; word-spacing: 0px;" class=""><span style="color: rgb(31, 73, 125);" class="">Could you please review and sponsor this patch.<o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif; background-color: white; orphans: auto; text-align: start; widows: 1; -webkit-text-stroke-width: 0px; word-spacing: 0px;" class=""><span style="color: rgb(31, 73, 125);" class=""> <o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif; background-color: white; orphans: auto; text-align: start; widows: 1; -webkit-text-stroke-width: 0px; word-spacing: 0px;" class=""><span style="color: rgb(31, 73, 125);" class="">Bug-id:<o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif; background-color: white;" class=""><span style="color: rgb(31, 73, 125);" class=""><a href="https://bugs.openjdk.java.net/browse/JDK-8152907" style="color: purple; text-decoration: underline;" class="">https://bugs.openjdk.java.net/browse/JDK-8152907</a><br class="">webrev:<o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif; background-color: white;" class=""><span style="color: rgb(31, 73, 125);" class=""><a href="http://cr.openjdk.java.net/~vdeshpande/libm_tanlog10/8152907/webrev.00/" style="color: purple; text-decoration: underline;" class="">http://cr.openjdk.java.net/~vdeshpande/libm_tanlog10/8152907/webrev.00/</a><o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif; background-color: white;" class=""><span style="color: rgb(31, 73, 125);" class=""><o:p class=""> </o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif; background-color: white;" class=""><span style="color: rgb(31, 73, 125); background-color: white; background-position: initial initial; background-repeat: initial initial;" class="">Thanks and regards,</span><span style="color: rgb(31, 73, 125);" class=""><o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif; background-color: white; orphans: auto; text-align: start; widows: 1; -webkit-text-stroke-width: 0px; word-spacing: 0px;" class=""><span style="color: rgb(31, 73, 125); background-color: white; background-position: initial initial; background-repeat: initial initial;" class="">Vivek</span></div></div></div></blockquote></div><br class=""></div></body></html>