Skip to content

Correct x86 instruction metadata - #136120

Merged
tannergooding merged 1 commit into
dotnet:mainfrom
tannergooding:tannergooding-issue-135844-fix
Oct 20, 2026
Merged

tannergooding merged 1 commit into
dotnet:mainfrom
tannergooding:tannergooding-issue-135844-fix

Conversation

@tannergooding

Copy link
Copy Markdown
Member

Correct x86 instruction metadata used by the JIT for memory sizing, EVEX compressed displacement, masking, and operand mapping.

VPMULDQ and VPMULUDQ use a 64-bit broadcast memory operand, so the JIT must scale their compressed displacements by eight bytes. Add signed and unsigned AVX-512 regression coverage for the adjacent-field case. The remaining changes correct related instruction metadata and add quarter-width tuple handling for EVEX sizing and lowering.

Resolves #135844

Note

This PR description was generated with GitHub Copilot.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@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 Oct 20, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 5 pipeline(s).
11 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch
See info in area-owners.md if you want to be subscribed.

Comment thread src/coreclr/jit/lowerxarch.cpp
@tannergooding
tannergooding merged commit a3109fb into dotnet:main Oct 20, 2026
135 of 137 checks passed
@tannergooding
tannergooding deleted the tannergooding-issue-135844-fix branch October 20, 2026 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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: Incorrect displacement for VPMULUDQ with embedded broadcast

2 participants