NotificationsYou must be signed in to change notification settings
Fork34k
Star71.3k

GH-141794: Fix reference leaks introduced in GH-142228#142257

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

markshannon merged 1 commit intopython:mainfrommarkshannon:fix-leaks

Dec 4, 2025

Merged

GH-141794: Fix reference leaks introduced in GH-142228#142257

markshannon merged 1 commit intopython:mainfrommarkshannon:fix-leaks

Dec 4, 2025

Conversation

Copy link

Member

markshannon commentedDec 4, 2025•
edited by chris-eibl
Loading

Issue:Compiling executors can result in excessively large amounts of machine code. #141794

Fix reference leaks introduced inpythonGH-140800

fa5af0d

bedevere-appbot mentioned this pull request

Dec 4, 2025

JIT assembly optimizer leaves some redundant branches#139757

Open

bedevere-appbot added the awaiting core review label

Dec 4, 2025

markshannon added skip news 🔨 test-with-refleak-buildbotsTest PR w/ refleak buildbots; report in status section labels

Dec 4, 2025

Copy link

bedevere-bot commentedDec 4, 2025

🤖 New build scheduled with the buildbot fleet by@markshannon for commitfa5af0d 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F142257%2Fmerge

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

bedevere-bot removed the 🔨 test-with-refleak-buildbotsTest PR w/ refleak buildbots; report in status section label

Dec 4, 2025

markshannon mentioned this pull request

Dec 4, 2025

GH-141794: Limit size of generated machine code.#142228

Merged

diegorusso approved these changes

Dec 4, 2025

View reviewed changes

bedevere-appbot added awaiting merge and removed awaiting core review labels

Dec 4, 2025

Copy link

MemberAuthor

markshannon commentedDec 4, 2025

The s390x failures appear to timing related and aren't memory leaks.

markshannon merged commit6825d5c intopython:main

Dec 4, 2025

71 of 72 checks passed

bedevere-appbot removed the awaiting merge label

Dec 4, 2025

Copy link

bedevere-bot commentedDec 4, 2025

⚠️⚠️⚠️ Buildbot failure⚠️⚠️⚠️

Hi! The buildbotx86 Debian Installed with X 3.x (no tier) has failed when building commit6825d5c.

What do you need to do:

Don't panic.
Checkthe buildbot page in the devguide if you don't know what the buildbots are or how they work.
Go to the page of the buildbot that failed (https://buildbot.python.org/#/builders/1244/builds/6846) and take a look at the build logs.
Check if the failure is related to this commit (6825d5c) or if it is a false positive.
If the failure is related to this commit, please, reflect that on the issue and make a new Pull Request with a fix.

You can take a look at the buildbot page here:

https://buildbot.python.org/#/builders/1244/builds/6846

Summary of the results of the build (if available):

Click to see traceback logs

Traceback (most recent call last):  File �[35m"/buildbot/buildarea/3.x.ware-debian-x86.installed/build/target/lib/python3.15/test/test_annotationlib.py"�[0m, line �[35m160�[0m, in �[35mtest_nonexistent_attribute�[0m    �[31mself.assertEqual�[0m�[1;31m(epsilon_anno, support.EqualToForwardRef("some | {obj, module}",owner=f))�[0m    �[31m~~~~~~~~~~~~~~~~�[0m�[1;31m^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^�[0m�[1;35mAssertionError�[0m:�[35mForwardRef('some | {module, obj}', owner=[79 chars]028>) != EqualToForwardRef('some | {obj, module}',[86 chars]028>)�[0mTraceback (most recent call last):  File �[35m"/buildbot/buildarea/3.x.ware-debian-x86.installed/build/target/lib/python3.15/test/test_annotationlib.py"�[0m, line �[35m160�[0m, in �[35mtest_nonexistent_attribute�[0m    �[31mself.assertEqual�[0m�[1;31m(epsilon_anno, support.EqualToForwardRef("some | {obj, module}",owner=f))�[0m    �[31m~~~~~~~~~~~~~~~~�[0m�[1;31m^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^�[0m�[1;35mAssertionError�[0m:�[35mForwardRef('some | {module, obj}', owner=[79 chars]b80>) != EqualToForwardRef('some | {obj, module}',[86 chars]b80>)�[0m