Hi,
Thanks a lot for the reviews! I think I have addressed the review
comments from Andrew, Vladimir and Erik. This is the new webrev:
Full:
http://cr.openjdk.java.net/~njian/8231441/webrev.05/
Incremental:
http://cr.openjdk.java.net/~njian/8231441/webrev.05-vs-04/
Tests:
Tested with jtreg hotspot_all_no_apps, jdk_core and langtools:tier1 on
AArch64 systems with and without SVE as well as some x86_64 systems.
Mach5 submit test also reported passed.
Could you please help to take a look again? OK for jdk/jdk?
Thanks,
Ningsheng
On 8/19/20 5:53 PM, Ningsheng Jian wrote:
Hi Andrew,
I have updated the patch based on the review comments. Would you mind
taking another look? Thanks!
Full:
http://cr.openjdk.java.net/~njian/8231441/webrev.04/
Incremental:
http://cr.openjdk.java.net/~njian/8231441/webrev.04-vs-03/
Also add build-dev, as there's a makefile change.
And the split parts:
1) SVE feature detection:
http://cr.openjdk.java.net/~njian/8231441/webrev.04-feature
2) c2 register allocation:
http://cr.openjdk.java.net/~njian/8231441/webrev.04-ra
3) SVE c2 backend:
http://cr.openjdk.java.net/~njian/8231441/webrev.04-c2
Bug: https://bugs.openjdk.java.net/browse/JDK-8231441
CSR: https://bugs.openjdk.java.net/browse/JDK-8248742
