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]>

I think I like the general idea here, but many of the details are unclear to 
me. It is probably best to let the compiler team actually review this as these 
flags primarily affect generated code.

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

PR Review: https://git.openjdk.org/jdk/pull/31301#pullrequestreview-4415715279

Reply via email to