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

Rukmannagari, Shravya shravya.rukmannagari at intel.com
Tue Mar 29 00:58:35 UTC 2016

Hi Christian,
We would add separate files for each intrinsic. By splitting the CompilerDirectives, do you mean we have to add a separate file. Sorry I didn't exactly get it.

Shravya Rukmannagari.

From: Christian Thalinger [mailto:christian.thalinger at oracle.com]
Sent: Monday, March 28, 2016 5:18 PM
To: Deshpande, Vivek R <vivek.r.deshpande at intel.com>
Cc: hotspot compiler <hotspot-compiler-dev at openjdk.java.net>; Vladimir Kozlov <vladimir.kozlov at oracle.com>; Rukmannagari, Shravya <shravya.rukmannagari at intel.com>
Subject: Re: RFR (M): 8152907: Update for tan and log10 for x86

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<mailto: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.


Thanks and regards,

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

More information about the hotspot-compiler-dev mailing list