[8u20]: Request for approval - 8035394: PPC64: Make usage of intrinsic dsqrt depend on processor recognition.

[Vladimir Kozlov]

Sean,

We thought about requesting approval for pushing directly from 
ppc-aix-port/stage into jdk8u/jdk8u-dev/ because changes were 
extensively tested and also they have jdk changes.

But you may be right. Lets push it, when it is ready (passed all 
internal reviews/evaluations), into Hotspot's repo first jdk8u/hs-dev/. 
Do PIT testing and let Alejandro request usual approval for 8u20. Less 
work for me :)

Thanks,
Vladimir

On 2/24/14 1:49 PM, Seán Coffey wrote:
> Thanks for looking after this Vladimir. Yes - all hotspot related fixes
> should enter jdk8u via the hotspot team forest (hs-dev). Approval for
> those fixes are handled via bulk integration requests by hotspot
> gatekeeper.
>
> regards,
> Sean.
>
> On 24/02/14 19:59, Vladimir Kozlov wrote:
>> Goetz,
>>
>> I don't think we need to get approval to push into ppcaix stage repo.
>> We will prepare combined request for approval when we are ready to
>> push from ppc-aix stage repo into jdk8u-dev.
>> Thing about this as for other hotspot changes. We push reviewed
>> changeset into 8u hotspot repo and after PIT is done Alejandro is
>> sending approval request for all changes.
>>
>> I will push both these changes into ppc-aix-port/stage/hotspot later.
>>
>> Regards,
>> Vladimir
>>
>> On 2/24/14 2:18 AM, Lindenmaier, Goetz wrote:
>>> Hi,
>>>
>>> please review this change for backport to 8u via ppc-aix-port/stage.
>>>
>>> Only some PPC processors have a sqrt instruction.
>>> Currently, intrinsic for dsqrt is used if a corresponding rule is
>>> implemented in the .ad file (has_match_rule()).
>>> Instead, call match_ruole_supported(), which allows to check
>>> processor properties.
>>> Also adapt PPC64 .ad file to this.
>>>
>>> changeset: http://hg.openjdk.java.net/jdk9/hs/hotspot/rev/0f19095fd8c1
>>> bug:   https://bugs.openjdk.java.net/browse/JDK-8035394
>>> review:
>>> http://mail.openjdk.java.net/pipermail/hotspot-dev/2014-February/012642.html
>>>
>>>
>>> original webrev:
>>> http://cr.openjdk.java.net/~goetz/webrevs/8035394-1-fltRck/
>>>
>>> Thanks and best regards,
>>> Goetz.
>>>
>