8026049: (bf) Intrinsify ByteBuffer.put{Int, Double, Float, ...} methods
Andrew Haley
aph at redhat.com
Mon Mar 23 18:31:38 UTC 2015
On 03/21/2015 04:32 PM, Vladimir Kozlov wrote:
> On 3/21/15 4:36 AM, Andrew Haley wrote:
>> On 20/03/15 22:00, Vladimir Kozlov wrote:
>>> On 3/20/15 9:08 AM, Andrew Haley wrote:
>>>> Webrev at
>>>>
>>>> http://cr.openjdk.java.net/~aph/unaligned.hotspot.7/
>>>
>>> I don't see changes to VM's makefiles to set VM_BIG_ENDIAN. Or consider
>>> it is set if VM_LITTLE_ENDIAN is not set in prims/unsafe.cpp code.
>>
>> Darn, I already made that change once but missed it in this webrev. Sorry.
>
> I will wait new webrev.
>
>>
>>> In globals.hpp the empty line after flag description should not have '\'
>>> at the end.
>>
>> OK.
>>
>>> Otherwise it looks good.
>>>
>>>> http://cr.openjdk.java.net/~aph/unaligned.jdk.7/
>>>
>>> My only comment for jdk change is to use bigEndian name instead of BE in
>>> Unsafe.java. Someone from core libs should look on this.
>>
>> I would have used that, but I followed John Rose's suggestion. I
>> don't mind making changes but having to change it back-and-forth
>> really is too much.
>
> John have more authority than me :) so be it BE.
Ok. Sorry for being impatient.
http://cr.openjdk.java.net/~aph/unaligned.hotspot.8/
http://cr.openjdk.java.net/~aph/unaligned.jdk.7/
Andrew.
More information about the core-libs-dev
mailing list