[10] RFR(S): 8187684 - Intrinsify Math.multiplyHigh(long, long)

[Schmidt, Lutz]

Hi Dmitrij,
Your change looks good.
It works for my multiplyHigh implementation on s390 and ppc (not yet RFR’ed, delayed until your change is in).
Regards, 
Lutz 

 

On 02.10.2017, 15:47, "hotspot-compiler-dev on behalf of Dmitrij Pochepko" <hotspot-compiler-dev-bounces at openjdk.java.net on behalf of dmitrij.pochepko at bell-sw.com> wrote:

    Hi,
    
    please find rebased webrev here: 
    http://cr.openjdk.java.net/~dpochepk/8187684/webrev.newws.01/
    
    
    Thanks,
    
    Dmitij
    
    
    On 29.09.2017 02:40, Vladimir Kozlov wrote:
    > Dmitry,
    >
    > Please, update changes for new consolidated sources and send new 
    > patch/webrev.
    >
    > Thanks,
    > Vladimir
    >
    > On 9/25/17 9:42 AM, Vladimir Kozlov wrote:
    >> Yes, when repo will be opened.
    >>
    >> Please, send patch and add latest webrev link to the RFE.
    >>
    >> Thanks,
    >> Vladimir
    >>
    >> On 9/25/17 5:04 AM, Dmitrij Pochepko wrote:
    >>>
    >>> On 25.09.2017 14:04, Andrew Haley wrote:
    >>>> On 20/09/17 14:29, Andrew Haley wrote:
    >>>>> On 20/09/17 14:08, Dmitrij Pochepko wrote:
    >>>>>> please review small patch for enhancement: 8187684 - Intrinsify
    >>>>>> Math.multiplyHigh(long, long)
    >>>>> OK, thanks.
    >>>> Dmitrij, do you have a sponsor for this?  I'm sure Vladimir would
    >>>> be happy to help.  :-)
    >>>>
    >>> Hi,
    >>>
    >>> Vladimir, can you sponsor it?
    >>>
    >>> Thanks,
    >>> Dmitrij