RFR 8167065 - Adding support for double precision shifting for x86

[Paul Sandoz]

Hi,

> On 17 Oct 2016, at 11:09, Lupusoru, Razvan A <razvan.a.lupusoru at intel.com> wrote:
> 
> Paul,
> 
> Thanks again! I have implemented your suggestion and it definitely cleaned up the code a bit. The updated patch is available at http://cr.openjdk.java.net/~mcberg/8167065/jdk/webrev.03/
> 
> Can I please get another consideration of this patch? Thanks again!
> 

Previously you had in v02

3350             int numIter = magLen - nInts - 1;

and now it is in v03:

3267             int numIter = magLen - 1;

Which one is correct?

Paul.


> --Razvan
> 
> -----Original Message-----
> From: hotspot-compiler-dev [mailto:hotspot-compiler-dev-bounces at openjdk.java.net] On Behalf Of Lupusoru, Razvan A
> Sent: Friday, October 14, 2016 4:15 PM
> To: Paul Sandoz <paul.sandoz at oracle.com>
> Cc: Vladimir Kozlov <vladimir.kozlov at oracle.com>; Viswanathan, Sandhya <sandhya.viswanathan at intel.com>; hotspot-compiler-dev at openjdk.java.net
> Subject: RE: RFR 8167065 - Adding support for double precision shifting for x86
> 
> Paul,
> 
> Great suggestion! I will look into making the update and let you know if it works. Thanks for informing me about this option.
> 
> —
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 841 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20161017/1d319506/signature.asc>