RFR (M): 8152907: Update for tan and log10 for x86

Christian Thalinger christian.thalinger at oracle.com
Tue Mar 29 00:17:37 UTC 2016


I left this comment in the bug:

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: 

$ wc -l hotspot/src/cpu/x86/vm/macroAssembler_libm_x86_*.cpp 
    4571 hotspot/src/cpu/x86/vm/macroAssembler_libm_x86_32.cpp 
    3945 hotspot/src/cpu/x86/vm/macroAssembler_libm_x86_64.cpp 

Also, can we split out the CompilerDirectives changes?

> On Mar 28, 2016, at 1:52 PM, Deshpande, Vivek R <vivek.r.deshpande at intel.com> wrote:
> 
> Hi all
>  
> We would like to contribute a patch which optimizes tan and log10 X86 architecture using Intel LIBM library.
> Could you please review and sponsor this patch.
>  
> Bug-id:
> https://bugs.openjdk.java.net/browse/JDK-8152907 <https://bugs.openjdk.java.net/browse/JDK-8152907>
> webrev:
> http://cr.openjdk.java.net/~vdeshpande/libm_tanlog10/8152907/webrev.00/ <http://cr.openjdk.java.net/~vdeshpande/libm_tanlog10/8152907/webrev.00/>
>  
> Thanks and regards,
> Vivek

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20160328/daedf46b/attachment-0001.html>


More information about the hotspot-compiler-dev mailing list