[aarch64-port-dev ] RFR(S): 8210413: AArch64: Optimize div/rem by constant in C1
Yangfei (Felix)
felix.yang at huawei.com
Wed Sep 26 03:01:51 UTC 2018
Just eyeballed the changes, looks good.
Pushed: http://hg.openjdk.java.net/jdk/jdk/rev/e1368526699d
Thanks,
Felix
>
> Thanks for your code review. Could you help push this patch?
>
> --
> Thanks,
> Pengfei
>
>
> > -----Original Message-----
> >
> > On 09/20/2018 05:15 AM, Pengfei Li (Arm Technology China) wrote:
> > > Please find below new patch that added the same optimization for longs as
> > well as ints and also fixed an issue.
> > > http://cr.openjdk.java.net/~yzhang/8210413/webrev.01/
> > >
> > > Could you help look at it again?
> >
> > That's fine. I'm not exactly delighted by the amount of duplicated code for
> > long and int, but it's very hard to avoid in this case.
> > The patch is good for JDK/JDK.
> >
> > --
> > Andrew Haley
> > Java Platform Lead Engineer
> > Red Hat UK Ltd. <https://www.redhat.com>
> > EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671
More information about the aarch64-port-dev
mailing list