Skip to content

JIT: Ensure tail merging does not add preds to scratch block - #84353

Merged
jakobbotsch merged 1 commit into
dotnet:mainfrom
jakobbotsch:scratch-block-not-victim
Apr 5, 2023
Merged

JIT: Ensure tail merging does not add preds to scratch block#84353
jakobbotsch merged 1 commit into
dotnet:mainfrom
jakobbotsch:scratch-block-not-victim

Conversation

@jakobbotsch

Copy link
Copy Markdown
Member

Fix issue seen in #83388 where tail merging ended up adding new predecessors to the scratch block, making adding more "initialization IR" impossible for downstream phases.

Fix issue seen in dotnet#83388 where tail merging ended up adding new
predecessors to the scratch block, making adding more "initialization
IR" impossible for downstream phases.
@ghostghost added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Apr 5, 2023
@ghostghost assigned jakobbotschApr 5, 2023
@ghost

ghost commented Apr 5, 2023

Copy link
Copy Markdown

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

Issue Details

Fix issue seen in #83388 where tail merging ended up adding new predecessors to the scratch block, making adding more "initialization IR" impossible for downstream phases.

Author:jakobbotsch
Assignees:-
Labels:

area-CodeGen-coreclr

Milestone:-

@jakobbotsch

Copy link
Copy Markdown
MemberAuthor

cc @dotnet/jit-contrib PTAL @AndyAyersMS

@jakobbotsch
jakobbotsch merged commit 30f7511 into dotnet:mainApr 5, 2023
@jakobbotsch
jakobbotsch deleted the scratch-block-not-victim branch April 5, 2023 17:08
@ghostghost locked as resolved and limited conversation to collaborators May 5, 2023
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@AndyAyersMS