Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bpo-39320: Handle unpacking of **values in compiler #18141

Open
wants to merge 6 commits into
base: master
from

Conversation

@markshannon
Copy link
Contributor

markshannon commented Jan 23, 2020

This is the second part of the implementation of bpo-39320.
Part 1 was #17984

This PR replaces BUILD_MAP_UNPACK and BUILD_MAP_UNPACK_WITH_CALL
with DICT_MERGE and DICT_UPDATE.

DICT_UPDATE n calls item_n.update(tos) where item_n is the nth item on the stack and tos is popped from the stack.
DICT_MERGE n is the same, but raises an exception if there are duplicate keys.

https://bugs.python.org/issue39320

@markshannon markshannon force-pushed the markshannon:handle-dict-unpacking-in-compiler branch from b6ad237 to 95e421c Jan 23, 2020
@@ -0,0 +1,12 @@

Replace two complex bytecodes for building dicts with two simpler ones.

This comment has been minimized.

Copy link
@pablogsal

pablogsal Jan 23, 2020

Member

Different paragraphs render badly when joining the NEWS entries, sadly

@bedevere-bot

This comment has been minimized.

Copy link

bedevere-bot commented Jan 23, 2020

🤖 New build scheduled with the buildbot fleet by @markshannon for commit 2d2fe4c 🤖

If you want to schedule another build, you need to add the "🔨 test-with-buildbots" label again.

@pablogsal

This comment has been minimized.

Copy link
Member

pablogsal commented Jan 23, 2020

The failures in AMD64 Fedora Stable Clang Installed PR , buildbot/PPC64 Fedora PR and buildbot/x86 Gentoo Installed with X PR are unrelated so don't worry about them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.