Skip to content

Move optimize bools pass to a new file - #83768

Merged
jakobbotsch merged 1 commit into
dotnet:mainfrom
a74nh:runtime_file
Mar 22, 2023
Merged

Move optimize bools pass to a new file#83768
jakobbotsch merged 1 commit into
dotnet:mainfrom
a74nh:runtime_file

Conversation

@a74nh

Copy link
Copy Markdown
Contributor

No code changes, just moving.

No code changes, just moving.
@ghostghost added community-contribution Indicates that the PR has been added by a community member area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI labels Mar 22, 2023
@ghost

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

No code changes, just moving.

Author:a74nh
Assignees:-
Labels:

area-CodeGen-coreclr, community-contribution

Milestone:-

@a74nh

Copy link
Copy Markdown
ContributorAuthor

This change was originally in #79283, but was moved out for simplifying the review.

The optimize bools pass is now bigger than the if conversion pass, so it made sense to move to it's own file.

@a74nh

Copy link
Copy Markdown
ContributorAuthor

@kunalspathak and @jakobbotsch

@jakobbotschjakobbotsch left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@jakobbotsch
jakobbotsch merged commit 71b0ecd into dotnet:mainMar 22, 2023
@a74nh
a74nh deleted the runtime_file branch March 22, 2023 13:13
@ghostghost locked as resolved and limited conversation to collaborators Apr 21, 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 SuperPMIcommunity-contributionIndicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@a74nh@jakobbotsch