Skip to content

JIT: enable tail merge for try exits - #109661

Merged
AndyAyersMS merged 1 commit into
dotnet:mainfrom
AndyAyersMS:TailMergeTryExits
Nov 11, 2024
Merged

AndyAyersMS merged 1 commit into
dotnet:mainfrom
AndyAyersMS:TailMergeTryExits

Conversation

@AndyAyersMS

Copy link
Copy Markdown
Member

A try may have multiple blocks. Enable tail merging when these blocks have common suffixes, via cross-jumping.

Note there can only be one try entry point so this doesn't generalize to head merge.

A try may have multiple blocks. Enable tail merging when these blocks
have common suffixes, via cross-jumping.

Note there can only be one try entry point so this doesn't generalize
to head merge.
@ghost ghost added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Nov 9, 2024
@AndyAyersMS

AndyAyersMS commented Nov 9, 2024 •

Copy link
Copy Markdown
Member Author

Noticed this while working on some other changes. Seems like a decent number of diffs.

@jakobbotsch PTAL
cc @dotnet/jit-contrib

Diffs

@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.

@AndyAyersMS

Copy link
Copy Markdown
Member Author

@jakobbotsch ping

@AndyAyersMS
AndyAyersMS merged commit 06205b5 into dotnet:main Nov 11, 2024
mikelle-rogers pushed a commit to mikelle-rogers/runtime that referenced this pull request Dec 10, 2024
A try may have multiple blocks. Enable tail merging when these blocks
have common suffixes, via cross-jumping.

Note there can only be one try entry point so this doesn't generalize
to head merge.
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 12, 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