On Wed, 11 Oct 2023 20:10:12 GMT, iaroslavski wrote:
> > > > > Also @forceinline in these changes only works for case when new
> > > > > intrinsics are not used. I would suggest to adapt/update JMH
> > > > > benchmark to cover all cases and see effe
gt; > > intrinsics are disabled.
> > > Thanks, Vamsi
> >
> >
> > @vamsi-parasa Please revert changes (adding @forceinline to insertionSort
> > and mixedInsertionSort) - I checked: initinal version works faster.
>
> @iaroslavski Vladimir, have you u
On Wed, 11 Oct 2023 17:22:56 GMT, Srinivas Vamsi Parasa
wrote:
> > Also @forceinline in these changes only works for case when new intrinsics
> > are not used. I would suggest to adapt/update JMH benchmark to cover all
> > cases and see effect @forceinline without intrinsics. That will tell
On Fri, 6 Oct 2023 18:45:59 GMT, Srinivas Vamsi Parasa wrote:
>> My tier1-7 testing passed. Good.
>
>> My tier1-7 testing passed. Good.
>
> Thank you, Vladimir!
@vamsi-parasa
Please disrard my request to change high -> end. I find out a way to update
Java code only.
I updated my previous
On Fri, 6 Oct 2023 18:45:59 GMT, Srinivas Vamsi Parasa wrote:
>> My tier1-7 testing passed. Good.
>
>> My tier1-7 testing passed. Good.
>
> Thank you, Vladimir!
Hi @vamsi-parasa,
If DualPivotQuicksort.java is updated, can you improve `partitionDualPivot` and
`partitionSinglePivot` methods a
On Fri, 6 Oct 2023 18:45:59 GMT, Srinivas Vamsi Parasa wrote:
>> My tier1-7 testing passed. Good.
>
>> My tier1-7 testing passed. Good.
>
> Thank you, Vladimir!
Hi @vamsi-parasa,
May be too late but there is one question. We have 2 new methods
`private static void sort(Class elemType, A
On Wed, 20 Sep 2023 22:46:16 GMT, Srinivas Vamsi Parasa
wrote:
>> Srinivas Vamsi Parasa has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> change variable names of indexPivot* to pivotIndex*
>
> Hi Vladimir,
>
> Just trying to
one
> additional commit since the last revision:
>
> change variable names of indexPivot* to pivotIndex*
Hi Vamsi,
In this comment
https://github.com/openjdk/jdk/pull/13568#issuecomment-1728082819
Paul suggested comparing of performance.
Could you please run benchmarking of the followi
On Tue, 19 Sep 2023 01:57:44 GMT, Srinivas Vamsi Parasa
wrote:
>> The goal is to develop faster sort routines for x86_64 CPUs by taking
>> advantage of AVX512 instructions. This enhancement provides an order of
>> magnitude speedup for Arrays.sort() using int, long, float and double arrays.
On Tue, 19 Sep 2023 21:44:00 GMT, iaroslavski wrote:
>> Srinivas Vamsi Parasa has updated the pull request incrementally with two
>> additional commits since the last revision:
>>
>> - Update DualPivotQuicksort.java
>> - Rename arraySort and arrayPartition Java
On Tue, 19 Sep 2023 01:57:44 GMT, Srinivas Vamsi Parasa
wrote:
>> The goal is to develop faster sort routines for x86_64 CPUs by taking
>> advantage of AVX512 instructions. This enhancement provides an order of
>> magnitude speedup for Arrays.sort() using int, long, float and double arrays.
On Mon, 18 Sep 2023 18:54:07 GMT, Srinivas Vamsi Parasa
wrote:
>> The goal is to develop faster sort routines for x86_64 CPUs by taking
>> advantage of AVX512 instructions. This enhancement provides an order of
>> magnitude speedup for Arrays.sort() using int, long, float and double arrays.
On Mon, 18 Sep 2023 18:54:07 GMT, Srinivas Vamsi Parasa
wrote:
>> The goal is to develop faster sort routines for x86_64 CPUs by taking
>> advantage of AVX512 instructions. This enhancement provides an order of
>> magnitude speedup for Arrays.sort() using int, long, float and double arrays.
On Fri, 15 Sep 2023 22:11:44 GMT, Srinivas Vamsi Parasa
wrote:
>> Srinivas Vamsi Parasa has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Refactor the sort and partition intrinsics to accept method references for
>> fallback functions
>
On Thu, 14 Sep 2023 23:03:22 GMT, Paul Sandoz wrote:
> That neatly covers multiple element types and Java-based insertion sort
> algorithms (although I don't know why we need two since mixed insertion
> effectively embeds the other).
@PaulSandoz There are two insertion sorts in DPQ:
On Wed, 30 Aug 2023 15:10:45 GMT, Srinivas Vamsi Parasa
wrote:
>>> Hi Vladimir, Just verified that the test/jdk/java/util/Arrays/Sorting.java
>>> is triggering the intrinsic without additional flags
>>
>> Just to add that Sorting.java has short and long run modes. The default when
>> running
On Tue, 29 Aug 2023 16:57:11 GMT, Srinivas Vamsi Parasa
wrote:
> > My testing passed. But I am not sure correctness of code is fully tested.
> > For now we have only JMH benchmark for this new code. Do we have JDK test
> > which can check correctness of this code?
>
> Hi Vladimir, will add
On Mon, 28 Aug 2023 21:27:25 GMT, Srinivas Vamsi Parasa
wrote:
>> The goal is to develop faster sort routines for x86_64 CPUs by taking
>> advantage of AVX512 instructions. This enhancement provides an order of
>> magnitude speedup for Arrays.sort() using int, long, float and double arrays.
18 matches
Mail list logo