Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Remove some unused AbstractFlowPass parameters - #80931

#80931
Merged
jcouv merged 1 commit into
dotnet:maindotnet/roslyn:mainfrom
jcouv:remove-parametersjcouv/roslyn:remove-parametersCopy head branch name to clipboard
Oct 29, 2025
Merged

Remove some unused AbstractFlowPass parameters#80931
jcouv merged 1 commit into
dotnet:maindotnet/roslyn:mainfrom
jcouv:remove-parametersjcouv/roslyn:remove-parametersCopy head branch name to clipboard

Conversation

@jcouv

@jcouv jcouv commented Oct 28, 2025

Copy link
Copy Markdown
Contributor

Closes #42022

@jcouv jcouv self-assigned this Oct 28, 2025
CyrusNajmabadi
CyrusNajmabadi previously approved these changes Oct 28, 2025
@jcouv
jcouv dismissed CyrusNajmabadi’s stale review October 28, 2025 17:25

PR was a draft. I still have changes to force push

@jcouv
jcouv force-pushed the remove-parameters branch from 1caea39 to f425069 Compare October 28, 2025 17:58
@jcouv
jcouv force-pushed the remove-parameters branch from f425069 to 4be721a Compare October 28, 2025 18:36
@jcouv
jcouv marked this pull request as ready for review October 28, 2025 20:30
@jcouv
jcouv requested a review from a team as a code owner October 28, 2025 20:30

@AlekseyTs AlekseyTs left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM (commit 1)

@jcouv
jcouv merged commit 511a87e into dotnet:main Oct 29, 2025
24 checks passed
@jcouv
jcouv deleted the remove-parameters branch October 29, 2025 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Eliminate dead parameters in flow analysis

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.