On Wed, 3 Jun 2026 02:19:30 GMT, Ashutosh Mehra <[email protected]> wrote:
>> This patch is reorganizing the code to split `get_processor_features()` into >> multiple helper functions. Details are present in the issue >> https://bugs.openjdk.org/browse/JDK-8380448. >> It should not change the behavior of any VM flag, except >> `UseVectorizedHashCodeIntrinsic`. The earlier code always set >> `UseVectorizedHashCodeIntrinsic` to `true` if UseAVX >= 2. That means >> specifying `-XX:-UseVectorizedHashCodeIntrinsic` on command line wouldn't >> disable use of this intrinsic. This looks like a bug and has been fixed in >> this patch. >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > Ashutosh Mehra has updated the pull request incrementally with one additional > commit since the last revision: > > Fix setting of CopyAVX3Threshold > > Signed-off-by: Ashutosh Mehra <[email protected]> src/hotspot/cpu/x86/vm_version_x86.cpp line 856: > 854: // Note that it can be overridden by vendors in > set_vendor_specific_vm_config(). > 855: void VM_Version::set_vendor_agnostic_vm_config() { > 856: if (FLAG_IS_DEFAULT(IntelJccErratumMitigation)) { This doesn't sound like it is a vendor agnostic check?? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/31301#discussion_r3345948315
