Skip to content

gh-145854: Replace DELETE_NAME with PUSH_NULL; STORE_NAME#146006

Open
bkap123 wants to merge 3 commits intopython:mainfrom
bkap123:delete-name
Open

gh-145854: Replace DELETE_NAME with PUSH_NULL; STORE_NAME#146006
bkap123 wants to merge 3 commits intopython:mainfrom
bkap123:delete-name

Conversation

@bkap123
Copy link
Contributor

@bkap123 bkap123 commented Mar 16, 2026

Changes

  • Removed DELETE_NAME
  • When STORE_NAME receives a NULL value, it corresponds with deletion

📚 Documentation preview 📚: https://cpython-previews--146006.org.readthedocs.build/

Changes
- Removed `DELETE_NAME`
- When `STORE_NAME` receives a `NULL` value, it corresponds with
  deletion
@bkap123 bkap123 marked this pull request as draft March 16, 2026 02:31
@bkap123 bkap123 marked this pull request as ready for review March 16, 2026 10:39
@bkap123
Copy link
Contributor Author

bkap123 commented Mar 16, 2026

I am not sure why three of the CIFuzz tests fail. If anyone could help, that would be appreciated.

@markshannon
Copy link
Member

This will need a magic number bump (as do all changes that add or remove opcodes)

Copy link
Member

@markshannon markshannon left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks for doing this.

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.

3 participants