Skip to content

Remove DELETE_NAME instruction #145854

@markshannon

Description

@markshannon

Feature or enhancement

Proposal:

With a couple of small modifications to STORE_NAME it can be made to handle a NULL value.
Then DELETE_NAME can be replaced with PUSH_NULL; STORE_NAME

STORE_NAME is executed about 1 in 100k instructions and DELETE_NAME about 1 in a billion according to our stats, so we can save an opcode with no performance impact.

Has this already been discussed elsewhere?

This is a minor feature, which does not need previous discussion elsewhere

Links to previous discussion of this feature:

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    3.15new features, bugs and security fixesinterpreter-core(Objects, Python, Grammar, and Parser dirs)type-featureA feature request or enhancement

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions