On Sun, 10 May 2026 03:53:18 GMT, Ashutosh Mehra <[email protected]> wrote:

>> VM has a hardcoded requirement on the hardware to have support for SSE2. 
>> Without SSE2, VM won't even startup. Therefore there is no need to 
>> explicitly check for SSE2 and/or SSE support, after the initial hardcoded 
>> check for SSE2.
>> This patch clean up the such redundant checks for `supports_sse2()` and 
>> `supports_sse()`.
>> 
>> ---------
>> - [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 three 
> additional commits since the last revision:
> 
>  - Add required header file
>    
>    Signed-off-by: Ashutosh Mehra <[email protected]>
>  - Fix another test
>    
>    Signed-off-by: Ashutosh Mehra <[email protected]>
>  - Fix tests to not use -XX:UseSSE=0 or -XX:UseSSE=1
>    
>    Signed-off-by: Ashutosh Mehra <[email protected]>

Looks good. I submitted our testing.

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

PR Review: https://git.openjdk.org/jdk/pull/31045#pullrequestreview-4259649503

Reply via email to