Skip to content

Remove some unused AbstractFlowPass parameters#80931

Merged
jcouv merged 1 commit intodotnet:mainfrom
jcouv:remove-parameters
Oct 29, 2025
Merged

Remove some unused AbstractFlowPass parameters#80931
jcouv merged 1 commit intodotnet:mainfrom
jcouv:remove-parameters

Conversation

@jcouv
Copy link
Member

@jcouv jcouv commented Oct 28, 2025

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
Copy link
Contributor

@AlekseyTs AlekseyTs left a comment

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