Skip to content

Conversation

@tannergooding
Copy link
Member

This resolves #115493

Copilot AI review requested due to automatic review settings May 14, 2025 19:57
@github-actions github-actions bot added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label May 14, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR ensures that the NI_SSE2_Extract and NI_SSE41_Extract intrinsics are handled with a SIMD size of 16 and updates the lowering logic accordingly.

  • Adds a test project and corresponding test file to validate the intrinsic behavior.
  • Updates the lowering logic in coreclr/jit/lowerxarch.cpp with simdSize adjustments and new handling for specific intrinsic cases.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
src/tests/JIT/Regression/JitBlue/Runtime_115493/Runtime_115493.csproj New test project file enabling the test compilation and execution.
src/tests/JIT/Regression/JitBlue/Runtime_115493/Runtime_115493.cs Test file validating the SIMD intrinsic extraction behavior.
src/coreclr/jit/lowerxarch.cpp Updates to set the SIMD size to 16 and added conditions for additional intrinsic cases.

@tannergooding
Copy link
Member Author

tannergooding commented May 14, 2025

CC. @dotnet/jit-contrib, @kunalspathak for review

Copy link
Contributor

@kunalspathak kunalspathak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tannergooding tannergooding merged commit c7c961a into dotnet:main May 15, 2025
112 of 114 checks passed
@tannergooding tannergooding deleted the fix-115493 branch May 15, 2025 00:29
@github-actions github-actions bot locked and limited conversation to collaborators Jun 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

JIT: invalid vpextrw encoding on AVX-512 HW

2 participants