[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-hotspot-dev
Subject:    Re: [aarch64-port-dev ] RFR: 8129426: aarch64: add support for PopCount in C2
From:       Alejandro E Murillo <alejandro.murillo () oracle ! com>
Date:       2015-06-23 21:11:15
Message-ID: 5589CB73.7020509 () oracle ! com
[Download RAW message or body]

On 6/23/2015 7:51 AM, Edward Nevill wrote:
> On Tue, 2015-06-23 at 11:32 +0100, Andrew Dinn wrote:
>> On 23/06/15 11:10, Andrew Haley wrote:
>>> On 22/06/15 17:14, Andrew Dinn wrote:
>>>> . . .
>>>> Well, you might not want to take this risk and instead add an explicit
>>>> zero of the upper half. But I think we need to be clear what risk we are
>>>> taking.
>>> It's this: if we don't explicitly zero the upper half we'll have to
>>> audit all the code which might present a sign-extended value (instead
>>> of a zero-extended one) in a register that's supposed to contain a
>>> jint.
>> Ok, let's play safe. If Ed tweaks the patch to zero the upper word we
>> can always revise that later if/when we decide we are feeling lucky.
>>
> OK. New webrev at
>
> http://cr.openjdk.java.net/~enevill/8129426/webrev.02/
>
> All the best,
> Ed.
>
>
Hi,
to be consistent with similar integrations and to avoid potential 
merging problems,
going forward please work with  the hs-rt repo for this kind of changes,
as Volker has been doing.

I'm working on getting this info on an external wiki

Thanks
Alejandro

-- 
Alejandro

[prev in list] [next in list] [prev in thread] [next in thread]