-
-
Notifications
You must be signed in to change notification settings - Fork 34.4k
Remove DELETE_NAME instruction #145854
Copy link
Copy link
Open
Labels
3.15new features, bugs and security fixesnew features, bugs and security fixesinterpreter-core(Objects, Python, Grammar, and Parser dirs)(Objects, Python, Grammar, and Parser dirs)type-featureA feature request or enhancementA feature request or enhancement
Metadata
Metadata
Assignees
Labels
3.15new features, bugs and security fixesnew features, bugs and security fixesinterpreter-core(Objects, Python, Grammar, and Parser dirs)(Objects, Python, Grammar, and Parser dirs)type-featureA feature request or enhancementA feature request or enhancement
Feature or enhancement
Proposal:
With a couple of small modifications to
STORE_NAMEit can be made to handle aNULLvalue.Then
DELETE_NAMEcan be replaced withPUSH_NULL; STORE_NAMESTORE_NAMEis executed about 1 in 100k instructions andDELETE_NAMEabout 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
Linked PRs
DELETE_NAMEwithPUSH_NULL; STORE_NAME#146006