On Wed, 11 Oct 2023 22:40:20 GMT, Sandhya Viswanathan
wrote:
>> I see now that this is an unrelated change. In that case please avoid
>> changing whitespace in unrelated files for this PR.
>
> @erikj79 This space was inadvertently added as part of
> (https://github.com/openjdk/jdk/pull/14227)
On Wed, 11 Oct 2023 22:25:14 GMT, Erik Joelsson wrote:
>> Hi Erik (@erikj79),
>> BUILD_LIBFALLBACKLINKER is from different PR (#13079). If I understand
>> correctly, for LIB_SIMD_SORT, are you suggesting that we don't pad the lines
>> with spaces to align features into columns and instead just
On Wed, 11 Oct 2023 21:04:25 GMT, Srinivas Vamsi Parasa
wrote:
>> make/modules/java.base/Lib.gmk line 230:
>>
>>> 228: CFLAGS := $(CFLAGS_JDKLIB) $(LIBFFI_CFLAGS), \
>>> 229: LDFLAGS := $(LDFLAGS_JDKLIB) \
>>> 230: $(call SET_SHARED_LIBRARY_ORIGIN), \
>>
>> If you
On Wed, 11 Oct 2023 20:48:06 GMT, Erik Joelsson wrote:
>> Srinivas Vamsi Parasa has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Add @ForceInline annotation to insertion and mixedInsertion sort
>
> make/modules/java.base/Lib.gmk line 230:
On Wed, 11 Oct 2023 17:28:12 GMT, Srinivas Vamsi Parasa
wrote:
>> The goal of this PR is to address the follow-up comments to the SIMD
>> accelerated sort PR (#14227) which implemented AVX512 intrinsics for
>> Arrays.sort() methods.
>> The proposed changes are:
>>
>> 1) Restriction of the AVX
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 effect @forceinline without
> > > > > intrinsics.
On Wed, 11 Oct 2023 19:56:47 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
On Wed, 11 Oct 2023 17:28:12 GMT, Srinivas Vamsi Parasa
wrote:
>> The goal of this PR is to address the follow-up comments to the SIMD
>> accelerated sort PR (#14227) which implemented AVX512 intrinsics for
>> Arrays.sort() methods.
>> The proposed changes are:
>>
>> 1) Restriction of the AVX
On Wed, 11 Oct 2023 17:28:12 GMT, Srinivas Vamsi Parasa
wrote:
>> The goal of this PR is to address the follow-up comments to the SIMD
>> accelerated sort PR (#14227) which implemented AVX512 intrinsics for
>> Arrays.sort() methods.
>> The proposed changes are:
>>
>> 1) Restriction of the AVX
On Wed, 11 Oct 2023 17:28:12 GMT, Srinivas Vamsi Parasa
wrote:
>> The goal of this PR is to address the follow-up comments to the SIMD
>> accelerated sort PR (#14227) which implemented AVX512 intrinsics for
>> Arrays.sort() methods.
>> The proposed changes are:
>>
>> 1) Restriction of the AVX
> The goal of this PR is to address the follow-up comments to the SIMD
> accelerated sort PR (#14227) which implemented AVX512 intrinsics for
> Arrays.sort() methods.
> The proposed changes are:
>
> 1) Restriction of the AVX512 sort acceleration to only Intel CPUs. A
> performance regression (d
11 matches
Mail list logo