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

Merged

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.

@markshannon

This comment has been minimized.

Copy link
Contributor Author

markshannon commented Jan 27, 2020

OK, merging.

@markshannon markshannon merged commit 8a4cd70 into python:master Jan 27, 2020
41 of 44 checks passed
41 of 44 checks passed
Docs
Details
Windows (x86)
Details
Windows (x64)
Details
macOS
Details
Ubuntu
Details
buildbot/AMD64 Fedora Stable Clang Installed PR Build done.
Details
buildbot/PPC64 Fedora PR Build done.
Details
buildbot/x86 Gentoo Installed with X PR Build done.
Details
Azure Pipelines PR #20200123.28 succeeded
Details
bedevere/issue-number Issue number 39320 found
Details
bedevere/news News entry found in Misc/NEWS.d
buildbot/AMD64 Arch Linux TraceRefs PR Build done.
Details
buildbot/AMD64 Debian PGO PR Build done.
Details
buildbot/AMD64 Debian root PR Build done.
Details
buildbot/AMD64 Fedora Stable Clang PR Build done.
Details
buildbot/AMD64 Fedora Stable LTO + PGO PR Build done.
Details
buildbot/AMD64 Fedora Stable LTO PR Build done.
Details
buildbot/AMD64 Fedora Stable PR Build done.
Details
buildbot/AMD64 Fedora Stable Refleaks PR Build done.
Details
buildbot/AMD64 FreeBSD Non-Debug PR Build done.
Details
buildbot/AMD64 FreeBSD Shared PR Build done.
Details
buildbot/AMD64 RHEL7 LTO + PGO PR Build done.
Details
buildbot/AMD64 RHEL7 LTO PR Build done.
Details
buildbot/AMD64 RHEL7 PR Build done.
Details
buildbot/AMD64 RHEL7 Refleaks PR Build done.
Details
buildbot/AMD64 RHEL8 LTO + PGO PR Build done.
Details
buildbot/AMD64 RHEL8 LTO PR Build done.
Details
buildbot/AMD64 RHEL8 PR Build done.
Details
buildbot/AMD64 RHEL8 Refleaks PR Build done.
Details
buildbot/AMD64 Ubuntu Shared PR Build done.
Details
buildbot/AMD64 Windows10 PR Build done.
Details
buildbot/AMD64 Windows7 SP1 PR Build done.
Details
buildbot/AMD64 Windows8.1 Non-Debug PR Build done.
Details
buildbot/AMD64 Windows8.1 Refleaks PR Build done.
Details
buildbot/ARMv7 Debian buster PR Build done.
Details
buildbot/PPC64LE Fedora PR Build done.
Details
buildbot/s390x Debian PR Build done.
Details
buildbot/s390x RHEL PR Build done.
Details
buildbot/s390x SLES PR Build done.
Details
buildbot/x86 Gentoo Non-Debug with X PR Build done.
Details
buildbot/x86 Gentoo Refleaks PR Build done.
Details
buildbot/x86 Windows7 PR Build done.
Details
buildbot/x86-64 macOS PR Build done.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@markshannon markshannon deleted the markshannon:handle-dict-unpacking-in-compiler branch Jan 27, 2020
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.