RFR: 8317763: Follow-up to AVX512 intrinsics for Arrays.sort() PR [v2]

Srinivas Vamsi Parasa duke at openjdk.org
Wed Oct 11 20:35:16 UTC 2023


On Wed, 11 Oct 2023 07:10:57 GMT, iaroslavski <duke at openjdk.org> wrote:

> To have clear picture could you please run benchmarking to compare both cases: current implementation and implementation with Java insertionSort only?
> 
> see changes `sort(int.class, a, Unsafe.ARRAY_INT_BASE_OFFSET, low, high, DualPivotQuicksort::insertionSort);` -> `insertionSort(a, low, high);`

@iaroslavski 
Yes, the intrinsic is needed. 
I remember, the same question was asked in the original PR and data was provided in the original PR.
Even for a small say size=30, the AVX512 sort intrinsic is faster.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/16124#issuecomment-1758489003


More information about the build-dev mailing list