On Tue, 28 Nov 2023 12:23:00 GMT, Jatin Bhateja <jbhat...@openjdk.org> wrote:
>> Srinivas Vamsi Parasa has updated the pull request with a new target base >> due to a merge or a rebase. The incremental webrev excludes the unrelated >> changes brought in by the merge/rebase. The pull request contains 11 >> additional commits since the last revision: >> >> - Disable AVX2 sort for 64-bit types >> - Merge branch 'master' of https://git.openjdk.java.net/jdk into simdsort >> - fix jcheck failures due to windows encoding >> - fix carriage return and change insertion sort thresholds >> - fix formatting and white spaces >> - cleanup unused code >> - fix insertion sort thresholds >> - add insertion sort >> - fix headers >> - revert to xss-common-qsort >> - ... and 1 more: https://git.openjdk.org/jdk/compare/ae4d7f96...08307b6a > > src/hotspot/share/opto/library_call.cpp line 5391: > >> 5389: BasicType bt = elem_type->basic_type(); >> 5390: // Disable the intrinsic for 64-bit types with AVX2 >> 5391: if ((bt == T_LONG || bt == T_DOUBLE) && UseAVX == 2) { > > You can directly use convenience routine _is_double_word_type_ Thanks Jatin for the suggestion. Please see your suggestion incorporated in the new code upstreamed. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16534#discussion_r1408403056