[aarch64-port-dev ] RFR(M): 8196402: AARCH64: create intrinsic for Math.log
Dmitrij Pochepko
dmitrij.pochepko at bell-sw.com
Mon Jun 18 18:53:14 UTC 2018
Hi,
I added lots of documentation in a same manner as in Math::sin/cos patch.
Please take a look at updated webrev:
http://cr.openjdk.java.net/~dpochepk/8196402/webrev.05/
Thanks,
Dmitrij
On 04.06.2018 18:50, Dmitrij Pochepko wrote:
> On 04.06.2018 16:09, Dmitrij Pochepko wrote:
>> On 04.06.2018 15:53, Andrew Haley wrote:
>>> On 06/03/2018 03:03 PM, Dmitry Samersoff wrote:
>>>> This version of the fix looks good to me.
>>> Note: the same applies to log as cost and sin. Please make sure that
>>> the intrinsic is used for interp, C1, and C2.
>>>
>> Yes. Sin/cos change to enable intrinsics in intrp and C1 do it for
>> all math intrinsics. I'm currently testing it
> Please take a look at updated webrev. The only change is move log stub
> generation call from generate_all() to generate_initial(). It will be
> required for interpreter to use initialized stub routine address after
> JDK-8204289 is pushed
>
> http://cr.openjdk.java.net/~dpochepk/8196402/webrev.04/
>
> Thanks,
> Dmitrij
>
>
More information about the aarch64-port-dev
mailing list