Title: Update for fastcalls
Type: Stage: resolved
Components: Versions: Python 3.7, Python 3.6
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: Nosy List: inada.naoki, ned.deily, python-dev, vstinner
Priority: Keywords: patch

Created on 2016-11-22 11:42 by vstinner, last changed 2016-12-17 05:09 by ned.deily. This issue is now closed.

File name Uploaded Description Edit
gdb_fastcall.patch vstinner, 2016-11-22 11:42 review
gdb_fastcall-2.patch vstinner, 2016-11-22 11:44 review
Messages (10)
msg281465 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2016-11-22 11:42
Python 3.6 has a new C calling convention: "fast calls". was disabled when compact dict was merged, see issue #27350.

Sadly, I missed that fast calls also broke

Attached patch fixes, but I failed to fix With fast calls and patched, is now able to detect the Python frame of the call to the builtin id() function. It's a new feature compared to Python 3.5, but should be updated to handle that.

I will try to fix that later, but in the meanwhile I was interrupted because the fix for compact dict in failed on buildbots, see:
msg281522 - (view) Author: Roundup Robot (python-dev) Date: 2016-11-22 21:55
New changeset 752863f96fb8 by Victor Stinner in branch 'default':
Issue #28770: Update for fastcalls
msg281523 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2016-11-22 22:02
I keep the issue open until the 3.6 branch is reopened for regular bug fixes, like this one. Then I will backport the fix to 3.6.
msg282603 - (view) Author: INADA Naoki (inada.naoki) * (Python committer) Date: 2016-12-07 09:16
3.6 branch is reopened
msg282715 - (view) Author: Roundup Robot (python-dev) Date: 2016-12-08 16:17
New changeset f41d02d7da37 by Victor Stinner in branch '3.6':
Issue #28770: Update for fastcalls
msg282716 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2016-12-08 16:19
I fixed the issue in Python 3.6 (future 3.6.1).
msg283402 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2016-12-16 14:58
The change was cherry-picked in Python 3.6.0RC2.

The issue is now fixed, so I close the issue.
msg283407 - (view) Author: Ned Deily (ned.deily) * (Python committer) Date: 2016-12-16 15:11
Please leave this open until the cherry picking is completed.
msg283464 - (view) Author: Roundup Robot (python-dev) Date: 2016-12-17 04:02
New changeset b0efa88c4cf4 by Victor Stinner in branch '3.6':
Issue #28770: Update for fastcalls
msg283467 - (view) Author: Ned Deily (ned.deily) * (Python committer) Date: 2016-12-17 05:09
[cherrypicked for 3.6.0rc2]
Date User Action Args
2016-12-17 05:09:56ned.deilysetpriority: release blocker ->
status: open -> closed
messages: + msg283467

stage: resolved
2016-12-17 04:02:00python-devsetmessages: + msg283464
2016-12-16 15:11:38ned.deilysetstatus: closed -> open

messages: + msg283407
2016-12-16 14:58:48vstinnersetstatus: open -> closed

messages: + msg283402
2016-12-16 08:28:44ned.deilysetstatus: closed -> open
2016-12-16 08:27:49ned.deilysetpriority: normal -> release blocker
nosy: + ned.deily
2016-12-08 16:19:50vstinnersetstatus: open -> closed
resolution: fixed
messages: + msg282716
2016-12-08 16:17:28python-devsetmessages: + msg282715
2016-12-07 09:16:54inada.naokisetmessages: + msg282603
2016-11-22 22:02:21vstinnersetmessages: + msg281523
2016-11-22 21:55:36python-devsetnosy: + python-dev
messages: + msg281522
2016-11-22 11:44:51vstinnersetfiles: + gdb_fastcall-2.patch
2016-11-22 11:43:14vstinnersetnosy: + inada.naoki
2016-11-22 11:42:55vstinnercreate