RFR: 8277617: Adjust AVX3Threshold for copy/fill stubs
Sandhya Viswanathan
sviswanathan at openjdk.java.net
Tue Nov 30 00:43:04 UTC 2021
On Tue, 23 Nov 2021 06:49:07 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> @dholmes-ora I have implemented your review comments.
>
> Sorry @sviswa7 but could you explain in the comment why/how `avx3_threshold` reporting zero impacts the use 64-byte load/store - the connection is not at all obvious for anyone not fully conversant with AVX3 and how it is used by the code. Thanks.
@dholmes-ora @jatin-bhateja I have implemented your review comments.
I have used the direct formulation for avx3_threshold() method as suggested by David.
Reused the avx3_threshold() computation where possible as suggested by Jatin.
The tier1-tier3 testing passed on the platform where avx3_threshold() returns 0.
No additional observable overhead seen in SPECjvm2008 startup benchmarks on AVX512 platform.
Please let me know if the patch looks ok to you.
-------------
PR: https://git.openjdk.java.net/jdk/pull/6512
More information about the hotspot-compiler-dev
mailing list