FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-144741: Fix test_frame_pointer_unwind for libpython by vstinner · Pull Request #145499 · python/cpython · GitHub

Repository navigation

gh-144741: Fix test_frame_pointer_unwind for libpython - #145499

Merged
vstinner merged 1 commit into
python:mainfrom
vstinner:classify_address
Mar 4, 2026
Merged

vstinner merged 1 commit into
python:mainfrom
vstinner:classify_address

Conversation

vstinner commented Mar 4, 2026 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

Fix test_frame_pointer_unwind when Python is built with --enable-shared. Classify also libpython frames as "python".

Fix test_frame_pointer_unwind when Python is built with
--enable-shared. Classify also libpython frames as "python".

vstinner commented Mar 4, 2026

Copy link
Copy Markdown
Member Author

cc @diegorusso

vstinner commented Mar 4, 2026

Copy link
Copy Markdown
Member Author

cc @pablogsal

Copy link
Copy Markdown
Member

Great catch!

Copy link
Copy Markdown
Member

CC @diegorusso

diegorusso left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I agree, excellent stuff!

vstinner commented Mar 4, 2026

Copy link
Copy Markdown
Member Author

The JIT failures are unrelated and should be fixed by #145496 (which increases _Py_MAX_MANAGED_STATIC_BUILTIN_TYPES to fix the issue).

vstinner merged commit a003923 into python:main Mar 4, 2026
67 of 73 checks passed
vstinner deleted the classify_address branch March 4, 2026 17:29

vstinner commented Mar 4, 2026

Copy link
Copy Markdown
Member Author

Merged. Thanks for your reviews!

ljfp pushed a commit to ljfp/cpython that referenced this pull request Apr 25, 2026
…145499)

Fix test_frame_pointer_unwind when Python is built with
--enable-shared. Classify also libpython frames as "python".
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL