RFR: 8266431: Dual-Pivot Quicksort improvements (Radix sort) [v2]

Vladimir Yaroslavskiy duke at openjdk.org
Wed Oct 22 20:38:58 UTC 2025


On Sat, 11 Oct 2025 09:09:07 GMT, Tagir F. Valeev <tvaleev at openjdk.org> wrote:

>> Vladimir Yaroslavskiy has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   JDK-8266431: Dual-Pivot Quicksort improvements
>>   
>>   * Moved Radix sort out from sorting
>
> src/java.base/share/classes/java/util/DualPivotQuicksort.java line 152:
> 
>> 150:     @ForceInline
>> 151:     @IntrinsicCandidate
>> 152:     private static <T> void sort(Class<?> elemType, T a, long offset,
> 
> Why do we need this method? Offset and elemType are not used here. Probably it could be inlined?

These methods sort() and partition() were introduced in PR https://github.com/openjdk/jdk/pull/14227 where intrinsics for sorting and partitioning based on AVX512 instructions were added. The signature of these methods are in sync with native implementations from linux/native/libsimdsort package.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27411#discussion_r2453292847


More information about the core-libs-dev mailing list