On Thu, 30 Nov 2023 09:38:22 GMT, Magnus Ihse Bursie <i...@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
