[10] RFR (S): 8189177 - AARCH64: Improve _updateBytesCRC32C intrinsic

Dmitry Chuyko dmitry.chuyko at bell-sw.com
Fri Oct 20 17:45:47 UTC 2017


Hello,

Please review an improvement of CRC32C calculation on AArch64. It is 
done pretty similar to a change for JDK-8189176 described in [1].

MacroAssembler::kernel_crc32c gets unused table registers. They can be 
used to make neighbor loads and CRC calculations independent. Adding 
prologue and epilogue for main by-64 loop makes it applicable starting 
from len=128 so additional by-32 loop is added for smaller lengths.

rfe: https://bugs.openjdk.java.net/browse/JDK-8189177
webrev: http://cr.openjdk.java.net/~dchuyko/8189177/webrev.00/
benchmark: 
http://cr.openjdk.java.net/~dchuyko/8189177/crc32c/CRC32CBench.java

Results for T88 and A53 [2] are similar to CRC32 change (good), but 
again splitting pair loads may slow down other CPUs so measurements on 
different HW are welcome.

-Dmitry

[1] 
http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-October/027225.html
[2] 
https://bugs.openjdk.java.net/browse/JDK-8189177?focusedCommentId=14124535&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14124535



More information about the hotspot-compiler-dev mailing list