Hi Martin, Gustavo,

> most of our tests (including a subset of JCK) were running over night and I couldn’t see any problems related to this change.
Thank you so much for long-running tests.

>Pushed:
>http://hg.openjdk.java.net/jdk/jdk/rev/1c85328b7631
Thank you, Gustavo!


Best regards,
--
Michihiro,
IBM Research - Tokyo

Inactive hide details for "Gustavo Romero" ---2018/12/13 21:29:32---Hi Michi, Martin On 12/13/2018 07:24 AM, Doerr, Martin wrot"Gustavo Romero" ---2018/12/13 21:29:32---Hi Michi, Martin On 12/13/2018 07:24 AM, Doerr, Martin wrote:

From: "Gustavo Romero" <gromero@linux.vnet.ibm.com>
To: "Doerr, Martin" <martin.doerr@sap.com>, Michihiro Horie/Japan/IBM@IBMJP
Cc: "hotspot compiler" <hotspot-compiler-dev@openjdk.java.net>, "ppc-aix-port-dev@openjdk.java.net" <ppc-aix-port-dev@openjdk.java.net>, "Simonis, Volker" <volker.simonis@sap.com>
Date: 2018/12/13 21:29
Subject: Re: RFR: 8215262: PPC64: FMA Vectorization on PPC64




Hi Michi, Martin

On 12/13/2018 07:24 AM, Doerr, Martin wrote:
> most of our tests (including a subset of JCK) were running over night and I couldn’t see any problems related to this change.
>
> So I think you can push it if you want to get it in jdk12.

Pushed:
http://hg.openjdk.java.net/jdk/jdk/rev/1c85328b7631

Thank you.

Best regards,
Gustavo
 
> Best regards,
>
> Martin
>
> *From:*Michihiro Horie <HORIE@jp.ibm.com>
> *Sent:* Donnerstag, 13. Dezember 2018 00:05
> *To:* Gustavo Romero <gromero@linux.vnet.ibm.com>
> *Cc:* hotspot compiler <hotspot-compiler-dev@openjdk.java.net>; Doerr, Martin <martin.doerr@sap.com>; ppc-aix-port-dev@openjdk.java.net; Simonis, Volker <volker.simonis@sap.com>
> *Subject:* Re: RFR: 8215262: PPC64: FMA Vectorization on PPC64
>
> Hi Gustavo,
>
> Thanks a lot. It's great to have ~80% reduction on POWER9 Witherspoon.
>
>
> Best regards,
> --
> Michihiro,
> IBM Research - Tokyo
>
> Inactive hide details for "Gustavo Romero" ---2018/12/13 07:39:15---Hi Michi and Martin, On 12/12/2018 01:53 PM, Doerr, Martin "Gustavo Romero" ---2018/12/13 07:39:15---Hi Michi and Martin, On 12/12/2018 01:53 PM, Doerr, Martin wrote:
>
> From: "Gustavo Romero" <gromero@linux.vnet.ibm.com <mailto:gromero@linux.vnet.ibm.com>>
> To: "Doerr, Martin" <martin.doerr@sap.com <mailto:martin.doerr@sap.com>>, Michihiro Horie/Japan/IBM@IBMJP, "ppc-aix-port-dev@openjdk.java.net <mailto:ppc-aix-port-dev@openjdk.java.net>" <ppc-aix-port-dev@openjdk.java.net <mailto:ppc-aix-port-dev@openjdk.java.net>>, "hotspot compiler" <hotspot-compiler-dev@openjdk.java.net <mailto:hotspot-compiler-dev@openjdk.java.net>>
> Cc: "Simonis, Volker" <volker.simonis@sap.com <mailto:volker.simonis@sap.com>>
> Date: 2018/12/13 07:39
> Subject: Re: 8215262: PPC64: FMA Vectorization on PPC64
>
> ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
>
>
>
>
> Hi Michi and Martin,
>
> On 12/12/2018 01:53 PM, Doerr, Martin wrote:
>> this looks good to me. We should be able to test it since it is for Power8. Maybe Gustavo would like to test in addition.
>
> Michi, the change looks good to me too.
> I've tested it on POWER8 and on POWER9 and observed a reduction of ~30% on my
> POWER8, but on my POWER9 (Witherspoon) it was ~80%, so it can be even better on
> a Boston (POWER9 too) machine. Thus, thumbs up from my side.
>
> I can sponsor that change but I'll wait an ack. from SAP that it's ok for
> pushing. Maybe Martin wants to perform additional tests after his first review
> or others might want to review too...  I'm wondering if it would be reasonable
> to run JCK/TCK against it. Would it test Math.fma() since it's a 9+ method and
> needs a warmed up JVM? Unfortunately I can't run the JCK/TCK against it myself
> if it applies here.
>
> Thank you.
>
> Best regards,
> Gustavo
>
>> Best regards,
>>
>> Martin
>>
>> *From:*Michihiro Horie <HORIE@jp.ibm.com <mailto:HORIE@jp.ibm.com>>
>> *Sent:* Mittwoch, 12. Dezember 2018 15:50
>> *To:* ppc-aix-port-dev@openjdk.java.net <mailto:ppc-aix-port-dev@openjdk.java.net>; hotspot compiler <hotspot-compiler-dev@openjdk.java.net <mailto:hotspot-compiler-dev@openjdk.java.net>>
>> *Cc:* Gustavo Romero <gromero@linux.vnet.ibm.com <mailto:gromero@linux.vnet.ibm.com>>; Doerr, Martin <martin.doerr@sap.com <mailto:martin.doerr@sap.com>>; Simonis, Volker <volker.simonis@sap.com <mailto:volker.simonis@sap.com>>
>> *Subject:* RFR: 8215262: PPC64: FMA Vectorization on PPC64
>>
>> Dear all,
>>
>> Would you please review the following change?
>> This change supports FMA vectorization on PPC64.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8215262
>> Webrev: http://cr.openjdk.java.net/~mhorie/8215262/webrev.00/ <http://cr.openjdk.java.net/%7Emhorie/8215262/webrev.00/> <http://cr.openjdk.java.net/%7Emhorie/8215262/webrev.00/>
>>
>> Microbenchmark I used is http://cr.openjdk.java.net/~mhorie/8215262/VFma.java <http://cr.openjdk.java.net/%7Emhorie/8215262/VFma.java> <http://cr.openjdk.java.net/%7Emhorie/8215262/VFma.java>
>> It showed ~50% reduction in the execution time.
>>
>> I also tested with JTREG and there was no difference in the results compared to the original.
>>
>>
>> Best regards,
>> --
>> Michihiro,
>> IBM Research - Tokyo
>>
>
>