Skip to content

Fix early JITDUMP disasm in ARM/ARM64 - #66502

Merged
jakobbotsch merged 2 commits into
dotnet:mainfrom
jakobbotsch:fix-jitdump-arm
Mar 11, 2022
Merged

Fix early JITDUMP disasm in ARM/ARM64#66502
jakobbotsch merged 2 commits into
dotnet:mainfrom
jakobbotsch:fix-jitdump-arm

Conversation

@jakobbotsch

Copy link
Copy Markdown
Member

On ARM/ARM64 this code works a little differently than on x64 and I inadvertently broke the early codegen disasm we output in jit dumps when I fixed this for COMPlus_JitDisasm in #66328.

@ghostghost added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Mar 11, 2022
@ghost

Copy link
Copy Markdown

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

Issue Details

On ARM/ARM64 this code works a little differently than on x64 and I inadvertently broke the early codegen disasm we output in jit dumps when I fixed this for COMPlus_JitDisasm in #66328.

Author:jakobbotsch
Assignees:jakobbotsch
Labels:

area-CodeGen-coreclr

Milestone:-

@jakobbotsch

Copy link
Copy Markdown
MemberAuthor

cc @dotnet/jit-contrib

@jakobbotsch
jakobbotsch merged commit 8242a50 into dotnet:mainMar 11, 2022
@jakobbotsch
jakobbotsch deleted the fix-jitdump-arm branch March 11, 2022 20:36
@ghostghost locked as resolved and limited conversation to collaborators Apr 11, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-CodeGen-coreclrCLR 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.

2 participants

@jakobbotsch@BruceForstall