On Wed, 11 Oct 2023 17:22:56 GMT, Srinivas Vamsi Parasa <d...@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

Reply via email to