Skip to content

gh-144741: Fix test_frame_pointer_unwind for libpython#145499

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

gh-144741: Fix test_frame_pointer_unwind for libpython#145499
vstinner merged 1 commit intopython:mainfrom
vstinner:classify_address

Conversation

@vstinner
Copy link
Copy Markdown
Member

@vstinner vstinner commented Mar 4, 2026

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
Copy link
Copy Markdown
Member Author

vstinner commented Mar 4, 2026

cc @diegorusso

@vstinner
Copy link
Copy Markdown
Member Author

vstinner commented Mar 4, 2026

cc @pablogsal

@pablogsal
Copy link
Copy Markdown
Member

Great catch!

@pablogsal
Copy link
Copy Markdown
Member

CC @diegorusso

Copy link
Copy Markdown
Contributor

@diegorusso diegorusso left a comment

Choose a reason for hiding this comment

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

I agree, excellent stuff!

@vstinner
Copy link
Copy Markdown
Member Author

vstinner commented Mar 4, 2026

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

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

vstinner commented Mar 4, 2026

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".
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