Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JitDump and other minor improvements #96933

Merged

Conversation

BruceForstall
Copy link
Member

  1. Make DOTNET_JitDisasmAssemblies apply to JitDump as well.
  2. Fix JitDump of gsParamsToShadows: it calls fgMorphTree which requires compCurBB to be set, but apparently only for JitDump.
  3. Various typos and formatting nits

1. Make `DOTNET_JitDisasmAssemblies` apply to `JitDump` as well.
2. Fix JitDump of `gsParamsToShadows`: it calls `fgMorphTree` which requires
`compCurBB` to be set, but apparently only for JitDump.
3. Various typos and formatting nits
@ghost ghost assigned BruceForstall Jan 13, 2024
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Jan 13, 2024
@ghost
Copy link

ghost commented Jan 13, 2024

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

Issue Details
  1. Make DOTNET_JitDisasmAssemblies apply to JitDump as well.
  2. Fix JitDump of gsParamsToShadows: it calls fgMorphTree which requires compCurBB to be set, but apparently only for JitDump.
  3. Various typos and formatting nits
Author: BruceForstall
Assignees: BruceForstall
Labels:

area-CodeGen-coreclr

Milestone: -

@BruceForstall
Copy link
Member Author

@jakobbotsch PTAL
cc @dotnet/jit-contrib

@BruceForstall BruceForstall merged commit 6f9d656 into dotnet:main Jan 14, 2024
127 of 129 checks passed
@BruceForstall BruceForstall deleted the JitDumpAndOtherImprovements branch January 14, 2024 23:18
tmds pushed a commit to tmds/runtime that referenced this pull request Jan 23, 2024
1. Make `DOTNET_JitDisasmAssemblies` apply to `JitDump` as well.
2. Fix JitDump of `gsParamsToShadows`: it calls `fgMorphTree` which requires
`compCurBB` to be set, but apparently only for JitDump.
3. Various typos and formatting nits
@github-actions github-actions bot locked and limited conversation to collaborators Feb 14, 2024
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.

2 participants