Skip to content

gh-114058: Improve method information in redundancy eliminator - #115848

Merged
Fidget-Spinner merged 3 commits into
python:mainfrom
Fidget-Spinner:improve_methods
Feb 23, 2024
Merged

gh-114058: Improve method information in redundancy eliminator#115848
Fidget-Spinner merged 3 commits into
python:mainfrom
Fidget-Spinner:improve_methods

Conversation

@Fidget-Spinner

@Fidget-SpinnerFidget-Spinner commented Feb 23, 2024

Copy link
Copy Markdown
Member

@Fidget-Spinner

Copy link
Copy Markdown
MemberAuthor

Addresses @markshannon 's reviews in #115779 (comment).

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

I don't think you can remove sym_is_not_null

Comment threadPython/tier2_redundancy_eliminator_bytecodes.c Outdated
Comment threadPython/optimizer_analysis.c
@bedevere-app

Copy link
Copy Markdown

When you're done making the requested changes, leave the comment: I have made the requested changes; please review again.

@Fidget-Spinner

Copy link
Copy Markdown
MemberAuthor

I have made the requested changes; please review again

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

Looks good

@Fidget-Spinner
Fidget-Spinner merged commit 2ec50b4 into python:mainFeb 23, 2024
@Fidget-Spinner
Fidget-Spinner deleted the improve_methods branch February 23, 2024 15:41
woodruffw pushed a commit to woodruffw-forks/cpython that referenced this pull request Mar 4, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@Fidget-Spinner@markshannon