RFR: 8317763: Follow-up to AVX512 intrinsics for Arrays.sort() PR [v3]
iaroslavski
duke at openjdk.org
Wed Oct 11 19:27:35 UTC 2023
On Wed, 11 Oct 2023 17:22:56 GMT, Srinivas Vamsi Parasa <duke at openjdk.org> 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 us which @forceinline annotations are needed.
>
> Added `@ForceInline` annotations to `insertionSort` and `mixedInsertionSort` as it is helping arrays of small sizes when intrinsics are disabled.
>
> Thanks, Vamsi
@vamsi-parasa
Please revert changes (adding @ForceInline to insertionSort and mixedInsertionSort) - I checked: initinal version works faster.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/16124#issuecomment-1758387580
More information about the hotspot-compiler-dev
mailing list