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

JIT: extract BBJ_COND to BBJ_ALWAYS profile repair as utility #110494

Merged

Conversation

AndyAyersMS
Copy link
Member

and call it from a few places.

@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 Dec 6, 2024
@AndyAyersMS
Copy link
Member Author

@amanasifkhalid PTAL
cc @dotnet/jit-contrib

No diffs expected. I though there were more possible callers, so perhaps I've missed some. Will also be adding more with the deabstraction work.

@AndyAyersMS
Copy link
Member Author

@amanasifkhalid fyi

Copy link
Member

@amanasifkhalid amanasifkhalid left a comment

Choose a reason for hiding this comment

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

Sorry I'm late to this. Thanks for adding this in!

src/coreclr/jit/fgprofile.cpp Outdated Show resolved Hide resolved
src/coreclr/jit/fgprofile.cpp Outdated Show resolved Hide resolved
src/coreclr/jit/fgprofile.cpp Outdated Show resolved Hide resolved
src/coreclr/jit/fgprofile.cpp Outdated Show resolved Hide resolved
@AndyAyersMS AndyAyersMS merged commit f456716 into dotnet:main Dec 10, 2024
106 of 108 checks passed
hez2010 pushed a commit to hez2010/runtime that referenced this pull request Dec 14, 2024
…#110494)

JIT: extract BBJ_COND to BBJ_ALWAYS profile repair as utility

and call it from a few places.

Co-authored-by: Aman Khalid <[email protected]>
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.

2 participants