RFR: 8319577: x86_64 AVX2 intrinsics for Arrays.sort methods (int, float arrays) [v5]

Thu Nov 30 18:22:10 UTC 2023

On Thu, 30 Nov 2023 09:38:22 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:

>> Seems a bit odd to me too as the existing simd code seems to C code residing in .cpp files for some reason.
>
> Yeah, that is also a good point. Why setting C++ flags in CFLAGS, and not just CXXFLAGS?
> 
> Also @vamsi-parasa, did you check into if this really was needed and why? To be clear, I basically consider this an integration blocker.

Hi Magnus (@magicus),
The simdsort has a dependency on some C++ 17 features and would require a lot of changes to avoid that. 
Please see the latest code which removed the unused flags from CFLAGS.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/16534#discussion_r1411099219