[10] RFR: 8189176 - AARCH64: Improve _updateBytesCRC32 intrinsic
Dmitry Chuyko
dmitry.chuyko at bell-sw.com
Wed Nov 1 16:24:55 UTC 2017
As per Derek's suggestion I made private
MacroAssembler::kernel_crc32_using_crc32(). Register naming became more
clear, numeric results are the same.
webrev: http://cr.openjdk.java.net/~dchuyko/8189176/webrev.01/
-Dmitry
On 10/28/2017 10:52 AM, Andrew Haley wrote:
> On 27/10/17 23:31, White, Derek wrote:
>> - The use of temp registers in the UseCRC32 case is kind of muddled, using tmp, and table0..table3 as temp registers, and the name "table" is confusing in this case.
>> - Maybe it would be cleaner to refactor the UseCRC32 code into a separate kernel_crc32_using_crc32() subroutine (static or macro?). This would accept the main args and 4 registers for temps. The caller can supply some combination of table or tmp registers.
>> - This would shrink the size of kernel_crc32() by a lot too.
> That would be nice.
>
More information about the hotspot-compiler-dev
mailing list